Open shawkins opened 1 month ago
@shawkins : Is this issue related to Kubernetes Model? Or do we want to accommodate null
value for blockOwnerDeletion
while handing it in KubernetesClient?
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.
Is this issue related to Kubernetes Model? Or do we want to accommodate null value for blockOwnerDeletion while handing it in KubernetesClient?
@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 https://github.com/operator-framework/java-operator-sdk/issues/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