Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.
Red Hat Satellite engineering is moving the tracking of its product development work on Satellite to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "Satellite project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs will be migrated starting at the end of May. If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "Satellite project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/SAT-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.

Bug 1316710

Summary: `foreman-rake permissions:reset` emits warnings: "circular argument reference - connection" and more
Product: Red Hat Satellite Reporter: Jan Hutař <jhutar>
Component: OtherAssignee: Eric Helms <ehelms>
Status: CLOSED ERRATA QA Contact: Katello QA List <katello-qa-list>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 6.2.0CC: apagac, bbuckingham, bkearney, daviddavis, lpramuk, mmccune, pmoravec, sthirugn, tpapaioa
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: 2016-07-27 11:51:13 UTC Type: Bug
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: 1115190    

Description Jan Hutař 2016-03-10 21:14:49 UTC
Description of problem:
`foreman-rake permissions:reset` emits warnings: "circular argument reference - connection" and more


Version-Release number of selected component (if applicable):
Satellite-6.2.0-RHEL-7-20160308.1-Satellite-x86_64


How reproducible:
always


Steps to Reproduce:
1. # foreman-rake permissions:reset


Actual results:
# foreman-rake permissions:reset
/opt/theforeman/tfm/root/usr/share/gems/gems/docker-api-1.17.0/lib/docker.rb:103: warning: circular argument reference - connection
/opt/theforeman/tfm/root/usr/share/gems/gems/katello-3.0.0.6/config/routes/overrides.rb:3: warning: already initialized constant Katello::WhitelistConstraint::PATHS
/opt/theforeman/tfm/root/usr/share/gems/gems/katello-3.0.0.6/config/routes/overrides.rb:3: warning: previous definition of PATHS was here
Reset to user: admin, password: swHsxfeLqABk38cL


Expected results:
No warnings in the output

Comment 1 David Davis 2016-03-17 15:43:15 UTC
Og reported a similar warning message in his /var/log/foreman/cron.log:

/opt/theforeman/tfm/root/usr/share/gems/gems/docker-api-1.17.0/lib/docker.rb:103: warning: circular argument reference - connection

I've this this in development as well. Looks like the fix is here:

https://github.com/swipely/docker-api/commit/7e92801d345e5eaaec4730df7498bc896ee9a070

Which is included in the docker-api gem v1.18.0.

Comment 2 David Davis 2016-03-17 15:48:01 UTC
These are just warnings and not serious errors. So not harmful but it may confuse/mislead users.

Comment 3 Pavel Moravec 2016-04-09 13:16:11 UTC
(In reply to David Davis from comment #1)
> Og reported a similar warning message in his /var/log/foreman/cron.log:
> 
> /opt/theforeman/tfm/root/usr/share/gems/gems/docker-api-1.17.0/lib/docker.rb:
> 103: warning: circular argument reference - connection
> 
> I've this this in development as well. Looks like the fix is here:
> 
> https://github.com/swipely/docker-api/commit/
> 7e92801d345e5eaaec4730df7498bc896ee9a070
> 
> Which is included in the docker-api gem v1.18.0.

Hit it during Sat62 testing days. In my case, I got also mail to /var/spool/mail/root with content:

From root.brq.redhat.com  Sat Apr  9 14:53:19 2016
Return-Path: <root.brq.redhat.com>
X-Original-To: root
Delivered-To: root.brq.redhat.com
Received: by pmoravec-sat62-rhel7.gsslab.brq.redhat.com (Postfix, from userid 0)
	id D3DFC1594F4; Sat,  9 Apr 2016 14:53:19 +0200 (CEST)
From: Anacron <root.brq.redhat.com>
To: root.brq.redhat.com
Content-Type: text/plain; charset="UTF-8"
Subject: Anacron job 'cron.weekly' on pmoravec-sat62-rhel7.gsslab.brq.redhat.com
Message-Id: <20160409125319.D3DFC1594F4.brq.redhat.com>
Date: Sat,  9 Apr 2016 14:53:19 +0200 (CEST)

/etc/cron.weekly/katello-remove-orphans:

/opt/theforeman/tfm/root/usr/share/gems/gems/docker-api-1.17.0/lib/docker.rb:103: warning: circular argument reference - connection
/opt/theforeman/tfm/root/usr/share/gems/gems/katello-3.0.0.15/config/routes/overrides.rb:3: warning: already initialized constant Katello::WhitelistConstraint::PATHS
/opt/theforeman/tfm/root/usr/share/gems/gems/katello-3.0.0.15/config/routes/overrides.rb:3: warning: previous definition of PATHS was here



Any run of /etc/cron.weekly/katello-remove-orphans causes the same, i.e. any customer would be "affected".

Will it be fixed in Sat6.2 GA release or is it worth filing KCS for that until fix released?

Comment 4 David Davis 2016-04-14 12:24:28 UTC
I did a diff between v1.17.0 (the current version in Satellite) and v1.18.0 of the docker-api gem and there's only a few small changes. I don't see any reason why this can't be ready by GA. 

That said, I don't know the criteria for GA bugs at this point though. Paging Mike McCune.

Comment 7 Lukas Pramuk 2016-06-02 15:58:53 UTC
VERIFIED.

@satellite-6.2.0-14.1.el7sat.noarch

# foreman-rake permissions:reset
Reset to user: admin, password: qDLMLXp6RFMaDoYW


>>> no disturbing warnings

Comment 8 Bryan Kearney 2016-07-27 11:51:13 UTC
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-2016:1501