mhroncok | smoser: ping | 10:31 |
---|---|---|
smoser | bah. | 14:35 |
smoser | mhroncok, tennis, you can't just ask questions and then leave. | 14:35 |
* smoser is still on reduced computer hours | 14:36 | |
ericsnow | PTAL https://bugs.launchpad.net/cloud-init/+bug/1404311 | 15:19 |
smoser | i'll look at that now. :) | 15:46 |
ericsnow | thanks! | 16:18 |
wwitzel3 | smoser: o/ | 16:31 |
smoser | o/ | 16:31 |
smoser | looking at your MP for bug 1404311 | 16:31 |
smoser | https://code.launchpad.net/~wwitzel3/cloud-init/gce/+merge/245209 | 16:31 |
smoser | the LOG messages on line 29 of the diff there look wrong. | 16:32 |
smoser | LOG.warn... | 16:32 |
smoser | you have None, None | 16:32 |
smoser | but do not have a '%s' at all in the string | 16:32 |
wwitzel3 | smoser: yep, you are right | 16:34 |
wwitzel3 | smoser: I will correct that | 16:34 |
wwitzel3 | smoser: correction pushed up | 16:42 |
smoser | wwitzel3, ok. i merged. please do check what is in trunk now just to be sure but it shoudl be good. | 17:13 |
smoser | you had a pep8 error like this: {'key':'value'} | 17:13 |
smoser | which should have required white space between ':' but wasnt. | 17:13 |
wwitzel3 | smoser: hrmm, surprised my vim didn't warn or auto correct that | 17:14 |
smoser | and i didn't understand why ./tools/run-pep8 didn't catch it. now it does. | 17:14 |
smoser | thats why most of the noise in my commits before merge | 17:14 |
wwitzel3 | smoser: I probably messed something up when I install all the go-vim stuff | 17:14 |
wwitzel3 | smoser: thank you, so for testing this against GCE live .. how would I go about doing that? | 17:15 |
smoser | that is kind of hard isnt it. can you re-bundle an instance? | 17:15 |
smoser | ie, launch one, patch it, create a snapshot and then from there? | 17:16 |
wwitzel3 | smoser: hrmm, maybe? .. is there an easy way to re-run the cloud-init process on an instance? | 17:16 |
wwitzel3 | smoser: I could create and instance, patch, and just re-run the part of the scipt that consumes the metadata and generates the file | 17:17 |
wwitzel3 | I actually don't know anything about when cloud-init actually runs | 17:18 |
wwitzel3 | I can probably figure it out :) | 17:18 |
smoser | wwitzel3, yeah, you can do that. | 17:19 |
smoser | actually if you just | 17:19 |
smoser | rm -Rf /var/lib/cloud; reboot | 17:19 |
smoser | that shoudl be good | 17:19 |
smoser | after installing your new deb | 17:19 |
wwitzel3 | smoser: ok, awesome I will give that a shot | 17:20 |
wwitzel3 | smoser: thanks again | 17:20 |
wwitzel3 | smoser: confirmed the new behavior, thanks again :) | 18:04 |
=== zz_gondoi is now known as gondoi | ||
=== gondoi is now known as zz_gondoi | ||
wwitzel3 | smoser: so what will be the next steps to get the new update in to live gce? is there a schedule for releases? | 19:50 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!