| === kukacz_ is now known as kukacz | ||
| powersj | smoser: https://code.launchpad.net/~powersj/cloud-init/+git/cloud-init/+merge/334141 | 19:13 |
|---|---|---|
| powersj | enables bionic for integration tests | 19:13 |
| smoser | powersj: gracias. | 19:26 |
| powersj | looking at the image modification issue you brought up now | 19:26 |
| powersj | We use qemu-image create with the original qcow as a backing file | 19:27 |
| powersj | heh it does indeed look like we setup the image (i.e. install new deb) before creating the copy doh! | 19:34 |
| smoser | powersj: so i had thought someon this. | 19:35 |
| powersj | all ears :) | 19:36 |
| smoser | when you create an image it should create a qcow2 backed by the /srv/ file | 19:36 |
| 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:37 |
| 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:39 |
| * powersj goes and gets food | 19:42 | |
| smoser | https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+ref/fix/cii-kvmimage-preserve-original | 19:46 |
| powersj | smoser: looking | 20:12 |
| smoser | that was what i had started a while ago | 20:14 |
| 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:15 |
| powersj | smoser: that looks good | 20:18 |
| powersj | going to propose it? | 20:19 |
| powersj | oh yeah gotta change snapshot | 20:20 |
| smoser | powersj: i can see if i can do that quick | 20:20 |
| smoser | (this was a while ago... so my brain wsant too fresh) | 20:21 |
| smoser | ug. | 20:26 |
| smoser | htis is why i gave up. it gets ugly there. | 20:26 |
| smoser | powersj: https://code.launchpad.net/~smoser/cloud-init/+git/cloud-init/+merge/334147 | 21:50 |
| 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:51 |
| 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. | 21:52 |
| 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 | 22:29 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!