cloud-init-dev team mailing list archive

Thread
Date

Re: [Merge] ~larsks/cloud-init:lp/1665441 into cloud-init:master

To: Lars Kellogg-Stedman <lars@xxxxxxxxxx>
From: Scott Moser <smoser@xxxxxxxxxx>
Date: Fri, 17 Feb 2017 03:08:24 -0000
In-reply-to: <20170216205423.17789.86908.launchpad@ackee.canonical.com>
Reply-to: mp+317553@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Lars,
Thanks so much for investigating and finding a fix.

I hate it when I provide a fix for a bug in someone's code and they ask for a unit test.

That said...
A unit test here will avoid a regression in the future.  Could you add one?

If you don't add one, then I'm going to, and it will just take me longer to pull it.

-- 
https://code.launchpad.net/~larsks/cloud-init/+git/cloud-init/+merge/317553
Your team cloud init development team is requested to review the proposed merge of ~larsks/cloud-init:lp/1665441 into cloud-init:master.

References

[Merge] ~larsks/cloud-init:lp/1665441 into cloud-init:master
From: Lars Kellogg-Stedman, 2017-02-16