curtin

Code review comment for ~falcojr/curtin:update-manager-in-docs

  1. Git
  2. lp:~falcojr/curtin
  3. update-manager-in-docs
  4. Merge into master
Revision history for this message
Ryan Harper (raharper) wrote :

py3-flake8 runtests: commands[0] | /jenkins/servers/server/workspace/curtin-autoland-test/nodes/metal-amd64/curtin/.tox/py3-flake8/bin/python -m flake8 curtin tests/
curtin/block/bcache.py:322:31: E741 ambiguous variable name 'l'
curtin/block/bcache.py:323:30: E741 ambiguous variable name 'l'
curtin/block/lvm.py:26:40: E741 ambiguous variable name 'l'
curtin/block/__init__.py:1319:17: E741 ambiguous variable name 'l'
tests/unittests/test_block_dasd.py:20:21: E741 ambiguous variable name 'l'
tests/unittests/test_apt_source.py:961:26: E741 ambiguous variable name 'l'
tests/vmtests/test_fs_battery.py:168:27: E741 ambiguous variable name 'l'
tests/vmtests/test_network.py:111:28: E741 ambiguous variable name 'l'

is what autoland is showing.

@Chad we need to manually queue James's branch in CI, or add him to the correct lp group to have automatic CI.

I cannot reproduce the tox failure locally so something is different for autolander tox =(

« Back to merge proposal