Skip to content
This repository was archived by the owner on Nov 27, 2024. It is now read-only.

cd: build/push images#59

Merged
filariow merged 5 commits intokonflux-workspaces:mainfrom
sadlerap:cd-build-images
Apr 4, 2024
Merged

cd: build/push images#59
filariow merged 5 commits intokonflux-workspaces:mainfrom
sadlerap:cd-build-images

Conversation

@sadlerap
Copy link
Member

@sadlerap sadlerap commented Mar 28, 2024

Push built artifacts to quay.io for further distribution, either for testing or use in production.

Containers are tagged under the following naming convention:

  • Release artifacts are tagged with the first 8 characters of the latest commit. The most-recently-pushed container will also be tagged with latest.
  • Testing artifacts are tagged as pr-${PR_NUMBER}-${COMMIT}, where $PR_NUMBER is the pull request number, and $COMMIT is the first 8 characters on the latest commit. This naming convention distinguishes testing artifacts from release artifacts.

@sadlerap sadlerap marked this pull request as draft March 28, 2024 22:30
@sadlerap sadlerap marked this pull request as ready for review March 28, 2024 22:36
Push built artifacts to quay.io for further distribution, either for
testing or use in production.

Containers are tagged under the following naming convention:

- Release artifacts are tagged with the output of `git describe
  --always`.  The most-recently-pushed container will be tagged with
  `latest`.
- Testing artifacts are tagged as `pr-${PR_NUMBER}-${COMMIT}`, where
  $PR_NUMBER is the pull request number, and $COMMIT is the output of
  `git describe --always` on the latest commit.  This naming convention
  distinguishes testing artifacts from release artifacts.

Signed-off-by: Andy Sadler <ansadler@redhat.com>
Copy link
Member

@filariow filariow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly LGTM, just one nit

Co-authored-by: Francesco Ilario <filario@redhat.com>
Signed-off-by: Andy Sadler <ansadler@redhat.com>
@sadlerap sadlerap requested a review from filariow April 3, 2024 20:24
sadlerap and others added 2 commits April 3, 2024 15:29
This makes image tags more predictable and will let us pull pre-built
images in prow.

Signed-off-by: Andy Sadler <ansadler@redhat.com>
Copy link
Member

@filariow filariow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mostly lgtm, just some nits

Fix three errors:
- we should unconditionally use $GITHUB_SHA for fetching commit SHAs,
  rather than from the pull_request event.
- remove an outdated comment referring to ghcr.io; we don't push there.
- `docker push -a` takes an untagged image name, so we should leave them
  off.

Signed-off-by: Andy Sadler <ansadler@redhat.com>
@sadlerap sadlerap requested a review from filariow April 4, 2024 17:23
Copy link
Member

@filariow filariow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@filariow filariow merged commit f4f27ac into konflux-workspaces:main Apr 4, 2024
@sadlerap sadlerap deleted the cd-build-images branch April 5, 2024 12:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants