Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Docker Compose #33

Closed
wants to merge 23 commits into from
Closed

[WIP] Docker Compose #33

wants to merge 23 commits into from

Conversation

mic-max
Copy link
Contributor

@mic-max mic-max commented May 23, 2022

Fixes #24

Changes

docker compose build is working.

Error when trying to run

  • python email service SSL handshake error

For significant contributions please make sure you have completed the following items:

  • Appropriate CHANGELOG.md updated for non-trivial changes
  • Design discussion issue #

@mviitane
Copy link
Member

I tried this out and it seems to be compiling and running fine on both Apple silicon and x86.

@cartersocha
Copy link
Contributor

Hey @mic-max, should we close this issue now or is this related to the bug in the docker compose?

@jsirianni
Copy link
Member

I know this is still WIP but figured I would mention my results on Fedora amd64:

Fails with duplicate ports I believe.

Creating opentelemetry-demo-webstore_redis_1   ... done
Creating opentelemetry-demo-webstore_jaeger_1 ... done
Creating opentelemetry-demo-webstore_otelcol_1 ... done
Creating opentelemetry-demo-webstore_paymentservice_1 ... 
Creating opentelemetry-demo-webstore_paymentservice_1        ... error
Creating opentelemetry-demo-webstore_adservice_1             ... done
Creating opentelemetry-demo-webstore_emailservice_1   ... 
Creating opentelemetry-demo-webstore_shippingservice_1       ... done
Creating opentelemetry-demo-webstore_cartservice_1           ... done
Creating opentelemetry-demo-webstore_productcatalogservice_1 ... done
Creating opentelemetry-demo-webstore_emailservice_1          ... done

ERROR: for opentelemetry-demo-webstore_paymentservice_1  Cannot start service paymentservice: driver failed programming external connectivity on endpoint opentelemetry-demo-webstore_paymentservice_1 (50d572803a35d7acbd642c475Creating opentelemetry-demo-webstore_currencyservice_1       ... done
Creating opentelemetry-demo-webstore_recommendationservice_1 ... 
Creating opentelemetry-demo-webstore_recommendationservice_1 ... error

ERROR: for opentelemetry-demo-webstore_recommendationservice_1  Cannot start service recommendationservice: driver failed programming external connectivity on endpoint opentelemetry-demo-webstore_recommendationservice_1 (9d15b24ec9da177edc10d4b95fe9b0efda061c262dc5bf6d65795b54ded70930): Bind for 0.0.0.0:8080 failed: port is already allocated

ERROR: for paymentservice  Cannot start service paymentservice: driver failed programming external connectivity on endpoint opentelemetry-demo-webstore_paymentservice_1 (50d572803a35d7acbd642c47518a21f7407dfe1910291e4074f7a9580808bc3e): Bind for 0.0.0.0:50051 failed: port is already allocated

ERROR: for recommendationservice  Cannot start service recommendationservice: driver failed programming external connectivity on endpoint opentelemetry-demo-webstore_recommendationservice_1 (9d15b24ec9da177edc10d4b95fe9b0efda061c262dc5bf6d65795b54ded70930): Bind for 0.0.0.0:8080 failed: port is already allocated
ERROR: Encountered errors while bringing up the project.

@mic-max mic-max closed this Jun 2, 2022
@mic-max
Copy link
Contributor Author

mic-max commented Jun 2, 2022

Closing in favor of the above linked PR by @julianocosta89

The other changes on this branch will be extracted into other smaller PRs

smith pushed a commit to smith/opentelemetry-demo that referenced this pull request Aug 5, 2024
* remove unused daemonset.yaml file

* feat: add elasticsearch k8s secrets

* docs: add elasticsearch endpoint secrets

* set daemonset as default Helm deployment method

* Revert "set daemonset as default Helm deployment method"

This reverts commit 80612a5.

* Revert "remove unused daemonset.yaml file"

This reverts commit 8de4003.

* decouple elasticsearch exporter from deployment mode

* feat: add configmap for daemonset deployment

* rollback filelog pods exclusion

* feat: add deployment.environment attribute

* remove extra volumes in deployment mode

* remove security context from deployment mode

* add security context for daemonset deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate build to use docker compose
4 participants