Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1509163 - No recommended version of Open vSwitch for OCP 3.7
No recommended version of Open vSwitch for OCP 3.7
Status: CLOSED ERRATA
Product: OpenShift Container Platform
Classification: Red Hat
Component: Installer (Show other bugs)
3.7.0
Unspecified Unspecified
medium Severity medium
: ---
: 3.7.0
Assigned To: Miciah Dashiel Butler Masters
Wenkai Shi
:
Depends On:
Blocks: 1534922
  Show dependency treegraph
 
Reported: 2017-11-03 04:58 EDT by Wenkai Shi
Modified: 2018-01-16 04:57 EST (History)
7 users (show)

See Also:
Fixed In Version:
Doc Type: No Doc Update
Doc Text:
undefined
Story Points: ---
Clone Of:
: 1534922 (view as bug list)
Environment:
Last Closed: 2017-11-28 17:21:23 EST
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 Product Errata RHSA-2017:3188 normal SHIPPED_LIVE Moderate: Red Hat OpenShift Container Platform 3.7 security, bug, and enhancement update 2017-11-28 21:34:54 EST

  None (edit)
Description Wenkai Shi 2017-11-03 04:58:05 EDT
Description of problem:
No recommended version of Open vSwitch for OCP 3.7 during run ovs_version health check. Check failed.

Version-Release number of the following components:
openshift-ansible-3.7.0-0.190.0.git.0.129e91a.el7

How reproducible:
100%

Steps to Reproduce:
1. Run ovs_version health check.
# ansible-playbook -i inventory -v /usr/share/ansible/openshift-ansible/playbooks/byo/openshift-checks/adhoc.yml -e openshift_checks=ovs_version
2.
3.

Actual results:
# ansible-playbook -i inventory -v /usr/share/ansible/openshift-ansible/playbooks/byo/openshift-checks/adhoc.yml -e openshift_checks=ovs_version
...
TASK [Run health checks (adhoc)] *************************************************************************************************************************************************************

CHECK [ovs_version : ec2-xx-xx-xx-xx.compute-1.amazonaws.com] *****************************************************************************************************************************
fatal: [ec2-xx-xx-xx-xx.compute-1.amazonaws.com]: FAILED! => {"changed": false, "checks": {"ovs_version": {"failed": true, "failures": [["OpenShiftCheckException", "There is no recommended version of Open vSwitch for the current version of OpenShift: 3.7"]], "msg": "There is no recommended version of Open vSwitch for the current version of OpenShift: 3.7"}}, "failed": true, "msg": "One or more checks failed", "playbook_context": "adhoc"}

CHECK [ovs_version : ec2-xx-xx-xx-xx.compute-1.amazonaws.com] *****************************************************************************************************************************
fatal: [ec2-xx-xx-xx-xx.compute-1.amazonaws.com]: FAILED! => {"changed": false, "checks": {"ovs_version": {"failed": true, "failures": [["OpenShiftCheckException", "There is no recommended version of Open vSwitch for the current version of OpenShift: 3.7"]], "msg": "There is no recommended version of Open vSwitch for the current version of OpenShift: 3.7"}}, "failed": true, "msg": "One or more checks failed", "playbook_context": "adhoc"}
...

Expected results:
Should have recommended version for OCP 3.7

Additional info:
# cat /usr/share/ansible/openshift-ansible/roles/openshift_health_checker/openshift_checks/ovs_version.py
...
    openshift_to_ovs_version = {
        "3.6": ["2.6", "2.7"],
        "3.5": ["2.6", "2.7"],
        "3.4": "2.4",
    }
...
# cat /usr/share/ansible/openshift-ansible/roles/openshift_health_checker/openshift_checks/package_version.py
...
    openshift_to_ovs_version = {
        (3, 4): "2.4",
        (3, 5): ["2.6", "2.7"],
        (3, 6): ["2.6", "2.7"],
    }
...
Comment 1 Wenkai Shi 2017-11-05 21:01:43 EST
There should have a correct recommended version in 3.7 when release it.
Comment 2 Luke Meyer 2017-11-06 10:24:38 EST
I seem to recall there was some logic to use the most recent recommendation if there wasn't one. Maybe that was in a different check though. Apparently not here. We don't particularly want to have to update this for each release...
Comment 3 Miciah Dashiel Butler Masters 2017-11-08 11:09:27 EST
PR: https://github.com/openshift/openshift-ansible/pull/6023
Comment 5 Wenkai Shi 2017-11-08 23:44:58 EST
Verified with version openshift-ansible-3.7.4-1.git.0.254e849.el7. works well~

# ansible-playbook -i inventory -v /usr/share/ansible/openshift-ansible/playbooks/byo/openshift-checks/adhoc.yml -e openshift_checks=ovs_version
...
TASK [Run health checks (adhoc)] *************************************************************************************************************************************************************

CHECK [ovs_version : ec2-xx-xx-xx-xx.compute-1.amazonaws.com] ******************************************************************************************************************************

CHECK [ovs_version : ec2-xx-xx-xx-xx.compute-1.amazonaws.com] ******************************************************************************************************************************
ok: [ec2-xx-xx-xx-xx.compute-1.amazonaws.com] => {"changed": false, "checks": {"ovs_version": {"changed": false, "invocation": {"module_args": {"package_list": [{"name": "openvswitch", "version": ["2.6", "2.7", "2.8"]}]}}}}, "failed": false, "playbook_context": "adhoc"}
ok: [ec2-xx-xx-xx-xx.compute-1.amazonaws.com] => {"changed": false, "checks": {"ovs_version": {"changed": false, "invocation": {"module_args": {"package_list": [{"name": "openvswitch", "version": ["2.6", "2.7", "2.8"]}]}}}}, "failed": false, "playbook_context": "adhoc"}
...

# vim /usr/share/ansible/openshift-ansible/roles/openshift_health_checker/openshift_checks/ovs_version.py
...
    openshift_to_ovs_version = {
        "3.7": ["2.6", "2.7", "2.8"],
...

# vim /usr/share/ansible/openshift-ansible/roles/openshift_health_checker/openshift_checks/package_version.py
...
    openshift_to_ovs_version = {
        (3, 4): "2.4",
        (3, 5): ["2.6", "2.7"],
        (3, 6): ["2.6", "2.7", "2.8"],
        (3, 7): ["2.6", "2.7", "2.8"],
    }
...

# vim /usr/share/ansible/openshift-ansible/roles/openshift_health_checker/test/ovs_version_test.py
...
@pytest.mark.parametrize('openshift_release,expected_ovs_version', [
    ("3.7", ["2.6", "2.7", "2.8"]),
...
Comment 6 Miciah Dashiel Butler Masters 2017-11-10 10:10:53 EST
No need to document this since it is only an issue that arises from the increase in the version number.
Comment 9 errata-xmlrpc 2017-11-28 17:21:23 EST
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-2017:3188

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