GitHub PushEvent at 2014-12-01 16:50:19
Repository: nova
Actor: openstack-gerrit
Payload: {"shas":[["687e8664a2e27b8810ed23fc8db3cdd8bef084c8","6b837bdaf0c015056031915a48623d46c772077a@ericsson.com","Enable pep8 on ./tools directory\n\nThis patch fixes the pep8 issues of the tool scripts and\nconfigures tox to run pep8 on the tools directory.\n\nTwo hacking rules have been modified to ignore the tool directory as both\nrules are checking oslo usage which is not critical in case of tools\n- N310 use timeutils instead of datetime module\n- N324 use jsonutils instead of json module\n\nThe tools/xenserver directory is still excluded from pep8 as it contains\nscripts that are expected to run on dom0 with python2.4 therefore the\nfollowing rules cannot be meaningfully followed\n- H231  Python 3.x incompatible 'except x,y:' construct\n- H233  Python 3.x incompatible use of print operator\n\nChange-Id: Icfd9e708a2ca1e421676b410e41807b2e630718f","Balazs Gibizer",true],["e8b085293cf525d16570afa01e4a07da84e6fe8b","d95b56ce41a2e1ac4cecdd398defd7414407cc08@review.openstack.org","Merge \"Enable pep8 on ./tools directory\"","Jenkins",true]],"size":2,"ref":"refs/heads/master","head":"e8b085293cf525d16570afa01e4a07da84e6fe8b"}