-
Notifications
You must be signed in to change notification settings - Fork 39k
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
Re-add pkg/generated/bindata.go #65985
Re-add pkg/generated/bindata.go #65985
Conversation
/assign @thockin |
/retest |
/lgtm |
/lgtm yes please! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dims, ixdy, thockin 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:
Approvers can indicate their approval by writing |
@ixdy, Let's backport please! |
#66009 for cherrypick. |
Automatic merge from submit-queue (batch tested with PRs 65993, 65986, 65351, 65996, 65985). If you want to cherry-pick this change to another branch, please follow the instructions here. |
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Don't delete pkg/generated/bindata.go in make clean **What this PR does / why we need it**: follow-up to #65985: `pkg/generated/bindata.go` has been re-added to the repo, so `make clean` shouldn't delete it, as doing so makes the tree dirty. This is mostly problematic for jobs which run `make clean` followed by `make quick-release` or `make release` (like our CI build job), since those won't regenerate `pkg/generated/bindata.go`, as they run builds inside the build container. **Release note**: ```release-note NONE ```
…65986-#65985-upstream-release-1.11 Automatic merge from submit-queue. Automated cherry pick of #65882: Add script to verify generated files #65986: verify-generated-files: ensure git tree is clean #65985: Don't gitignore pkg/generated/bindata.go Cherry pick of #65882 #65986 #65985 #66076 on release-1.11. #65882: Add script to verify generated files #65986: verify-generated-files: ensure git tree is clean #65985: Re-add pkg/generated/bindata.go #66076: Don't delete pkg/generated/bindata.go in make clean ```release-note Re-adds `pkg/generated/bindata.go` to the repository to allow some parts of k8s.io/kubernetes to be go-vendorable. ``` Fixes #65968.
/sig release |
What this PR does / why we need it: I removed
pkg/generated/bindata.go
in #62432 as it had drifted out-of-date and was automatically generated bymake
andbazel
.This broke some downstream users who import
k8s.io/kubernetes
.It's unclear whether the project intends for
k8s.io/kubernetes
to be vendorable, as there are other generated files which are not saved in the repo, includingpkg/generated/openapi/zz_generated.openapi.go
andtest/e2e/generated/bindata.go
, butpkg/generated/bindata.go
seems to be a core dependency for current uses.The new verify check added in #65882 will ensure that this file stays up-to-date.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):This partially addresses #65968. We also need to cherry-pick this to release-1.11.
Release note: