astefanutti commented on a change in pull request #2535:
URL: https://github.com/apache/camel-k/pull/2535#discussion_r678981146
##########
File path: pkg/trait/owner.go
##########
@@ -78,17 +78,20 @@ func (t *ownerTrait) Apply(e *Environment) error {
}
e.Resources.VisitMetaObject(func(res metav1.Object) {
- references := []metav1.OwnerReference{
- {
- APIVersion: e.Integration.APIVersion,
- Kind: e.Integration.Kind,
- Name: e.Integration.Name,
- UID: e.Integration.UID,
- Controller: &controller,
- BlockOwnerDeletion: &blockOwnerDeletion,
- },
+ // Avoid setting owner references across namespaces (resources
are asynchronously refused by the api server)
Review comment:
hum that's a very good question :) I've never noticed it, probably
because of the behaviour you describe. It seems there have been some issue with
invalid references, like kubernetes/kubernetes#65200. I would not expect it to
have something to do with server-side apply, but more an issue with how invalid
references are handled asynchronously.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
