Re: [I] Polish trait conditions [camel-k]
squakez closed issue #5027: Polish trait conditions URL: https://github.com/apache/camel-k/issues/5027 -- 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
Re: [I] Polish trait conditions [camel-k]
squakez closed issue #5027: Polish trait conditions URL: https://github.com/apache/camel-k/issues/5027 -- 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
Re: [I] Polish trait conditions [camel-k]
squakez commented on issue #5027: URL: https://github.com/apache/camel-k/issues/5027#issuecomment-192355 We may also change the Build CR conditions for containers which reason is not following Kubernetes convention. -- 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
[I] Polish trait conditions [camel-k]
squakez opened a new issue, #5027: URL: https://github.com/apache/camel-k/issues/5027 ### Requirement In version 2.2.0 we're introducing a condition that provide information about trait execution (warnings, error, info, ...). For example: ``` - firstTruthyTime: "2024-01-05T14:59:28Z" lastTransitionTime: "2024-01-05T14:59:28Z" lastUpdateTime: "2024-01-05T14:59:31Z" message: 'explicitly disabled by the platform: container image was not built via Camel K operator' reason: jvmTraitConfiguration status: "True" type: TraitInfo ``` However, it may require to be adjusted according the expected k8s conditions API: https://pkg.go.dev/k8s.io/apimachinery@v0.23.0/pkg/apis/meta/v1#Condition Something that is happening is that we have a single type, whilst we probably need to have a different one for each trait. ### Problem a ### Proposal _No response_ ### Open questions _No response_ -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org