Add check for AZURE_FEDERATED_TOKEN_FILE environment variable in Velero deployment test

Signed-off-by: Tiger Kaovilai <[email protected]>

Author: kaovilai

api/v1alpha1/cloudstorage_types.go

@@ -42,7 +42,7 @@ type CloudStorageSpec struct {
 	// region for the bucket to be in, will be us-east-1 if not set.
 	Region string `json:"region,omitempty"`
 	// provider is the provider of the cloud storage
-	// +kubebuilder:validation:Enum=aws
+	// +kubebuilder:validation:Enum=aws;azure;gcp
 	Provider CloudStorageProvider `json:"provider"`
 
 	// https://pkg.go.dev/github.com/Azure/azure-sdk-for-go/sdk/storage/[email protected]#section-readme

bundle/manifests/oadp.openshift.io_cloudstorages.yaml

@@ -75,6 +75,8 @@ spec:
                 description: provider is the provider of the cloud storage
                 enum:
                 - aws
+                - azure
+                - gcp
                 type: string
               region:
                 description: region for the bucket to be in, will be us-east-1 if

config/crd/bases/oadp.openshift.io_cloudstorages.yaml

@@ -75,6 +75,8 @@ spec:
                 description: provider is the provider of the cloud storage
                 enum:
                 - aws
+                - azure
+                - gcp
                 type: string
               region:
                 description: region for the bucket to be in, will be us-east-1 if

internal/controller/velero_test.go

@@ -2579,6 +2579,9 @@ func TestDPAReconciler_buildVeleroDeploymentWithAzureWorkloadIdentity(t *testing
 				if !foundClientIDEnvVar {
 					t.Errorf("Expected AZURE_CLIENT_ID environment variable to be set")
 				}
+				if !foundTokenFileEnvVar {
+					t.Errorf("Expected AZURE_FEDERATED_TOKEN_FILE environment variable to be set")
+				}
 			} else {
 			}
 		})

pkg/credentials/stsflow/stsflow.go

@@ -385,4 +385,3 @@ func GetGCPCredentialsFromSecret(clientset kubernetes.Interface, namespace strin
 
 	return string(serviceAccountJSON), nil
 }
-