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.

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

Replace NetworkExplorer kernel-collector-podsecuritypolicy since PodSecurityPolicy was removed in K8s v1.25 #896

Closed
jvoravong opened this issue Aug 24, 2023 · 5 comments
Labels
enhancement New feature or request

Comments

@jvoravong
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Description

Steps to Reproduce

Try deploying the network explorer to a Kubernetes +1.25 cluster.

Expected Result

Should be able to deploy the Network Explorer to current Kubernetes versions.

Actual Result

Can only deploy to older versions of Kubernetes.

Tested with the chart v0.83.0

Describe the solution you'd like

Refactor the kernel-collector K8s resources to use proper K8s objects instead of a PodSecurityPolicy.

Describe alternatives you've considered

No response

Additional context

No response

@jvoravong jvoravong added the enhancement New feature or request label Aug 24, 2023
@kumachop2
Copy link

I have tested with the chart v0.85.0. Please do advice on timelines.

@kumachop2
Copy link

kumachop2 commented Oct 20, 2023

@jvoravong I'm still getting error with Pod Sec Policy and here is error.

K8s Version: v1.25.6
Helm Chart: 0.86.0

2023-10-20T22:56:52.7227095Z Error: INSTALLATION FAILED: unable to build kubernetes objects from release manifest: resource mapping not found for name: "-splunk-otel-collector-kernel-collector" namespace: "" from "": no matches for kind "PodSecurityPolicy" in version "policy/v1beta1"
2023-10-20T22:56:52.7230520Z ensure CRDs are installed first
2023-10-20T22:56:52.7233232Z helm.go:84: [debug] resource mapping not found for name: "
-splunk-otel-collector-kernel-collector" namespace: "" from "": no matches for kind "PodSecurityPolicy" in version "policy/v1beta1"
2023-10-20T22:56:52.7235682Z ensure CRDs are installed first

@kumachop2
Copy link

@jvoravong Please do respond with your findings. Thanks in advance.

@jinja2
Copy link
Collaborator

jinja2 commented Oct 25, 2023

@kumachop2 the error is due to how helm works. The psp object is in the stored helm release state and it can't determine the current state of psp from the last deployed manifest when it attempts to create a diff for upgrade. Refer to this helm document on how you can fix the helm state and successfully upgrade.

@kumachop2
Copy link

@jinja2 Thank You for your response and inputs. I'm running the helm chart against local repo rather internet due to external access issues. Some reason local repo had PodSecurityPolicy template and deleted it to resolve.

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

No branches or pull requests

3 participants