1) Had to increase the timeout as 120 seconds was not enough. I bumped to 600.
2) the tox target "cittest_run" actually does a "tree_run" In order to not be confused between run versus tree_run I think this should really be "cittest_tree_run"
Tested by running the following on my local system with pylxd 2.1.3:
lxc list was clean afterwards as expected. I believe I only missed a ppa test.
The only warning message I got other than those for pylxd's 2.2 deprecation was:
/home/powersj/Work/repos/cloud-init-wesley/tests/cloud_tests/collect.py:48: DeprecationWarning: The 'warn' method is deprecated, use 'warning' instead
LGTM with two comments:
1) Had to increase the timeout as 120 seconds was not enough. I bumped to 600.
2) the tox target "cittest_run" actually does a "tree_run" In order to not be confused between run versus tree_run I think this should really be "cittest_tree_run"
Tested by running the following on my local system with pylxd 2.1.3:
# lint archive. ubuntu. com/ubuntu/ yakkety main' 0.7.9-0ubuntu1_ all.deb tests/configs/ modules/ write_files. yaml tests/configs/ modules/ timezone. yaml
$ tox
# runs with options
$ python3 -m tests.cloud_tests run -v -n trusty
$ python3 -m tests.cloud_tests run -v -n xenial
$ python3 -m tests.cloud_tests run -v -n yakkety --repo 'deb http://
$ python3 -m tests.cloud_tests run -v -n xenial --deb cloud-init_
# new tox based runs + tree_run
$ tox -e citest_run
$ tox -e citest -- tree_run -n zesty -t tests/cloud_
$ python3 -m tests.cloud_tests tree_run -v -n zesty -t tests/cloud_
lxc list was clean afterwards as expected. I believe I only missed a ppa test.
The only warning message I got other than those for pylxd's 2.2 deprecation was: Work/repos/ cloud-init- wesley/ tests/cloud_ tests/collect. py:48: DeprecationWarning: The 'warn' method is deprecated, use 'warning' instead
/home/powersj/