Re: [Cloud-init-dev] [Merge] ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master

2018-05-16 Thread Stephen Ford
I have now signed the Canonical Contributor Agreement. -- https://code.launchpad.net/~thenarcotk/cloud-init/+git/cloud-init/+merge/345608 Your team cloud-init commiters is requested to review the proposed merge of ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master.

[Cloud-init-dev] [Merge] ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master

2018-05-16 Thread Stephen Ford
Stephen Ford has proposed merging ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master. Commit message: Fix spelling mistakes in a cc_mounts.py log and error messages This also makes some of the messages more consistent. Requested reviews: cloud-init commiters

[Cloud-init-dev] [Merge] ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master

2018-05-16 Thread Stephen Ford
The proposal to merge ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master has been updated. Status: Work in progress => Superseded For more details, see: https://code.launchpad.net/~thenarcotk/cloud-init/+git/cloud-init/+merge/345608 -- Your team cloud-init commiters

Re: [Cloud-init-dev] [Merge] ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master

2018-05-16 Thread Stephen Ford
Thanks Chad for the feedback, a couple more spelling or wording mistakes too have been corrected. -- https://code.launchpad.net/~thenarcotk/cloud-init/+git/cloud-init/+merge/345608 Your team cloud-init commiters is requested to review the proposed merge of

Re: [Cloud-init-dev] [Merge] ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master

2018-05-15 Thread Chad Smith
Stephen, thank you so much for this patch and making cloud-init better. This project is governed by the contributor's license agreement. Once you are able to confirm signing the CLA, we'll land this branch immediately. Here are the details below. I've marked this branch 'work in progress' and

Re: [Cloud-init-dev] [Merge] ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master

2018-05-15 Thread Chad Smith
Diff comments: > diff --git a/cloudinit/config/cc_mounts.py b/cloudinit/config/cc_mounts.py > index f14a4fc..6bd713a 100644 > --- a/cloudinit/config/cc_mounts.py > +++ b/cloudinit/config/cc_mounts.py > @@ -337,7 +337,7 @@ def handle(_name, cfg, cloud, log, _args): > start =

[Cloud-init-dev] [Merge] ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master

2018-05-15 Thread Chad Smith
The proposal to merge ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master has been updated. Status: Needs review => Work in progress For more details, see: https://code.launchpad.net/~thenarcotk/cloud-init/+git/cloud-init/+merge/345608 -- Your team cloud-init

[Cloud-init-dev] [Merge] ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master

2018-05-15 Thread Stephen Ford
Stephen Ford has proposed merging ~thenarcotk/cloud-init:fix-spelling-error-cc-mounts into cloud-init:master. Commit message: Fix a spelling mistake in a cc_mounts.py error message The error message has a spelling mistake when a nonexistent named mount is used. Requested reviews: cloud-init