squakez commented on PR #4942: URL: https://github.com/apache/camel-k/pull/4942#issuecomment-1832351973
> quick comment before looking at the code: we can maybe simplify it by using a static label `camel.apache.org/supervise=true` and take the name of the deployment/etc/ Unfortunately not. Or at least, it would not be that easy. The way we are watching objects is quite limited to the presence of the `camel.apache.org/integration` label. If we remove such a requirement, we'd need to watch all objects. You will see later that we already had to remove the filter from Pods, in order to be able to watch also the non managed ones. I think that if we remove this constraint we may degrade the performance too much. -- 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: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
