feat: Add validation for mounted pvc

Author: wg102

frontend/src/app/resource-form/volume/volume.component.ts

@@ -1,8 +1,10 @@
 import { Component, OnInit, Input, OnDestroy } from "@angular/core";
-import { FormGroup, Validators } from "@angular/forms";
+import { FormGroup, Validators, ValidatorFn, AbstractControl } from "@angular/forms";
 import { Volume } from "src/app/utils/types";
 import { Subscription } from "rxjs";
-import {TranslateService} from "@ngx-translate/core";
+import { TranslateService } from "@ngx-translate/core";
+import { NamespaceService } from "src/app/services/namespace.service";
+import { KubernetesService } from "src/app/services/kubernetes.service";
 
 @Component({
   selector: "app-volume",
@@ -12,6 +14,7 @@ import {TranslateService} from "@ngx-translate/core";
 export class VolumeComponent implements OnInit, OnDestroy {
   private _notebookName = "";
   private _defaultStorageClass: boolean;
+  private mountedVolumes: Set<string> = new Set<string>();
 
   currentPVC: Volume;
   existingPVCs: Set<string> = new Set();
@@ -101,14 +104,21 @@ export class VolumeComponent implements OnInit, OnDestroy {
   }
 
   // ----- Component Functions -----
-  constructor(private translate: TranslateService) {}
+  constructor(
+    private translate: TranslateService,
+    private k8s: KubernetesService,
+    private ns: NamespaceService) { }
 
   ngOnInit() {
     this.volume
       .get("name")
-      .setValidators([Validators.required, Validators.pattern(/^[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*$/)]);
-    
-   this.subscriptions.add(
+      .setValidators([
+        Validators.required,
+        this.isMountedValidator(),
+        Validators.pattern(/^[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*$/)
+      ]);
+
+    this.subscriptions.add(
       this.volume.get("type").valueChanges.subscribe((type: string) => {
         this.setVolumeType(type);
       })
@@ -122,6 +132,18 @@ export class VolumeComponent implements OnInit, OnDestroy {
         this.updateVolInputFields();
       })
     );
+
+    // Get the list of mounted volumes of the existing Notebooks in the selected Namespace
+    this.subscriptions.add(
+      this.ns.getSelectedNamespace().subscribe(ns => {
+        this.k8s.getResource(ns).subscribe(notebooks => {
+          this.mountedVolumes.clear();
+          notebooks.map(nb => nb.volumes.map(v => {
+            this.mountedVolumes.add(v)
+          }));
+        });
+      })
+    );
   }
 
   ngOnDestroy() {
@@ -166,12 +188,23 @@ export class VolumeComponent implements OnInit, OnDestroy {
 
   showNameError() {
     const volumeName = this.volume.get("name");
-    
+
     if (volumeName.hasError("required")) {
       return this.translate.instant("volume.errorNameRequired");
     }
     if (volumeName.hasError("pattern")) {
       return this.translate.instant("volume.errorNamePattern");
     }
+    if (volumeName.hasError("isMounted")) {
+      return this.translate.instant("volume.errorMountedVolume");
+    }
+  }
+
+  //Method that disables selecting a mounted pvc
+  private isMountedValidator(): ValidatorFn {
+    return (control: AbstractControl): { [key: string]: any } => {
+      const exists = this.mountedVolumes.has(control.value);
+      return exists ? { isMounted: true } : null;
+    };
   }
 }

frontend/src/assets/i18n/en.json

@@ -126,6 +126,7 @@
     "lblName": "Name",
     "errorNameRequired": "The volume name can't be empty",
     "errorNamePattern": "The volume name can only contain lowercase alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character",
+    "errorMountedVolume":"The volume is already mounted to another notebook and cannot be currently selected",
     "lblSize": "Size",
     "lblMode": "Mode",
     "optReadWriteOnce": "ReadWriteOnce",

frontend/src/assets/i18n/fr.json

@@ -127,6 +127,7 @@
     "lblName": "Nom",
     "errorNameRequired": "Le nom du volume ne peut pas être vide",
     "errorNamePattern": "Le nom du volume ne peut contenir que des caractères alphanumériques en minuscule, '-' ou '.', et doit commencer et se terminer par un caractères alphanumériques",
+    "errorMountedVolume":"Le volume est déjà monté sur un autre bloc note et ne peut être sélectionné actuellement",
     "lblSize": "Taille",
     "lblMode": "Mode",
     "optReadWriteOnce": "ReadWriteOnce",