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

2017-05-22 Thread Chad Smith
Review: Approve I think we agreed in standup that since you've fixed this with a workaround for pylint's bug, you could probably add a structured comment or something that mark that we should look at pylint tip in the future to see if that issue is fixed. For bugs on external dependencies

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

2017-05-22 Thread Server Team CI bot
Review: Approve continuous-integration PASSED: Continuous integration, rev:9c33cb24b8a834aa8034db4c989725901a0814c6 https://jenkins.ubuntu.com/server/job/cloud-init-ci/372/ Executed test runs: SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/372 SUCCESS:

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

2017-05-22 Thread Server Team CI bot
Review: Approve continuous-integration PASSED: Continuous integration, rev:b01ae9627652e9ef03c68049dcee345755b5e806 https://jenkins.ubuntu.com/server/job/cloud-init-ci/370/ Executed test runs: SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/370 SUCCESS:

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

2017-05-22 Thread Server Team CI bot
Review: Approve continuous-integration PASSED: Continuous integration, rev:d6518ba988e24f939687d83d5946bb9a3ce2751a https://jenkins.ubuntu.com/server/job/cloud-init-ci/367/ Executed test runs: SUCCESS: https://jenkins.ubuntu.com/server/job/cloud-init-ci/nodes=metal-amd64/367 SUCCESS: