search

radanalyticsio / spark-operator

Operator for managing the Spark clusters on Kubernetes and OpenShift.
Apache License 2.0
157 stars 61 forks source link

Add status subblock updatest to supported CRs #275

Closed tmckayus closed 4 years ago

tmckayus commented 4 years ago

Description

This adds basic status updates to the CRs managed by the spark operator. Status includes a state string and a lastTransitionTime field. This PR is dependent on a change to the AbstractOperator in https://github.com/jvm-operators/abstract-operator/pull/62

As an initial pass, the status block available is fixed (string, date) and the changes should not break any existing subclasses of AbstractOperator class. In the future we can figure out how to add more general status.

Related Issue

The following issue is partly related. This is a first step toward general status, but it is mostly to satisfy failing CI in the community-operators repo because of the lack of a status block.

https://github.com/radanalyticsio/spark-operator/issues/216

Types of changes

:sparkles: New feature (non-breaking change which adds functionality)

tmckayus commented 4 years ago

@jkremser @elmiko ptal

Note this can't be mereged into the corresponding change in AbstactOperators is merged and released.

elmiko commented 4 years ago

this seems like a fine starting point for the status work, +1