-
Notifications
You must be signed in to change notification settings - Fork 3
Expire images #29
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
base: main
Are you sure you want to change the base?
Expire images #29
Conversation
Signed-off-by: dd <4183971+didierofrivia@users.noreply.github.com>
Signed-off-by: dd <4183971+didierofrivia@users.noreply.github.com>
| echo "Setting expiry to 'never' for release tag ${{ github.ref_name }}" | ||
| else | ||
| echo "expiry=3w" >> $GITHUB_OUTPUT | ||
| echo "Setting expiry to '3w' for non-release ref ${{ github.ref_name }}" |
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.
We could change this and make it open and read the value from input
| - name: Determine quay image expiry | ||
| id: quay-expiry | ||
| run: | | ||
| if [[ "${{ github.ref_type }}" == "tag" && "${{ github.ref_name }}" =~ ^v[0-9]+\.[0-9]+\.[0-9]+(-.*)?$ ]]; then |
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.
We could also just change the Makefile value
developer-portal-controller/Makefile
Line 11 in 16e5f8e
| QUAY_IMAGE_EXPIRY ?= never |
QUAY_IMAGE_EXPIRY to never when triggering the release workflow, and let 3w as the default... what do you reckon?
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.
Same way it's changing the VERSION here
developer-portal-controller/.github/workflows/automated-release.yaml
Lines 52 to 59 in 16e5f8e
| - name: Update VERSION in Makefile | |
| run: | | |
| VERSION="${{ github.event.inputs.version }}" | |
| sed -i "s/^VERSION ?= .*/VERSION ?= $VERSION/" Makefile | |
| echo "Updated Makefile VERSION to $VERSION" | |
| grep "^VERSION" Makefile | |
Closes #8
TODO