Description of problem: byo failed on atomic host with containerized=True: docker_image_availability Version-Release number of the following components: rpm -q openshift-ansible [root@installer-jenkins-slave-1 private-openshift-ansible]# git log --oneline -1 9920ea8 Merge pull request #5588 from IshentRas/fix-typo rpm -q ansible # rpm -q ansible ansible-2.3.2.0-2.el7.noarch ansible --version # ansible --version ansible 2.3.2.0 config file = /home/slave3/workspace/Launch Environment Flexy/private-openshift-ansible/ansible.cfg configured module search path = Default w/o overrides python version = 2.7.13 (default, Jan 12 2017, 17:59:37) [GCC 6.3.1 20161221 (Red Hat 6.3.1-1)] How reproducible: Steps to Reproduce: 1. ansible-playbook openshift-ansible/playbooks/byo/config.yml the inv. file and the output are attached. 2. 3. Actual results: Please include the entire output from the last TASK line through the end of output if an error is generated Expected results: Additional info: Please attach logs from ansible-playbook with the -vvv flag
It looks like this is the relevant error line: Checked by: timeout 10 skopeo inspect --tls-verify=false docker://{registry}/{image} To me, this indicates that the image test is no longer reading the registry nor image variables properly. @Luke, to your knowledge has there been any updates related to this check that would cause expansion to fail?
Expansion isn't failing, that's just the generic way of telling the user what command it's using so they can replicate; it runs multiple times -- several images against all registries. If they want to see the individual invocations there's -vvv or -e openshift_checks_output_dir ... There have been changes in this area, and I'll look into whether they're relevant, but it could also be other refactoring. I think openshift_docker_additional_registries is supposed to be an array and in true pythonic fashion, the string is being interpreted as an array of characters. Just have to find out why (specifically, why now and not before).
Looks like the l2_docker_additional_registries (etc..) may be the new variables to use for registries as they properly force the results into a list depending on the original import (single string or CSV string): b35272a794 (Michael Gugino 2017-08-24 17:24:41 -0400 14) # The l2_docker_* variables convert csv strings to lists, if b35272a794 (Michael Gugino 2017-08-24 17:24:41 -0400 15) # necessary. These variables should be used in place of their respective b35272a794 (Michael Gugino 2017-08-24 17:24:41 -0400 16) # openshift_docker_* counterparts to ensure the properly formatted lists are b35272a794 (Michael Gugino 2017-08-24 17:24:41 -0400 17) # utilized.
PR: https://github.com/openshift/openshift-ansible/pull/5619
Merged.
Verified with [fedora@ip-172-31-33-174 openshift-ansible]$ git log --oneline -1 04a17c289 (HEAD -> master, origin/master, origin/HEAD) Merge pull request #5630 from abutcher/etcd-scaleup-target [fedora@ip-172-31-33-174 openshift-ansible]$ ansible --version ansible 2.3.2.0 config file = /home/fedora/openshift-ansible/ansible.cfg configured module search path = Default w/o overrides python version = 2.7.13 (default, Sep 5 2017, 08:53:59) [GCC 7.1.1 20170622 (Red Hat 7.1.1-3)] The error in the bz did not show up. The installation of byo succeeded.
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:3464