Review: Approve continuous-integration
PASSED: Continuous integration, rev:29d0bb1cffd00ec3df0c2effd4a649af07390911
https://jenkins.ubuntu.com/server/job/cloud-init-ci/388/
Executed test runs:
SUCCESS:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/388
SUCCESS:
htt
Review: Approve continuous-integration
PASSED: Continuous integration, rev:768e7ed3a8ebddacd7112afccb94e8fe864bcdd0
https://jenkins.ubuntu.com/server/job/cloud-init-ci/387/
Executed test runs:
SUCCESS:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/387
SUCCESS:
htt
Chad Smith has proposed merging ~chad.smith/cloud-init:cc-ntp-schema into
cloud-init:master with ~chad.smith/cloud-init:cc-ntp-testing as a prerequisite.
Commit message:
cc_ntp: Add schema definition and passive schema validation logging warnings.
cloud-config files are very flexible and permiss
Review: Needs Fixing continuous-integration
FAILED: Continuous integration, rev:15a8bc4dbb75ff9968c344cda907e0fd6ed3a0fd
https://jenkins.ubuntu.com/server/job/cloud-init-ci/386/
Executed test runs:
FAILURE:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/386/console
@scott Addressed your review comments thanks a lot. So unittest runtimes
1m30sec versus 2m20sec? Do we want to make the switch to have all CiTestCase
subclasses logged?
Diff comments:
> diff --git a/cloudinit/config/cc_ntp.py b/cloudinit/config/cc_ntp.py
> index 225f898..257bb40 100644
>
Given that these are all generally acceptable changes I'm going to pull this.
Even if its possible we'd find fallout of the hacking issue.
--
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/324497
Your team cloud-init commiters is requested to review the proposed merge of
~
The proposal to merge ~smoser/cloud-init:feature/flake8-upgrade into
cloud-init:master has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/324497
--
Your team cloud-init commiters is requested to review
https://bugs.launchpad.net/hacking/+bug/1607942
is a 'hacking' upstream bug on versions.
--
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/324497
Your team cloud-init commiters is requested to review the proposed merge of
~smoser/cloud-init:feature/flake8-upgrade into clou
Review: Approve continuous-integration
PASSED: Continuous integration, rev:2825a917e5fa130818c0d77219f32961b99a057f
https://jenkins.ubuntu.com/server/job/cloud-init-ci/385/
Executed test runs:
SUCCESS:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/385
SUCCESS:
htt
Hm..
So after doing this I noticed that pep8 is still in my tox flake8 env and
wondered why.
$ pip check
hacking 0.13.0 has requirement flake8<2.6.0,>=2.5.4, but you have flake8 3.3.0.
hacking 0.13.0 has requirement mccabe==0.2.1, but you have mccabe 0.6.1.
hacking 0.13.0 has requirement pyflakes
Review: Approve continuous-integration
PASSED: Continuous integration, rev:194e15a6785f695ed5d228fdf6601837df3fb781
https://jenkins.ubuntu.com/server/job/cloud-init-ci/384/
Executed test runs:
SUCCESS:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/384
SUCCESS:
htt
Review: Approve continuous-integration
PASSED: Continuous integration, rev:d0e1e415ef282ff28e8a142a0ff8389dbbb83567
https://jenkins.ubuntu.com/server/job/cloud-init-ci/383/
Executed test runs:
SUCCESS:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/383
SUCCESS:
htt
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
Review: Approve continuous-integration
PASSED: Continuous integration, rev:b6c5ed3b04a00bba7e49ebb58947926b4f121e88
https://jenkins.ubuntu.com/server/job/cloud-init-ci/382/
Executed test runs:
SUCCESS:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/382
SUCCESS:
htt
Review: Approve continuous-integration
PASSED: Continuous integration, rev:49c4eb85daa77cb20a027e3199a92e3dbc773f53
https://jenkins.ubuntu.com/server/job/cloud-init-ci/381/
Executed test runs:
SUCCESS:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/381
SUCCESS:
htt
I've pushed some new commits, please see and review.
thanks.
Diff comments:
> diff --git a/cloudinit/sources/DataSourceEc2.py
> b/cloudinit/sources/DataSourceEc2.py
> index 2f9c7ed..818a639 100644
> --- a/cloudinit/sources/DataSourceEc2.py
> +++ b/cloudinit/sources/DataSourceEc2.py
probably.
I
Diff comments:
> diff --git a/cloudinit/config/cc_ntp.py b/cloudinit/config/cc_ntp.py
> index 225f898..257bb40 100644
> --- a/cloudinit/config/cc_ntp.py
> +++ b/cloudinit/config/cc_ntp.py
> @@ -69,18 +67,12 @@ def handle(name, cfg, cloud, log, _args):
> " but not a d
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
some responses in line. i'll fix some things too. thanks for review.
Diff comments:
> diff --git a/cloudinit/sources/DataSourceEc2.py
> b/cloudinit/sources/DataSourceEc2.py
> index 2f9c7ed..818a639 100644
> --- a/cloudinit/sources/DataSourceEc2.py
> +++ b/cloudinit/sources/DataSourceEc2.py
> @@
some comments in line.
I'm generally in agreement here.
Diff comments:
> diff --git a/cloudinit/config/cc_ntp.py b/cloudinit/config/cc_ntp.py
> index 225f898..257bb40 100644
> --- a/cloudinit/config/cc_ntp.py
> +++ b/cloudinit/config/cc_ntp.py
> @@ -53,14 +53,12 @@ distros = ['centos', 'debian'
The proposal to merge ~chad.smith/cloud-init:cc-ntp-testing into
cloud-init:master has been updated.
Commit Message changed to:
cc_ntp: Restructure cc_ntp unit tests add with_logs class attr to unit tests.
Any CiTestCase subclass can now set a class attribute with_logs = True and
tests can now
Review: Approve continuous-integration
PASSED: Continuous integration, rev:a4f3aab9cdc3c5b04c8df2ed768dca4a3cc06149
https://jenkins.ubuntu.com/server/job/cloud-init-ci/380/
Executed test runs:
SUCCESS:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/380
SUCCESS:
htt
for quick reference, H304 shows:
http://paste.ubuntu.com/24635484/
--
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/324497
Your team cloud-init commiters is requested to review the proposed merge of
~smoser/cloud-init:feature/flake8-upgrade into cloud-init:master.
I expect :vi info in files (H102)
for H304, I dont have strong feelings. we could fix these.
--
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/324497
Your team cloud-init commiters is requested to review the proposed merge of
~smoser/cloud-init:feature/flake8-upgrade into c
The proposal to merge ~smoser/cloud-init:feature/flake8-upgrade into
cloud-init:master has been updated.
Commit Message changed to:
flake8: move the pinned version of flake8 up to 3.3.0
This just moves flake8 and related tools up to newer versions and fixes the
complaints associated with that.
The proposal to merge ~smoser/cloud-init:feature/flake8-upgrade into
cloud-init:master has been updated.
Commit Message changed to:
flake8: move the pinned version of flake8 up to 3.3.0
This just moves flake8 and related tools up to newer versions and fixes the
complaints associated with that.
Scott Moser has proposed merging ~smoser/cloud-init:feature/flake8-upgrade into
cloud-init:master.
Requested reviews:
cloud-init commiters (cloud-init-dev)
For more details, see:
https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/324497
--
Your team cloud-init commiters is r
Review: Approve
+1 land it
--
https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/324490
Your team cloud-init commiters is requested to review the proposed merge of
~powersj/cloud-init:fix-blank-line into cloud-init:master.
___
Maili
Diff comments:
> diff --git a/cloudinit/config/cc_ntp.py b/cloudinit/config/cc_ntp.py
> index 225f898..257bb40 100644
> --- a/cloudinit/config/cc_ntp.py
> +++ b/cloudinit/config/cc_ntp.py
> @@ -53,14 +53,12 @@ distros = ['centos', 'debian', 'fedora', 'opensuse',
> 'ubuntu']
>
>
> def handl
Review: Approve continuous-integration
PASSED: Continuous integration, rev:ea4c7fcc21f1c3b2be5ab1d69892fe09edd34498
https://jenkins.ubuntu.com/server/job/cloud-init-ci/379/
Executed test runs:
SUCCESS:
https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/379
SUCCESS:
htt
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: "tests: pycode
31 matches
Mail list logo