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

GitHub Actions runs publish_container.yml periodically from forked project #813

Closed
lahwaacz opened this issue Feb 11, 2025 · 3 comments · Fixed by #814
Closed

GitHub Actions runs publish_container.yml periodically from forked project #813

lahwaacz opened this issue Feb 11, 2025 · 3 comments · Fixed by #814

Comments

@lahwaacz
Copy link
Contributor

I have a forked project and just noticed that GitHub Actions runs something periodically:

https://github.com/lahwaacz/hyperqueue/actions/runs/13252379042

It failed on the Tag and push image step because I obviously don't have access to push to ghcr.io/it4innovations/hyperqueue 🙈

Please reconfigure it such that it does not run from fork projects.

@Kobzol
Copy link
Collaborator

Kobzol commented Feb 11, 2025

Hi, you can disable CI on forks, in fact I think that it's disabled by default.

@lahwaacz
Copy link
Contributor Author

I don't remember enabling CI in this fork. I have probably forked it a long time ago and recently just synchronized the content, including this new CI configuration...

In any case, this CI workflow feels very weird. Some forks may want to enable CI, except for this one workflow that pushes to your image registry, and forcing them to change the configuration is not very fork-friendly 🤷

@Kobzol
Copy link
Collaborator

Kobzol commented Feb 11, 2025

When I need to run CI on a fork, I usually just modify CI to suit my needs and avoid things like this. But it doesn't really cost us anything to solve this upstream, so here you go :) #814

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 a pull request may close this issue.

2 participants