Unexpected update in setup script for functional test

Bug #1879303 reported by Yasufumi Ogawa
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tacker
Fix Released
Low
Naoaki Horie

Bug Description

There are some placeholder files for running functional test in zuul. The contents of the files, such as `auth_url `, are updated while running devstack `stack.sh`.

$ git status
    modified: tacker/tests/etc/samples/local-vim.yaml
    modified: tools/test-setup-default-vim.sh

There are not expected changes included in git. It is annoying while developing, or might make something mistakes.

Yasufumi Ogawa (yasufum)
Changed in tacker:
importance: Undecided → Low
assignee: nobody → Yasufumi Ogawa (yasufum)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tacker (master)

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

Changed in tacker:
assignee: Yasufumi Ogawa (yasufum) → Naoaki Horie (naoaki-horie)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tacker (master)

Reviewed: https://review.opendev.org/733721
Committed: https://git.openstack.org/cgit/openstack/tacker/commit/?id=fa373795ba1000eb6ee369a21b412c5d32826e0f
Submitter: Zuul
Branch: master

commit fa373795ba1000eb6ee369a21b412c5d32826e0f
Author: Naoaki Horie <email address hidden>
Date: Fri Jun 5 00:14:46 2020 +0000

    Unexpected update in setup script for functional test

    There are some placeholder files for running functional test in zuul.
    The contents of the files, such as `auth_url` of the following files,
    are updated while running devstack `stack.sh`.

    * tacker/tests/etc/samples/local-vim.yaml
    * tools/test-setup-default-vim.sh

    These Git management files including the above, should not be
    modified. Therefore, I will be fix as follow.

    The need files to be change for Zuul are deploy by Ansible,
    so the deploy process is run with Ansible instead of the
    `devstack/lib/tacker` script.

    Allow `tools/test-setup-default-vim.sh` script to be executed
    by itself, so the git management files aren't updated by running
    devstack `stack.sh`.

    Also, as other improvements, replace the file path included in
    Ansible role to definition value.

    Change-Id: Iad88c7adfe56e926ee0324d94787577ba066989b
    Closes-Bug: #1879303

Changed in tacker:
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.