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

Applying policy to existing helm releases #54

Closed
falken01 opened this issue Jul 8, 2024 · 2 comments
Closed

Applying policy to existing helm releases #54

falken01 opened this issue Jul 8, 2024 · 2 comments

Comments

@falken01
Copy link

falken01 commented Jul 8, 2024


Issues on GitHub are intended to be related to bugs or feature requests with provider codebase,
so we recommend using our other community resources instead of asking here 👍.


Hello,

My question is if that feature works only for newly created pods? Unfortunately for me after creating a new release with helm the pod runs into error with following information in logs:

"error":"pod resources reversion failed: Pod \"[pod's name]" is invalid: spec: Forbidden: pod updates may not change fields other than spec.containers[].image,spec.initContainers[].image,spec.activeDeadlineSeconds,spec.tolerations (only additions to existing tolerations) (...)

To reproduce:

  1. Installation: kubectl apply -f https://github.com/google/kube-startup-cpu-boost/releases/download/v0.9.0/manifests.yaml
  2. Creating object within my namespace with proper values
  3. Recreating pods with command: helm upgrade --namespace xxx --install --values (...)

I ask that question bcs every example I found uses kubectl commands where pods are entirely deleted. But I havent found example with helm

@mikouaj
Copy link
Member

mikouaj commented Jul 11, 2024

Hi @falken01, thank you for rising this.
The error message says that the controller is not able to revert the POD CPU resources to the original values. The reason of that is most likely that your cluster has no InPlacePodVerticalScaling feature gate enabled. This feature makes Pod resource requirements mutable and is a pre-requisite for Kube Startup CPU Boost. Can you please verify this?

@falken01
Copy link
Author

You;re right i havent read docs carefully enought. Thanks

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

No branches or pull requests

2 participants