Bug 782562 - "force" checkbox necessary on the Red Hat import manifest upload
Summary: "force" checkbox necessary on the Red Hat import manifest upload
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: WebUI
Version: 6.0.1
Hardware: Unspecified
OS: Unspecified
unspecified
medium vote
Target Milestone: Unspecified
Assignee: Jordan OMara
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks: katello-blockers
TreeView+ depends on / blocked
 
Reported: 2012-01-17 17:35 UTC by Tom McKay
Modified: 2019-09-26 13:28 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-22 18:20:35 UTC
Target Upstream Version:


Attachments (Terms of Use)

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


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