[Cloud-init-dev] [Merge] ~tlashchova/cloud-init:fix-bug-add-unittest into cloud-init:master

2018-02-01 Thread Tatiana Kholkina
Tatiana Kholkina has proposed merging ~tlashchova/cloud-init:fix-bug-add-unittest into cloud-init:master. Requested reviews: cloud-init commiters (cloud-init-dev) For more details, see: https://code.launchpad.net/~tlashchova/cloud-init/+git/cloud-init/+merge/337003 -- Your team cloud-init

[Cloud-init-dev] [Merge] ~tlashchova/cloud-init:fix-bug-add-unittest into cloud-init:master

2018-02-01 Thread Tatiana Kholkina
The proposal to merge ~tlashchova/cloud-init:fix-bug-add-unittest into cloud-init:master has been updated. Commit Message changed to: Fix ssh keys validation in ssh_util For more details, see: https://code.launchpad.net/~tlashchova/cloud-init/+git/cloud-init/+merge/337003 -- Your team

Re: [Cloud-init-dev] [Merge] ~tlashchova/cloud-init:fix-bug-add-unittest into cloud-init:master

2018-02-01 Thread Scott Moser
Tatiana, I'll get our c-i bot to test your branch, and pending success there, looks good! -- https://code.launchpad.net/~tlashchova/cloud-init/+git/cloud-init/+merge/337003 Your team cloud-init commiters is requested to review the proposed merge of ~tlashchova/cloud-init:fix-bug-add-unittest

Re: [Cloud-init-dev] [Merge] ~tlashchova/cloud-init:fix-bug-add-unittest into cloud-init:master

2018-02-01 Thread Scott Moser
improve your commit message, just have a 'messag'e part as well as subject. Ie: Do not write invalid keys to authorized_keys This fixes a bug where invalid keys would sneak into authorized_keys. Diff comments: > diff --git a/cloudinit/ssh_util.py b/cloudinit/ssh_util.py > index

Re: [Cloud-init-dev] [Merge] ~tlashchova/cloud-init:fix-bug-add-unittest into cloud-init:master

2018-02-01 Thread Tatiana Kholkina
> improve your commit message, just have a 'messag'e part as well as subject. > > Ie: >Do not write invalid keys to authorized_keys > >This fixes a bug where invalid keys would sneak into authorized_keys. Done --

[Cloud-init-dev] [Merge] ~tlashchova/cloud-init:fix-bug-add-unittest into cloud-init:master

2018-02-01 Thread Tatiana Kholkina
The proposal to merge ~tlashchova/cloud-init:fix-bug-add-unittest into cloud-init:master has been updated. Commit Message changed to: Fix ssh keys validation in ssh_util This fixes a bug where invalid keys would sneak into authorized_keys. For more details, see: