Get namespace from admission request in pod mutator (kubeflow#430)

* Set pod namespace from SERVING_NAMESPACE env

* Get namespace from admission request

pkg/controller/inferenceservice/resources/credentials/service_account_credentials.go

@@ -80,7 +80,8 @@ func (c *CredentialBuilder) CreateSecretVolumeAndEnv(namespace string, serviceAc
 	err := c.client.Get(context.TODO(), types.NamespacedName{Name: serviceAccountName,
 		Namespace: namespace}, serviceAccount)
 	if err != nil {
-		log.Error(err, "Failed to find service account", "ServiceAccountName", serviceAccountName)
+		log.Error(err, "Failed to find service account", "ServiceAccountName", serviceAccountName,
+			"Namespace", namespace)
 		return nil
 	}
 

pkg/webhook/admission/pod/mutator.go

@@ -54,6 +54,9 @@ func (mutator *Mutator) Handle(ctx context.Context, req types.Request) types.Res
 		return admission.ErrorResponse(http.StatusInternalServerError, err)
 	}
 
+	// For some reason pod namespace is always empty when coming to pod mutator, need to set from admission request
+	pod.Namespace = req.AdmissionRequest.Namespace
+
 	if err := mutator.mutate(pod, configMap); err != nil {
 		return admission.ErrorResponse(http.StatusInternalServerError, err)
 	}

pkg/webhook/admission/pod/storage_initializer_injector.go

@@ -147,7 +147,6 @@ func (mi *StorageInitializerInjector) InjectStorageInitializer(pod *v1.Pod) erro
 		ReadOnly:  true,
 	}
 	userContainer.VolumeMounts = append(userContainer.VolumeMounts, sharedVolumeReadMount)
-
 	// Change the CustomSpecStorageUri env variable value to the default model path if present
 	for index, envVar := range userContainer.Env {
 		if envVar.Name == constants.CustomSpecStorageUriEnvVarKey && envVar.Value != "" {