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

Clean up e2e Dockerfiles #1393

Merged
merged 14 commits into from
Dec 13, 2024
Merged

Clean up e2e Dockerfiles #1393

merged 14 commits into from
Dec 13, 2024

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Dec 9, 2024

As part of #1140, the Dockerfiles are clean up.

  • Ensure the docker builds can be cached
  • Unify the entrypoint of the e2e binary to be located in a Linux filesystem appropriate location
  • Unify the Go application to be the main program that runs the testing application to prevent logic split when running
    • Update the kafka e2e test to call the Kafka setup script from the Go program

@MrAlias MrAlias force-pushed the e2e-dockerfiles branch 4 times, most recently from 7b83aa3 to 708b5b7 Compare December 9, 2024 21:43
@MrAlias MrAlias force-pushed the e2e-dockerfiles branch 3 times, most recently from d9d43b4 to d951331 Compare December 9, 2024 22:22
@MrAlias MrAlias marked this pull request as ready for review December 9, 2024 22:22
@MrAlias MrAlias requested a review from a team as a code owner December 9, 2024 22:22
@MrAlias MrAlias added this to the v0.20.0-alpha milestone Dec 11, 2024
@MrAlias MrAlias merged commit f1a441d into open-telemetry:main Dec 13, 2024
27 checks passed
@MrAlias MrAlias deleted the e2e-dockerfiles branch December 13, 2024 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants