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
Default owner reference blockOwnerDeletion to true #5838
Comments
@shawkins : Is this issue related to Kubernetes Model? Or do we want to accommodate |
If some will need this to be configurable, we can just override it on our side. But having this defaulted to true is probably the expected way to go IMO. |
@rohanKanojia This is referring to the HasMetadata.addOwnerReference helper method, it's defaulting to null but true would be a better choice. |
Would this be a breaking change? i.e. can we add this in 6.13, seems like low hanging fruit. |
Strictly speaking yes, but practially speaking no - it's what users should want by default. |
Is your enhancement related to a problem? Please describe
Relates to operator-framework/java-operator-sdk#2318 - the fabric8 logic is currently defaulting to null. the operator sdk is ok if the default behavior changes in fabric8. They don't expect that behavior will need to be configurable.
Describe the solution you'd like
For the fabric8 ownerReference creation method to default blockOwnerDeletion to true - that matches the default expected behavior of kubernetes.
Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: