-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat: add DockerHub and ScaleWay registries #85
Conversation
you can try yourself
|
Thoughts on making a simple dockerfile for this repo that excutes hello world of something so that this workflow runs on this repo? |
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do like it! 🔥 🪨
DockerHub CI Access Token has been created and stored in a password manager. |
should this be an org level secret so that we aren't adding them one by one? or are they repo specific access tokens? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like there is some left over bash code that needs to be removed.
We should also submit the docker test as a separate PR to merge in so that we can see the test here on this PR.
Co-authored-by: Matthew Sevey <[email protected]>
Co-authored-by: Matthew Sevey <[email protected]>
40dee0d
to
d2f7700
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merge conflicts need to be resolved so that we can verify the CI.
one conversation still needs resolution.
@MSevey they are org level wise (For dockerhub registry I am using my own as it's a free tier plan 😓 ) |
@Bidon15 sounds good, lmk if you need any help here. Sounds like we just need to create a celestia dockerhub registry account? and then migrate from your account to the company account? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
workflow file has an error https://github.com/celestiaorg/.github/actions/runs/6866669808
Ok, I thought that was fixed, but I think I misread that you need to do a lot of workarounds to make this happen 😢 Will revert to just main |
Love learning new limitations on github action |
@MSevey please grant proper rights to the job testing is done in #88 |
@Bidon15 this isnt a permission that we grant. You have to specific the permissions required in the workflow. i.e.
like:
|
Closing this PR bc #88 is so much better and works 🖖 |
Overview
Part of https://github.com/celestiaorg/devops/issues/124
With this common workflow update, we will have 3 ways where we can fetch newly built containers
In order to fully activate this - we need to revive secrets from the celestiaorg common account in DockerHub and create a new token
https://hub.docker.com/u/celestiaorg
For Scaleway, the secrets are created in 1pass and we only need to transfer the secrets to respective repositories, like:
In addition to that, common workflows should be called with inheritance like I did here in my own repo
https://github.com/Bidon15/celestia-node/blob/main/.github/workflows/docker-build-publish.yml
Other note is caching. We need it to make builds sub 10 minutes. I've tested them thoroughly in my forks and it works fine as of now in celestia-node.
Meaning that other repos should be fine too
Checklist
closes: https://github.com/celestiaorg/devops/issues/562