Bug 798339

Summary: If promotion fails, please rollback
Product: Red Hat Satellite Reporter: Og Maciel <omaciel>
Component: Content ManagementAssignee: Lukas Zapletal <lzap>
Status: CLOSED WONTFIX QA Contact: Katello QA List <katello-qa-list>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0.0CC: mmccune
Target Milestone: UnspecifiedKeywords: 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: 2014-03-12 22:53:10 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 Og Maciel 2012-02-28 16:49:52 UTC
Description of problem:

If I'm promoting content to an environment for the first time (or even if not the first time for that matter) and something causes this process to fail somewhere along the way, pulp will leave content from this partial promotion in its local filesystem, which to me is not the best thing to do, not to say that it could lead to complications in the future, when the promotion does work eventually. I'd like to see promotions and syncs behave more like a database transaction, where partial successful operations get "rolled back" and only 100% successful operations get "committed" to the system.

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


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

See BZs https://bugzilla.redhat.com/show_bug.cgi?id=798299 and https://bugzilla.redhat.com/show_bug.cgi?id=798301 for more information on how I came to see partial committed content to my system. Since my promotion failed, I should not see content for my custom SystemEngine -> Hourly repository but I did have content as shown here:

/var/lib/pulp/repos/SaoPaulo/Dev1/custom/SystemEngine/Hourly/packages/

Comment 1 Mike McCune 2012-02-29 00:21:44 UTC
ideal but time consuming to implement.  Advocating we fix in V1+

Comment 2 Lukas Zapletal 2014-03-12 10:04:20 UTC
Bug is irrelevant because orchestration is being rewritten.

Comment 3 Bryan Kearney 2014-03-12 22:53:10 UTC
Closing old bugs which are not relevant based on new UI and CLI