[stable/stein] Neutron-tempest-plugin jobs are failing due to wrong python version

Bug #1873776 reported by Slawek Kaplonski
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Critical
Bernard Cafarelli

Bug Description

Issue like in https://b5dda6b7d9fe09cc6972-899ca8db232a7bf5fd9d6d869408b0bd.ssl.cf1.rackcdn.com/720051/1/check/neutron-tempest-plugin-api-stein/5a092fb/controller/logs/devstacklog.txt happens in all neutron-tempest-plugin jobs in stable/stein branch.

Error:

2020-04-20 00:19:32.058 | Ignoring typed-ast: markers 'python_version == "3.6"' don't match your environment
2020-04-20 00:19:32.120 | Obtaining file:///opt/stack/neutron-tempest-plugin
2020-04-20 00:19:32.987 | neutron-tempest-plugin requires Python '>=3.6' but the running Python is 2.7.17
2020-04-20 00:19:33.080 | You are using pip version 9.0.3, however version 20.0.2 is available.
2020-04-20 00:19:33.080 | You should consider upgrading via the 'pip install --upgrade pip' command.

I think it should be run with python 3.6 or we should use tag 0.9.0 which is last version with support for python 2.7

Revision history for this message
Bernard Cafarelli (bcafarel) wrote :

I wonder if we lack some configuration change like
https://bugs.launchpad.net/tempest/+bug/1871327
https://review.opendev.org/#/q/I60949fb735c82959fb2cfcb6aeef9e33fb0445b6

Or what changed recently, I think it was still passing on April 17/18 2020

For supported stable branches, I guess we should have it under py3 venv (to run on latest tests)

Revision history for this message
Ghanshyam Mann (ghanshyammann) wrote :

this is same problem what we had in devstack for tempest to install the tempest on system wide not in venv.

- https://github.com/openstack/neutron-tempest-plugin/blob/5719f992d146505eb593d1fd68d02e4945e2e07f/devstack/plugin.sh#L8

We need to disable the neutron-tempest-plugins installation on systemwide.

Changed in neutron:
status: Confirmed → Won't Fix
status: Won't Fix → In Progress
assignee: nobody → Bernard Cafarelli (bcafarel)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron-tempest-plugin (master)

Fix proposed to branch: master
Review: https://review.opendev.org/721277

tags: added: neutron-proactive-backport-potential
tags: removed: neutron-proactive-backport-potential
Revision history for this message
Rodolfo Alonso (rodolfo-alonso-hernandez) wrote :

Hello:

I think the problem described in this bug is due to [1]. Master branch and Train is not affected by this problem because the tempest-plugin CI jobs are using python3.6, but older branches (<=Stein) are using python2.7.

This is why this check in "setup.cfg" is failing:
  python-requires = >=3.6

Because we don't have branches in neutron-tempest-plugin, we should remove this limitation until all stable branches tested in the CI use python>=3.6.

Regards.

[1]https://github.com/openstack/neutron-tempest-plugin/commit/cad2c81b5427af6a26293742d9b2605b73eebfa0

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron-tempest-plugin (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/721795

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron-tempest-plugin (master)

Change abandoned by Rodolfo Alonso Hernandez (<email address hidden>) on branch: master
Review: https://review.opendev.org/721795

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron-tempest-plugin (master)

Reviewed: https://review.opendev.org/721277
Committed: https://git.openstack.org/cgit/openstack/neutron-tempest-plugin/commit/?id=0bdc63d7c1c3ed192b7206ec9ac25e4380fabbd2
Submitter: Zuul
Branch: master

commit 0bdc63d7c1c3ed192b7206ec9ac25e4380fabbd2
Author: Bernard Cafarelli <email address hidden>
Date: Mon Apr 20 16:31:14 2020 +0200

    Only install plugin system-wide if INSTALL_TEMPEST is set

    Similarly to main tempest installation check, we should only install
    plugin on master branches (where INSTALL_TEMPEST is True), and not on
    stable branches

    Also drop all-plugin use for tox_envlist in stadium jobs that were using
    it, it is deprecated and breaks with this new variable

    Change-Id: Ia7238c2044053d77b559de544c22909c50e1a11f
    Co-Authored-By: Ghanshyam Mann <email address hidden>
    Closes-Bug: #1873776

Changed in neutron:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.