Skip to content

Conversation

@puckpuck
Copy link
Contributor

Changes

Copies template file into image, and uses an ENTRYPOINT to start. This removes the need to mount volumes or specify a command for the image to start, enabling easier deployment with Helm

Also sets the port to 8080 instead of 80 due to the need for privileged access in some scenarios.

# Conflicts:
#	docker-compose.yml
#	src/frontendproxy/envoy.tmpl.yaml
#	src/frontendproxy/envoy.yaml.tmpl
@puckpuck puckpuck requested a review from a team October 22, 2022 03:06
@austinlparker austinlparker merged commit 541dae3 into open-telemetry:main Oct 22, 2022
@puckpuck puckpuck deleted the frontendproxy-packaging branch October 22, 2022 03:17
jmichalak9 pushed a commit to jmichalak9/opentelemetry-demo that referenced this pull request Mar 22, 2024
* top button goes to /feature

* package frontendproxy image

* set frontendproxy port to 8080

* use proxy url instead

* use proxy url instead

* use proxy url instead
mohamed3637 added a commit to mohamed3637/opentelemetry-demo that referenced this pull request Oct 7, 2025
* top button goes to /feature

* package frontendproxy image

* set frontendproxy port to 8080

* use proxy url instead

* use proxy url instead

* use proxy url instead
cloud-hb pushed a commit to cloud-hb/opentelemetry-demo that referenced this pull request Nov 17, 2025
* top button goes to /feature

* package frontendproxy image

* set frontendproxy port to 8080

* use proxy url instead

* use proxy url instead

* use proxy url instead
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.

2 participants