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: build and push Docker images on release #1

Merged
merged 1 commit into from
Apr 19, 2024
Merged

Conversation

vayan
Copy link
Member

@vayan vayan commented Apr 18, 2024

Also refactor the "publish to pypi" workflow into a global "release" workflow.
because the images depend on the package being published.

Run sample:

Screenshot 2024-04-18 at 13 08 29

@vayan vayan requested a review from CCP-Zeulix April 18, 2024 07:49
@CCP-Zeulix
Copy link
Contributor

Sheer brilliance!

Looks really good and if it works, feel free to "ship-it" :D

The only thing that might be worth looking into is to check if it's possible in the pipelines to check if the release was flagged as a "pre-release" and if so, skip the image building part (or make it optional and triggered by manual button pressing for pre-releases). We basically only want images with stable versions as they're used as base images for production services.

Also refactor the "publish to pypi" workflow into a global
"release" workflow.
because the images depend on the package being published.
@vayan vayan force-pushed the add-gha-docker-images branch from 5de41aa to 416ce64 Compare April 18, 2024 08:45
@vayan
Copy link
Member Author

vayan commented Apr 18, 2024

Oh you're right! Added a "prereleased" workflow that will only publish to pypi and updated the previous one (pypi+images) to only trigger on "released" events.

@vayan vayan marked this pull request as ready for review April 19, 2024 02:03
@vayan vayan merged commit ede3e5c into main Apr 19, 2024
10 checks passed
@vayan vayan deleted the add-gha-docker-images branch April 19, 2024 02:03
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