In s390x publish centos9 and kubevirtci jobs, do GCS calls without auth where possible #3945
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does / why we need it:
In s390x publish centos9 and kubevirtci jobs, do GCS calls to check for existence and contents of the publicly available files under kubevirt-prow/release/kubevirt/kubevirtci/ without authentication
As there are limits to number of refresh tokens that are issued per client-user combination in GCS, and beyond limits it may make old token to stop working, so we are reducing the token usage with this fix.
Ref: https://developers.google.com/identity/protocols/oauth2#5.-refresh-the-access-token,-if-necessary.
#3812
This PR is adds to the previous PR 3812 to fix the issue
This should address prow job failures like:
periodic-kubevirtci-bump-centos-base-s390x/1871405677058985984
publish-kubevirtci-s390x/1886882224930820096
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
@brianmcarey As you suggested earlier, I'm using the curl without auth for publicly available URLs. Apologies for implementing it late. Please review.
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.
Release note: