[QA][CI] The job 'gate-fuel-qa-pylint' fails with new version of pylint

Bug #1600132 reported by Artem Panchenko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Artem Panchenko
Mitaka
Fix Released
High
Artem Panchenko

Bug Description

New version of pylint 1.6 contains new checkers:

https://github.com/PyCQA/pylint/blob/pylint-1.6.0/ChangeLog#L55-L56

This causes failure of 'fuel-qa' gate job:

2016-07-08 07:02:06.368468 | ************* Module fuelweb_test.testrail.report
2016-07-08 07:02:06.368654 | C: 76,17: Consider iterating the dictionary directly instead of calling .keys() (consider-iterating-dictionary)
2016-07-08 07:02:06.368689 | ************* Module fuelweb_test.testrail.generate_failure_group_statistics
2016-07-08 07:02:06.368726 | C:443,32: Consider iterating the dictionary directly instead of calling .keys() (consider-iterating-dictionary)
2016-07-08 07:02:06.368769 | C:444,20: Consider iterating the dictionary directly instead of calling .keys() (consider-iterating-dictionary)
2016-07-08 07:02:06.368788 | ************* Module fuelweb_test.helpers.checkers
2016-07-08 07:02:06.368821 | C:511,21: Consider iterating the dictionary directly instead of calling .keys() (consider-iterating-dictionary)
2016-07-08 07:02:06.368854 | C:571,24: Consider iterating the dictionary directly instead of calling .keys() (consider-iterating-dictionary)
2016-07-08 07:02:06.368887 | C:587,26: Consider iterating the dictionary directly instead of calling .keys() (consider-iterating-dictionary)
2016-07-08 07:02:06.368924 | ************* Module fuelweb_test.helpers.patching
2016-07-08 07:02:06.368958 | C:399,15: Consider iterating the dictionary directly instead of calling .keys() (consider-iterating-dictionary

Tags: area-qa
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (master)

Fix proposed to branch: master
Review: https://review.openstack.org/339391

Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Artem Panchenko (apanchenko-8)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (master)

Reviewed: https://review.openstack.org/339391
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=fa2d058d49a6c3786db50ee24a459e43ee43200b
Submitter: Jenkins
Branch: master

commit fa2d058d49a6c3786db50ee24a459e43ee43200b
Author: Artem Panchenko <email address hidden>
Date: Fri Jul 8 10:23:16 2016 +0300

    Don't use .keys() method while iterating over dict

    Closes-bug: #1600132

    Change-Id: I49ee06d795280f572db07c9537310433f07bd0ab

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/339483

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/mitaka)

Reviewed: https://review.openstack.org/339483
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=e3c61048d37270d0ef20d85e0cc39e88e79cf5b5
Submitter: Jenkins
Branch: stable/mitaka

commit e3c61048d37270d0ef20d85e0cc39e88e79cf5b5
Author: Artem Panchenko <email address hidden>
Date: Fri Jul 8 10:23:16 2016 +0300

    Don't use .keys() method while iterating over dict

    Closes-bug: #1600132

    Change-Id: I49ee06d795280f572db07c9537310433f07bd0ab
    (cherry picked from commit fa2d058d49a6c3786db50ee24a459e43ee43200b)

Changed in fuel:
status: Fix Committed → Fix Released
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.