Skip to content

[release-v0.37.x] cherry-pick: fix alibabacloud-go/cr-20160607 license - #3018

Merged
tekton-robot merged 1 commit into
tektoncd:release-v0.37.xfrom
divyansh42:fix-alibabacloud-license-release-v0.37.x
Jul 9, 2026
Merged

[release-v0.37.x] cherry-pick: fix alibabacloud-go/cr-20160607 license#3018
tekton-robot merged 1 commit into
tektoncd:release-v0.37.xfrom
divyansh42:fix-alibabacloud-license-release-v0.37.x

Conversation

@divyansh42

Copy link
Copy Markdown
Member

Summary

  • Cherry-pick of 028e665 from main to fix the "Check generated code" CI failure on release-v0.37.x
  • Uses a replace directive in go.mod to point alibabacloud-go/cr-20160607 to vdemeester/cr-20160607 (a fork that includes the LICENSE file)
  • Removes the manual LICENSE copy workaround from hack/update-deps.sh since the fork handles it

Context

alibabacloud-go/cr-20160607@v1.0.1 ships without a LICENSE file. On main, this was fixed by replacing it with a fork. On release-v0.37.x, a manual copy workaround in update-deps.sh was used instead, but go mod vendor strips these files, causing "Check generated code" failures whenever vendor is regenerated without running the full update-deps.sh script.

This cherry-pick aligns release-v0.37.x with main by using the same replace directive approach.

Made with Cursor

Using a fork + replace, hopefully it will work ?

Signed-off-by: Vincent Demeester <vdemeest@redhat.com>
@tekton-robot tekton-robot added do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jul 9, 2026
@divyansh42

Copy link
Copy Markdown
Member Author

/release-note-none

@tekton-robot tekton-robot added release-note-none Denotes a PR that doesnt merit a release note. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Jul 9, 2026
@divyansh42 divyansh42 changed the title cherry-pick: fix alibabacloud-go/cr-20160607 license [release-v0.37.x] [release-v0.37.x] cherry-pick: fix alibabacloud-go/cr-20160607 license Jul 9, 2026

@vdemeester vdemeester left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

@tekton-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vdemeester

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

The pull request process is described here

Details Needs approval from an approver in each of these files:

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

@tekton-robot tekton-robot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jul 9, 2026
@divyansh42

Copy link
Copy Markdown
Member Author

/retest

3 similar comments
@divyansh42

Copy link
Copy Markdown
Member Author

/retest

@divyansh42

Copy link
Copy Markdown
Member Author

/retest

@divyansh42

Copy link
Copy Markdown
Member Author

/retest

@tekton-robot
tekton-robot merged commit 944a71a into tektoncd:release-v0.37.x Jul 9, 2026
36 of 44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesnt merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants