Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1147049 - Passing empty values to katello-installer proxy settings creates invalid JSON
Passing empty values to katello-installer proxy settings creates invalid JSON
Status: CLOSED ERRATA
Product: Red Hat Satellite 6
Classification: Red Hat
Component: Installer (Show other bugs)
6.0.4
Unspecified Unspecified
unspecified Severity medium (vote)
: Beta
: Unused
Assigned To: Eric Helms
Corey Welton
: Reopened, Triaged
: 1278085 (view as bug list)
Depends On:
Blocks: 1132363
  Show dependency treegraph
 
Reported: 2014-09-26 12:52 EDT by David Gurtner
Modified: 2016-07-27 07:22 EDT (History)
7 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2016-07-27 07:22:39 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)


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Knowledge Base (Solution) 2201011 None None None 2016-03-16 09:11 EDT
Red Hat Product Errata RHSA-2015:1592 normal SHIPPED_LIVE Important: Red Hat Satellite 6.1.1 on RHEL 6 2015-08-12 05:04:35 EDT

  None (edit)
Description David Gurtner 2014-09-26 12:52:54 EDT
Description of problem:

Running katello-installer --katello-proxy-port="" generates broken Pulp Plugins files under /etc/pulp/server/plugins.conf.d/{yum,puppet,iso}_importer.json

The Files will contain the following line:

"proxy_port": ,

instead of 

"proxy_port": "",

This prevents celeryd workers from starting.

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


How reproducible:

always


Steps to Reproduce:
1. katello-installer --katello-proxy-port="" --katello-proxy-url=""

Actual results:

/etc/pulp/server/plugins.conf.d/{yum,puppet,iso}_importer.json

Contain the line

"proxy_port": ,


Expected results:

"proxy_port": NULL,


Additional info:

Removing the values from answer file or passing NULL works.
Comment 1 RHEL Product and Program Management 2014-09-26 13:03:19 EDT
Since this issue was entered in Red Hat Bugzilla, the release flag has been
set to ? to ensure that it is properly evaluated for this release.
Comment 3 Eric Helms 2014-11-04 16:03:50 EST
This issue has been fully addressed by - https://github.com/Katello/katello-installer/pull/84
Comment 6 Elyézer Rezende 2015-04-06 15:32:10 EDT
Failed on: Satellite-6.1.0-RHEL-7-20150331.1

Steps:

1. Run katello-installer --katello-proxy-port="" --katello-proxy-url=""
2. Checked the config files

# cat /etc/pulp/server/plugins.conf.d/{yum,puppet,iso}_importer.json
{
    "proxy_host": "",

    "proxy_port": ,

    "proxy_username": "",
    "proxy_password": ""
}
{
    "proxy_host": "",

    "proxy_port": ,

    "proxy_username": "",
    "proxy_password": ""
}
{
    "proxy_host": "",

    "proxy_port": ,

    "proxy_username": "",
    "proxy_password": ""
}
Comment 8 errata-xmlrpc 2015-08-12 01:17:25 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/RHSA-2015:1592
Comment 9 sthirugn@redhat.com 2015-08-14 16:46:20 EDT
Accidentally closed with 6.1.1 errata
Comment 10 Brad Buckingham 2015-11-16 16:47:25 EST
*** Bug 1278085 has been marked as a duplicate of this bug. ***
Comment 11 Corey Welton 2016-06-24 10:54:00 EDT
POSSIBLY Verified in Snap 16:

[root@rhsm-qe-3 ~]# cat /etc/pulp/server/plugins.conf.d/{yum,puppet,iso}_importer.json
{
    
    "proxy_host": null,
    
    
    "proxy_port": null,
    
        "proxy_username": "admin",
    
        "proxy_password": "password"
    
}
{
    
    "proxy_host": null,
    
    
    "proxy_port": null,
    
        "proxy_username": "admin",
    
        "proxy_password": "password"
    
}
{
    
    "proxy_host": null,
    
    
    "proxy_port": null,
    
        "proxy_username": "admin",
    
        "proxy_password": "password"
    
}


Should we be clearing username/password  if proxy settings themselves are cleared?
Comment 12 Corey Welton 2016-06-24 10:55:25 EDT
To clarify, does having user/password set affect things if there are no values otherwise for host and port?
Comment 13 Corey Welton 2016-06-24 11:33:19 EDT
Tested and even if the user/passwd remain, sync continues to work.

That said, there is no reason I can think of to keep user/passwd around if proxy and port are gone.  Will write up a separate RFE for removal.
Comment 15 Bryan Kearney 2016-07-27 07:22:39 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-2016:1501

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