Comment 6 for bug 1851249

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla-ansible (stable/stein)

Reviewed: https://review.opendev.org/692989
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=9df6ee216a30111e744d1a5bce2eea1238b7694f
Submitter: Zuul
Branch: stable/stein

commit 9df6ee216a30111e744d1a5bce2eea1238b7694f
Author: Martin Chlumsky <email address hidden>
Date: Mon Nov 4 11:54:58 2019 -0500

    Fix empty match while setting supported_policy_files

    When supported_policy_files gets set under python 3.7 [1], the regex '(.*)'
    matches twice, once for the policy file name and once more for the empty string
    that follows the policy file name. This is new behavior under python
    3.7. [2]

    This leads to the replacement string being written out twice resulting
    in something like this: "nova_policy.yamlnova_".

    This patch changes the regex to '(.+)' ensuring there is no match success
    against the empty string.

    [1]:
    - set_fact:
        supported_policy_files: "{{ supported_policy_format_list | map('regex_replace', '(.*)', '{{ project_name }}_\\1') | list }}"

    [2]: https://docs.python.org/3/library/re.html#re.sub

    Change-Id: Ie5278832e293364c66d53ddb07dff9c5409f0cc6
    Closes-Bug: 1851249
    (cherry picked from commit 9d0ccad15aa358cdc6dc2f690bf181cb8354d150)