Merge branch 'mmalvezz' into 'master'

fix bug 35287543

See merge request rac-docker-dev/oracle-database-operator!244

Author: mmalvezz

controllers/database/pdb_controller.go

@@ -490,6 +490,7 @@ func (r *PDBReconciler) callAPI(ctx context.Context, req ctrl.Request, pdb *dbap
 		return "", err
 	}
 	webUser := string(secret.Data[cdb.Spec.WebServerUser.Secret.Key])
+        webUser = strings.TrimSpace(webUser)
 
 	// Get Web Server User Password
 	secret = &corev1.Secret{}
@@ -503,6 +504,8 @@ func (r *PDBReconciler) callAPI(ctx context.Context, req ctrl.Request, pdb *dbap
 		return "", err
 	}
 	webUserPwd := string(secret.Data[cdb.Spec.WebServerPwd.Secret.Key])
+        webUserPwd = strings.TrimSpace(webUserPwd)
+
 
 	var httpreq *http.Request
 	if action == "GET" {

oracle-database-operator.yaml

@@ -853,6 +853,131 @@ status:
 ---
 apiVersion: apiextensions.k8s.io/v1
 kind: CustomResourceDefinition
+metadata:
+  annotations:
+    controller-gen.kubebuilder.io/version: v0.6.1
+  creationTimestamp: null
+  name: dataguardbrokers.database.oracle.com
+spec:
+  group: database.oracle.com
+  names:
+    kind: DataguardBroker
+    listKind: DataguardBrokerList
+    plural: dataguardbrokers
+    singular: dataguardbroker
+  scope: Namespaced
+  versions:
+  - additionalPrinterColumns:
+    - jsonPath: .status.primaryDatabase
+      name: Primary
+      type: string
+    - jsonPath: .status.standbyDatabases
+      name: Standbys
+      type: string
+    - jsonPath: .spec.protectionMode
+      name: Protection Mode
+      type: string
+    - jsonPath: .status.clusterConnectString
+      name: Cluster Connect Str
+      priority: 1
+      type: string
+    - jsonPath: .status.externalConnectString
+      name: Connect Str
+      type: string
+    - jsonPath: .spec.primaryDatabaseRef
+      name: Primary Database
+      priority: 1
+      type: string
+    - jsonPath: .status.status
+      name: Status
+      type: string
+    name: v1alpha1
+    schema:
+      openAPIV3Schema:
+        description: DataguardBroker is the Schema for the dataguardbrokers API
+        properties:
+          apiVersion:
+            description: 'APIVersion defines the versioned schema of this representation of an object. Servers should convert recognized schemas to the latest internal value, and may reject unrecognized values. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#resources'
+            type: string
+          kind:
+            description: 'Kind is a string value representing the REST resource this object represents. Servers may infer this from the endpoint the client submits requests to. Cannot be updated. In CamelCase. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#types-kinds'
+            type: string
+          metadata:
+            type: object
+          spec:
+            description: DataguardBrokerSpec defines the desired state of DataguardBroker
+            properties:
+              fastStartFailOver:
+                properties:
+                  enable:
+                    type: boolean
+                  strategy:
+                    items:
+                      description: FSFO strategy
+                      properties:
+                        sourceDatabaseRef:
+                          type: string
+                        targetDatabaseRefs:
+                          type: string
+                      type: object
+                    type: array
+                type: object
+              loadBalancer:
+                type: boolean
+              nodeSelector:
+                additionalProperties:
+                  type: string
+                type: object
+              primaryDatabaseRef:
+                type: string
+              protectionMode:
+                enum:
+                - MaxPerformance
+                - MaxAvailability
+                type: string
+              setAsPrimaryDatabase:
+                type: string
+              standbyDatabaseRefs:
+                items:
+                  type: string
+                type: array
+            required:
+            - primaryDatabaseRef
+            - protectionMode
+            - standbyDatabaseRefs
+            type: object
+          status:
+            description: DataguardBrokerStatus defines the observed state of DataguardBroker
+            properties:
+              clusterConnectString:
+                type: string
+              externalConnectString:
+                type: string
+              primaryDatabase:
+                type: string
+              primaryDatabaseRef:
+                type: string
+              protectionMode:
+                type: string
+              standbyDatabases:
+                type: string
+              status:
+                type: string
+            type: object
+        type: object
+    served: true
+    storage: true
+    subresources:
+      status: {}
+status:
+  acceptedNames:
+    kind: ""
+    plural: ""
+  conditions: []
+  storedVersions: []
+---
+apiVersion: apiextensions.k8s.io/v1
+kind: CustomResourceDefinition
 metadata:
   annotations:
     controller-gen.kubebuilder.io/version: v0.6.1
@@ -1915,7 +2040,7 @@ spec:
                 type: object
               conditions:
                 items:
-                  description: "Condition contains details for one aspect of the current state of this API Resource. --- This struct is intended for direct use as an array at the field path .status.conditions.  For example, type FooStatus struct{     // Represents the observations of a foo's current state.     // Known .status.conditions.type are: \"Available\", \"Progressing\", and \"Degraded\"     // +patchMergeKey=type     // +patchStrategy=merge     // +listType=map     // +listMapKey=type     Conditions []metav1.Condition `json:\"conditions,omitempty\" patchStrategy:\"merge\" patchMergeKey:\"type\" protobuf:\"bytes,1,rep,name=conditions\"` \n     // other fields }"
+                  description: "Condition contains details for one aspect of the current state of this API Resource. --- This struct is intended for direct use as an array at the field path .status.conditions.  For example, \n \ttype FooStatus struct{ \t    // Represents the observations of a foo's current state. \t    // Known .status.conditions.type are: \"Available\", \"Progressing\", and \"Degraded\" \t    // +patchMergeKey=type \t    // +patchStrategy=merge \t    // +listType=map \t    // +listMapKey=type \t    Conditions []metav1.Condition `json:\"conditions,omitempty\" patchStrategy:\"merge\" patchMergeKey:\"type\" protobuf:\"bytes,1,rep,name=conditions\"` \n \t    // other fields \t}"
                   properties:
                     lastTransitionTime:
                       description: lastTransitionTime is the last time the condition transitioned from one status to another. This should be when the underlying condition changed.  If that is not known, then using the time when the API field changed is acceptable.
@@ -2015,26 +2140,29 @@ spec:
     - jsonPath: .status.edition
       name: Edition
       type: string
+    - jsonPath: .status.sid
+      name: Sid
+      priority: 1
+      type: string
     - jsonPath: .status.status
       name: Status
       type: string
     - jsonPath: .status.role
       name: Role
-      priority: 1
       type: string
     - jsonPath: .status.releaseUpdate
       name: Version
       type: string
     - jsonPath: .status.connectString
       name: Connect Str
       type: string
-    - jsonPath: .status.tcpsConnectString
-      name: TCPS Connect Str
-      type: string
     - jsonPath: .status.pdbConnectString
       name: Pdb Connect Str
       priority: 1
       type: string
+    - jsonPath: .status.tcpsConnectString
+      name: TCPS Connect Str
+      type: string
     - jsonPath: .status.tcpsPdbConnectString
       name: TCPS Pdb Connect Str
       priority: 1
@@ -2077,6 +2205,10 @@ spec:
                 type: string
               cloneFrom:
                 type: string
+              createAsStandby:
+                type: boolean
+              dgBrokerConfigured:
+                type: boolean
               edition:
                 enum:
                 - standard
@@ -2142,6 +2274,8 @@ spec:
                   volumeName:
                     type: string
                 type: object
+              primaryDatabaseRef:
+                type: string
               readinessCheckPeriod:
                 type: integer
               replicas:
@@ -2185,7 +2319,7 @@ spec:
                 type: string
               conditions:
                 items:
-                  description: "Condition contains details for one aspect of the current state of this API Resource. --- This struct is intended for direct use as an array at the field path .status.conditions.  For example, type FooStatus struct{     // Represents the observations of a foo's current state.     // Known .status.conditions.type are: \"Available\", \"Progressing\", and \"Degraded\"     // +patchMergeKey=type     // +patchStrategy=merge     // +listType=map     // +listMapKey=type     Conditions []metav1.Condition `json:\"conditions,omitempty\" patchStrategy:\"merge\" patchMergeKey:\"type\" protobuf:\"bytes,1,rep,name=conditions\"` \n     // other fields }"
+                  description: "Condition contains details for one aspect of the current state of this API Resource. --- This struct is intended for direct use as an array at the field path .status.conditions.  For example, \n \ttype FooStatus struct{ \t    // Represents the observations of a foo's current state. \t    // Known .status.conditions.type are: \"Available\", \"Progressing\", and \"Degraded\" \t    // +patchMergeKey=type \t    // +patchStrategy=merge \t    // +listType=map \t    // +listMapKey=type \t    Conditions []metav1.Condition `json:\"conditions,omitempty\" patchStrategy:\"merge\" patchMergeKey:\"type\" protobuf:\"bytes,1,rep,name=conditions\"` \n \t    // other fields \t}"
                   properties:
                     lastTransitionTime:
                       description: lastTransitionTime is the last time the condition transitioned from one status to another. This should be when the underlying condition changed.  If that is not known, then using the time when the API field changed is acceptable.
@@ -2237,6 +2371,8 @@ spec:
               datafilesPatched:
                 default: "false"
                 type: string
+              dgBrokerConfigured:
+                type: boolean
               edition:
                 type: string
               flashBack:
@@ -2293,6 +2429,8 @@ spec:
                 type: object
               prebuiltDB:
                 type: boolean
+              primaryDatabase:
+                type: string
               releaseUpdate:
                 type: string
               replicas:
@@ -2636,6 +2774,32 @@ rules:
   - get
   - patch
   - update
+- apiGroups:
+  - database.oracle.com
+  resources:
+  - dataguardbrokers
+  verbs:
+  - create
+  - delete
+  - get
+  - list
+  - patch
+  - update
+  - watch
+- apiGroups:
+  - database.oracle.com
+  resources:
+  - dataguardbrokers/finalizers
+  verbs:
+  - update
+- apiGroups:
+  - database.oracle.com
+  resources:
+  - dataguardbrokers/status
+  verbs:
+  - get
+  - patch
+  - update
 - apiGroups:
   - database.oracle.com
   resources:
@@ -2965,6 +3129,27 @@ webhooks:
     resources:
     - cdbs
   sideEffects: None
+- admissionReviewVersions:
+  - v1
+  - v1beta1
+  clientConfig:
+    service:
+      name: oracle-database-operator-webhook-service
+      namespace: oracle-database-operator-system
+      path: /mutate-database-oracle-com-v1alpha1-dataguardbroker
+  failurePolicy: Fail
+  name: mdataguardbroker.kb.io
+  rules:
+  - apiGroups:
+    - database.oracle.com
+    apiVersions:
+    - v1alpha1
+    operations:
+    - CREATE
+    - UPDATE
+    resources:
+    - dataguardbrokers
+  sideEffects: None
 - admissionReviewVersions:
   - v1
   - v1beta1
@@ -3137,6 +3322,27 @@ webhooks:
     resources:
     - cdbs
   sideEffects: None
+- admissionReviewVersions:
+  - v1
+  - v1beta1
+  clientConfig:
+    service:
+      name: oracle-database-operator-webhook-service
+      namespace: oracle-database-operator-system
+      path: /validate-database-oracle-com-v1alpha1-dataguardbroker
+  failurePolicy: Fail
+  name: vdataguardbroker.kb.io
+  rules:
+  - apiGroups:
+    - database.oracle.com
+    apiVersions:
+    - v1alpha1
+    operations:
+    - CREATE
+    - UPDATE
+    resources:
+    - dataguardbrokers
+  sideEffects: None
 - admissionReviewVersions:
   - v1
   - v1beta1