Bug 1337696 - Remove PVC from webui doesn't remove it from DC
Summary: Remove PVC from webui doesn't remove it from DC
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: OKD
Classification: Red Hat
Component: Deployments
Version: 3.x
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
: ---
Assignee: Michail Kargakis
QA Contact: zhou ying
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-05-19 20:35 UTC by Grant Shipley
Modified: 2016-05-27 14:10 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-05-27 14:10:15 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
image (41.17 KB, image/png)
2016-05-19 20:35 UTC, Grant Shipley
no flags Details
image2 (73.86 KB, image/png)
2016-05-19 20:36 UTC, Grant Shipley
no flags Details
image3 (10.34 KB, image/png)
2016-05-19 20:36 UTC, Grant Shipley
no flags Details
image4 (64.71 KB, image/png)
2016-05-19 20:37 UTC, Grant Shipley
no flags Details
image5 (41.17 KB, image/png)
2016-05-19 20:37 UTC, Grant Shipley
no flags Details

Description Grant Shipley 2016-05-19 20:35:55 UTC
Created attachment 1159673 [details]
image

Description of problem:
After adding a volume and making a claim and then removing the claim from the webui the claim is not actually remove from the Deployment config.

See attachments showing no claims but a mount point still.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.  Add volume
2.  Get a claim
3.  Remove storage from webui
4.  See that it was removed (oc get pvc)
5.  See that it wasn't removed from DeploymentConfig

Actual results:


Expected results:


Additional info:

Comment 1 Grant Shipley 2016-05-19 20:36:30 UTC
Created attachment 1159674 [details]
image2

Comment 2 Grant Shipley 2016-05-19 20:36:48 UTC
Created attachment 1159675 [details]
image3

Comment 3 Grant Shipley 2016-05-19 20:37:06 UTC
Created attachment 1159676 [details]
image4

Comment 4 Grant Shipley 2016-05-19 20:37:28 UTC
Created attachment 1159677 [details]
image5

Comment 5 Dan Mace 2016-05-20 18:26:02 UTC
This isn't a bug in Online. Origin doesn't re-write deploymentConfigs in this manner in response to a PVC delete operation. I'm not even sure that it should. This is probably better discussed in an RFE or GitHub issue for Origin.

Comment 6 Dan Mace 2016-05-20 19:15:10 UTC
Reassigning to origin/deployments for now and cc'ing Michalis. I don't know that there's any precedent for any sort of API-level protection against deleting resources  another resource "depends" on. The use case extends to other things, like secrets or image stream tags referenced by a trigger.

Comment 7 Michail Kargakis 2016-05-27 14:10:15 UTC
I am not sure what to do with this. We cannot just rewrite a template because an object it depends on does not exist. As Dan said, this use extends on a bunch of other things. The correct thing to do at the moment is to edit your deployment config and remove the obsolete reference. I am going to close this bug and open a RFE on Github so it can get more eyes.

Comment 8 Michail Kargakis 2016-05-27 14:10:35 UTC
Github issue: https://github.com/openshift/origin/issues/9059


Note You need to log in before you can comment on or make changes to this bug.