Re: [Cloud-init-dev] [Merge] ~raharper/cloud-init:fix/cloud-tests-lxd-retry-delete into cloud-init:master

2019-10-11 Thread Server Team CI bot
Review: Approve continuous-integration PASSED: Continuous integration, rev:aaf0c0e11ed07b65d15f8759d907483561c40334 https://jenkins.ubuntu.com/server/job/cloud-init-ci/1207/ Executed test runs: SUCCESS: Checkout SUCCESS: Unit & Style Tests SUCCESS: Ubuntu LTS: Build SUCCESS:

Re: [Cloud-init-dev] [Merge] ~raharper/cloud-init:fix/cloud-tests-lxd-retry-delete into cloud-init:master

2019-10-11 Thread Paride Legovini
Review: Approve Right. In the usual failure mode retrying should work even without sleeping, but it's way better to add a sleep(1) there. -- https://code.launchpad.net/~raharper/cloud-init/+git/cloud-init/+merge/374043 Your team cloud-init Commiters is requested to review the proposed merge of

Re: [Cloud-init-dev] [Merge] ~raharper/cloud-init:fix/cloud-tests-lxd-retry-delete into cloud-init:master

2019-10-11 Thread Server Team CI bot
Review: Needs Fixing continuous-integration FAILED: Continuous integration, rev:c6bdca59c2b7afdf19b9bc7a4df8367584016627 https://jenkins.ubuntu.com/server/job/cloud-init-ci/1206/ Executed test runs: SUCCESS: Checkout FAILED: Unit & Style Tests Click here to trigger a rebuild:

Re: [Cloud-init-dev] [Merge] ~raharper/cloud-init:fix/cloud-tests-lxd-retry-delete into cloud-init:master

2019-10-11 Thread Ryan Harper
I need to add a sleep after the failure -- https://code.launchpad.net/~raharper/cloud-init/+git/cloud-init/+merge/374043 Your team cloud-init Commiters is requested to review the proposed merge of ~raharper/cloud-init:fix/cloud-tests-lxd-retry-delete into cloud-init:master.

Re: [Cloud-init-dev] [Merge] ~raharper/cloud-init:fix/cloud-tests-lxd-retry-delete into cloud-init:master

2019-10-11 Thread Paride Legovini
Review: Approve LGTM -- https://code.launchpad.net/~raharper/cloud-init/+git/cloud-init/+merge/374043 Your team cloud-init Commiters is requested to review the proposed merge of ~raharper/cloud-init:fix/cloud-tests-lxd-retry-delete into cloud-init:master.

[Cloud-init-dev] [Merge] ~raharper/cloud-init:fix/cloud-tests-lxd-retry-delete into cloud-init:master

2019-10-11 Thread Ryan Harper
Ryan Harper has proposed merging ~raharper/cloud-init:fix/cloud-tests-lxd-retry-delete into cloud-init:master. Commit message: cloud_test/lxd: Retry container delete a few times LXD integration tests fail sometimes due to failure to delete the container, usually related to ZFS backend. This is