using a listOP with config_template in an INI file will break openstack services

Bug #1543588 reported by Kevin Carter
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
Fix Released
Medium
Kevin Carter
Kilo
Fix Released
Medium
Kevin Carter
Liberty
Fix Released
Medium
Kevin Carter

Bug Description

The config template can be used to override or overload an option into a configuration file. This ability supports string, int, list, and dict formats which are rendered as standard INI values. Sadly OpenStack services can not deal with configuration options that are in list format following the [1,2,3] syntax. These options MUST be in simple comma or new line separated values. To achieve this, the config_template action plugin must load and write list options when using an ini config format as comma separated lists.

Changed in openstack-ansible:
status: Triaged → In Progress
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible-plugins (master)

Reviewed: https://review.openstack.org/277579
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-plugins/commit/?id=852ab822622928f20d963f7ffb450bcce72c5a69
Submitter: Jenkins
Branch: master

commit 852ab822622928f20d963f7ffb450bcce72c5a69
Author: Kevin Carter <email address hidden>
Date: Mon Feb 8 14:26:45 2016 -0600

    Updated config parsing for listOPS

    This change updates the list parsing for all list options passed through the
    config_template module using the INI configuration type. This change is needed
    so that when passing a list into a configuration file for use within an
    OpenStack service oslo.config properlly handles the lists as a comma seperated
    value when rendered.

    Closes-Bug: #1543588
    Change-Id: If8055b3362f1d9d1f274baa447fac094a6a91481
    Signed-off-by: Kevin Carter <email address hidden>

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

Reviewed: https://review.openstack.org/277661
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=43b01a7897bfa1d4d6880b5f62d5110646987b8f
Submitter: Jenkins
Branch: kilo

commit 43b01a7897bfa1d4d6880b5f62d5110646987b8f
Author: Kevin Carter <email address hidden>
Date: Mon Feb 8 20:30:57 2016 -0600

    Updated config parsing for listOPS

    This change updates the list parsing for all list options passed through the
    config_template module using the INI configuration type. This change is needed
    so that when passing a list into a configuration file for use within an
    OpenStack service oslo.config properlly handles the lists as a comma seperated
    value when rendered.

    Closes-Bug: #1543588
    Change-Id: If106ef16349a88f3fcd363927bd4b27d11cbbbf3
    Depends-On: If8055b3362f1d9d1f274baa447fac094a6a91481
    Signed-off-by: Kevin Carter <email address hidden>

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

Reviewed: https://review.openstack.org/277660
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible/commit/?id=864ba64c6c74c9481d8441c143c36387beb70fb5
Submitter: Jenkins
Branch: liberty

commit 864ba64c6c74c9481d8441c143c36387beb70fb5
Author: Kevin Carter <email address hidden>
Date: Mon Feb 8 20:30:57 2016 -0600

    Updated config parsing for listOPS

    This change updates the list parsing for all list options passed through the
    config_template module using the INI configuration type. This change is needed
    so that when passing a list into a configuration file for use within an
    OpenStack service oslo.config properlly handles the lists as a comma seperated
    value when rendered.

    Closes-Bug: #1543588
    Change-Id: If106ef16349a88f3fcd363927bd4b27d11cbbbf3
    Depends-On: If8055b3362f1d9d1f274baa447fac094a6a91481
    Signed-off-by: Kevin Carter <email address hidden>

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

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

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 12.0.8

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

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 11.2.12

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

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 12.0.9

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

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

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

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.

Revision history for this message
Doug Hellmann (doug-hellmann) wrote : Fix included in openstack/openstack-ansible 11.2.15

This issue was fixed in the openstack/openstack-ansible 11.2.15 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.