lburgazzoli commented on issue #766: Add conditions to camel-k CRs
URL: https://github.com/apache/camel-k/pull/766#issuecomment-508717975
 
 
   > Love it!
   > 
   > Two general comments:
   > 
   > * Currently, conditions are shared among CRs, so it is possible to mix 
them, I wonder whether there should be dedicated conditions types per CR
   
   Yes, that’s what I’m working on. It is a little bit of a pain as we 
basically need to duplicate all the code but that’t the way to *go* :)
   
   > * It seems there is a larger need to report the owned resources and I 
wonder to what extend it generalises the proposed `DeploymentAvailable` and 
`ExposureAvailable` conditions
   
   Yes, the example here is just a placeholder to show how it will look but the 
final goal is to report any owned resource
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to