Change PVC access mode to RWM (#496)

kubevirt · Jun 18, 2019 · d445c1c · d445c1c
1 parent ae8c9e8
commit d445c1c
Show file tree

Hide file tree

Showing 14 changed files with 23 additions and 21 deletions.
diff --git a/src/components/VmConsoles/tests/__snapshots__/DesktopViewerSelector.test.js.snap b/src/components/VmConsoles/tests/__snapshots__/DesktopViewerSelector.test.js.snap
@@ -75,7 +75,7 @@ exports[`<DesktopViewerSelector /> renders correctly 1`] = `
               "spec": Object {
                 "pvc": Object {
                   "accessModes": Array [
-                    "ReadWriteOnce",
+                    "ReadWriteMany",
                   ],
                   "resources": Object {
                     "requests": Object {

diff --git a/src/constants/index.js b/src/constants/index.js
@@ -36,6 +36,7 @@ export const PROVISION_SOURCE_IMPORT = 'Import';
 export const PROVISION_SOURCE_CLONED_DISK = 'Cloned Disk'; // PVC or upload image to PVC
 
 export const PVC_ACCESSMODE_RWO = 'ReadWriteOnce';
+export const PVC_ACCESSMODE_RWM = 'ReadWriteMany';
 
 export const TEMPLATE_API_VERSION = 'template.openshift.io/v1';
 export const TEMPLATE_FLAVOR_LABEL = 'flavor.template.kubevirt.io';

diff --git a/src/k8s/objects/pvc/pvc.js b/src/k8s/objects/pvc/pvc.js
@@ -1,4 +1,5 @@
 import { PersistentVolumeClaimModel } from '../../../models/index';
+import { PVC_ACCESSMODE_RWM } from '../../../constants';
 
 export const buildPvc = ({ generateName, namespace, size, unit, storageClass }) => ({
   apiVersion: PersistentVolumeClaimModel.apiVersion,
@@ -8,7 +9,7 @@ export const buildPvc = ({ generateName, namespace, size, unit, storageClass })
     generateName: `${generateName}-`,
   },
   spec: {
-    accessModes: ['ReadWriteOnce'],
+    accessModes: [PVC_ACCESSMODE_RWM],
     volumeMode: 'Filesystem',
     resources: {
       requests: {

diff --git a/src/k8s/vmBuilder.js b/src/k8s/vmBuilder.js
@@ -3,10 +3,10 @@ import { get, last } from 'lodash';
 import {
   PROVISION_SOURCE_PXE,
   BOOT_ORDER_FIRST,
-  PVC_ACCESSMODE_RWO,
   BOOT_ORDER_SECOND,
   DEVICE_TYPE_DISK,
   DEVICE_TYPE_INTERFACE,
+  PVC_ACCESSMODE_RWM,
 } from '../constants';
 
 import {
@@ -320,7 +320,7 @@ export const getDataVolumeTemplateSpec = (storage, dvSource) => {
     },
     spec: {
       pvc: {
-        accessModes: [PVC_ACCESSMODE_RWO],
+        accessModes: [PVC_ACCESSMODE_RWM],
         resources: {
           requests: {
             storage: `${storage.size}Gi`,

diff --git a/src/tests/mocks/dataVolume/index.js b/src/tests/mocks/dataVolume/index.js
@@ -15,7 +15,7 @@ export const dataVolumes = {
         },
       },
       pvc: {
-        accessModes: ['ReadWriteOnce'],
+        accessModes: ['ReadWriteMany'],
         resources: {
           requests: {
             storage: '1G',
@@ -38,7 +38,7 @@ export const dataVolumes = {
         },
       },
       pvc: {
-        accessModes: ['ReadWriteOnce'],
+        accessModes: ['ReadWriteMany'],
         resources: {
           requests: {
             storage: '1G',
@@ -62,7 +62,7 @@ export const dataVolumes = {
         blank: {},
       },
       pvc: {
-        accessModes: ['ReadWriteOnce'],
+        accessModes: ['ReadWriteMany'],
         resources: {
           requests: {
             storage: '1G',
@@ -86,7 +86,7 @@ export const dataVolumes = {
         },
       },
       pvc: {
-        accessModes: ['ReadWriteOnce'],
+        accessModes: ['ReadWriteMany'],
         resources: {
           requests: {
             storage: '1G',
@@ -110,7 +110,7 @@ export const dataVolumes = {
         },
       },
       pvc: {
-        accessModes: ['ReadWriteOnce'],
+        accessModes: ['ReadWriteMany'],
         resources: {
           requests: {
             storage: '1G',

diff --git a/src/tests/mocks/user_template/pvc.mock.js b/src/tests/mocks/user_template/pvc.mock.js
@@ -44,7 +44,7 @@ export const pvcTemplate = {
             },
             spec: {
               pvc: {
-                accessModes: ['ReadWriteOnce'],
+                accessModes: ['ReadWriteMany'],
                 resources: {
                   requests: {
                     storage: '15Gi',

diff --git a/src/tests/mocks/user_template/pxe-dv.mock.js b/src/tests/mocks/user_template/pxe-dv.mock.js
@@ -44,7 +44,7 @@ export const pxeDataVolumeTemplate = {
             },
             spec: {
               pvc: {
-                accessModes: ['ReadWriteOnce'],
+                accessModes: ['ReadWriteMany'],
                 resources: {
                   requests: {
                     storage: '15Gi',

diff --git a/src/tests/mocks/user_template/url-flavor.mock.js b/src/tests/mocks/user_template/url-flavor.mock.js
@@ -36,7 +36,7 @@ export const urlCustomFlavorTemplate = {
             },
             spec: {
               pvc: {
-                accessModes: ['ReadWriteOnce'],
+                accessModes: ['ReadWriteMany'],
                 resources: {
                   requests: {
                     storage: '10Gi',

diff --git a/src/tests/mocks/user_template/url-nonetwork.mock.js b/src/tests/mocks/user_template/url-nonetwork.mock.js
@@ -36,7 +36,7 @@ export const urlNoNetworkTemplate = {
             },
             spec: {
               pvc: {
-                accessModes: ['ReadWriteOnce'],
+                accessModes: ['ReadWriteMany'],
                 resources: {
                   requests: {
                     storage: '10Gi',

diff --git a/src/tests/mocks/user_template/url.mock.js b/src/tests/mocks/user_template/url.mock.js
@@ -98,7 +98,7 @@ export const urlTemplateDataVolume = {
   },
   spec: {
     pvc: {
-      accessModes: ['ReadWriteOnce'],
+      accessModes: ['ReadWriteMany'],
       resources: {
         requests: {
           storage: '5Gi',

diff --git a/src/tests/mocks/vm/vm.mock.js b/src/tests/mocks/vm/vm.mock.js
@@ -47,7 +47,7 @@ export const fullVm = {
             },
           },
           pvc: {
-            accessModes: ['ReadWriteOnce'],
+            accessModes: ['ReadWriteMany'],
             resources: {
               requests: {
                 storage: '1G',

diff --git a/src/tests/mocks/vm/vm_validation.mock.js b/src/tests/mocks/vm/vm_validation.mock.js
@@ -29,7 +29,7 @@ export const vm1 = {
             },
           },
           pvc: {
-            accessModes: ['ReadWriteOnce'],
+            accessModes: ['ReadWriteMany'],
             resources: {
               requests: {
                 storage: '1G',
@@ -114,7 +114,7 @@ export const vm2 = {
             },
           },
           pvc: {
-            accessModes: ['ReadWriteOnce'],
+            accessModes: ['ReadWriteMany'],
             resources: {
               requests: {
                 storage: '1G',

diff --git a/src/utils/patches.js b/src/utils/patches.js
@@ -6,10 +6,10 @@ import {
   ANNOTATION_FIRST_BOOT,
   BOOT_ORDER_FIRST,
   BOOT_ORDER_SECOND,
-  PVC_ACCESSMODE_RWO,
   TEMPLATE_FLAVOR_LABEL,
   deviceTypeToPathKey,
   DEVICE_TYPE_INTERFACE,
+  PVC_ACCESSMODE_RWM,
 } from '../constants';
 import { NETWORK_TYPE_POD } from '../components/Wizard/CreateVmWizard/constants';
 import { assignBootOrderIndex, getBootableDevicesInOrder, getDevices, addBindingToInterface } from '../k8s/vmBuilder';
@@ -69,7 +69,7 @@ export const getAddDiskPatch = (vm, storage) => {
     },
     spec: {
       pvc: {
-        accessModes: [PVC_ACCESSMODE_RWO],
+        accessModes: [PVC_ACCESSMODE_RWM],
         resources: {
           requests: {
             storage: `${storage.size}Gi`,

diff --git a/src/utils/tests/patches.test.js b/src/utils/tests/patches.test.js
@@ -4,7 +4,7 @@ import {
   ANNOTATION_FIRST_BOOT,
   BOOT_ORDER_SECOND,
   BOOT_ORDER_FIRST,
-  PVC_ACCESSMODE_RWO,
+  PVC_ACCESSMODE_RWM,
   TEMPLATE_FLAVOR_LABEL,
   POD_NETWORK,
   DISK_PATH_KEY,
@@ -86,7 +86,7 @@ const dataVolumeTemplate = {
   },
   spec: {
     pvc: {
-      accessModes: [PVC_ACCESSMODE_RWO],
+      accessModes: [PVC_ACCESSMODE_RWM],
       resources: {
         requests: {
           storage: `${storageNoClass.size}Gi`,