search

apache / camel-k

Apache Camel K is a lightweight integration platform, born on Kubernetes, with serverless superpowers
https://camel.apache.org/camel-k
Apache License 2.0
868 stars 348 forks source link

Consider using metav1.Condition instead of own condition structs #5191

Open christophd opened 8 months ago

christophd commented 8 months ago

Requirement

Use the default metav1.Condition as conditions on Camel K custom resources.

Problem

The conditions used in the Camel K custom resources usually are structs that alias to strings.

Instead of this we could try to just reuse metav1.Condition, which would allow to manipulate conditions using some k8s utilities.

The downside is that we won't have the specific condition type, but given 99% of the time is just an alias to strings and there are no real enums in go, I don't see anymore a very strong benefit.

Proposal

No response

Open questions

No response

github-actions[bot] commented 5 months ago

This issue has been automatically marked as stale due to 90 days of inactivity. It will be closed if no further activity occurs within 15 days. If you think that’s incorrect or the issue should never stale, please simply write any comment. Thanks for your contributions!