Bug 1330437

Summary: Requesting dynamically provisioned storageis not disabled when setting dynamicProvisioningEnabled: false
Product: OpenShift Online Reporter: mdong
Component: StorageAssignee: Mark Turansky <mturansk>
Status: CLOSED CURRENTRELEASE QA Contact: Jianwei Hou <jhou>
Severity: high Docs Contact:
Priority: high    
Version: 3.xCC: aos-bugs, mturansk, xtian
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-06-23 17:31:36 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description mdong 2016-04-26 09:00:58 UTC
Description of problem:
In dev-preview-int 3rd iteration, requesting Dynamically Provisioned Storage dose not be disabled when setting dynamicProvisioningEnabled: false 



Package versions:
oc v3.2.0.16
kubernetes v1.2.0-36-g4a3f9c5
docker 1.9.1 (78ee77d)
kernel 3.10.0-327.13.1.el7.x86_64


How reproducible:
Always

Steps to Reproduce:
1. Check master-config.yaml in dev-preview-int
[root@dev-preview-int-master-78c04 ~]# cat /etc/origin/master/master-config.yaml
volumeConfig:
  dynamicProvisioningEnabled: false

2. Create a project in dev-preview-int. 
3. Create pvc to request dynamically provision 

   [dongm@dhcp-136-41 tmp]$ cat 1.yaml
    kind: "PersistentVolumeClaim"
    apiVersion: "v1"
    metadata:
      name: "claim2"
      annotations:
        volume.alpha.kubernetes.io/storage-class: "foo"
    spec:
      accessModes:
        - "ReadWriteOnce"
      resources:
        requests:
          storage: "5Gi"
    [dongm@dhcp-136-41 tmp]$ oc create -f 1.yaml
    persistentvolumeclaim "claim2" created
    [dongm@dhcp-136-41 tmp]$ oc get pvc
    NAME      STATUS    VOLUME         CAPACITY   ACCESSMODES   AGE
    claim2    Bound     pv-aws-w3kpj   5Gi        RWO           4s


Actual results:
Requesting dynamically provisioned storage  is successfully

Expected results:
Requesting dynamically provisioned storage should be disabled as setting in master-config.yaml in dev-preview-int

Additional info:

Comment 1 Mark Turansky 2016-04-26 12:17:17 UTC
I am taking ownership of this BZ. It's a config issue in Online.

Comment 2 Mark Turansky 2016-04-26 13:32:39 UTC
Confirmed the flag is in 3.2.0.16 -- https://github.com/openshift/ose/compare/v3.2.0.16...master

Comment 3 Mark Turansky 2016-04-26 13:40:54 UTC
Actually, I'm not sure now.  I see the commit in the 3.2.016 link above but I don't see it in my local OSE repo:

$ git tag --contains e05671447c08ed6ed20ec59064dea36e94e7f513
v3.2.0.17
v3.2.0.18
v3.2.0.19
v3.2.0.20

Comment 4 Mark Turansky 2016-04-26 13:45:43 UTC
I am moving this to ON_QA because it will be fixed in the next version (3.2.0.17).

Comment 5 mdong 2016-04-29 03:14:38 UTC
Verified on openshift v3.2.0.20

When setting dynamicProvisioningEnabled: false,dynamic pv can't be created.
Will verify withe next dev-preview-int env later.

Comment 6 mdong 2016-05-16 09:45:52 UTC
Verified against dev-preview-stg(openshift v3.2.0.44)