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

Improved release workflow #38

Merged
merged 1 commit into from
Feb 27, 2024
Merged

Improved release workflow #38

merged 1 commit into from
Feb 27, 2024

Conversation

halostatue
Copy link
Member

We maintain three docker images:

  • ansible
  • sqitch / pgTAP
  • aws-cli with SessionManager plugin

The release flows are basically the same, but there were unnecessary
differences.

  • We now no longer conditionally create the release in parallel to the
    image build and push. This would result in orphan releases if the
    build failed. The release creation is done at the end, immediately before
    updating the Dockerhub description.

  • We add annotations to all metadata configuration.

  • We set env.REPOSITORY based on GITHUB_REPOSITORY, converting entirely to
    lowercase and removing the leading docker-.

  • We use kineticcafe/${{ env.REPOSITORY }} for image builds and Dockerhub
    description updates.

We maintain three docker images:

- ansible
- sqitch / pgTAP
- aws-cli with SessionManager plugin

The release flows are basically the same, but there were *unnecessary*
differences.

- We now no longer conditionally create the release in parallel to the
  image build and push. This would result in orphan releases if the
  build failed. The release creation is done at the end, immediately before
  updating the Dockerhub description.

- We add `annotations` to all metadata configuration.

- We set `env.REPOSITORY` based on `GITHUB_REPOSITORY`, converting entirely to
  lowercase and removing the leading `docker-`.

- We use `kineticcafe/${{ env.REPOSITORY }}` for image builds and Dockerhub
  description updates.
@halostatue halostatue requested review from mikestok and ohnit February 27, 2024 17:49
@halostatue halostatue merged commit 11ef0c2 into main Feb 27, 2024
1 check passed
@halostatue halostatue deleted the improved-release-workflow branch February 27, 2024 20:08
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