Bug 795928

Summary: [RFE] Environment renaming should be enabled
Product: [Retired] Subscription Asset Manager Reporter: Tom McKay <tomckay>
Component: katelloAssignee: Jordan OMara <jomara>
Status: CLOSED WONTFIX QA Contact: SAM QE List <sam-qe-list>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 1.3CC: athomas, bkearney, jomara, lzap, mmccune, pchalupa, sthirugn, taw
Target Milestone: rcKeywords: FutureFeature, Triaged
Target Release: 1.3   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of:
: 882248 996562 (view as bug list) Environment:
Last Closed: 2013-08-13 20:09:07 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 833466, 882248, 996562    

Description Tom McKay 2012-02-21 20:01:55 UTC
See 787226 for reasons renaming was disabled. This disable, though, is only a short-term fix to avoid larger issues and needs to be resolved.

-----
Since we base the repoid in pulp off of the environment name, we really
shouldn't be able to rename it, as it could cause problems. 

One such scenario that could cause issues:


1. create environment Dev
2. promote repo X to dev
3. rename environment Dev to Foo
4. Create a new environment called Dev after Foo
5. promote repo X to Dev


Since the repo was already created in an environment called Dev, the repo
promotion would fail.  


Ideally we wouldn't be basing repo_id off of environment name.  There was some
discussion on having an immutable label field, which should allow us to have a
name that is changeable, but this is probably post V1.


We need to block this in the UI and the api.

Comment 1 Lukas Zapletal 2012-12-18 15:14:16 UTC
Ok this can be closed too: https://bugzilla.redhat.com/show_bug.cgi?id=807055

Comment 2 Justin Sherrill 2012-12-18 15:29:31 UTC
*** Bug 807055 has been marked as a duplicate of this bug. ***

Comment 3 Tom McKay 2012-12-18 18:44:40 UTC
https://github.com/Katello/katello/pull/1281

Comment 4 Bryan Kearney 2013-06-06 20:50:15 UTC
Moving all POST / MODIFIED bugs to ON_QA due to the new builds.

Comment 6 sthirugn@redhat.com 2013-08-13 20:09:07 UTC
Environments are not valid in headpin/sam anymore.  https://bugzilla.redhat.com/show_bug.cgi?id=996562 will be used to track this change in Satellite 6.