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
Splunk Operator: controller doesn't detect if statefulset deleted for standalone type #1284
Comments
in description there was warning: but, it shouldn't be the case, as before deleting statefulset it was working |
can you add CR spec here, we will retry on our end and let you know |
broken example, which creates warning and blocking creating resources:
correct example:
so, all in all, expectation that |
Hi @yaroslav-nakonechnikov , in every custom resource's reconcile logic we do a validation of its spec and error out if any CR spec is erroneous(same for all CRs). For standalone we validate its spec here. Another example of an indexer cluster here. As mentioned here already, Splunk operator records a Additionally we log an error here to indicate an erroneous config. Please let us know if the explanation above is acceptable. |
Hi @yaroslav-nakonechnikov , please let us know if the explanation above is acceptable. |
explanation doesn't fix problem. i expect that controller overrides stateful set (and or recreate it) in case of any changes in statefulset. otherwise, behavior is not consistent and leads to additional issues, which are not expected. |
Hey @yaroslav-nakonechnikov , currently we are looking to set the Phase as Error in the CR status in such scenarios to indicate that the CR spec is invalid i.e
In the future we are looking to add a Message field in the CR status to indicate such errors i.e
Please let us know if that solution works. |
yes, adding nice log messages will help a lot! |
Please select the type of request
Bug
Tell us more
Describe the request
if there are some changes done in stateful set for standalone instance - controlled doesn't recreate it as it is done for other types
Expected behavior
controller creates statefulset from crd
The text was updated successfully, but these errors were encountered: