Rally

rally verify start convert keys and values to lowercase

Bug #1881456 reported by Xing Zhang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Rally
Fix Released
Undecided
Xing Zhang

Bug Description

`rally verify start` converts options to lower case which case some tests runs with unexpected result. Only run one or more test can reproduce.

Here is an example:

octavia-tempest-plugin has an option RBAC_test_type in load_balancer section:
[load_balancer]
RBAC_test_type = owner_or_admin

rally verify configure-verifier --extend extra-options.conf (Fixed in https://bugs.launchpad.net/rally/+bug/1877930)

grep -i rbac_ .rally/verification/verifier-59e386d2-7c5a-4ba1-b36b-bba1d4e5a7e4/for-deployment-b3aac897-c7e2-4797-b92c-ed26fd1fe0d9/tempest.conf
RBAC_test_type = owner_or_admin
(UUID may different in your environment)

rally verify start --pattern some_pattern

grep -i rbac_ .rally/verification/verifier-59e386d2-7c5a-4ba1-b36b-bba1d4e5a7e4/for-deployment-b3aac897-c7e2-4797-b92c-ed26fd1fe0d9/tempest.conf
rbac_test_type = owner_or_admin

Xing Zhang (xingzhang)
Changed in rally:
assignee: nobody → Xing Zhang (xingzhang)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to rally-openstack (master)

Reviewed: https://review.opendev.org/732002
Committed: https://git.openstack.org/cgit/openstack/rally-openstack/commit/?id=7b5bb0b4369245148b4dd8948a96fdc202dcb0cc
Submitter: Zuul
Branch: master

commit 7b5bb0b4369245148b4dd8948a96fdc202dcb0cc
Author: Xing Zhang <email address hidden>
Date: Sun May 31 11:08:15 2020 +0800

    Make config parser case sensitivity in rally-openstack

    rally verify start will convert config options to lowercase
    by using configparser, which cannot set the right config
    for oslo.config.

    Options in config file should be case sensitive, some projects
    like octavia-tempest-plugin can't set some value as expected:
    [load_balancer]
    RBAC_test_type = owner_or_admin

    Use `conf.optionxform = str` to prevent case transformation[1].

    [1] https://docs.python.org/3/library/configparser.html

    Change-Id: I553c962d40ab73e50c1e3f5b229ab553c17fda55
    Closes-Bug: #1881456

Changed in rally:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.