lburgazzoli commented on PR #4942:
URL: https://github.com/apache/camel-k/pull/4942#issuecomment-1832365761

   > > 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.
   
   mh, I see. 
   we may need to think how to make the process simpler 


-- 
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

Reply via email to