inventory-manage.py should automatically load inventory file

Bug #1454324 reported by Bjoern
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Low
git-harry
Juno
Fix Released
Low
Jesse Pretorius
Kilo
Fix Released
Low
git-harry
Trunk
Fix Released
Low
git-harry

Bug Description

The script should load the inventory the same way as the dynamic inventory generator and we only would need -f to override the file.
That change makes the mange script more user friendly IMHO

Changed in openstack-ansible:
status: New → Confirmed
importance: Undecided → Low
git-harry (git-harry)
Changed in openstack-ansible:
assignee: nobody → git-harry (git-harry)
Changed in openstack-ansible:
status: Confirmed → In Progress
Changed in openstack-ansible:
status: In Progress → Fix Committed
Revision history for this message
Darren Birkett (darren-birkett) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-ansible-deployment (kilo)

Reviewed: https://review.openstack.org/199149
Committed: https://git.openstack.org/cgit/stackforge/os-ansible-deployment/commit/?id=4e6938d719be00c9c984b2a64aeec2ecaeee19ed
Submitter: Jenkins
Branch: kilo

commit 4e6938d719be00c9c984b2a64aeec2ecaeee19ed
Author: git-harry <email address hidden>
Date: Tue Jul 7 11:13:03 2015 +0100

    Set default inventory file for inventory-manage.py

    The script inventory-manage.py takes a file argument. The default name
    for this file, in the project, is openstack_inventory.json. Not only is
    it the default but the rest of the project assumes that it will be that
    name.

    This commit sets the default argument for the file parameter to this
    assumed file name - 'openstack_inventory.json'.

    Change-Id: I635122d22bb0467aad21f48dccda1bc0444518c1
    Closes-bug: #1454324
    (cherry picked from commit 576831481262b93dfdf72f078a4b620a7ac130fb)

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

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

commit 21d32b32f7b2972bc8e71141433dd5494a1dd2fa
Author: git-harry <email address hidden>
Date: Tue Jul 7 11:13:03 2015 +0100

    Set default inventory file for inventory-manage.py

    The script inventory-manage.py takes a file argument. The default name
    for this file, in the project, is rpc_inventory.json. Not only is
    it the default but the rest of the project assumes that it will be that
    name.

    This commit sets the default argument for the file parameter to this
    assumed file name - 'rpc_inventory.json'.

    Change-Id: I635122d22bb0467aad21f48dccda1bc0444518c1
    Closes-bug: #1454324
    (manual backport from commit 576831481262b93dfdf72f078a4b620a7ac130fb)

Revision history for this message
Davanum Srinivas (DIMS) (dims-v) wrote : Fix included in openstack/openstack-ansible 11.2.14

This issue was fixed in the openstack/openstack-ansible 11.2.14 release.

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.