Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1337089 - [Behind Proxy] gitHTTPSProxy and HTTPS_PROXY should be included in openshift_builddefaults_json variable of example file
[Behind Proxy] gitHTTPSProxy and HTTPS_PROXY should be included in openshift_...
Status: CLOSED ERRATA
Product: OpenShift Container Platform
Classification: Red Hat
Component: Installer (Show other bugs)
3.2.0
Unspecified Unspecified
medium Severity low
: ---
: 3.3.1
Assigned To: Scott Dodson
Wenkai Shi
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2016-05-18 05:24 EDT by Gan Huang
Modified: 2017-01-21 21:47 EST (History)
6 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Previously, the example advanced configuration hosts file documented 'openshift_builddefaults_json' without specifying all the possible options. Now the example value has been updated to express all the possible options currently available.
Story Points: ---
Clone Of:
Environment:
Last Closed: 2016-10-27 12:12:30 EDT
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
sdodson: needinfo-


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2016:2122 normal SHIPPED_LIVE OpenShift Container Platform atomic-openshift-utils bug fix update 2016-10-27 16:11:30 EDT

  None (edit)
Description Gan Huang 2016-05-18 05:24:09 EDT
Description of problem:
The variable defined in example hosts file is:
[root@xxx ~]# grep "openshift_builddefaults_json" docs/example-inventories/hosts.ose.example
#openshift_builddefaults_json='{"BuildDefaults":{"configuration":{"apiVersion":"v1","env":[{"name":"HTTP_PROXY","value":"http://proxy.example.com.redhat.com:3128"},{"name":"NO_PROXY","value":"ose3-master.example.com"}],"gitHTTPProxy":"http://proxy.example.com:3128","kind":"BuildDefaultsConfig"}}}'

gitHTTPSProxy and HTTPS_PROXY are not included in the example variable, actually they're needed to do S2I build in 99% secenarios.

Version-Release number of selected component (if applicable):
openshift-ansible-docs-3.0.90-1.git.0.a077b68.el7.noarch

How reproducible:
100%

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:
gitHTTPSProxy and HTTPS_PROXY should be included in example variable openshift_builddefaults_json 

Additional info:
Comment 2 Scott Dodson 2016-09-28 14:08:56 EDT
(In reply to Gan Huang from comment #0)
> Description of problem:
> The variable defined in example hosts file is:
> [root@xxx ~]# grep "openshift_builddefaults_json"
> docs/example-inventories/hosts.ose.example
> #openshift_builddefaults_json='{"BuildDefaults":{"configuration":
> {"apiVersion":"v1","env":[{"name":"HTTP_PROXY","value":"http://proxy.example.
> com.redhat.com:3128"},{"name":"NO_PROXY","value":"ose3-master.example.com"}],
> "gitHTTPProxy":"http://proxy.example.com:3128","kind":
> "BuildDefaultsConfig"}}}'
> 
> gitHTTPSProxy and HTTPS_PROXY are not included in the example variable,
> actually they're needed to do S2I build in 99% secenarios.

This variable is really an advanced configuration value. Why do you say it's needed in 99% of the s2i scenarios? Is it still necessary with all the changes that've happened between then and now?
Comment 3 Scott Dodson 2016-09-28 15:38:18 EDT
Nevermind, I see what you mean.

https://github.com/openshift/openshift-ansible/pull/2521
Comment 5 Wenkai Shi 2016-10-09 05:31:34 EDT
There is gitHTTPSProxy without HTTPS_PROXY in the example variable. If install like this, it will be failure.

[root@xxx ~]# grep "openshift_builddefaults_json" /usr/share/doc/openshift-ansible-docs-3.3.32/docs/example-inventories/hosts.ose.example 
#openshift_builddefaults_json='{"BuildDefaults":{"configuration":{"kind":"BuildDefaultsConfig","apiVersion":"v1","gitHTTPSProxy":"http://proxy.example.com.redhat.com:3128","gitHTTPProxy":"http://proxy.example.com.redhat.com:3128","env":[{"name":"HTTP_PROXY","value":"http://proxy.example.com.redhat.com:3128"},{"name":"NO_PROXY","value":"ose3-master.example.com"}]}}'

...
TASK [openshift_builddefaults : Set builddefaults config structure] ************
Sunday 09 October 2016  03:53:00 -0400 (0:00:06.931)       0:12:00.481 ******** 
fatal: [xxx.example.com]: FAILED! => {"changed": false, "failed": true, "module_stderr": "Shared connection to xxx.example.com closed.\r\n", "module_stdout": "Traceback (most recent call last):\r\n  File \"/tmp/ansible_iwewXK/ansible_module_openshift_facts.py\", line 2214, in <module>\r\n    main()\r\n  File \"/tmp/ansible_iwewXK/ansible_module_openshift_facts.py\", line 2195, in main\r\n    protected_facts_to_overwrite)\r\n  File \"/tmp/ansible_iwewXK/ansible_module_openshift_facts.py\", line 1645, in __init__\r\n    protected_facts_to_overwrite)\r\n  File \"/tmp/ansible_iwewXK/ansible_module_openshift_facts.py\", line 1710, in generate_facts\r\n    facts = set_proxy_facts(facts)\r\n  File \"/tmp/ansible_iwewXK/ansible_module_openshift_facts.py\", line 1467, in set_proxy_facts\r\n    facts['master']['kube_admission_plugin_config'].update(builddefaults['config'])\r\nValueError: dictionary update sequence element #0 has length 1; 2 is required\r\n", "msg": "MODULE FAILURE"}
...
Comment 7 Wenkai Shi 2016-10-11 02:29:27 EDT
It is already fix.

[root@xxx ~]# grep "openshift_builddefaults_json" /usr/share/doc/openshift-ansible-docs-3.3.34/docs/example-inventories/hosts.ose.example
#openshift_builddefaults_json='{"BuildDefaults":{"configuration":{"kind":"BuildDefaultsConfig","apiVersion":"v1","gitHTTPSProxy":"http://proxy.example.com.redhat.com:3128","gitHTTPProxy":"http://proxy.example.com.redhat.com:3128","env":[{"name":"HTTP_PROXY","value":"http://proxy.example.com.redhat.com:3128"},{"name":"HTTPS_PROXY","value":"http://proxy.example.com.redhat.com:3128"},{"name":"NO_PROXY","value":"ose3-master.example.com"}]}}'
Comment 8 Wenkai Shi 2016-10-13 05:14:17 EDT
The version of fixed rpm is:
openshift-ansible-docs-3.3.34-1.git.0.5b998bb.el7.noarch.rpm
Comment 10 errata-xmlrpc 2016-10-27 12:12:30 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:2122

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