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

Request for Multiarch Support in Telemetrygen Container Image. #30072

Closed
IshwarKanse opened this issue Dec 19, 2023 · 2 comments · Fixed by #29905
Closed

Request for Multiarch Support in Telemetrygen Container Image. #30072

IshwarKanse opened this issue Dec 19, 2023 · 2 comments · Fixed by #29905
Assignees
Labels
cmd/telemetrygen telemetrygen command enhancement New feature or request

Comments

@IshwarKanse
Copy link

Component(s)

cmd/telemetrygen

Is your feature request related to a problem? Please describe.

The OpenTelemetry Operator and Collector are compatible with multiple architectures including linux/amd64, linux/arm64, linux/ppc64le, and linux/s390x. However, the E2E tests in the operator repository rely on Telemetrygen, which currently lacks multiarch support.

To enable testing on all supported platforms, it’s crucial that the Telemetrygen image is built with multiarch support. This enhancement will allow us to run our E2E tests across all supported architectures, ensuring comprehensive coverage and compatibility.

We propose that the Telemetrygen image be built directly from the project repository, which would streamline the process and maintain consistency. This change would be a significant improvement for our testing infrastructure.

Describe the solution you'd like

Update the Telemetrygen workflow to build and publish multiarch images.

Describe alternatives you've considered

Build the multiarch images from the forked repo and publish it to a container registry. Replace the image in tests.

Additional context

No response

@IshwarKanse IshwarKanse added enhancement New feature or request needs triage New item requiring triage labels Dec 19, 2023
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@crobert-1
Copy link
Member

Sounds like a good idea to me! A code owner has also approved the PR, so I'm removing the needs triage label.

@crobert-1 crobert-1 removed the needs triage New item requiring triage label Dec 19, 2023
bogdandrutu pushed a commit that referenced this issue Dec 19, 2023
…29905)

**Description:** <Describe what has changed.>
This PR updates the Telemetrygen workflow to build and publish multiarch
images.

Resolves
#30072

**Testing:** <Describe what testing was performed and which tests were
added.>

ToDo

**Documentation:** <Describe the documentation added.>

---------

Signed-off-by: Ishwar Kanse <ikanse@redhat.com>
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this issue Jan 10, 2024
…pen-telemetry#29905)

**Description:** <Describe what has changed.>
This PR updates the Telemetrygen workflow to build and publish multiarch
images.

Resolves
open-telemetry#30072

**Testing:** <Describe what testing was performed and which tests were
added.>

ToDo

**Documentation:** <Describe the documentation added.>

---------

Signed-off-by: Ishwar Kanse <ikanse@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmd/telemetrygen telemetrygen command enhancement New feature or request
Projects
None yet
2 participants