Bug 782562

Summary: "force" checkbox necessary on the Red Hat import manifest upload
Product: Red Hat Satellite Reporter: Tom McKay <tomckay>
Component: WebUIAssignee: Jordan OMara <jomara>
Status: CLOSED CURRENTRELEASE QA Contact: Katello QA List <katello-qa-list>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.0.1CC: athomas, kbidarka, 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: 2012-08-22 18:20:35 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: 747354    

Description Tom McKay 2012-01-17 17:35:04 UTC
To force import of older manifests, the "force" option must be passed to the import API. This needs to be controlled via UI and CLI.

diff --git a/src/app/controllers/providers_controller.rb b/src/app/controllers/providers_controller.rb
index ffbe968..d0f3c24 100644
--- a/src/app/controllers/providers_controller.rb
+++ b/src/app/controllers/providers_controller.rb
@@ -65,7 +65,7 @@ class ProvidersController < ApplicationController
         temp_file = File.new(File.join(dir, "import_#{SecureRandom.hex(10)}.zip"), 'w+', 0600)
         temp_file.write params[:provider][:contents].read
         temp_file.close
-        @provider.import_manifest File.expand_path(temp_file.path)
+        @provider.import_manifest(File.expand_path(temp_file.path), {:force => "true"})
         notice _("Subscription manifest uploaded successfully for provider '%{name}'. Please enable t
       rescue Exception => error
         display_message = parse_display_message(error.response)

Comment 1 Jordan OMara 2012-01-24 18:21:54 UTC
committed 9b5f3e90692f5bf1453f50f8418214d3af95ea46

Comment 2 Jordan OMara 2012-01-24 18:50:00 UTC
Note: my commit does not address the CLI, but this feature is already present

Comment 3 Mike McCune 2012-01-26 19:07:24 UTC
mass ON_QA move

Comment 5 Kedar Bidarkar 2012-01-31 09:41:29 UTC
Force checkbox is now available under RedHat Content Provider Section

Comment 7 Mike McCune 2013-08-16 18:18:12 UTC
getting rid of 6.0.0 version since that doesn't exist