Deprecate non-working KSMStatus

This patch moves status data about kube-state-metrics
to CeilometerStatus, because currently KSMStatus does not update
properly and if helper.PatchInstance is made to update KSMStatus
then Ceilometer objects ends in never ending reconciliation loop.

Author: paramite

api/bases/telemetry.openstack.org_ceilometers.yaml

@@ -44,7 +44,9 @@ spec:
               More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds
             type: string
           ksmStatus:
-            description: KSMStatus defines the observed state of kube-state-metrics
+            description: |-
+              NOTE(mmagr): remove with API version increment
+              Deprecated
             properties:
               conditions:
                 description: Conditions
@@ -136,6 +138,10 @@ spec:
                 type: object
               ipmiImage:
                 type: string
+              ksmEnabled:
+                default: true
+                description: Whether kube-state-metrics should be deployed
+                type: boolean
               ksmImage:
                 type: string
               ksmTls:
@@ -295,6 +301,15 @@ spec:
                   type: string
                 description: Map of hashes to track e.g. job status
                 type: object
+              ksmHash:
+                additionalProperties:
+                  type: string
+                description: Map of hashes to track e.g. job status
+                type: object
+              ksmReadyCount:
+                description: ReadyCount of kube-state-metrics instances
+                format: int32
+                type: integer
               mysqldExporterExportedGaleras:
                 description: List of galera CRs, which are being exported with mysqld_exporter
                 items:

api/bases/telemetry.openstack.org_telemetries.yaml

@@ -433,6 +433,10 @@ spec:
                     type: boolean
                   ipmiImage:
                     type: string
+                  ksmEnabled:
+                    default: true
+                    description: Whether kube-state-metrics should be deployed
+                    type: boolean
                   ksmImage:
                     type: string
                   ksmTls:

api/v1beta1/ceilometer_types.go

@@ -113,6 +113,11 @@ type CeilometerSpecCore struct {
 	// NetworkAttachmentDefinitions list of network attachment definitions the service pod gets attached to
 	NetworkAttachmentDefinitions []string `json:"networkAttachmentDefinitions,omitempty"`
 
+	// Whether kube-state-metrics should be deployed
+	// +kubebuilder:validation:optional
+	// +kubebuilder:default=true
+	KSMEnabled *bool `json:"ksmEnabled,omitempty"`
+
 	// Whether mysqld_exporter should be deployed
 	// +kubebuilder:validation:optional
 	MysqldExporterEnabled *bool `json:"mysqldExporterEnabled,omitempty"`
@@ -177,9 +182,16 @@ type CeilometerStatus struct {
 	// List of galera CRs, which are being exported with mysqld_exporter
 	// +listType=atomic
 	MysqldExporterExportedGaleras []string `json:"mysqldExporterExportedGaleras,omitempty"`
+
+	// ReadyCount of kube-state-metrics instances
+	KSMReadyCount int32 `json:"ksmReadyCount,omitempty"`
+
+	// Map of hashes to track e.g. job status
+	KSMHash map[string]string `json:"ksmHash,omitempty"`
 }
 
-// KSMStatus defines the observed state of kube-state-metrics
+// NOTE(mmagr): remove with API version increment
+// Deprecated
 type KSMStatus struct {
 	// ReadyCount of ksm instances
 	ReadyCount int32 `json:"readyCount,omitempty"`
@@ -223,8 +235,7 @@ type CeilometerList struct {
 
 // IsReady - returns true if Ceilometer is reconciled successfully
 func (instance Ceilometer) IsReady() bool {
-	return instance.CeilometerStatus.Conditions.IsTrue(condition.ReadyCondition) &&
-		instance.KSMStatus.Conditions.IsTrue(condition.ReadyCondition)
+	return instance.CeilometerStatus.Conditions.IsTrue(condition.ReadyCondition)
 }
 
 func init() {

api/v1beta1/conditions.go

@@ -56,8 +56,17 @@ const (
 
 	DashboardDefinitionReadyCondition condition.Type = "DashboardDefinitionReady"
 
-	// KSMReadyCondition Status=True condition which indicates if the KSM is configured and operational
-	KSMReadyCondition condition.Type = "KSMReady"
+	// KSMTLSInputReadyCondition Status=True condition when required TLS sources are ready for KSM
+	KSMTLSInputReadyCondition condition.Type = "KSMTLSInputReady"
+
+	// KSMDeploymentReadyCondition Status=True condition when KSM statefulset created ok
+	KSMDeploymentReadyCondition condition.Type = "KSMDeploymentReady"
+
+	// KSMServiceConfigReadyCondition Status=True Condition which indicates that all service config got rendered ok
+	KSMServiceConfigReadyCondition condition.Type = "KSMServiceConfigReady"
+
+	// KSMCreateServiceReadyCondition Status=True condition when k8s service for the KSM created ok
+	KSMCreateServiceReadyCondition condition.Type = "KSMCreateServiceReady"
 
 	// MysqldExporter conditions
 	MysqldExporterDBReadyCondition condition.Type = "MysqldExporterDBReady"
@@ -207,17 +216,8 @@ const (
 	//
 	// KSMReady condition messages
 	//
-	// KSMReadyInitMessage
-	KSMReadyInitMessage = "KSM not started"
-
-	// KSMReadyMessage
-	KSMReadyMessage = "KSM completed"
-
-	// KSMReadyErrorMessage
-	KSMReadyErrorMessage = "KSM error occured %s"
-
-	// KSMReadyRunningMessage
-	KSMReadyRunningMessage = "KSM in progress"
+	// KSMDisabledMessage
+	KSMDisabledMessage = "kube-state-metrics is disabled"
 
 	//
 	// mysqld_exporter condition messages

api/v1beta1/zz_generated.deepcopy.go

@@ -44,7 +44,9 @@ spec:
               More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds
             type: string
           ksmStatus:
-            description: KSMStatus defines the observed state of kube-state-metrics
+            description: |-
+              NOTE(mmagr): remove with API version increment
+              Deprecated
             properties:
               conditions:
                 description: Conditions
@@ -136,6 +138,10 @@ spec:
                 type: object
               ipmiImage:
                 type: string
+              ksmEnabled:
+                default: true
+                description: Whether kube-state-metrics should be deployed
+                type: boolean
               ksmImage:
                 type: string
               ksmTls:
@@ -295,6 +301,15 @@ spec:
                   type: string
                 description: Map of hashes to track e.g. job status
                 type: object
+              ksmHash:
+                additionalProperties:
+                  type: string
+                description: Map of hashes to track e.g. job status
+                type: object
+              ksmReadyCount:
+                description: ReadyCount of kube-state-metrics instances
+                format: int32
+                type: integer
               mysqldExporterExportedGaleras:
                 description: List of galera CRs, which are being exported with mysqld_exporter
                 items:

config/crd/bases/telemetry.openstack.org_ceilometers.yaml

@@ -433,6 +433,10 @@ spec:
                     type: boolean
                   ipmiImage:
                     type: string
+                  ksmEnabled:
+                    default: true
+                    description: Whether kube-state-metrics should be deployed
+                    type: boolean
                   ksmImage:
                     type: string
                   ksmTls: