-
Notifications
You must be signed in to change notification settings - Fork 47
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
Node Tolerations and Node Selector on componets #52
Comments
The configuration looks good it is a possible bug |
we are running operator version 0.1.0. Here is our objectstore manifest
|
+1 Kinda critical thing , will be very appreciate for fix |
@mitchellmaler i looked into the code and find out the problem. With config below work just fine for all resources
Pls check from ur side and we can close the issue :) |
+1 This is also an issue for me. I have tried what @Ultrafenrir has posted and it did not work. Operator version 0.1.3 I saw that this had changed to
|
@tarokkk Any thoughts on this? Unfortunately, this is blocking us and I might have to build this out a different way |
We will release this week a completely rewritten override functionality that should solve this issue as well. |
I'm doing a test for this scenario with the latest overrides mechanisms introduced in master. |
@kody-abe master works fine for me with the above config. Could you please give it a try in a dev/staging environment with the current master image? |
@pepov Can confirm, master works as expected with the above config! Look forward to the release. |
@pepov Spoke too soon. This did not work for rule or storeGateway in the Thanos object. But, did work for query.
|
For this to work you have to use the latest CRDs from master. Also note, that rule does not have a |
Yes! That was the ticket. This now works as expected for all types. Thanks @pepov! Looking forward to the release. |
Is your feature request related to a problem? Please describe.
Currently we are trying to deploy the compactor to a specific set of nodes with extra disk space. We tried to add the workloadOverrides with nodeSelector and tolerations but that does not seem to be picked up. It's possible we are just using the overrides incorrectly too since the documentation is limited on how to use them.
Describe the solution you'd like to see
Either have specific fields to add these to the components (compactor and others) or allow the overrides to pull them in.
Describe alternatives you've considered
Patching the deployment after the operator creates it.
The text was updated successfully, but these errors were encountered: