[RFE] Allow setting defaults for ansible deploy driver_info parameters in ironic.conf

Bug #1736409 reported by Dmitry Tantsur
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Wishlist
Pavlo Shchelokovskyy
puppet-ironic
Fix Released
Undecided
Dmitry Tantsur

Bug Description

In some cases, especially in single-purpose clouds like TripleO, it make sense to set these options per deployment. This will simplify enrolling nodes in such use cases.

Introduce the following options in the [ansible] section:

 username - default user name
 key_file - path to SSH private key
 deploy_playbook, shutdown_playbook, clean_playbook - default playbooks
 clean_steps_config - file with default clean steps configuration

The values in driver_info will override these values.

Tags: rfe
Changed in ironic:
assignee: nobody → Pavlo Shchelokovskyy (pshchelo)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to ironic-specs (master)

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

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to ironic-specs (master)

Reviewed: https://review.openstack.org/525708
Committed: https://git.openstack.org/cgit/openstack/ironic-specs/commit/?id=9197c5b1624b4db1611b01ab417c493f428e16cb
Submitter: Zuul
Branch: master

commit 9197c5b1624b4db1611b01ab417c493f428e16cb
Author: Pavlo Shchelokovskyy <email address hidden>
Date: Tue Dec 5 19:23:14 2017 +0200

    Amend ansible deploy interface spec

    add proposed config options for default values for "ansible_*"
    driver_info fields supported by this interface.

    Change-Id: I4ed0cedcadbee3c74ae8ccba29dc72804f229ccb
    Related-Bug: #1736409

Dmitry Tantsur (divius)
Changed in puppet-ironic:
assignee: nobody → Dmitry Tantsur (divius)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-ironic (master)

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

Changed in puppet-ironic:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/525710
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=ce7119eaf3e32ac7d27df11c7f0a1f65159d255f
Submitter: Zuul
Branch: master

commit ce7119eaf3e32ac7d27df11c7f0a1f65159d255f
Author: Pavlo Shchelokovskyy <email address hidden>
Date: Tue Dec 5 18:10:52 2017 +0200

    [ansible] add defaults to config

    In a homogeneous hardware deployment that uses the same ramdisk for all
    nodes, and possibly the same playbooks for actions with ansible deploy
    interface, it is benefitial to make defaults of various `ansible_`
    driver_info fields to be configurable via ironic configuration file
    to simplify node enrollment.

    This patch also deprecates `ansible_deploy_username` and
    `ansible_deploy_key_file` options in node's driver_info in favor of
    `ansible_username` and `ansible_key_file` respectively.

    Change-Id: Ib198c07c1d414c0d78950e5d98a5176e12a7df13
    Closes-Bug: #1736409

Changed in ironic:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-ironic (master)

Reviewed: https://review.openstack.org/529599
Committed: https://git.openstack.org/cgit/openstack/puppet-ironic/commit/?id=ca56d8237a70506965e287a3f411705ada2e6da2
Submitter: Zuul
Branch: master

commit ca56d8237a70506965e287a3f411705ada2e6da2
Author: Dmitry Tantsur <email address hidden>
Date: Thu Dec 21 15:29:18 2017 +0100

    ansible: allow configuring default_* configuration parameters

    Change-Id: Icf3304b6d47eb13f2cd214d385d714b6287d6828
    Depends-On: Ib198c07c1d414c0d78950e5d98a5176e12a7df13
    Closes-Bug: #1736409

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

This issue was fixed in the openstack/ironic 10.0.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-ironic 12.2.0

This issue was fixed in the openstack/puppet-ironic 12.2.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.