[RFE][tripleo-validations] tripleo-ansible-inventory can only group computes and controllers

Bug #1685544 reported by John Fulton
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
Wishlist
Florian Fuchs

Bug Description

The tripleo-ansible-inventory in the tripleo-validations repository is only able to support the Ansible host groups control or compute [1]. Would it benefit the tripleo-validations project if it could also do the following?

1. It could create an Ansible host group for any role that a user could compose?
2. It could determine which nodes fit a role based on services whose name matches a pattern?

If so I could work on a patch to tripleo-ansible-inventory to make it do this and track that effort under this bug. However, I want to check with the tripleo-validations community first to be sure it would be a useful patch.

One place it would be useful outside of tripleo-validations is for the Ansible inventory portion of this spec:

 https://specs.openstack.org/openstack/tripleo-specs/specs/pike/tripleo-ceph-ansible-integration.html

I want to use the tripleo-validations tripleo-ansible-inventory so that ceph-ansible can configure different Ceph services on any overcloud node which is running a Ceph service (including whatever role or hostgroup a user might have composed). This process would be similar to running:

 grep -i ceph /usr/share/openstack-tripleo-heat-templates/roles_data.yaml

and then setting up Ansible inventory aliases for mons, osds, and other host groups that ceph-ansible uses [2].

Feel free to close this RFE if it wouldn't be of use or assign it to me and I will follow up with you.

[1] https://github.com/openstack/tripleo-validations/blob/master/scripts/tripleo-ansible-inventory#L150-L165
[2] https://github.com/ceph/ceph-ansible/blob/master/site.yml.sample

Tags: validations
Changed in tripleo:
milestone: none → pike-2
importance: Undecided → Wishlist
tags: added: validations
Changed in tripleo:
status: New → Triaged
Revision history for this message
John Fulton (jfulton-org) wrote :

I see a patch related to this RFE already in the works at https://review.openstack.org/#/c/450233/

Changed in tripleo:
assignee: nobody → Florian Fuchs (flo-fuchs)
status: Triaged → In Progress
Revision history for this message
John Fulton (jfulton-org) wrote :

I see the second part of this bug:

> 2. It could determine which nodes fit a role based on services whose name matches a pattern?

is partially addressed in the following:

 https://review.openstack.org/#/c/457972/

perhaps pattern matching isn't necessary if all subsets of nodes are created per service.

Changed in tripleo:
assignee: Florian Fuchs (flo-fuchs) → Steven Hardy (shardy)
Changed in tripleo:
assignee: Steven Hardy (shardy) → Florian Fuchs (flo-fuchs)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-validations (master)

Reviewed: https://review.openstack.org/450233
Committed: https://git.openstack.org/cgit/openstack/tripleo-validations/commit/?id=9461adfa244793cba91f87fcee25a6887a9b7368
Submitter: Jenkins
Branch: master

commit 9461adfa244793cba91f87fcee25a6887a9b7368
Author: Florian Fuchs <email address hidden>
Date: Mon Mar 27 15:47:36 2017 +0200

    Show all roles in inventory

    This patch updates the tripleo-ansible-inventory to show all roles in
    the host list, not just compute and controller nodes.

    The retrieval of IPs and stack outputs is slightly refactored: Instead of
    retrieving them from Nova, the host IPs are now read from the stacks new
    RoleNetIpMap output. The stack outputs are fetched from heat via a new
    `StackOutputs` class that ensures outputs are only fetched when needed.

    Closes-Bug: #1685544
    Closes-Bug: #1689789
    Change-Id: Ic8a19f15dd45b383a3f3d403653e3eaef2c1865b

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-validations 7.1.0

This issue was fixed in the openstack/tripleo-validations 7.1.0 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.