You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The text was updated successfully, but these errors were encountered:
csviri
changed the title
KubernetesDependentResource should log warning if owner reference not added but PrimaryToSecondaryMapper is not implementedKubernetesDependentResource should log warning if owner reference not added but SecondaryToPrimaryMapper is not implemented
Apr 19, 2022
Discussed this on weekly, we could also approach this problem in a different way. That Kubernetes dependent resource might support in addition to owner reference handling also adding labels if the owner reference cannot be added (resources are in different namespace).
Basically with these two different approaches we cover every case. So should be supported out of the box. But also there should be a possibility for a custom approach.
csviri
changed the title
KubernetesDependentResource should log warning if owner reference not added but SecondaryToPrimaryMapper is not implemented
Improved out of the box SecondaryToPrimary mapping for KubernetesDependentResource.
Apr 22, 2022
It basically won't work with default mapper.
Throw event an exception?
see: https://github.com/java-operator-sdk/java-operator-sdk/blob/6724b05702b8ade2551aa2a54b2cf8593f8e19fa/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/dependent/kubernetes/KubernetesDependentResource.java#L59-L60
The text was updated successfully, but these errors were encountered: