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

CI steps fail publishing images and logging into docker #3113

Closed
gdfast opened this issue Jul 8, 2024 · 3 comments
Closed

CI steps fail publishing images and logging into docker #3113

gdfast opened this issue Jul 8, 2024 · 3 comments
Labels
discuss-at-sig This issue or PR should be discussed at the next SIG meeting

Comments

@gdfast
Copy link
Contributor

gdfast commented Jul 8, 2024

Component(s)

No response

Describe the issue you're reporting

I've been doing the typical contributor flow of making commits to branches on my fork – https://github.com/gdfast/opentelemetry-operator – and then opening pull requests again the main repo. When I updated PR #3080 today after merging in new upstream changes from the main branch, I got a series of CI failures, including:

All of these workflows fail at the "Log into Docker.io" step with

Error: Username and password required

A little later, I also got the failure

which failed in each workflow's Build and push step with

Error: buildx failed with: ERROR: failed to solve: failed to push ghcr.io/open-telemetry/opentelemetry-operator/e2e-test-app-bridge-server:main: unexpected status from POST request to https://ghcr.io/v2/open-telemetry/opentelemetry-operator/e2e-test-app-bridge-server/blobs/uploads/: 403 Forbidden

My fork was sync'ed to this commit: 1289c63

I acknowledge the OTel Operator is under active development and I may have missed a new setup step that wasn't previously required.

@pavolloffay
Copy link
Member

All these workflows are from your fork https://github.com/gdfast/opentelemetry-operator. We don't guarantee that GHA workflows will work in forked repositories.

@jaronoff97
Copy link
Contributor

@pavolloffay i wonder if we should either disable workflows on user forks by default or have it use their repo for pushing images. That being said, idk why we do a push at all for this?

@jaronoff97 jaronoff97 added discuss-at-sig This issue or PR should be discussed at the next SIG meeting and removed needs triage labels Jul 9, 2024
@pavolloffay
Copy link
Member

pavolloffay commented Jul 10, 2024

I would not object if someone wants to use generic org-name/repo-name in GHA e.g.

ghcr.io/open-telemetry/opentelemetry-operator/autoinstrumentation-java

but I would treat it as a best effort. It's problematic to test and maintain.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss-at-sig This issue or PR should be discussed at the next SIG meeting
Projects
None yet
Development

No branches or pull requests

3 participants