Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1455154 - Error refreshing manifest. wrong number of arguments (2 for 1)
Error refreshing manifest. wrong number of arguments (2 for 1)
Status: CLOSED ERRATA
Product: Red Hat Satellite 6
Classification: Red Hat
Component: Subscription Management (Show other bugs)
6.2.10
Unspecified Unspecified
high Severity high (vote)
: 6.2.10
: Unused
Assigned To: Justin Sherrill
Roman Plevka
: Regression, Triaged
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2017-05-24 07:20 EDT by Evgeni Golov
Modified: 2017-06-20 13:24 EDT (History)
5 users (show)

See Also:
Fixed In Version: rubygem-katello-3.0.0.135-1
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2017-06-20 13:24:07 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
log of the issue (15.50 KB, text/plain)
2017-05-24 07:20 EDT, Evgeni Golov
no flags Details


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2017:1553 normal SHIPPED_LIVE Satellite 6.2.10 Async Bug Release 2017-06-20 17:19:07 EDT

  None (edit)
Description Evgeni Golov 2017-05-24 07:20:40 EDT
Created attachment 1281920 [details]
log of the issue

Description of problem:
When trying to refresh a manifest on 6.2.10-1.0, the UI just shows "Error refreshing manifest. wrong number of arguments (2 for 1)"
production.log excerpt is attached.

The problem seems to be that the refresh is calling the owner_regenerate_upstream_certificates function, which is defined as:
def owner_regenerate_upstream_certificates(upstream)
but we're calling it with two options
:action => [self, :owner_regenerate_upstream_certificates, upstream, options]

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

How reproducible:
100%

Steps to Reproduce:
1. refresh manifest

Actual results:


Expected results:
manifest is refreshed

Additional info:
Comment 2 Evgeni Golov 2017-05-24 08:29:32 EDT
The obvious patch that *seems* to fix it is:

% git diff
diff --git a/app/models/katello/glue/provider.rb b/app/models/katello/glue/provider.rb
index ea408f2..2101aba 100644
--- a/app/models/katello/glue/provider.rb
+++ b/app/models/katello/glue/provider.rb
@@ -223,7 +223,7 @@ module Katello
             zip_file_path = "/tmp/#{rand}.zip"
             upstream = options[:upstream]
             pre_queue.create(:name => "regenerate upstream certificates for: #{self.organization.name}",
-                             :priority => 1, :action => [self, :owner_regenerate_upstream_certificates, upstream, options],
+                             :priority => 1, :action => [self, :owner_regenerate_upstream_certificates, upstream],
                              :action_rollback => nil)
             pre_queue.create(:name => "export upstream manifest for owner: #{self.organization.name}",
                              :priority => 2, :action => [self, :owner_upstream_update, upstream, options],
Comment 4 Roman Plevka 2017-05-29 06:52:53 EDT
VERIFIED on 6.2.10-2 by QA automation:

No issues detected while refreshing a manifest
Comment 6 errata-xmlrpc 2017-06-20 13:24:07 EDT
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2017:1553

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