Skip to content

add r2 release creds to gcp release-builder publish jobs for 1.29 and 1.30 - #6061

Merged
istio-testing merged 2 commits into
istio:masterfrom
PetrMc:petrmc/gcp-release-builder-r2-creds
Sep 21, 2026
Merged

istio-testing merged 2 commits into
istio:masterfrom
PetrMc:petrmc/gcp-release-builder-r2-creds

Conversation

@PetrMc

@PetrMc PetrMc commented Sep 21, 2026

Copy link
Copy Markdown
Member

publish.sh on release-1.29/1.30 uploads to R2 since istio/release-builder#2419 and istio/release-builder#2420 but the gcp publish jobs never got cf-r2-istio-release-auth, so 1.29.8 and 1.30.5 publish failed with CF_CREDENTIALS unbound. Matches the release-1.31 aws job.

@PetrMc
PetrMc requested a review from a team as a code owner September 21, 2026 17:05
@istio-policy-bot

Copy link
Copy Markdown

😊 Welcome @PetrMc! This is either your first contribution to the Istio test-infra repo, or it's been
a while since you've been here.

You can learn more about the Istio working groups, Code of Conduct, and contribution guidelines
by referring to Contributing to Istio.

Thanks for contributing!

Courtesy of your friendly welcome wagon.

@istio-testing istio-testing added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 21, 2026
… 1.30

Signed-off-by: Petr McAllister <petr.mcallister@gmail.com>
@PetrMc

PetrMc commented Sep 21, 2026

Copy link
Copy Markdown
Member Author

also dropped the istio-ecosystem main/1.31 sail-operator dashboards from testgrid: #6041 moved those jobs to aws but the configurator (both the presubmit check and the trusted upload to gs://istio-testgrid) only reads prow/gcp/cluster/jobs, so they had no tabs and the check has been failing since sep 16. reverted the config_test.go merge from #6041 so the go test matches what prod generates, same as #6008 did for the istio jobs

@istio-testing istio-testing added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Sep 21, 2026
Signed-off-by: Petr McAllister <petr.mcallister@gmail.com>

@fjglira fjglira left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lgtm. Let's rerun and check

@istio-testing
istio-testing merged commit 1fe3b02 into istio:master Sep 21, 2026
7 checks passed
@istio-testing

Copy link
Copy Markdown
Collaborator

@PetrMc: Updated the job-config configmap in namespace default at cluster default using the following files:

  • key istio.release-builder.release-1.29.gen.yaml using file prow/gcp/cluster/jobs/istio/release-builder/istio.release-builder.release-1.29.gen.yaml
  • key istio.release-builder.release-1.30.gen.yaml using file prow/gcp/cluster/jobs/istio/release-builder/istio.release-builder.release-1.30.gen.yaml
Details

In response to this:

publish.sh on release-1.29/1.30 uploads to R2 since istio/release-builder#2419 and istio/release-builder#2420 but the gcp publish jobs never got cf-r2-istio-release-auth, so 1.29.8 and 1.30.5 publish failed with CF_CREDENTIALS unbound. Matches the release-1.31 aws job.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants