Bug 794799

Summary:	Deleting env from middle of path causes inconsistent state
Product:	Red Hat Satellite	Reporter:	Jeff Weiss <jweiss>
Component:	Content Management	Assignee:	Katello Bug Bin <katello-bugs>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Katello QA List <katello-qa-list>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	6.0.0	CC:	bkearney, dajohnso, gstoecke, inecas, jsherril, mmccune, tstrachota
Target Milestone:	Unspecified	Keywords:	TestBlocker, Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-08-22 18:27:50 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Jeff Weiss 2012-02-17 17:08:49 UTC

Description of problem:


Version-Release number of selected component (if applicable):
katello-0.1.238-4.el6.noarch

How reproducible:


Steps to Reproduce:
1. Create env path Development->QA->Production
2. Remove env QA
3. 
  
Actual results:
Both QA and Production are removed from the displayed list on the Org page.  However you can still select Production as a prior when you create a new environment.  If you do, the new environment is not shown.

Expected results:
Production should not be selectable as prior after having been implicitly removed.

Additional info:

Comment 1 Jeff Weiss 2012-02-17 19:34:07 UTC

You cannot create a new environment with the same name as the previous one either.  "Validation failed: Name must be unique within one organization" - so clearly it's not being removed completely.

Comment 2 Justin Sherrill 2012-03-02 21:13:25 UTC

Hrmmmm   you aren't supposed to be able to remove an environment if its in the middle.  Only ones at the end.

Comment 3 Justin Sherrill 2012-03-05 18:55:40 UTC

7294092190295fb1b5fbee015f9112fbecf5392e

disabled remove link (and added tipsy) if environment isn't the last one in its promotion path.

Also added validation in model to prevent deletes via api/cli.

Comment 7 Mike McCune 2012-03-07 23:43:32 UTC

mass move ON_QA after brewing

Comment 8 Jeff Weiss 2012-03-08 17:31:03 UTC

Verified, 0.2.8-1.git.11.033f96d.el6 - can no longer delete envs unless it's at the end of the path.  Remove link is disabled and has help tip.