Skip to content

operator opentelemetry-operator (0.125.0) #6181

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

opentelemetrybot
Copy link
Contributor

Release opentelemetry-operator 0.125.0.

cc @pavolloffay @frzifus @yuriolisa @jaronoff97 @TylerHelmuth @swiatekm @iblancasa

Signed-off-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
Copy link
Contributor

@frzifus,@iblancasa,@yuriolisa,@pavolloffay , please approve as you are original reviewer(s).

  • To add more contributors to ci.yaml follow instruction here.
  • To approve follow instruction here

@openshift-ci openshift-ci bot requested review from haripate and mporrato May 15, 2025 12:18
Copy link
Contributor

Dockerfile or bundle.Dockerfile is added/changed. Note that for security reasons none of these files are going to be used when building bundle. Docker file will be generated and all label information is taken from annotations.yaml.

@github-actions github-actions bot changed the title Update the opentelemetry to 0.125.0 operator opentelemetry-operator (0.125.0) May 15, 2025
@framework-automation
Copy link
Collaborator

/merge possible

@framework-automation
Copy link
Collaborator

/merge possible

Copy link
Contributor

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

1 similar comment
Copy link
Contributor

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

@viktordienes
Copy link

@iblancasa @jaronoff97 @frzifus @swiatekm

This PR is ready to be merged automatically, but it's missing the authorized-changes label for the following versions:

Could one of you please add the necessary labels so we can proceed with the merge?

Thank you!

@jaronoff97
Copy link

the comment from the github actions goes nowhere, @viktordienes what should we do?

@dbiro
Copy link

dbiro commented Jun 5, 2025

Hi @jaronoff97,

According to logs about this run of operator CI check operator changes has been detected which is not allowed by default. If it was not intended see troubleshooting otherwise follow steps in maintainer docs.

Have you checked the above things? I assume you have the right permissions as a maintainer to fix it. Am I right?

Cheers,
Daniel

@swiatekm
Copy link

swiatekm commented Jun 5, 2025

@dbiro looking at the maintainer documentation, it sounds like the authorized_changes label should be added automatically upon approval from a reviewer from ci.yaml, but that doesn't seem to actually be happening. I don't think any of the opentelemetry-operator maintainers has permission to directly alter labels on this PR.

@dbiro
Copy link

dbiro commented Jun 5, 2025

@dbiro looking at the maintainer documentation, it sounds like the authorized_changes label should be added automatically upon approval from a reviewer from ci.yaml, but that doesn't seem to actually be happening. I don't think any of the opentelemetry-operator maintainers has permission to directly alter labels on this PR.

@swiatekm I have tried to dig deeper to figure out why this label is missing but still not clear for me. I understood that authorized_changes label should have been added automatically after approval bit I couldn't find the root cause. I have (re)created this PR (just for a retry) but It didn't worked.

Who could help in debugging?

@pavolloffay
Copy link
Contributor

This PR can be closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.