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
code-generator: Go native codegen for gen-helpers #122236
base: master
Are you sure you want to change the base?
Conversation
Please note that we're already in Test Freeze for the Fast forwards are scheduled to happen every 6 hours, whereas the most recent run was: Fri Dec 8 10:14:44 UTC 2023. |
Welcome @cardil! |
Hi @cardil. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: cardil The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/ok-to-test |
87a6908
to
c4ce601
Compare
de65f5f
to
63c80bb
Compare
/cc @thockin |
65a1d2b
to
ed5922d
Compare
644a5be
to
1cd3950
Compare
Gentle ping |
1cd3950
to
cb10688
Compare
870a5f8
to
1ade5f6
Compare
f64d961
to
57629ce
Compare
63846df
to
701eb12
Compare
* Go-native code-gen entrypoint structure bcedd4a * Parsing of gen-client flags 1160fc7 * Structure for all commands of the K8s code-gen db6a5d7 * Helpers-gen calls deepcopy-gen aaf0d26 * Remove unneeded Printer interface eb6748c * Setup a default boilerplate 67bcbb0 * Helper-gen calls defaulter-gen 5b524d1 * Streamline logging b596957 * Fixing tests 323501a * Helpers-gen calls the conversion-gen 2831fdd * Replace shell code with golang tooling 0615e85 * Fixing lint errors 30a4a7e * Auto-skip if git isn't available 79a1e6a * Add k8s.io/code-generator to examples go.mod b760ba5 * Printing stderr when exec.Command fails 7280dd2 * Update boilerplate 5c27b82 * Calculate sourcedir from regular sources, not test ones 8bfa3c5 * Migrate off using `git-*` shell outs to search Go files 7b61cf6 * Auto-skip failing test when executed in bad GOPATH 079f581 * Fix lint errors 41d722d * Use deepcopy gen fn for deepcopy-gen main 5be3f70 * Ensure to run codegen main with GO modules 1cd3950
701eb12
to
59892fc
Compare
/test pull-kubernetes-verify It looks like a flaky test (job #1765465013083967488)? |
PR needs rebase. 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. |
Given https://kubernetes.slack.com/archives/C0EG7JC6T/p1705698354201949 discussion, this is on hold, right? |
Yes. On hold. Likely to be closed, soon. |
What type of PR is this?
/kind feature
/kind cleanup
What this PR does / why we need it:
This PR rewrites the shell code from
kube_codegen.sh
into a Go code with both executable, and packages which may be used downstream. I purposely limited the scope for now to justkube::codegen::gen_helpers
. The follow-up PRs are, of course, expected.The reasons for this are explained in the connected issue: kubernetes/code-generator#163.
Which issue(s) this PR fixes:
Fixes kubernetes/code-generator#163 (partially)
Special notes for your reviewer:
This is the first PR for this issue. Expect PRs for other shell functions:
kube::codegen::gen_openapi
andkube::codegen::gen_client
.The idea is not to change the logic of the generator, or the flags, and to keep 100% compatibility.
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: