Remove all reference to gcr.io tekton projects #8885

vdemeester · 2025-07-15T09:23:38Z

Changes

There was still a few references that makes the CI fail. This should fix it.

/cc @afrittoli @twoGiants @waveywaves

/kind cleanup

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
Has Tests included if any functionality added or changed
pre-commit Passed
Follows the commit message standard
Meets the Tekton contributor standards (including functionality, content, code)
Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes

NONE

tekton-robot · 2025-07-15T09:23:43Z

@vdemeester: GitHub didn't allow me to request PR reviews from the following users: twoGiants.

Note that only tektoncd members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

Changes

There was still a few references that makes the CI fail. This should fix it.

/cc @afrittoli @twoGiants @waveywaves

/kind cleanup

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps

Has Tests included if any functionality added or changed

pre-commit Passed

Follows the commit message standard

Meets the Tekton contributor standards (including functionality, content, code)

Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep

Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.

Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

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/test-infra repository.

twoGiants

Looks good! Approved.

vdemeester · 2025-07-15T13:38:47Z

/retest

twoGiants · 2025-07-15T14:13:17Z

/kind cleanup

vdemeester · 2025-07-16T11:35:00Z

hmm. I don't understand the failures now…

vdemeester · 2025-07-16T11:35:04Z

/retest

khrm

@vdemeester I think we need to fix step action. It refers to git-init catalog from gcr.

PS: I just saw that it was updated in catalog.

vdemeester · 2025-07-16T11:50:52Z

@vdemeester I think we need to fix step action. It refers to git-init catalog from gcr.

It is fixed on catalog now.

vdemeester · 2025-07-16T12:29:35Z

ahhh I messed up the parameters… it should be green now 🙃

afrittoli · 2025-07-16T13:57:07Z

arendelle-rn6gp              48s         Normal    Started                        taskrun/http-resolver-kz4pn-http-resolver                                 
arendelle-rn6gp              48s         Warning   Failed                         taskrun/http-resolver-kz4pn-http-resolver                                 [User error] declared workspace "output" is required but has not been bound
arendelle-rn6gp              48s         Warning   InternalError                  taskrun/http-resolver-kz4pn-http-resolver                                 declared workspace "output" is required but has not been bound

and later on

default                      62s         Warning   FreeDiskSpaceFailed            node/kind-control-plane                                                   Failed to garbage collect required amount of images. Attempted to free 10449369497 bytes, but only found 3052318 bytes eligible to free.
default                      25m         Normal    Starting                       node/kind-worker                                                          Starting kubelet.
default                      25m         Warning   InvalidDiskCapacity            node/kind-worker                                                          invalid capacity 0 on image filesystem

khrm · 2025-07-16T14:35:27Z

Http resolver examples are failing.

afrittoli · 2025-07-16T14:42:13Z

/retest

examples/v1/pipelineruns/beta/http-resolver-credentials.yaml

afrittoli · 2025-07-16T22:04:18Z

/retest

afrittoli · 2025-07-16T22:40:00Z

Now the http-resolver fails:

    examples_test.go:128: exit status 1 Output: Error from server (BadRequest): error when creating "STDIN": admission webhook "webhook.pipeline.tekton.dev" denied the request: mutation failed: cannot decode incoming new object: json: unknown field "value"

examples/v1/pipelineruns/beta/http-resolver.yaml

There was still a few references that makes the CI fail. This should fix it. Signed-off-by: Vincent Demeester <vdemeest@redhat.com>

vdemeester · 2025-07-17T10:18:35Z

It is finally green 🎉

afrittoli

/approve
/lgtm

tekton-robot · 2025-07-17T10:20:22Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: afrittoli, twoGiants

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

~~OWNERS~~ [afrittoli]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

vdemeester · 2025-07-25T07:43:18Z

This needs to be cherry-picked to old releases for the CI..

vdemeester · 2025-07-25T07:43:24Z

/cherrypick release-v0.65.x

vdemeester · 2025-07-25T07:43:31Z

/cherrypick release-v0.68.x

vdemeester · 2025-07-25T07:43:48Z

/cherrypick release-v1.0.x

vdemeester · 2025-07-25T07:43:57Z

/cherrypick release-v1.2.x

tekton-robot · 2025-07-25T07:44:01Z

@vdemeester: #8885 failed to apply on top of branch "release-v0.65.x":

Applying: Remove all reference to gcr.io tekton projects
Using index info to reconstruct a base tree...
M	config/300-crds/300-pipelinerun.yaml
M	config/300-crds/300-taskrun.yaml
M	pkg/apis/pipeline/v1/openapi_generated.go
M	pkg/apis/pipeline/v1/swagger.json
M	pkg/apis/pipeline/v1beta1/openapi_generated.go
M	pkg/apis/pipeline/v1beta1/swagger.json
M	tekton/release-pipeline.yaml
M	test/resolvers_test.go
M	test/tektonbundles_test.go
Falling back to patching base and 3-way merge...
Auto-merging test/tektonbundles_test.go
Auto-merging test/resolvers_test.go
Auto-merging tekton/release-pipeline.yaml
Auto-merging pkg/apis/pipeline/v1beta1/swagger.json
Auto-merging pkg/apis/pipeline/v1beta1/openapi_generated.go
Auto-merging pkg/apis/pipeline/v1/swagger.json
Auto-merging pkg/apis/pipeline/v1/openapi_generated.go
Auto-merging config/300-crds/300-taskrun.yaml
CONFLICT (content): Merge conflict in config/300-crds/300-taskrun.yaml
Auto-merging config/300-crds/300-pipelinerun.yaml
CONFLICT (content): Merge conflict in config/300-crds/300-pipelinerun.yaml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Remove all reference to gcr.io tekton projects
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherrypick release-v0.65.x

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/test-infra repository.

tekton-robot · 2025-07-25T07:44:08Z

@vdemeester: #8885 failed to apply on top of branch "release-v0.68.x":

Applying: Remove all reference to gcr.io tekton projects
Using index info to reconstruct a base tree...
M	config/300-crds/300-pipelinerun.yaml
M	config/300-crds/300-taskrun.yaml
M	pkg/apis/pipeline/v1/openapi_generated.go
M	pkg/apis/pipeline/v1/swagger.json
M	pkg/apis/pipeline/v1beta1/openapi_generated.go
M	pkg/apis/pipeline/v1beta1/swagger.json
M	tekton/release-pipeline.yaml
M	test/resolvers_test.go
M	test/tektonbundles_test.go
Falling back to patching base and 3-way merge...
Auto-merging test/tektonbundles_test.go
Auto-merging test/resolvers_test.go
Auto-merging tekton/release-pipeline.yaml
Auto-merging pkg/apis/pipeline/v1beta1/swagger.json
Auto-merging pkg/apis/pipeline/v1beta1/openapi_generated.go
Auto-merging pkg/apis/pipeline/v1/swagger.json
Auto-merging pkg/apis/pipeline/v1/openapi_generated.go
Auto-merging config/300-crds/300-taskrun.yaml
CONFLICT (content): Merge conflict in config/300-crds/300-taskrun.yaml
Auto-merging config/300-crds/300-pipelinerun.yaml
CONFLICT (content): Merge conflict in config/300-crds/300-pipelinerun.yaml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Remove all reference to gcr.io tekton projects
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherrypick release-v0.68.x

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/test-infra repository.

tekton-robot · 2025-07-25T07:44:26Z

@vdemeester: new pull request created: #8918

In response to this:

/cherrypick release-v1.0.x

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/test-infra repository.

tekton-robot · 2025-07-25T07:44:35Z

@vdemeester: new pull request created: #8919

In response to this:

/cherrypick release-v1.2.x

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/test-infra repository.

waveywaves · 2025-07-29T19:48:57Z

/cherrypick release-v1.1.x

tekton-robot · 2025-07-29T19:49:37Z

@waveywaves: new pull request created: #8929

In response to this:

/cherrypick release-v1.1.x

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/test-infra repository.

github-project-automation bot added this to Tekton Community Roadmap Jul 15, 2025

github-project-automation bot moved this to Todo in Tekton Community Roadmap Jul 15, 2025

tekton-robot added release-note-none Denotes a PR that doesnt merit a release note. kind/cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. labels Jul 15, 2025

tekton-robot requested review from afrittoli and waveywaves July 15, 2025 09:23

tekton-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jul 15, 2025

twoGiants approved these changes Jul 15, 2025

View reviewed changes

vdemeester force-pushed the fix-e2e-rm-gcrio branch 3 times, most recently from 849de30 to ff031c0 Compare July 16, 2025 09:09

tekton-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jul 16, 2025

vdemeester force-pushed the fix-e2e-rm-gcrio branch 3 times, most recently from 16d43a7 to ff00683 Compare July 16, 2025 10:56

vdemeester mentioned this pull request Jul 16, 2025

Update kubernetes version of the CI #8758

Merged

8 tasks

khrm reviewed Jul 16, 2025

View reviewed changes

vdemeester force-pushed the fix-e2e-rm-gcrio branch from ff00683 to 19edf77 Compare July 16, 2025 12:29

khrm reviewed Jul 16, 2025

View reviewed changes

examples/v1/pipelineruns/beta/http-resolver-credentials.yaml Show resolved Hide resolved

afrittoli reviewed Jul 16, 2025

View reviewed changes

examples/v1/pipelineruns/beta/http-resolver.yaml Outdated Show resolved Hide resolved

vdemeester force-pushed the fix-e2e-rm-gcrio branch from 55a1d27 to 85b9778 Compare July 17, 2025 07:41

Remove all reference to gcr.io tekton projects

bdbe0a2

There was still a few references that makes the CI fail. This should fix it. Signed-off-by: Vincent Demeester <vdemeest@redhat.com>

vdemeester force-pushed the fix-e2e-rm-gcrio branch from 85b9778 to bdbe0a2 Compare July 17, 2025 09:45

afrittoli reviewed Jul 17, 2025

View reviewed changes

tekton-robot assigned afrittoli Jul 17, 2025

tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Jul 17, 2025

tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 17, 2025

tekton-robot merged commit 550b357 into tektoncd:main Jul 17, 2025
20 checks passed

github-project-automation bot moved this from Todo to Done in Tekton Community Roadmap Jul 17, 2025

vdemeester deleted the fix-e2e-rm-gcrio branch July 17, 2025 10:25

twoGiants mentioned this pull request Jul 22, 2025

Switch from Tekton Hub to Artifacthub in examples and docs. #8904

Closed

tekton-robot mentioned this pull request Jul 25, 2025

[release-v1.0.x] Remove all reference to gcr.io tekton projects #8918

Merged

tekton-robot mentioned this pull request Jul 25, 2025

[release-v1.2.x] Remove all reference to gcr.io tekton projects #8919

Merged

tekton-robot mentioned this pull request Jul 29, 2025

[release-v1.1.x] Remove all reference to gcr.io tekton projects #8929

Closed

Uh oh!

Remove all reference to gcr.io tekton projects #8885

Remove all reference to gcr.io tekton projects #8885

Uh oh!

Conversation

vdemeester commented Jul 15, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Submitter Checklist

Release Notes

Uh oh!

tekton-robot commented Jul 15, 2025

Changes

Submitter Checklist

Release Notes

Uh oh!

twoGiants left a comment

Choose a reason for hiding this comment

Uh oh!

vdemeester commented Jul 15, 2025

Uh oh!

twoGiants commented Jul 15, 2025

Uh oh!

vdemeester commented Jul 16, 2025

Uh oh!

vdemeester commented Jul 16, 2025

Uh oh!

khrm left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

vdemeester commented Jul 16, 2025

Uh oh!

vdemeester commented Jul 16, 2025

Uh oh!

afrittoli commented Jul 16, 2025

Uh oh!

khrm commented Jul 16, 2025

Uh oh!

afrittoli commented Jul 16, 2025

Uh oh!

Uh oh!

afrittoli commented Jul 16, 2025

Uh oh!

afrittoli commented Jul 16, 2025

Uh oh!

Uh oh!

vdemeester commented Jul 17, 2025

Uh oh!

afrittoli left a comment

Choose a reason for hiding this comment

Uh oh!

tekton-robot commented Jul 17, 2025

Uh oh!

Uh oh!

vdemeester commented Jul 25, 2025

Uh oh!

vdemeester commented Jul 25, 2025

Uh oh!

vdemeester commented Jul 25, 2025

Uh oh!

vdemeester commented Jul 25, 2025

Uh oh!

vdemeester commented Jul 25, 2025

Uh oh!

tekton-robot commented Jul 25, 2025

Uh oh!

tekton-robot commented Jul 25, 2025

Uh oh!

tekton-robot commented Jul 25, 2025

Uh oh!

tekton-robot commented Jul 25, 2025

Uh oh!

waveywaves commented Jul 29, 2025

Uh oh!

tekton-robot commented Jul 29, 2025

Uh oh!

Reviewers

Assignees

Labels

vdemeester commented Jul 15, 2025 •

edited

Loading

khrm left a comment •

edited

Loading