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

GH Actions: Stop publishing to quay.io #4177

Merged

Conversation

davemarchevsky
Copy link
Collaborator

This hasn't succeeded in quite some time. Let's turn it off for now.

Signed-off-by: Dave Marchevsky [email protected]

This hasn't succeeded in quite some time. Let's turn it off for now.

Signed-off-by: Dave Marchevsky <[email protected]>
@chantra
Copy link
Contributor

chantra commented Aug 14, 2022

This would be interesting to get working though. It would seriously speed up the CI by essentially skipping the build container step, which last I checked was about 6 min.

@davemarchevsky
Copy link
Collaborator Author

@chantra there's a "Publish to Dockerhub" step which is likely doing the same thing and has working creds. Neither quay.io nor dockerhub publish steps have been truly maintained for a while and consistently make master commit signal red.

I'm totally on board for building container off master when new commit is added, then using that for PR tests. But until that's working I'd rather have master commit signal be useful.

@davemarchevsky davemarchevsky merged commit 3e90aa4 into iovisor:master Aug 14, 2022
fengjixuchui added a commit to fengjixuchui/bcc that referenced this pull request Aug 15, 2022
Merge pull request iovisor#4177 from davemarchevsky/davemarchevsky_no…
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.

2 participants