feat: opsdefinition env support optional setting (#9438)

Author: xuriwuyun

apis/operations/v1alpha1/opsdefinition_types.go

@@ -134,6 +134,11 @@ type OpsEnvVar struct {
 	//
 	// +kubebuilder:validation:Required
 	ValueFrom *OpsVarSource `json:"valueFrom"`
+
+	// Specify whether the ENV must be defined.
+	//
+	// +optional
+	Optional *bool `json:"optional,omitempty"`
 }
 
 // +kubebuilder:validation:XValidation:rule="has(self.envRef) || has(self.fieldPath)", message="either fieldPath and envRef."

apis/operations/v1alpha1/zz_generated.deepcopy.go

@@ -7973,6 +7973,9 @@ spec:
                               Specifies the name of the environment variable to be injected into Pods executing OpsActions.
                               It must conform to the C_IDENTIFIER format, which includes only alphanumeric characters and underscores, and cannot begin with a digit.
                             type: string
+                          optional:
+                            description: Specify whether the ENV must be defined.
+                            type: boolean
                           valueFrom:
                             description: Specifies the source of the environment variable's
                               value.

config/crd/bases/operations.kubeblocks.io_opsdefinitions.yaml

@@ -7973,6 +7973,9 @@ spec:
                               Specifies the name of the environment variable to be injected into Pods executing OpsActions.
                               It must conform to the C_IDENTIFIER format, which includes only alphanumeric characters and underscores, and cannot begin with a digit.
                             type: string
+                          optional:
+                            description: Specify whether the ENV must be defined.
+                            type: boolean
                           valueFrom:
                             description: Specifies the source of the environment variable's
                               value.

deploy/helm/crds/operations.kubeblocks.io_opsdefinitions.yaml

@@ -1881,6 +1881,18 @@ OpsVarSource
 <p>Specifies the source of the environment variable&rsquo;s value.</p>
 </td>
 </tr>
+<tr>
+<td>
+<code>optional</code><br/>
+<em>
+bool
+</em>
+</td>
+<td>
+<em>(Optional)</em>
+<p>Specify whether the ENV must be defined.</p>
+</td>
+</tr>
 </tbody>
 </table>
 <h3 id="operations.kubeblocks.io/v1alpha1.OpsExecAction">OpsExecAction

docs/developer_docs/api-reference/operations.md

@@ -176,6 +176,9 @@ func buildEnvVars(reqCtx intctrlutil.RequestCtx,
 			envVar, err = buildVarWithFieldPath(targetPod, vars[i].ValueFrom.FieldRef.FieldPath)
 		}
 		if envVar == nil {
+			if vars[i].Optional != nil && *vars[i].Optional {
+				continue
+			}
 			return nil, intctrlutil.NewFatalError(fmt.Sprintf(`can not find the env "%s" in the container "%s"`, envVarRef.EnvName, envVarRef.TargetContainerName))
 		}
 		envVar.Name = vars[i].Name