neutronclient gate broken following VPNaaS infra changes

Bug #1484148 reported by Assaf Muller
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-neutronclient
Expired
High
Unassigned
Assaf Muller (amuller)
summary: - stable/kilo and neutronclient gates broken following VPNaaS infra
- changes
+ neutronclient gate broken following VPNaaS infra changes
description: updated
tags: added: gate-failure
tags: added: vpnaas
Revision history for this message
Paul Michali (pcm) wrote :

Have https://review.openstack.org/#/c/209887/ and https://review.openstack.org/#/c/210021/1 to add plugin support (need to be updated with bug#)

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

We need to get 209887 updated in project-config to enable the use of a gate hook. Once upstreamed, 210021 can then use a gate-hook to enable the plugin. This will then allow future changes (if any are needed) to be done inside of the neutronclient project.

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

These is a workaround in place (VPN tests are skipped). Do need to determine approach for solution with the test jobs.

Some choices are:

A) One test job that enables the plugins and tests everything
B) Two jobs one for core, one for adv svc commands
C) Three/Four jobs, one for core, and then one for each advanced service, as they switch to plugins.

Revision history for this message
Kyle Mestery (mestery) wrote :

What is the current status of this bug Paul? Is it still critical?

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

Currently, the VPN tests are being skipped, so that the neutron client tests will pass.

There are two commits out to restore VPN tests (not sure why they are not showing up here).

https://review.openstack.org/#/c/209887/ Creates two experiment jobs for neutron-client that will use a gate hook to run tests; one for core neutron client tests and one for advanced services tests. The jobs are neutronclient-test-dsvm-functional and neutronclient-test-dsvm-functional-adv-svcs.

https://review.openstack.org/#/c/214587/ Modifies neutron client to split out the test modules into core and advanced service test modules (VPN only currently), and uses the gate hook to enable VPN devstack plugin for the advanced services job.

We need to get 209887 upstreamed, so that 214587 can then be tested in Jenkins.

Revision history for this message
Kyle Mestery (mestery) wrote :

Targeting Liberty-RC1 now and hoping we can land these two reviews.

Changed in neutron:
milestone: none → liberty-rc1
Kyle Mestery (mestery)
Changed in neutron:
importance: Critical → High
Kyle Mestery (mestery)
Changed in python-neutronclient:
status: New → In Progress
importance: Undecided → High
assignee: nobody → Paul Michali (pcm)
no longer affects: neutron
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-neutronclient (master)

Reviewed: https://review.openstack.org/214587
Committed: https://git.openstack.org/cgit/openstack/python-neutronclient/commit/?id=b915201dbfd81b7dbb62c51a30576d596c3a3fa7
Submitter: Jenkins
Branch: master

commit b915201dbfd81b7dbb62c51a30576d596c3a3fa7
Author: Paul Michali <email address hidden>
Date: Wed Aug 19 12:16:05 2015 +0000

    Enable VPN test cases

    With the gate hook changes from Ieeaca1375d68705509f4e05f10cb35c0fa0b9582 and
    new jobs from I2db939bf99288c0cdec06cdd49fec3bdc72e5253, this commit does
    several things:

        * Moves test modules to tests/functional/core/
        * Moves VPN test cases to new module & places in tests/functional/adv-svcs
        * Modifies tox.ini to support 'functional' and 'functional-adv-svcs' test

    This commit will be used to test the experimental jobs, one that runs the
    same tests as before, one that runs VPN tests using the VPN DevStack plugin
    via the gate-hook.sh.

    Modified post test hook so that test results are properly created (was not
    seeing results from testr, prior to this).

    Goal is to place all advanced services into tests/functional/adv-svcs/ and
    use plugins, as needed in the gate hook.

    Change-Id: I1e3d19e51a1cbd1bc947bbf9927260cd4d73841a
    Depends-On: I2db939bf99288c0cdec06cdd49fec3bdc72e5253
    Partial-Bug: 1484148

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

Update:

VPN tests were "skipped" so that job would work (so this bug won't affect release). Since then, new jobs have been created and tested as experimental and, then as non-voting in check queue (https://review.openstack.org/227232). Once this gets some test time with commits, we can move it to voting and close this bug out completely (and remove the old job, as one of the new ones replaces it).

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Ok, so to close this we need a project-config change rather than any neutron tree related change.

Changed in python-neutronclient:
milestone: none → 2.4.1
milestone: 2.4.1 → none
Revision history for this message
Paul Michali (pcm) wrote :

Correct. Once we are comfortable with the non-voting results, I can make another project-config commit to make this voting.

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

This bug is > 180 days without activity. We are unsetting assignee and milestone and setting status to Incomplete in order to allow its expiry in 60 days.

If the bug is still valid, then update the bug status.

Changed in python-neutronclient:
assignee: Paul Michali (pcm) → nobody
status: In Progress → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for python-neutronclient because there has been no activity for 60 days.]

Changed in python-neutronclient:
status: Incomplete → Expired
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.