neutron-vpnaas: Enable python34 support

Bug #1480326 reported by Kyle Mestery
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Li Ma

Bug Description

The following files are currently failing and need to be fixed for "tox -epy34" to work in this repository:

test_plugin.py
test_netns_wrapper.py
test_cisco_ipsec.py (in device drivers)
test_cisco_csr_rest_client.py
test_vpn_db.py

Kyle Mestery (mestery)
Changed in neutron:
importance: Undecided → Low
tags: added: low-hanging-fruit vpnaas
Li Ma (nick-ma-z)
Changed in neutron:
assignee: nobody → Li Ma (nick-ma-z)
assignee: Li Ma (nick-ma-z) → nobody
Li Ma (nick-ma-z)
Changed in neutron:
assignee: nobody → Li Ma (nick-ma-z)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron-vpnaas (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/211472

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Related fix proposed to branch: master
Review: https://review.openstack.org/215515

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Related fix proposed to branch: master
Review: https://review.openstack.org/215520

Revision history for this message
Li Ma (nick-ma-z) wrote :

test_plugin.py and test_vpn_db.py is dependent upon neutron-py34 blueprint.

Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron-vpnaas (master)

Reviewed: https://review.openstack.org/211472
Committed: https://git.openstack.org/cgit/openstack/neutron-vpnaas/commit/?id=5052bf7e8c5ab96efd36974455d9737db2845af4
Submitter: Jenkins
Branch: master

commit 5052bf7e8c5ab96efd36974455d9737db2845af4
Author: Nick <email address hidden>
Date: Tue Aug 11 17:12:14 2015 +0800

    Add test_netns_wrapper.py module for py34 support

    Change-Id: I15f43db0244a20c7509085804b4c9bdfa5ba5000
    Related-Bug: #1480326

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/215515
Committed: https://git.openstack.org/cgit/openstack/neutron-vpnaas/commit/?id=ae24b1a65a0d33eb9ca926040c962da09cbbc911
Submitter: Jenkins
Branch: master

commit ae24b1a65a0d33eb9ca926040c962da09cbbc911
Author: Nick <email address hidden>
Date: Fri Aug 21 16:38:39 2015 +0800

    Add test_cisco_ipsec.py module for py34 support

    Change-Id: I02b229721d0f52d7a4e04e9e91c7a3b4664b2d02
    Related-Bug: #1480326

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Reviewed: https://review.openstack.org/215520
Committed: https://git.openstack.org/cgit/openstack/neutron-vpnaas/commit/?id=51a6bec639ebf09aaa599e0bd221d8440b44c6df
Submitter: Jenkins
Branch: master

commit 51a6bec639ebf09aaa599e0bd221d8440b44c6df
Author: Nick <email address hidden>
Date: Fri Aug 21 16:45:39 2015 +0800

    Add cisco_csr_rest_client.py module for py34 support

    Change-Id: I864ecb982ac65afcc0819f851b0136dcce5de561
    Related-Bug: #1480326

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron-vpnaas (master)

Fix proposed to branch: master
Review: https://review.openstack.org/238084

Revision history for this message
Paul Michali (pcm) wrote :

In progress - just needs core review/approval.

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

Reviewed: https://review.openstack.org/238084
Committed: https://git.openstack.org/cgit/openstack/neutron-vpnaas/commit/?id=841fe76364021cc1770dd8f60e7eb8bd93e5229b
Submitter: Jenkins
Branch: master

commit 841fe76364021cc1770dd8f60e7eb8bd93e5229b
Author: Li Ma <email address hidden>
Date: Wed Oct 21 21:43:07 2015 +0800

    Update list of modules supporting py34 tests

    Currently, as neutron project supports py34 to a certain degree,
    test_plugin and test_vpn_db modules also support py34.

    All the modules in neutron-vpnaas support py34 finally.
    So, we remove the enumeration of modules and test all modules.

    Change-Id: Iba5cf0065ec9243fad90efcb192a66236a21bb2c
    Closes-Bug: #1480326

Changed in neutron:
status: In Progress → Fix Committed
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/neutron-vpnaas 8.0.0.0b1

This issue was fixed in the openstack/neutron-vpnaas 8.0.0.0b1 development milestone.

Changed in neutron:
status: Fix Committed → 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.