Over-indented Pep8 issue hit during keystone build

Bug #1813926 reported by Shrey bhatnagar
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Invalid
Undecided
Shrey bhatnagar

Bug Description

There are few lines of code that are "over-indented" in below files,
1) keystone/tests/unit/backend/core_ldap.py
2) keystone/identity/core.py
3) keystone/cmd/cli.py

Please find below the error that we see during build:
./keystone/tests/unit/backend/core_ldap.py:30:9: E117 over-indented (comment)
        # Create the groups base entry (ou=Groups,cn=example,cn=com)
        ^
./keystone/tests/unit/backend/core_ldap.py:31:9: E117 over-indented
        group_api = identity_api.driver.group
        ^
./keystone/tests/unit/backend/core_ldap.py:68:13: E117 over-indented
            user_dn = (
            ^
./keystone/identity/core.py:750:21: E117 over-indented
                    LOG.warning('Found multiple domains being mapped to a '
                    ^
./keystone/cmd/cli.py:967:25: E117 over-indented
                        sections[group][option] = sections[group][option][0]
                        ^

Changed in keystone:
assignee: nobody → Shrey bhatnagar (shrebhat)
Revision history for this message
Colleen Murphy (krinkle) wrote :

Hi Shrey, this should already be addressed by https://review.openstack.org/633800

You can recheck any patches you have open and the CI error should be gone now, or rebase your local checkout against latest master.

Colleen Murphy (krinkle)
Changed in keystone:
status: New → Invalid
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.