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

Fix OTEL Operator version for E2E Operator EKS tests and revert previous fix attempt #1655

Merged
merged 2 commits into from
Jul 11, 2024

Conversation

thpierce
Copy link
Contributor

Description:
We recently observed that main build was failing in ADOT Java repo, and failures started after May 20th, when OTEL Operator v0.100.0 was released, adding finalizer logic that we believe to be incompatible with our test framework. As a short term measure, we are fixing the OTEL Operator version to v0.99.0, the last known healthy version (as the test was passing on May 17th, when v0.99.0 was latest).

Also in this PR, we are reverting a previous change that attempted to rename the Collector, as we thought there was some problem with a duplicate naming, though this only introduced new problems.

Link to tracking Issue: N/A

Testing: N/A

Documentation: N/A

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

thpierce added 2 commits July 11, 2024 08:29
This reverts commit c0ad0f8, reversing
changes made to 45baf87.
We recently observed that main build was failing in ADOT Java repo, and failures started after May 20th, when OTEL Operator v0.100.0 was released, adding finalizer logic that we believe to be incompatible with our test framework. As a short term measure, we are fixing the OTEL Operator version to v0.99.0, the last known healthy version (as the test was passing on May 17th, when v0.99.0 was latest).
@thpierce thpierce requested a review from a team as a code owner July 11, 2024 15:40
@Aneurysm9 Aneurysm9 merged commit 0a771ab into aws-observability:terraform Jul 11, 2024
10 checks passed
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.

3 participants