Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:coverage into cloud-init:master

2016-10-20 Thread Joshua Powers
Example tox run: https://paste.ubuntu.com/23356016/ -- https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/308975 Your team cloud init development team is requested to review the proposed merge of ~powersj/cloud-init:coverage into cloud-init:master. ___

[Cloud-init-dev] [Merge] ~powersj/cloud-init:coverage into cloud-init:master

2016-10-20 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:coverage into cloud-init:master. Requested reviews: cloud init development team (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/308975 Adds in coverage collections to cloud

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/1635350-tests-read-cfg.d into cloud-init:master

2016-10-24 Thread Joshua Powers
Review: Approve Can confirm this merge request fixes the issue that I was seeing. I can now build as a non-root user as expected. -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/309124 Your team cloud init development team is requested to review the proposed merge of ~sm

[Cloud-init-dev] [Merge] ~powersj/cloud-init:integration-testing into cloud-init:master

2016-12-08 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:integration-testing into cloud-init:master has been updated. Description changed to: * initial commit of integration test framework * added tests.rst to Read the Docs and updated index.rst * updated tox.ini to not run our integration unit test scripts wh

Re: [Cloud-init-dev] [Merge] ~wesley-wiedenmeier/cloud-init:integration-testing-invocation-cleanup into cloud-init:master

2017-01-11 Thread Joshua Powers
Review: Approve 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

Re: [Cloud-init-dev] [Merge] ~wesley-wiedenmeier/cloud-init:integration-testing-invocation-cleanup into cloud-init:master

2017-01-12 Thread Joshua Powers
Review: Approve LGTM Thanks for the changes! Ran the following successfully with no changes to timeouts: $ tox $ tox -e citest_tree_run $ tox -e citest -- tree_run -n zesty -t tests/cloud_tests/configs/modules/write_files.yaml $ python3 -m tests.cloud_tests tree_run -v -n zesty -t tests/cloud

Re: [Cloud-init-dev] [Merge] ~wesley-wiedenmeier/cloud-init:integration-testing into cloud-init:master

2017-01-17 Thread Joshua Powers
@wesley, per the commit message you enabled additional distros. It looks like wheezy, jessie and centros66, and centos70. Do these require the newer version of pylxd for running? Should they work as expected? Trying to understand how to test these. -- https://code.launchpad.net/~wesley-wiedenme

Re: [Cloud-init-dev] [Merge] ~wesley-wiedenmeier/cloud-init:integration-testing into cloud-init:master

2017-01-18 Thread Joshua Powers
Review: Approve LGTM, with two minor comments below. Reusing images has really sped up execution and printing the cloud-init version is great. I ran through the following tests: $ tox $ tox -e citest -- run -v -n zesty -t tests/cloud_tests/configs/modules/write_files.yaml $ tox -e citest_new_p

[Cloud-init-dev] [Merge] ~powersj/cloud-init:make-tox-revamp into cloud-init:master

2017-01-30 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:make-tox-revamp into cloud-init:master. Commit message: Remove style checking during build and add latest style checks to tox - make check will no longer run the style checks - created style-check make file target to continue to run pep8

[Cloud-init-dev] [Merge] ~powersj/cloud-init:pycodestyle into cloud-init:master

2017-01-31 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:pycodestyle into cloud-init:master. Commit message: pycodestyle cleanup Requested reviews: cloud init development team (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/316042

[Cloud-init-dev] [Merge] ~powersj/cloud-init:jlp into cloud-init:master

2017-02-23 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:jlp into cloud-init:master. Requested reviews: cloud init development team (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/318152 ~=~=~ DO NOT MERGE ~=~=~ -- Your team cloud

[Cloud-init-dev] [Merge] ~powersj/cloud-init:update_cii_tox_docs into cloud-init:master

2017-03-10 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:update_cii_tox_docs into cloud-init:master has been updated. Commit Message changed to: doc: add dependencies and tox instructions for integration tests Add instructions on how to run the cii tests via tox as well as the basic system dependencies for ru

[Cloud-init-dev] [Merge] ~powersj/cloud-init:update_cii_tox_docs into cloud-init:master

2017-03-10 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:update_cii_tox_docs into cloud-init:master. Commit message: doc: add dependencies and tox instructions for integration tests Add instructions on how to run the cii tests via tox as well as the basic system dependencies for running the cii

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:update_cii_tox_docs into cloud-init:master

2017-03-10 Thread Joshua Powers
tox -e doc output: https://paste.ubuntu.com/24153451/ -- https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/319617 Your team cloud init development team is requested to review the proposed merge of ~powersj/cloud-init:update_cii_tox_docs into cloud-init:master. __

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:update_cii_tox_docs into cloud-init:master

2017-03-10 Thread Joshua Powers
I am fine with removing the dependency section, running via tox is much simpler anyway. Update made, only have the run via tox method now. -- https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/319617 Your team cloud init development team is requested to review the proposed merg

[Cloud-init-dev] [Merge] ~powersj/cloud-init:update_cii_tox_docs into cloud-init:master

2017-03-10 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:update_cii_tox_docs into cloud-init:master has been updated. Commit Message changed to: doc: add running via tox instructions for integration tests For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/319617 -- Y

[Cloud-init-dev] [Merge] ~powersj/cloud-init:gitignore-tox-doc-html into cloud-init:master

2017-03-10 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:gitignore-tox-doc-html into cloud-init:master. Commit message: gitignore: adding doc/rtd_html When running tox -e doc it produces the complete read the docs output and places it in doc/rtd_html. This is not ignored by git and shows up as

[Cloud-init-dev] [Merge] ~powersj/cloud-init:test_timezone into cloud-init:master

2017-03-13 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:test_timezone into cloud-init:master. Commit message: test: add daylight savings timezone as valid Requested reviews: cloud init development team (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git

[Cloud-init-dev] [Merge] ~powersj/cloud-init:test-set-password-list into cloud-init:master

2017-03-14 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:test-set-password-list into cloud-init:master has been updated. Status: Work in progress => Superseded For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/319877 -- Your team cloud init development team is re

[Cloud-init-dev] [Merge] ~powersj/cloud-init:test-set-password-list into cloud-init:master

2017-03-14 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:test-set-password-list into cloud-init:master. Commit message: test: Add test for setting password as list This adds an integration test for setting passwords when given as a list, rather than a string. This also updates the docs and tests

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:test-set-password-list into cloud-init:master

2017-03-16 Thread Joshua Powers
Pulled in smoser's changes from https://paste.ubuntu.com/24191229/ and applied them to both set_password_list and set_password_list_string -- https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/319878 Your team cloud init development team is requested to review the proposed merg

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:test-set-password-list into cloud-init:master

2017-03-17 Thread Joshua Powers
I added class/subclass method you proposed, which I like a lot. There is no reason for duplicate code. My only concern is base.py filling up with a bunch of code, but I think that is ok. However, I had to modify the parameters slightly so nose actually realizes there are tests to run and added

[Cloud-init-dev] [Merge] ~powersj/cloud-init:1674317_fix_is_file into cloud-init:master

2017-03-20 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:1674317_fix_is_file into cloud-init:master. Commit message: net: Fix incorrect call to isfile Previous commit 5beecdf introduced a regression by calling os.path.is_file, a non-existent function. This changes that call to use os.path.isfile

Re: [Cloud-init-dev] [Merge] ~bbaude/cloud-init:azure_bounce into cloud-init:master

2017-03-20 Thread Joshua Powers
After adding commit 9040e78, this ran integration tests successfully: https://paste.ubuntu.com/24217659/ -- https://code.launchpad.net/~bbaude/cloud-init/+git/cloud-init/+merge/320411 Your team cloud init development team is requested to review the proposed merge of ~bbaude/cloud-init:azure_boun

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:fix-test_netconfig into cloud-init:master

2017-03-21 Thread Joshua Powers
Review: Approve Tested and working on torkoal. Thanks! -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/320495 Your team cloud init development team is requested to review the proposed merge of ~smoser/cloud-init:fix-test_netconfig into cloud-init:master.

[Cloud-init-dev] [Merge] ~powersj/cloud-init:enable-pylint into cloud-init:master

2017-03-21 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:enable-pylint into cloud-init:master. Commit message: test: enable pylint support in tox Requested reviews: cloud init development team (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud

[Cloud-init-dev] [Merge] ~powersj/cloud-init:enable-snap into cloud-init:master

2017-03-24 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:enable-snap into cloud-init:master. Commit message: snap: allows for creating cloud-init snap Add a basic snapcraft.yaml file to allow the creation of cloud-init as a snap. This will always pull down the latest source from master for the

[Cloud-init-dev] [Merge] ~powersj/cloud-init:enable-snap into cloud-init:master

2017-03-24 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:enable-snap into cloud-init:master has been updated. Description changed to: To test: 1. git clone -b enable-snap git+ssh://git.launchpad.net/~powersj/cloud-init 2. cd cloud-init 3. Because the required setup.py change is not in master, you need to edit

[Cloud-init-dev] [Merge] ~powersj/cloud-init:change-pylint-style into cloud-init:master

2017-04-06 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:change-pylint-style into cloud-init:master. Commit message: pylint: update the output style to be easier to read Requested reviews: cloud init development team (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-log-warn into cloud-init:master

2017-04-07 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:fix-log-warn into cloud-init:master has been updated. Status: Merged => Needs review For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/322137 -- Your team cloud init development team is requested to review

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-log-warn into cloud-init:master

2017-04-07 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:fix-log-warn into cloud-init:master has been updated. Status: Work in progress => Merged For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/322137 -- Your team cloud init development team is requested to rev

[Cloud-init-dev] [Merge] ~powersj/cloud-init:pylint-fix-backslash into cloud-init:master

2017-04-07 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:pylint-fix-backslash into cloud-init:master. Requested reviews: cloud init development team (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/322164 pylint: fix various

[Cloud-init-dev] [Merge] ~powersj/cloud-init:pylint-warnings-cleanup into cloud-init:master

2017-04-07 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:pylint-warnings-cleanup into cloud-init:master. Requested reviews: cloud init development team (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/322163 Cleanup of various pylint

[Cloud-init-dev] [Merge] ~powersj/cloud-init:add-rtd-examples into cloud-init:master

2017-04-10 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:add-rtd-examples into cloud-init:master. Commit message: Doc: add additional RTD examples Includes missing examples for RTD LP: #1459604 Requested reviews: cloud init development team (cloud-init-dev) Related bugs: Bug #159604 in

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-debian-backports into cloud-init:master

2017-04-10 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:fix-debian-backports into cloud-init:master has been updated. Commit Message changed to: template: Update debian backports template Debian backports are contains in the main repo and not somewhere seperate. Thanks to Charles Plessy. LP: #1627293 For m

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-debian-backports into cloud-init:master

2017-04-10 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix-debian-backports into cloud-init:master. Commit message: template: Update debian backports template Debian backports are contains in the main repo and not somewhere seperate. Thanks to Charles Plessy. LP: #1627293 Requested reviews

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-rsyslog into cloud-init:master

2017-04-10 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix-rsyslog into cloud-init:master. Commit message: rsyslog: replace ~ with stop The discard action (tilde character) has been replaced by the “stop” RainerScript directive. It is considered more intuitive and offers slightly better

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/1636531-altcloud-test-used-blkid into cloud-init:master

2017-04-11 Thread Joshua Powers
Review: Approve LGTM -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/322386 Your team cloud init development team is requested to review the proposed merge of ~smoser/cloud-init:bug/1636531-altcloud-test-used-blkid into cloud-init:master. ___

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-log-warn into cloud-init:master

2017-04-12 Thread Joshua Powers
ok so what I can do is propose a new pylintrc that excludes all the other warning numbers, except these two. And as I fix more warnings remove them from the excludes list. Something like: [MESSAGES CONTROL] # Errors & warnings only # W0223 - abstract-method # W0511 - fixme # W0612 - unused-vari

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:pylint-fix-backslash into cloud-init:master

2017-04-12 Thread Joshua Powers
Diff comments: > diff --git a/cloudinit/config/cc_apt_configure.py > b/cloudinit/config/cc_apt_configure.py > index 06804e8..a1ceb7c 100644 > --- a/cloudinit/config/cc_apt_configure.py > +++ b/cloudinit/config/cc_apt_configure.py > @@ -6,7 +6,7 @@ > # > # This file is part of cloud-init. See

[Cloud-init-dev] [Merge] ~powersj/cloud-init:pylint-warnings-cleanup into cloud-init:master

2017-04-13 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:pylint-warnings-cleanup into cloud-init:master has been updated. Status: Needs review => Work in progress For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/322163 -- Your team cloud init development team is

[Cloud-init-dev] [Merge] ~powersj/cloud-init:pylint-fix-backslash into cloud-init:master

2017-04-13 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:pylint-fix-backslash into cloud-init:master has been updated. Status: Needs review => Work in progress For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/322164 -- Your team cloud init development team is re

[Cloud-init-dev] [Merge] ~powersj/cloud-init:enable-artful-int into cloud-init:master

2017-04-26 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:enable-artful-int into cloud-init:master. Commit message: tests: enable Ubuntu 17.10 'artful' in integration tests. Requested reviews: cloud init development team (cloud-init-dev) For more details, see: https://code.lau

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-centos-unittests into cloud-init:master

2017-04-27 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix-centos-unittests into cloud-init:master. Commit message: unittests: fix unittests run on centos. Some unit tests were broken when running on centos. This fixes the unit test, with a small re-work of apt_configure. Also in 't

[Cloud-init-dev] [Merge] ~powersj/cloud-init:integration-test-revamp into cloud-init:master

2017-05-08 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:integration-test-revamp into cloud-init:master. Commit message: Integration Testing: tox env, pyxld 2.2.3, and revamp framework Massive update to clean up and greatly enhance the integration testing framework developed by Wesley Wiedenmeier

[Cloud-init-dev] [Merge] ~powersj/cloud-init:integration-test-revamp into cloud-init:master

2017-05-08 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:integration-test-revamp into cloud-init:master has been updated. Description changed to: Tests preformed: $ tox $ tox -e citest -- bddeb $ tox -e citest -- bddeb --deb output.deb $ tox -e citest -- run -v -n xenial $ tox -e citest -- run -v -n zesty --de

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:integration-test-revamp into cloud-init:master

2017-05-16 Thread Joshua Powers
@chad.smith: recall a tree_run will build your local tree. To do that it will be default use a xenial container to build cloud-init, produce the deb, and then use that deb on your artful test. I'm sure if you look at the full output you will see a acquiring image for artful and xenial. -- htt

[Cloud-init-dev] [Merge] ~powersj/cloud-init:integration-test-revamp into cloud-init:master

2017-05-16 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:integration-test-revamp into cloud-init:master has been updated. Status: Needs review => Superseded For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/323588 -- Your team cloud-init commiters is requested to

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-test-mkfs into cloud-init:master

2017-05-17 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix-test-mkfs into cloud-init:master. Commit message: tests: fix hardcoded path to mkfs.ext4 A recent merge that added a mkfs.ext4 tests has a hard coded location for the binary of mkfs.ext4. On CentOS 7 the test failed because the command

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:integration-test-revamp into cloud-init:master

2017-05-17 Thread Joshua Powers
Made Chad's spelling and doc string changes. Looked into the tmpdir changes and agreed that it is useless so removed the function and updated the lxd calls to call tempfile.mkdtemp directly. Thanks for the review! -- https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/324136 Y

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-test-mkfs into cloud-init:master

2017-05-17 Thread Joshua Powers
With mock method per smoser and tested on zesty and centos 7. -- https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/324201 Your team cloud-init commiters is requested to review the proposed merge of ~powersj/cloud-init:fix-test-mkfs into cloud-init:master.

[Cloud-init-dev] [Merge] ~powersj/cloud-init:integration-test-revamp into cloud-init:master

2017-05-18 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:integration-test-revamp into cloud-init:master has been updated. Commit Message changed to: Integration Testing: tox env, pyxld 2.2.3, and revamp framework Massive update to clean up and greatly enhance the integration testing framework developed by We

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:integration-test-revamp into cloud-init:master

2017-05-18 Thread Joshua Powers
I made an update to the commit message. That was a combination of the original messages before we had pulled in that tox change earlier. -- https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/324136 Your team cloud-init commiters is requested to review the proposed merge of ~po

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:pylint/1444-pylint-tip-ignore-e1101-from-contextlib into cloud-init:master

2017-05-22 Thread Joshua Powers
Why are there systemd changes at all? I am probably the last person who should comment on a merge request including more than what the change log says, but why are those necessary? -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/324401 Your team cloud-init commiters is re

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:pylint/1444-pylint-tip-ignore-e1101-from-contextlib into cloud-init:master

2017-05-22 Thread Joshua Powers
Review: Approve LGTM -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/324401 Your team cloud-init commiters is requested to review the proposed merge of ~smoser/cloud-init:pylint/1444-pylint-tip-ignore-e1101-from-contextlib into cloud-init:master. ___

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-integration-ntp into cloud-init:master

2017-05-22 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix-integration-ntp into cloud-init:master. Commit message: tests: Fix ntp integration test regression LP: #1692686 Requested reviews: cloud-init commiters (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-integration-snappy into cloud-init:master

2017-05-22 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:fix-integration-snappy into cloud-init:master has been updated. Commit Message changed to: tests: Fix snapd test case for non-started service Snapd does not start on artful or on the versions in proposed. This changes the behavior of the test to confirm

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-integration-snappy into cloud-init:master

2017-05-22 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix-integration-snappy into cloud-init:master. Commit message: tests: Fix snapd test case for non-started service Snapd does not start on artful or on the versions in proposed. This changes the behavior of the test to confirm that snapd is

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-integration-snappy into cloud-init:master

2017-05-22 Thread Joshua Powers
We could have a version-based check, however the version that breaks this, 2.26, is in proposed and about to migrate to all supported releases at any day. I certainly expect this to be fixed unless someone tells me that what we are doing (e.g. running snapd in a container) isn't allowed. My ori

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-blank-line into cloud-init:master

2017-05-23 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix-blank-line into cloud-init:master. Commit message: tests: cleanup pycodestyle error of missing blank line LP: #1692960 Requested reviews: cloud-init commiters (cloud-init-dev) Related bugs: Bug #1692960 in cloud-init: "

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:feature/flake8-upgrade into cloud-init:master

2017-05-23 Thread Joshua Powers
Review: Approve LGTM thank you for updating the versions as well. The tip commands all pass, tox passes, and the versions all look to be good to go. -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/324497 Your team cloud-init commiters is requested to review the proposed

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-integration-snappy into cloud-init:master

2017-05-23 Thread Joshua Powers
Updated the test content to check that snapd is installed using dpkg output rather than relying on --version output. -- https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/324430 Your team cloud-init commiters is requested to review the proposed merge of ~powersj/cloud-init:fix

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/package-build-no-stylechecks into cloud-init:master

2017-05-24 Thread Joshua Powers
+1 passes CI, build test working again, all tox envs, except centos6, look good. Thanks for doing this. The tox centos6 env, tested on centos 6 lxd, fail with the following, and I can get these cleaned up in a separate merge. =

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/package-build-no-stylechecks into cloud-init:master

2017-05-24 Thread Joshua Powers
Review: Approve -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/324541 Your team cloud-init commiters is requested to review the proposed merge of ~smoser/cloud-init:bug/package-build-no-stylechecks into cloud-init:master. ___

[Cloud-init-dev] [Merge] ~powersj/cloud-init:create-centos-tests into cloud-init:master

2017-06-01 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:create-centos-tests into cloud-init:master. Requested reviews: cloud-init commiters (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/324982 Looking for a review on this. It

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:create-centos-tests into cloud-init:master

2017-06-02 Thread Joshua Powers
@smoser: thanks for the template! @rharper: responses below two big points: 1) Knowning when network is up is a better solution than the sleep. May try to see if cloud-init is finished if cloud-init is in the images or some other solution. 2) Knowing when we need to use the proxy. When this is

Re: [Cloud-init-dev] [Merge] ~chad.smith/cloud-init:skip-jsonschema-unittest-when-missing-deps into cloud-init:master

2017-06-02 Thread Joshua Powers
Review: Approve LGTM passes tox locally and in my centos6 container. -- https://code.launchpad.net/~chad.smith/cloud-init/+git/cloud-init/+merge/325024 Your team cloud-init commiters is requested to review the proposed merge of ~chad.smith/cloud-init:skip-jsonschema-unittest-when-missing-deps in

[Cloud-init-dev] [Merge] ~powersj/cloud-init:create-centos-tests into cloud-init:master

2017-06-05 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:create-centos-tests into cloud-init:master has been updated. Status: Needs review => Work in progress For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/324982 -- Your team cloud-init commiters is requested

[Cloud-init-dev] [Merge] ~powersj/cloud-init:create-centos-tests into cloud-init:master

2017-06-06 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:create-centos-tests into cloud-init:master. Requested reviews: Server Team CI bot (server-team-bot): continuous-integration cloud-init commiters (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git

[Cloud-init-dev] [Merge] ~powersj/cloud-init:create-centos-tests into cloud-init:master

2017-06-06 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:create-centos-tests into cloud-init:master has been updated. Description changed to: For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/324982 -- Your team cloud-init commiters is requested to review the propo

[Cloud-init-dev] [Merge] ~powersj/cloud-init:create-centos-tests into cloud-init:master

2017-06-06 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:create-centos-tests into cloud-init:master has been updated. Commit Message changed to: tools: add centos scripts to build and test * Creates centos 6 or 7 lxd container * Sets http_proxy variable for yum if set locally * Creates centos user * P

[Cloud-init-dev] [Merge] ~powersj/cloud-init:create-centos-tests into cloud-init:master

2017-06-06 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:create-centos-tests into cloud-init:master has been updated. Description changed to: Read for review For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/324982 -- Your team cloud-init commiters is requested to r

[Cloud-init-dev] [Merge] ~powersj/cloud-init:style-fix into cloud-init:master

2017-06-07 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:style-fix into cloud-init:master. Commit message: test: Fix lint check of unused import The jsonschema package is used only when available, but the lint check thinks the import is unused across pyflakes and flake8. In order to avoid having

[Cloud-init-dev] [Merge] ~powersj/cloud-init:snap-remove-pipe into cloud-init:master

2017-06-07 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:snap-remove-pipe into cloud-init:master. Commit message: snap: Removing snapcraft plug line The snap is a classic snap and does not need to specify any plugs because it is already unconfined. LP: #1695333 Requested reviews: cloud-init

[Cloud-init-dev] [Merge] ~powersj/cloud-init:integration-test-revamp into cloud-init:master

2017-06-08 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:integration-test-revamp into cloud-init:master has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/324136 -- Your team cloud-init commiters is requested to rev

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/fix-ci-redhat into cloud-init:master

2017-06-14 Thread Joshua Powers
Review: Approve Only reviewed run-centos: clean up looks good and runs as expected locally. Thanks for taking the time to convert over to using read-dependencies. -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/325679 Your team cloud-init commiters is requested to revi

[Cloud-init-dev] [Merge] ~powersj/cloud-init:test-ci into cloud-init:master

2017-06-15 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:test-ci into cloud-init:master has been updated. Description changed to: DO NOT MERGE This is a test for the new CI system only! For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/325777 -- Your team cloud-init

[Cloud-init-dev] [Merge] ~powersj/cloud-init:test-ci into cloud-init:master

2017-06-15 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:test-ci into cloud-init:master. Commit message: doc: test Requested reviews: cloud-init commiters (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/325777 DO NOT MERGE This is

Re: [Cloud-init-dev] [Merge] ~mb-methot/cloud-init:bug/1699282 into cloud-init:master

2017-06-20 Thread Joshua Powers
Hi Marc Based on a quick look, it appears that in Python3 six.StringIO exists, but not in Python2. As you have corrected, io.BytesIO exists in both. Also CI looks good. Can I get you to do the following: 1. Set a commit message with a few details: Summary More information ... LP:

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix_ntp_tests into cloud-init:master

2017-06-26 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix_ntp_tests into cloud-init:master. Requested reviews: cloud-init commiters (cloud-init-dev) For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/326312 tests: update ntp tests after sntp added

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix_ntp_tests into cloud-init:master

2017-06-26 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:fix_ntp_tests into cloud-init:master has been updated. Commit Message changed to: tests: update ntp tests after sntp added A recent change to ntp in artful has added the sntp package whenever ntp is installed. The tests, rather poorly, did a `dpkg -l` i

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix_ntp_tests into cloud-init:master

2017-06-26 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:fix_ntp_tests into cloud-init:master has been updated. Description changed to: Tested with: * python3 -m tests.cloud_tests run --os-name artful --preserve-data --data-dir results --verbose -t modules/ntp -t modules/ntp_servers -t modules/ntp_pools -

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix_ssh_import_id into cloud-init:master

2017-06-26 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix_ssh_import_id into cloud-init:master. Commit message: Tests: Simplify the check on ssh-import-id I want to be able to add additional SSH keys to my account, therefore I should not be limiting these tests to look for one specific key

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-apt-keyserver into cloud-init:master

2017-07-06 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix-apt-keyserver into cloud-init:master. Commit message: test: fix incorrect keyid The test is currently importing the incorrect keyid. It specifies the curtin developers ppa, rather than the cloud-init ppa. On Artful this causes failures

[Cloud-init-dev] [Merge] ~powersj/cloud-init:update-pylxd-version into cloud-init:master

2017-07-10 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:update-pylxd-version into cloud-init:master has been updated. Commit Message changed to: tests: Update version of pylxd With the upgrade to lxd 2.15, pylxd version 2.2.3 broke. Upgrading to version 2.2.4 fixes issues with missing attributes. For more d

[Cloud-init-dev] [Merge] ~powersj/cloud-init:update-pylxd-version into cloud-init:master

2017-07-10 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:update-pylxd-version into cloud-init:master. Commit message: tests: Update version of pylxd With the upgrade to lxd 2.15, pylxd version 2.2.3 broke. Upgrading to version 2.2.4 fixes issues with missing attributes. Requested reviews

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:bug/1703697-python3.6-ubuntu into cloud-init:master

2017-07-13 Thread Joshua Powers
Review: Approve LGTM thank you for the fix -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/327344 Your team cloud-init commiters is requested to review the proposed merge of ~smoser/cloud-init:bug/1703697-python3.6-ubuntu into cloud-init:master. _

[Cloud-init-dev] [Merge] ~powersj/cloud-init:cii-kvm into cloud-init:master

2017-07-18 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:cii-kvm into cloud-init:master. Commit message: test: Enable the KVM platform on integration tests The KVM platform includes: * Ubuntu images from daily stream * Image setup via mount-image-callback * Generation and injection

[Cloud-init-dev] [Merge] ~powersj/cloud-init:cii-kvm into cloud-init:master

2017-07-18 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:cii-kvm into cloud-init:master has been updated. Description changed to: Would like an initial review, please! Example command: $ python3 -m tests.cloud_tests run --verbose --platform kvm --os-name xenial -t modules/locale Note, not all existing tests

[Cloud-init-dev] [Merge] ~powersj/cloud-init:cii-kvm into cloud-init:master

2017-07-19 Thread Joshua Powers
The proposal to merge ~powersj/cloud-init:cii-kvm into cloud-init:master has been updated. Status: Needs review => Work in progress For more details, see: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/327646 -- Your team cloud-init commiters is requested to review th

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-arch-hostname into cloud-init:master

2017-07-21 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix-arch-hostname into cloud-init:master. Commit message: archlinux: fix set hostname cloud-init fails to set the hostname on Arch Linux becuase that _write_hostname passes conf instead of str(conf) to util.write_file. LP: #1705306

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-arch-dns into cloud-init:master

2017-07-21 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix-arch-dns into cloud-init:master. Commit message: archlinux: Fix empty dns on writing network config cloud-init fails when dns-nameservers is None. LP: #1663045 Requested reviews: Server Team CI bot (server-team-bot): continuous

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:fix/disable-yakkety-tests into cloud-init:master

2017-07-26 Thread Joshua Powers
Review: Approve LGTM one comment below. Diff comments: > diff --git a/tests/cloud_tests/releases.yaml b/tests/cloud_tests/releases.yaml > index 45deb58..c8dd142 100644 > --- a/tests/cloud_tests/releases.yaml > +++ b/tests/cloud_tests/releases.yaml > @@ -143,19 +143,6 @@ releases: >

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:fix/disable-yakkety-tests into cloud-init:master

2017-07-26 Thread Joshua Powers
Dropping sounds good to me then. -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/328098 Your team cloud-init commiters is requested to review the proposed merge of ~smoser/cloud-init:fix/disable-yakkety-tests into cloud-init:master. ___

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:fix/locale-test-update into cloud-init:master

2017-07-26 Thread Joshua Powers
Review: Approve -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/328095 Your team cloud-init commiters is requested to review the proposed merge of ~smoser/cloud-init:fix/locale-test-update into cloud-init:master. ___ Mailing l

[Cloud-init-dev] [Merge] ~powersj/cloud-init:fix-cii-tree into cloud-init:master

2017-07-28 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:fix-cii-tree into cloud-init:master. Commit message: tests: Fix build tree integration tests The build deb command was no longer working because it had assumed that you were in the root of the cloud-init directory. This changes where the

[Cloud-init-dev] [Merge] ~powersj/cloud-init:cii-kvm into cloud-init:master

2017-08-16 Thread Joshua Powers
Joshua Powers has proposed merging ~powersj/cloud-init:cii-kvm into cloud-init:master. Commit message: test: Enable the KVM platform on integration tests The KVM platform includes: * Ubuntu images from daily stream * Image setup via mount-image-callback * Generation and injection

Re: [Cloud-init-dev] [Merge] ~powersj/cloud-init:cii-kvm into cloud-init:master

2017-08-18 Thread Joshua Powers
I have been leaving integration test requirements out of requirements-test.txt. For example, pylxd is not in there either. I consider that file what is required for unit tests only. Thoughts? -- https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/327646 Your team cloud-init com

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:feature/run-opensuse into cloud-init:master

2017-08-25 Thread Joshua Powers
2 comments inline Diff comments: > diff --git a/tools/run-opensuse b/tools/run-opensuse > new file mode 100755 > index 000..e974f8c > --- /dev/null > +++ b/tools/run-opensuse > @@ -0,0 +1,273 @@ > +#!/bin/bash > +# This file is part of cloud-init. See LICENSE file for license information. > +

Re: [Cloud-init-dev] [Merge] ~smoser/cloud-init:fix/tox-xenial-3.5 into cloud-init:master

2017-08-30 Thread Joshua Powers
I think we should accept the comment diff change (i.e. python version change + jsonpatch version change). This way we can keep it in the default environments to continue to get coverage of those modules versions. -- https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/329926 Your

  1   2   3   >