-
Notifications
You must be signed in to change notification settings - Fork 8
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
use quay images in stepAction #156
base: main
Are you sure you want to change the base?
Conversation
tekton/cache-fetch.yaml
Outdated
@@ -75,7 +75,8 @@ spec: | |||
- name: BLOB_QUERY_PARAMS | |||
value: $(params.blobQueryParams) | |||
# FIXME: use a released version once something is released :) | |||
image: ghcr.io/openshift-pipelines/tekton-caches/cache:latest | |||
# image: ghcr.io/openshift-pipelines/tekton-caches/cache:latest |
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.
Do we need to comment it or straight away we can remove it
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.
@vdemeester
one of our customer is not able to use this stepAction as we are referring the images from ghcr.
As per their Org Policy ghcr.io is not accessible in their cluster.
@PuneetPunamiya
We need to remove this but I have this in comment for reference only. will remove this before merging this PR
use quay images in stepAction Signed-off-by: Pramod Bindal <[email protected]>
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.
The problem right now is that those images are expiring at some point. The image we refer here need to work tomorrow as well as in a year or 2.
i think only on-pr images expires. on-push images stay forever. isn't that true ? secondly I am trying to nudge the action as well so as soon as we have new image konflux should update the digest |
No description provided.