VPNaaS: Revise functional jobs to only setup DevStack

Bug #1445052 reported by Paul Michali
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Paul Michali

Bug Description

In a manner similar to what is done for Neutron, the two functional jobs need to be modified to only set up DevStack, and not actually run DevStack.

Tags: vpnaas
Paul Michali (pcm)
Changed in neutron:
assignee: nobody → Paul Michali (pcm)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron-vpnaas (master)

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

commit d7f65a2dbce4d25b1c5d76500a5365da5f9dfffd
Author: Paul Michali <email address hidden>
Date: Thu Mar 26 14:52:07 2015 -0400

    VPNaaS: Revise functional test hooks

    This commit does several things to allow functional tests to run, using a
    DevStack configuration, but without stacking.

    First, the gate_hook.sh is modified to specified environment variables
    needed for processing, and will call configure_vpn_for_func_testing.sh.

    The new configure_for_vpn_func_testing.sh script will first call the Neutron
    configure_for_func_testing.sh script (as modified in [1]) to setup DevStack.
    The script then installs the IPSec package, based on the functional job, and
    will create the vpn_agent.ini file, in case it is needed for tests.

    Minor changes are made to the post-test_hook.sh to indicate the location for
    the Neutron and VPN repos.

    Lastly, tox.ini is modified to place the rootwrap related files into the
    virtual environment, copy in the VPN rootwrap filter, set up the rootwrap
    commands, and copy the (customized) rootwrap.conf to /etc/neutron.

    This last step is needed, because the neutron-vpn-netns-wrapper script that
    is (currently) used for StrongSwan, uses /etc/neutron/rootwrap.conf and
    doesn't allow overriding. This copying is a simple approach that will work
    when multiple functional jobs are running, as the same config is used.

    As a future follow-up commit, we could look into modifying the VPN device
    drivers to pass the full path to the rootwrap.conf as a argument to the
    neutron-vpn-netns-wrapper script (--rootwrap_config). The setting could
    be added to the vpnagent.ini config file, available via config, to allow
    this to be customized to something other than /etc/neutron (for production
    use), and the functional tests could override it to point to the right
    virtual environment.

    To configure the environment for local run of the functional test, one
    can do:
        tools/configure_for_vpn_func_testing.sh <devstack-repo> [-i]

    where:
       devstack-repo... Path to devstack repo (/opt/stack/new/devstack
                        when run by gate)
       -i.............. Indicates to install Neutron package dependencies

    For StrongSwan, prefix the command with VENV=dsvm-functional-sswan

    Ref: [1] https://review.openstack.org/#/c/176064/

    Change-Id: I2243b06b3cdbb352dac7bc47d03c830efb87be3c
    Closes-Bug: #1445052
    Depends-On: Ie490e89c1a65e126ad0a57d062076e472762837e

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
milestone: none → liberty-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: liberty-1 → 7.0.0
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.