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

[operator] bump operator version to v0.104.0 #1263

Conversation

frzifus
Copy link
Member

@frzifus frzifus commented Jul 18, 2024

No description provided.

@frzifus frzifus requested a review from Allex1 as a code owner July 18, 2024 12:51
@frzifus frzifus requested a review from a team July 18, 2024 12:51
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. chart:operator Issue related to opentelemetry-operator helm chart labels Jul 18, 2024
Copy link
Contributor

@jaronoff97 jaronoff97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@frzifus you need to bump the chart version to 0.65.0

@frzifus frzifus force-pushed the opentelemetry-operator_release_v0.104.0 branch from 0ea0f7f to 77ae75d Compare July 18, 2024 14:23
@frzifus frzifus requested a review from jaronoff97 July 18, 2024 14:23
@frzifus frzifus force-pushed the opentelemetry-operator_release_v0.104.0 branch from 77ae75d to 1d70d58 Compare July 18, 2024 14:25
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Jul 18, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 18, 2024
@TylerHelmuth
Copy link
Member

@frzifus can you handle the conflicts

@frzifus frzifus force-pushed the opentelemetry-operator_release_v0.104.0 branch from 1d70d58 to bd0fca3 Compare July 31, 2024 10:46
@frzifus
Copy link
Member Author

frzifus commented Jul 31, 2024

Resolved @jaronoff97 @TylerHelmuth

Signed-off-by: Benedikt Bongartz <bongartz@klimlive.de>
@frzifus frzifus force-pushed the opentelemetry-operator_release_v0.104.0 branch from bd0fca3 to bcf5b45 Compare July 31, 2024 12:35
@TylerHelmuth
Copy link
Member

@frzifus one of the tests is failing and rerunning isn't working :(

@TylerHelmuth
Copy link
Member

did something change in 0.104.0?

@frzifus
Copy link
Member Author

frzifus commented Aug 1, 2024

Nothing I am aware about. The output off chainsaw/smoke-pod-dns-config (365.29s) does also not rly help.

@TylerHelmuth
Copy link
Member

@frzifus does the test pass locally?

@jaronoff97
Copy link
Contributor

this seems to be the issue (ran it locally)

⫸ k logs poddnsconfig-collector-6d9c74b776-xsh9q
Error: failed to get config: cannot unmarshal the configuration: 1 error(s) decoding:

* error decoding 'exporters': unknown type: "logging" for id: "logging" (valid values: [nop otlp otlphttp file loadbalancing debug])
2024/08/01 15:25:52 collector server run finished with error: failed to get config: cannot unmarshal the configuration: 1 error(s) decoding:

* error decoding 'exporters': unknown type: "logging" for id: "logging" (valid values: [nop otlp otlphttp file loadbalancing debug])

@jaronoff97
Copy link
Contributor

this is because we're using the k8s distro which no longer has the logging exporter. @TylerHelmuth I can push a fix in the operator repo for the test there and we can ignore this test failure until the 106 release. @frzifus can you add this to the exclude regex here?

@frzifus
Copy link
Member Author

frzifus commented Aug 1, 2024

@TylerHelmuth TylerHelmuth merged commit b32c0b0 into open-telemetry:main Aug 1, 2024
4 checks passed
@frzifus frzifus deleted the opentelemetry-operator_release_v0.104.0 branch August 1, 2024 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart:operator Issue related to opentelemetry-operator helm chart lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants