Skip to content

Conversation

ianbabysre
Copy link
Contributor

@ianbabysre ianbabysre commented Feb 13, 2025

I'm opening a PR to add a feature to the deployment :

This feature adds the following

Possibility to set a tolerations and/or nodeSelector to the Deployment

I tested it, see screenshot
image

Copy link

Click here to review and test in web IDE: Contribute

Copy link

Click here to review and test in web IDE: Contribute

Copy link

Click here to review and test in web IDE: Contribute

Copy link

Click here to review and test in web IDE: Contribute

@tolusha
Copy link
Contributor

tolusha commented Feb 14, 2025

@ianbabysre
Thank you for contribution, could you provide values.yaml you used for testing ?

@ianbabysre
Copy link
Contributor Author

ianbabysre commented Feb 14, 2025

Sure thing ! I tested the templating using by setting theses values in values.yaml
Then used helm template -f values.yaml --debug . for templating

nodeSelector:
  dedicated: compute
tolerations:
  - key: dedicated
  operator: Equal
  value: Compute
  effect: NoSchedule

I left empty values by default

@ianbabysre
Copy link
Contributor Author

@tolusha Let me know if that's okay with you or not !

@tolusha
Copy link
Contributor

tolusha commented Feb 17, 2025

Hi.
Unfortunately I failed to deploy helmcharts, could you double check please?

-> % helm install kubernetes-image-puller deploy/helm -n n9
Error: INSTALLATION FAILED: 1 error occurred:
	* Deployment in version "v1" cannot be handled as a Deployment: json: cannot unmarshal object into Go struct field PodSpec.spec.template.spec.tolerations of type []v1.Toleration

values.yaml

tolerations:
  - key: dedicated
    operator: Equal
    value: Compute
    effect: NoSchedule

Copy link

Click here to review and test in web IDE: Contribute

@ianbabysre
Copy link
Contributor Author

@tolusha Silly mistake on my end sorry, it should work correctly now. - I installed it the same way you did.
Let me know if that work on your end

Copy link
Contributor

@tolusha tolusha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good.

@tolusha tolusha merged commit e82df07 into che-incubator:main Feb 17, 2025
2 checks passed
@devspacesbuild
Copy link

Build 3.20 :: imagepuller_3.x/152: Console, Changes, Git Data

@devspacesbuild
Copy link

@devspacesbuild
Copy link

@devspacesbuild
Copy link

Build 3.20 :: get-sources-rhpkg-container-build_3.x/8916: FAILURE

imagepuller : 3.x :: Failed in 66726190 : BREW:BUILD/STATUS:UNKNOWN
FAILURE:; copied to quay

@ianbabysre
Copy link
Contributor Author

@tolusha Thanks a lot ! Have a great day

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.

3 participants