PUCM Maintenance Signals

pkg/api/admin/openshiftcluster.go

@@ -54,6 +54,7 @@ type OpenShiftClusterProperties struct {
 	ImageRegistryStorageAccountName string                  `json:"imageRegistryStorageAccountName,omitempty"`
 	InfraID                         string                  `json:"infraId,omitempty"`
 	HiveProfile                     HiveProfile             `json:"hiveProfile,omitempty"`
+	PucmPending                     bool                    `json:"pucmPending,omitempty"`
 }
 
 // ProvisioningState represents a provisioning state.

pkg/api/openshiftcluster.go

@@ -157,6 +157,8 @@ type OpenShiftClusterProperties struct {
 	RegistryProfiles []*RegistryProfile `json:"registryProfiles,omitempty"`
 
 	HiveProfile HiveProfile `json:"hiveProfile,omitempty"`
+
+	PucmPending bool `json:"pucmPending,omitempty"`
 }
 
 // ProvisioningState represents a provisioning state

pkg/backend/openshiftcluster.go

@@ -161,6 +161,7 @@ func (ocb *openShiftClusterBackend) handle(ctx context.Context, log *logrus.Entr
 		if err != nil {
 			return ocb.endLease(ctx, log, stop, doc, api.ProvisioningStateFailed, err)
 		}
+		doc.OpenShiftCluster.Properties.PucmPending = false
 		return ocb.endLease(ctx, log, stop, doc, api.ProvisioningStateSucceeded, nil)
 
 	case api.ProvisioningStateUpdating:
@@ -361,3 +362,6 @@ func (ocb *openShiftClusterBackend) emitMetrics(doc *api.OpenShiftClusterDocumen
 		"newProvisioningState": string(provisioningState),
 	})
 }
+
+func (ocb *openShiftClusterBackend) updatePUCMPendingState(doc *api.OpenShiftClusterDocument) *api.OpenShiftClusterDocument {
+}

pkg/monitor/cluster/cluster.go

@@ -153,6 +153,7 @@ func (mon *Monitor) Monitor(ctx context.Context) (errs []error) {
 		return
 	}
 	for _, f := range []func(context.Context) error{
+		mon.emitPucmState,
 		mon.emitAroOperatorHeartbeat,
 		mon.emitAroOperatorConditions,
 		mon.emitNSGReconciliation,

pkg/monitor/cluster/maintenance.go

@@ -0,0 +1,89 @@
+package cluster
+
+// Copyright (c) Microsoft Corporation.
+// Licensed under the Apache License 2.0.
+
+import (
+	"context"
+
+	"github.com/Azure/ARO-RP/pkg/api"
+)
+
+/**************************************************************
+	Possible PUCM states:
+
+	(1) PUCM pending
+		- We will do PUCM, so emit a maintenance pending signal
+		- Conditions:
+			* Field pucmPending is true
+			* Don't meet below conditions for in progress maintenance
+
+	(2) Planned PUCM in progress
+		- Emit a planned maintenance in progress signal.
+		- If first PUCM attempt fails, leave cluster in this state
+		  because we will need to retry PUCM in at a later time.
+		- Conditions:
+			* Field pucmPending is true
+			* One of: (a) provisoning state AdminUpdate or (2) AdminUpdate err is not nil
+
+	(3) Unplanned PUCM in progress
+		- Emit an unplanned maintenance in progress signal.
+		- If first PUCM attempt fails, leave cluster in this state
+		  because we will need to retry PUCM in at a later time.
+		- Conditions:
+			* Field pucmPending is false
+			* One of: (a) provisoning state AdminUpdate or (2) AdminUpdate err is not nil
+
+	(4) No ongoinig or scheduled PUCM
+		- Don't emit a signal
+		- Conditions:
+			* Field pucmPending is false
+			* Provisioning state is not AdminUpdate and AdminUpdate err is not nil
+**************************************************************/
+
+type pucmState string
+
+func (p pucmState) String() string {
+	return string(p)
+}
+
+const (
+	pucmNone             pucmState = "none"
+	pucmPending          pucmState = "pending"
+	pucmPlannedOngoing   pucmState = "planned_ongoing"
+	pucmUnplannedOngoing pucmState = "unplanned_ongoing"
+)
+
+func (mon *Monitor) emitPucmState(ctx context.Context) error {
+	state := getPucmState(mon.oc.Properties)
+	mon.emitGauge("cluster.maintenance.pucm", 1, map[string]string{
+		"state": state.String(),
+	})
+
+	return nil
+}
+
+func getPucmState(clusterProperties api.OpenShiftClusterProperties) pucmState {
+	var state pucmState
+
+	if pucmOngoing(clusterProperties) {
+		if clusterProperties.PucmPending {
+			state = pucmPlannedOngoing
+		} else {
+			state = pucmUnplannedOngoing
+		}
+	} else {
+		if clusterProperties.PucmPending {
+			state = pucmPending
+		} else {
+			state = pucmNone
+		}
+	}
+
+	return state
+}
+
+func pucmOngoing(clusterProperties api.OpenShiftClusterProperties) bool {
+	return clusterProperties.ProvisioningState == api.ProvisioningStateAdminUpdating ||
+		clusterProperties.LastAdminUpdateError != ""
+}

pkg/monitor/cluster/maintenance_test.go

@@ -0,0 +1,91 @@
+package cluster
+
+// Copyright (c) Microsoft Corporation.
+// Licensed under the Apache License 2.0.
+
+import (
+	"context"
+	"testing"
+
+	"github.com/Azure/ARO-RP/pkg/api"
+	mock_metrics "github.com/Azure/ARO-RP/pkg/util/mocks/metrics"
+	"github.com/golang/mock/gomock"
+)
+
+func TestEmitPucmState(t *testing.T) {
+	ctx := context.Background()
+
+	controller := gomock.NewController(t)
+	defer controller.Finish()
+
+	m := mock_metrics.NewMockEmitter(controller)
+
+	// Unplanned ongoing
+	oc := &api.OpenShiftCluster{
+		Properties: api.OpenShiftClusterProperties{
+			ProvisioningState: api.ProvisioningStateAdminUpdating,
+		},
+	}
+	mon := &Monitor{
+		m:  m,
+		oc: oc,
+	}
+	m.EXPECT().EmitGauge("cluster.maintenance.pucm", int64(1), map[string]string{
+		"state": pucmUnplannedOngoing.String(),
+	})
+
+	err := mon.emitPucmState(ctx)
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	// Planned ongoing
+	oc = &api.OpenShiftCluster{
+		Properties: api.OpenShiftClusterProperties{
+			ProvisioningState: api.ProvisioningStateAdminUpdating,
+			PucmPending:       true,
+		},
+	}
+
+	m.EXPECT().EmitGauge("cluster.maintenance.pucm", int64(1), map[string]string{
+		"state": pucmPlannedOngoing.String(),
+	})
+
+	err = mon.emitPucmState(ctx)
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	// Pending
+	oc = &api.OpenShiftCluster{
+		Properties: api.OpenShiftClusterProperties{
+			ProvisioningState: api.ProvisioningStateSucceeded,
+			PucmPending:       true,
+		},
+	}
+
+	m.EXPECT().EmitGauge("cluster.maintenance.pucm", int64(1), map[string]string{
+		"state": pucmPending.String(),
+	})
+
+	err = mon.emitPucmState(ctx)
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	// None
+	oc = &api.OpenShiftCluster{
+		Properties: api.OpenShiftClusterProperties{
+			ProvisioningState: api.ProvisioningStateSucceeded,
+		},
+	}
+
+	m.EXPECT().EmitGauge("cluster.maintenance.pucm", int64(1), map[string]string{
+		"state": pucmNone.String(),
+	})
+
+	err = mon.emitPucmState(ctx)
+	if err != nil {
+		t.Fatal(err)
+	}
+}