openstack/ovn-octavia-provider
Code Issues Proposed changes

Fix functional tests job

Browse Source 
This patch fixes functional tests job by doing 2 things:

* Configure correct db user in the functional tests job
  This was changed in the Neutron in commit [1] and should
  be updated also in the configure_functional_tests role in
  this repo.
* Remove run-devstack role from run_functional_job playbook
  There is no need to run that role in the functional
  job and not running it can save few minutes in each job's run.
  The same change was done in Neutron in patch [2].

[1] https://review.opendev.org/c/openstack/neutron/+/814009
[2] https://review.opendev.org/c/openstack/neutron/+/816354

Depends-On: https://review.opendev.org/c/openstack/devstack/+/819402

Closes-bug: #1952357
Change-Id: I0f3f93cdb361575591cd1278da74075d83611cae
This commit is contained in:
Slawek Kaplonski 2021-11-25 21:52:39 +01:00
parent 9665da4c4e
commit 1ee61f955f
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
playbooks/run_functional_job.yaml
View File

@ -1,7 +1,5 @@
- hosts: all - hosts: all
roles: roles:
- run-devstack
# Run bindep and test-setup after devstack so that they won't interfere
- role: bindep - role: bindep
bindep_profile: test bindep_profile: test
bindep_dir: "{{ zuul_work_dir }}" bindep_dir: "{{ zuul_work_dir }}"

2
roles/configure_functional_tests/tasks/main.yaml
View File

@ -15,6 +15,8 @@
STACK_USER=stack STACK_USER=stack
OVS_BRANCH={{ OVS_BRANCH }} OVS_BRANCH={{ OVS_BRANCH }}
OVN_BRANCH={{ OVN_BRANCH }} OVN_BRANCH={{ OVN_BRANCH }}
# This is DB USER used in e.g. pgsql db
DATABASE_USER=openstack_citest
source $DEVSTACK_PATH/functions source $DEVSTACK_PATH/functions
source $DEVSTACK_PATH/lib/neutron_plugins/ovs_source source $DEVSTACK_PATH/lib/neutron_plugins/ovs_source