=== kukacz_ is now known as kukacz
[19:13] <powersj> smoser: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/334141
[19:13] <powersj> enables bionic for integration tests
[19:26] <smoser> powersj: gracias.
[19:26] <powersj> looking at the image modification issue you brought up now
[19:27] <powersj> We use qemu-image create with the original qcow as a backing file
[19:34] <powersj> heh it does indeed look like we setup the image (i.e. install new deb) before creating the copy doh!
[19:35] <smoser> powersj: so i had thought someon this.
[19:36] <powersj> all ears :)
[19:36] <smoser> when you create an image it should create a qcow2 backed by the /srv/ file
[19:37] <smoser> then, the snapshot can qcow2 off of that.. the issue wth that is that ew kind of need snapshot to be persistent
[19:37] <smoser> so maybe when we snapshot it should "realize" the image.. rather than snapshost-qcow2 -> image->qcow2 -> /srv
[19:37] <smoser> as the image thing can be deleted.
[19:39] <powersj> Snapshot today calls platforms/nocloudkvm.py which creates a qcow backed by the /srv/ file and you are saying what needs to change is image, which should create a a qcow
[19:42]  * powersj goes and gets food
[19:46] <smoser> https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+ref/fix/cii-kvmimage-preserve-original
[20:12] <powersj> smoser: looking
[20:14] <smoser> that was what i had started a while ago
[20:15] <smoser> powersj: rad the commit message and the 'also have to modify'...
[20:15] <smoser> i think it'd be sufficient if we just di da copy (shutil.copy)
[20:15] <smoser> that way the snapshto woudl be backed by the imagein /srv/
[20:18] <powersj> smoser: that looks good
[20:19] <powersj> going to propose it?
[20:20] <powersj> oh yeah gotta change snapshot
[20:20] <smoser> powersj: i can see if i can do that quick
[20:21] <smoser> (this was a while ago... so my brain wsant too fresh)
[20:26] <smoser> ug.
[20:26] <smoser> htis is why i gave up. it gets ugly there.
[21:50] <smoser> powersj: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/334147
[21:51] <smoser> i'm running tox here... it did run a quick test of nocloud kvm for me.
[21:51] <smoser> but not throughogh testing
[21:52] <powersj> smoser: ok I'll play with it and let you know by eod
[21:52] <powersj> thanks!
[21:52] <smoser> well, k'm about to eod.
[21:52] <powersj> really appreciate it!
[21:52] <smoser> but yeah, give some feedback. thanks.
[22:29] <smoser> powersj: merged yoru cloud-init bionic so we'll hopefully get results of that tonight.
[22:29] <smoser> thanks. and with that, i do go afk
[22:29] <powersj> awesome
[22:29] <powersj> o/ cya