gate: ensure that aio script works with all branches

Bug #1413606 reported by Jesse Pretorius
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Low
Jesse Pretorius
Icehouse
Fix Released
High
Jesse Pretorius
Juno
Fix Released
Low
Jesse Pretorius

Bug Description

The os-ansible-aio-check script makes use of /etc/rpc_deploy/conf.d/ which does not exist (nor is supported) currently in the icehouse branch. As this script is used for all gating and the script used should ideally be consistent between branches, it would be best if the script could be adjusted to work regardless of the branch used.

Changed in openstack-ansible:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-ansible-deployment (master)

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

Changed in openstack-ansible:
importance: High → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on os-ansible-deployment (icehouse)

Change abandoned by Jesse Pretorius (<email address hidden>) on branch: icehouse
Review: https://review.openstack.org/149213
Reason: This patch was an investigation to figure out what needed to go into the branch in order to get the gate operational. Now that we know, we can cherry-pick the patches individually.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (master)

Reviewed: https://review.openstack.org/149279
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=8a21f096577a20be1aafdb2e3656cf5b06ff2f90
Submitter: Jenkins
Branch: master

commit 8a21f096577a20be1aafdb2e3656cf5b06ff2f90
Author: Jesse Pretorius <email address hidden>
Date: Thu Jan 22 14:29:00 2015 +0000

    Adjust os-ansible-aio-check to work for all branches

    The icehouse branch does not currently have support for using the conf.d
    directory for additional user configuration. This patch implements all
    the configuration, except swift (which is not available in the icehouse
    branch) into a monolothic user configuration file. This ensures
    compatibility of the configuration process for all branches.
    This patch also ensures that the plays executed are appropriate for the
    branches - the icehouse branch has different names for some of the
    plays.

    Change-Id: Iffad2ffce93e8173da68863210afba0aa105622d
    Closes-Bug: #1413606

Changed in openstack-ansible:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-ansible-deployment (icehouse)

Fix proposed to branch: icehouse
Review: https://review.openstack.org/149676

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-ansible-deployment (juno)

Fix proposed to branch: juno
Review: https://review.openstack.org/149681

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (juno)

Reviewed: https://review.openstack.org/149681
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=52913558fdad7392f01ab13d19c47a5ec600e1f7
Submitter: Jenkins
Branch: juno

commit 52913558fdad7392f01ab13d19c47a5ec600e1f7
Author: Jesse Pretorius <email address hidden>
Date: Thu Jan 22 14:29:00 2015 +0000

    Adjust os-ansible-aio-check to work for all branches

    The icehouse branch does not currently have support for using the conf.d
    directory for additional user configuration. This patch implements all
    the configuration, except swift (which is not available in the icehouse
    branch) into a monolothic user configuration file. This ensures
    compatibility of the configuration process for all branches.
    This patch also ensures that the plays executed are appropriate for the
    branches - the icehouse branch has different names for some of the
    plays.

    Change-Id: Iffad2ffce93e8173da68863210afba0aa105622d
    Closes-Bug: #1413606
    (cherry picked from commit 8a21f096577a20be1aafdb2e3656cf5b06ff2f90)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (icehouse)

Reviewed: https://review.openstack.org/149676
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=24c2e9e224d3562e9a8a4338f2ff15189c04a12b
Submitter: Jenkins
Branch: icehouse

commit 24c2e9e224d3562e9a8a4338f2ff15189c04a12b
Author: Jesse Pretorius <email address hidden>
Date: Thu Jan 22 14:29:00 2015 +0000

    Adjust os-ansible-aio-check to work for all branches

    The icehouse branch does not currently have support for using the conf.d
    directory for additional user configuration. This patch implements all
    the configuration, except swift (which is not available in the icehouse
    branch) into a monolothic user configuration file. This ensures
    compatibility of the configuration process for all branches.
    This patch also ensures that the plays executed are appropriate for the
    branches - the icehouse branch has different names for some of the
    plays.

    Change-Id: Iffad2ffce93e8173da68863210afba0aa105622d
    Closes-Bug: #1413606
    (cherry picked from commit 8a21f096577a20be1aafdb2e3656cf5b06ff2f90)

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