[fuel-devops] Add support for printing VM ips in dhcp admin-network to dos.py

[Matthew Mosesohn]

fuel-devops dos.py should be able to support the following use case:
. fuel-devops-3.x/bin/activate
export $necessary_vars
dos.py create myenv.yaml
dos.py net-list $myenv
dos.py net-list $myenv --node-ips admin

or alternately
dos.py list-node-ips $myenv admin

It is possible and has been implemented by QA team and former Solar team to call this function in fuel-devops by an external python client. This should reduce code drift and unmaintained standalone implementations.

[Andrey Nikitin (heos)]

Fuel QA team is responsible for the "fuel devops" framework.

[OpenStack Infra (hudson-openstack)]

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

[Dennis Dmitriev (ddmitriev)]

Matthew: merged as-is to unblock you, but the options should be more user-friendly, let's fix it in next patches.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/424147
Committed: https://git.openstack.org/cgit/openstack/fuel-devops/commit/?id=f774496e5d8a74ef5a1c7de581e6d32473b38887
Submitter: Jenkins
Branch: master

commit f774496e5d8a74ef5a1c7de581e6d32473b38887
Author: Anton Studenov <email address hidden>
Date: Tue Jan 24 13:59:53 2017 +0300

    Add logic to show slave node IPS

    New command dos.py slave-ips-list shows ALL IPs assigned
    to ALL nodes in an environment that have dhcp enabled.

    Change-Id: I9c5095dbba55a23660f9f20fab525de538ae8b08
    Closes-Bug: #1658630

[Dennis Dmitriev (ddmitriev)]

The option should be extended to use YAML as a default output format:

* default behaviour:
dos.py slave-ip-list tcpcloud-mk22
admin-pool01:
  cfg01:172.16.10.100
  ctl01:172.16.10.101
  ctl02:172.16.10.102
  ...
private-pool01:
  cfg01:192.168.10.100
  ctl01:192.168.10.101
  ctl02:192.168.10.102
  ...

* filtered by specified address pool:
dos.py slave-ip-list tcpcloud-mk22 --address-pool=admin-pool01
cfg01:172.16.10.100
ctl01:172.16.10.101
ctl02:172.16.10.102
...

* inventory mode:
dos.py slave-ip-list tcpcloud-mk22 --address-pool=admin-pool01 --ip-only
- 172.16.10.100
- 172.16.10.101
- 172.16.10.102
...

* inventory mode (single line for easy parsing from shell):
dos.py slave-ip-list tcpcloud-mk22 --address-pool=admin-pool01 --ip-only-plain
172.16.10.100 172.16.10.101 172.16.10.102 ...

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/436136

[Dmitry Belyaninov (dbelyaninov)]

Please take a look https://bugs.launchpad.net/fuel/+bug/1585261

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/436136
Committed: https://git.openstack.org/cgit/openstack/fuel-devops/commit/?id=9d9b495c5b9ced42a69446d0bf18612d7eb43315
Submitter: Jenkins
Branch: master

commit 9d9b495c5b9ced42a69446d0bf18612d7eb43315
Author: Dennis Dmitriev <email address hidden>
Date: Mon Feb 20 20:24:15 2017 +0200

    Extend 'slave-ip-list' CLI options

    - show address pool names or specify an --address-pool-name=<...>
    - show node names or set --ip-only flag

    Output examples:

    $ dos.py slave-ip-list ac_linear_test
    public-pool01: master,10.100.0.128 slave-0,10.100.0.129 slave-1,10.100.0.130
    neutron-pool01: master,10.100.2.2 slave-0,10.100.2.3 slave-1,10.100.2.4
    private-pool01: master,10.100.1.128 slave-0,10.100.1.129 slave-1,10.100.1.130

    $ dos.py slave-ip-list ac_linear_test --ip-only
    public-pool01: 10.100.0.128 10.100.0.129 10.100.0.130
    neutron-pool01: 10.100.2.2 10.100.2.3 10.100.2.4
    private-pool01: 10.100.1.128 10.100.1.129 10.100.1.130

    $ dos.py slave-ip-list ac_linear_test --address-pool-name=public-pool01
    master,10.100.0.128 slave-0,10.100.0.129 slave-1,10.100.0.130

    $ dos.py slave-ip-list ac_linear_test --address-pool-name=public-pool01 --ip-only
    10.100.0.128 10.100.0.129 10.100.0.130

    Change-Id: I75635f73b90f37d9e50de1772bc7da6720c41cec
    Related-Bug:#1658630