openstack
/
openstack
Code Issues Proposed changes

Update git submodules 

* Update openstack-ansible-os_cinder from branch 'master'
  - Merge "Fix evaluation of cinder_scheduler_default_filters, remove confusing comment"
  - Fix evaluation of cinder_scheduler_default_filters, remove confusing comment
    
    Change I12859167d19b9f40e3378ac08fed094a42f40bc7, merged just today,
    put the evaluation of cinder_scheduler_default_filters inside an
    {% if } block that it wasn't meant to be inside. As a consequence,
    that setting would be ignored more often that it would be
    honored.
    
    What threw me (sorry about that) was that I had tested this patch
    on a Queens box, and stubbornly copied into the template on master.
    
    And changes I02d2bae8712c0ca223cafb5a43304806c4b83125 and
    Ib5a128e82e5251077e341b5f428eb097bcc17590 had left the
    template in a somewhat confusing state: the template had retained the
    "## Cinder API's enabled" comment despite the fact that there were
    no settings left to enable or disable any APIs.
    
    So, with this change the evaluation of cinder_scheduler_default_filters
    goes to the right place (i.e. outside the {% if } block), *and*
    the confusing API comment is removed.
    
    Change-Id: Ic5707615571e62ba2326e2ad436333bac246c8dd
This commit is contained in:
Zuul 2019-02-23 13:19:08 +00:00 committed by Gerrit Code Review
parent ee42852088
commit 3b48f95cc2
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
openstack-ansible-os_cinder

@ -1 +1 @@
Subproject commit 7f528040adfb1e3becb20b9c7aca8af0ea6254ba
Subproject commit 34bb90e7558b5be8eddbe806067372e2d105bd21