stackabletech / issues

This repository is only for issues that concern multiple repositories or don't fit into any specific repository
2 stars 0 forks source link

Align Stackable conditions with K8s conditions #489

Open sbernauer opened 10 months ago

sbernauer commented 10 months ago

While trying to parse our status.conditions list as Vec<Condition> serde_json complained about the field lastTransitionTime not being present (in case the Trino cluster is stopped). This is because we have our hand-crafted version of a Condition here

Ideally we should use the k8s-openapi Conditions struct internally, similar to

pub struct ClusterCondition {
    #[serde(flatten)]
    pub condition: Condition,

    /// The last time this condition was updated.
    pub last_update_time: Time,
}