Skip to content

K8s 1.33 #287

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

K8s 1.33 #287

wants to merge 2 commits into from

Conversation

liangyuanpeng
Copy link
Contributor

@liangyuanpeng liangyuanpeng commented Apr 24, 2025

  • generate k8s 1.33 module
  • delete some files

NOTE: deleted scripts/k8s/processed-swagger.json and scripts/k8s/swagger.json, i think it's not need here, remind me if i missed something.

1. Please confirm that you have read the document before PR submitted

2. Contact Information(Optional)

If it is convenient, please provide your contact information so we can reach you when processing the PR:

  • Email:
  • HomePage:

Signed-off-by: liangyuanpeng <gcslyp@gmail.com>
Signed-off-by: liangyuanpeng <gcslyp@gmail.com>
@Peefy
Copy link
Contributor

Peefy commented Apr 24, 2025

Could you help add some unit test suites for the k8s package? Thank you!

@liangyuanpeng
Copy link
Contributor Author

Sure, i'll open a PR to add tests for k8s package and let's continue with this PR after merging the tested PRS.

@liangyuanpeng
Copy link
Contributor Author

seems like generater have some problem:

error[E2L23]: CompileError
 --> /home/vscode/.kcl/kpm/k8s_1.28/api/networking/v1/ingress_backend.k:6:1
  |
6 | import api.core.v1
  | ^ the name 'v1' is defined multiple times, 'v1' must be defined only once
  |


error[E2G22]: TypeError
  --> /home/vscode/.kcl/kpm/k8s_1.28/api/networking/v1/ingress_backend.k:22:16
   |
22 |     resource?: v1.TypedLocalObjectReference
   |                ^ attribute 'TypedLocalObjectReference' not found in 'module 'k8s.apimachinery.pkg.apis.meta.v1''
   |

ref: https://cloud-native.slack.com/archives/C05TC96NWN8/p1745517886457219

and i have reproduced it in this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants