-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
operator: move most of discover pod setting to cm #12691
Conversation
27026ae
to
7385048
Compare
7385048
to
f36d038
Compare
Tested with few settings # added toleration from
023-08-16 08:46:09.325812 I | op-k8sutil: DISCOVER_TOLERATIONS="- effect: NoSchedule\n key: hostname\n operator: Equal\n value: minikube\n" (configmap)
2023-08-16 08:46:09.325921 I | op-discover: tolerations: [{hostname Equal minikube NoSchedule <nil>}] # Also, disabled discover pod from cm
2023-08-16 09:26:43.744031 I | op-k8sutil: ROOK_ENABLE_DISCOVERY_DAEMON="false" (configmap) |
f36d038
to
79f8841
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@travisn do I need to modify settings for helm chart also,? currently, I don't see discover pod settings in helm settings so I have not added yet.
The helm settings for discovery settings are in the helm chart, just in the env vars. We can leave them in the env vars too for now in the helm chart. |
79f8841
to
6748ccf
Compare
It's better to move most of discover daemon setting from env to configmap rook-ceph-operator-config. Although, we are moving to configmap, we keep reading settings from env but the priority will be configmap settings. Signed-off-by: subhamkrai <srai@redhat.com>
6748ccf
to
fb39580
Compare
operator: move most of discover pod setting to cm (backport #12691)
Description of your changes:
operator: move most of discover pod setting to cm
It's better to move most of discover daemon setting
from env to configmap rook-ceph-operator-config.
Which issue is resolved by this Pull Request:
Resolves #12651
Checklist:
skip-ci
on the PR.