StarlingX

  1. Bug #1952400
  2. Comment #11

Comment 11 for bug 1952400

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

Reviewed: https://review.opendev.org/c/starlingx/config/+/820180
Committed: https://opendev.org/starlingx/config/commit/f5d836d161f11aa451a66e2c5f01821411932e09
Submitter: "Zuul (22348)"
Branch: master

commit f5d836d161f11aa451a66e2c5f01821411932e09
Author: Thiago Brito <email address hidden>
Date: Thu Dec 2 11:50:45 2021 -0300

    Fixing variable renaming

    On commit [1] the 3rd parameter for _find_ilvg() was renamed to fix a
    lint issue, but the usage of that parameter wasn't correctly renamed and
    a problem arised since there is a class in the same module with that
    name (that was being shadowed before). This commit fixes that and add
    some unitests for this method so we don't merge this kind of error in
    the future.

    [1] https://opendev.org/starlingx/config/commit/5923349485da0ea32042db1c60b0303f7d12e36d

    Partial-Bug: #1952400
    Signed-off-by: Thiago Brito <email address hidden>
    Change-Id: I328c3f5976f1903e621ff170a9da1a929983af18