Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Bump k8s to v0.29.0 and remove k8s.podSecurityPolicies #2883

Open
czunker opened this issue Dec 19, 2023 · 2 comments · May be fixed by #3954
Open

Bump k8s to v0.29.0 and remove k8s.podSecurityPolicies #2883

czunker opened this issue Dec 19, 2023 · 2 comments · May be fixed by #3954

Comments

@czunker
Copy link
Contributor

czunker commented Dec 19, 2023

Describe the bug

{"level":"info","file":"providers/k8s/resources/k8s.lr.manifest.yaml","time":"2023-12-19T15:56:03Z","message":"write file"}
--> [k8s] generate CLI json
go: downloading k8s.io/apiextensions-apiserver v0.29.0
go: downloading k8s.io/kube-openapi v0.0.0-20231214164306-ab13479f8bf8
go: downloading github.com/go-openapi/swag v0.22.5
go: downloading github.com/go-openapi/jsonreference v0.20.3
go: downloading github.com/go-openapi/jsonpointer v0.20.1
# go.mondoo.com/cnquery/v9/providers/k8s/resources
Error: resources/podsecuritypolicy.go:19:22: undefined: policyv1beta1.PodSecurityPolicy
Error: resources/podsecuritypolicy.go:31:39: undefined: policyv1beta1.PodSecurityPolicy
Error: resources/podsecuritypolicy.go:64:34: cannot infer T (/home/runner/_work/cnquery/cnquery/providers-sdk/v1/util/convert/map.go:7:24)
Error: resources/podsecuritypolicy.go:68:34: cannot infer T (/home/runner/_work/cnquery/cnquery/providers-sdk/v1/util/convert/map.go:7:24)
make: *** [Makefile:310: providers/build/k8s] Error 1
Error: Process completed with exit code 2.

To Reproduce
Steps to reproduce the behavior:

  1. Bump k8s to v0.29.0
  2. Run pipeline
  3. Note the error

Expected behavior
Should work and pass pipeline.
@czunker czunker mentioned this issue Dec 19, 2023
Merged
@czunker
Copy link
Contributor Author

czunker commented Dec 20, 2023 •
edited

Hold for now. But at some point we have to upgrade.

@czunker czunker changed the title Bump k8s to v0.29.0 and fix policyv1beta1.PodSecurityPolicy Bump k8s to v0.29.0 and remove k8s.podSecurityPolicies Dec 20, 2023
@czunker czunker mentioned this issue Feb 13, 2024
Closed
@czunker
Copy link
Contributor Author

czunker commented May 8, 2024

PSPs were removed in v1.25:
https://kubernetes.io/docs/concepts/security/pod-security-policy/

czunker added a commit that referenced this issue May 10, 2024
@czunker
🧹 Remove k8s PSPs and bump deps to v0.30
6e14993 
Fixed: #2883

Signed-off-by: Christian Zunker <christian@mondoo.com>
@czunker czunker linked a pull request May 10, 2024 that will close this issue
Draft
czunker added a commit that referenced this issue May 10, 2024
@czunker
🧹 Deprecate k8s PSPs and bump deps to latest v0.28 patch version
53822fd 
Preparation for #2883

Signed-off-by: Christian Zunker <christian@mondoo.com>
@czunker czunker mentioned this issue May 10, 2024
Merged
czunker added a commit that referenced this issue May 10, 2024
@czunker
🧹 Deprecate k8s PSPs and bump deps to latest v0.28 patch version
3030f35 
Preparation for #2883

Signed-off-by: Christian Zunker <christian@mondoo.com>
czunker added a commit that referenced this issue May 10, 2024
@czunker
🧹 Deprecate k8s PSPs and bump deps to latest v0.28 patch version (#3959)
e877e0f 
Preparation for #2883

Signed-off-by: Christian Zunker <christian@mondoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Merge after v12 is released] 🧹 Remove k8s PSPs and bump deps to v0.30 mondoohq/cnquery
1 participant
@czunker