Merge pull request #177 from paramite/no-prom-by-default

Avoid installing Prometheus by default
openstack-k8s-operators · Sep 13, 2023 · b908779 · b908779
2 parents 1c3d089 + 70cb56a
commit b908779
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 7 deletions.
diff --git a/api/bases/telemetry.openstack.org_autoscalings.yaml b/api/bases/telemetry.openstack.org_autoscalings.yaml
@@ -43,7 +43,7 @@ spec:
                 description: Specification of which prometheus to use for autoscaling
                 properties:
                   deployPrometheus:
-                    default: true
+                    default: false
                     description: Enables the deployment of autoscaling prometheus
                     type: boolean
                   host:

diff --git a/api/bases/telemetry.openstack.org_telemetries.yaml b/api/bases/telemetry.openstack.org_telemetries.yaml
@@ -47,7 +47,7 @@ spec:
                     description: Specification of which prometheus to use for autoscaling
                     properties:
                       deployPrometheus:
-                        default: true
+                        default: false
                         description: Enables the deployment of autoscaling prometheus
                         type: boolean
                       host:

diff --git a/api/v1beta1/autoscaling_types.go b/api/v1beta1/autoscaling_types.go
@@ -17,14 +17,14 @@ limitations under the License.
 package v1beta1
 
 import (
-	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	condition "github.com/openstack-k8s-operators/lib-common/modules/common/condition"
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
 
 // Prometheus defines which prometheus to use for Autoscaling
 type Prometheus struct {
 	// Enables the deployment of autoscaling prometheus
-	// +kubebuilder:default=true
+	// +kubebuilder:default=false
 	DeployPrometheus bool `json:"deployPrometheus,omitempty"`
 
 	// Host of user deployed prometheus if deployPrometheus is set to false
@@ -84,7 +84,6 @@ func (instance Autoscaling) IsReady() bool {
 	return instance.Status.Conditions.IsTrue(condition.ReadyCondition)
 }
 
-
 func init() {
 	SchemeBuilder.Register(&Autoscaling{}, &AutoscalingList{})
 }

diff --git a/config/crd/bases/telemetry.openstack.org_autoscalings.yaml b/config/crd/bases/telemetry.openstack.org_autoscalings.yaml
@@ -43,7 +43,7 @@ spec:
                 description: Specification of which prometheus to use for autoscaling
                 properties:
                   deployPrometheus:
-                    default: true
+                    default: false
                     description: Enables the deployment of autoscaling prometheus
                     type: boolean
                   host:

diff --git a/config/crd/bases/telemetry.openstack.org_telemetries.yaml b/config/crd/bases/telemetry.openstack.org_telemetries.yaml
@@ -47,7 +47,7 @@ spec:
                     description: Specification of which prometheus to use for autoscaling
                     properties:
                       deployPrometheus:
-                        default: true
+                        default: false
                         description: Enables the deployment of autoscaling prometheus
                         type: boolean
                       host: