Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-04
15:40:18 mdbooth Anyway, just my 2c, and I totally get that pragmatism might be required here.
15:40:46 mdbooth But it raised a flag for me. We'll need to unwind it eventually.
15:41:20 hrw also commit message got rewritten
15:41:21 openstackgerrit Marcin Juszkiewicz proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965
15:41:26 mriedem yeah i think passing a boolean through attach_volume to get_config is easy, it's live migrate that i'm worried about
15:41:48 ildikov mdbooth: not the first time it came up, but we didn't manage to have this as a top priority problem as of yet to find a better way :(
15:42:33 mdbooth ildikov: At the very least we'll need to be able to find it in order to unwind it.
15:43:12 kashyap mriedem: Oh the previous point about capablities, libvirt upstream says, if I file a bug they could add it - as it shouldn't be too difficult
15:43:29 kashyap mriedem: That'd be cleaner for us (Nova), isn't it?
15:43:50 mdbooth ildikov: Incidentally, does service_uuid indicate a shared pool of volumes?
15:44:10 mdbooth ildikov: e.g. multiple volumes on the same NFS mount?
15:44:28 ildikov mdbooth: no, that's independent
15:44:40 mdbooth ildikov: Where can I read what it means?
15:44:55 mriedem kashyap: long-term that would be cleaner yes,
15:45:01 mriedem but not something that is going to help me in queens
15:45:37 kashyap Right. /me imagines: If you add the version check, and then the capability comes along later, no one will remember to swap that, until prompted by something
15:46:13 mriedem kashyap: feel free to file a bug if you want :)
15:46:17 kashyap ildikov: Can you point to the latest URL of the multi-attach specification, please?
15:46:27 openstackgerrit Matthew Booth proposed openstack/nova master: Pass DriverBlockDevice to driver.attach_volume https://review.openstack.org/528363
15:46:27 openstackgerrit Matthew Booth proposed openstack/nova master: Use real block_device_info data in libvirt tests https://review.openstack.org/527916
15:46:28 openstackgerrit Matthew Booth proposed openstack/nova master: Fix libvirt volume tests passing invalid disk_info https://review.openstack.org/529328
15:46:28 openstackgerrit Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329
15:46:29 openstackgerrit Matthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config https://review.openstack.org/530786
15:46:29 openstackgerrit Matthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume https://review.openstack.org/530787
15:46:30 openstackgerrit Matthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver https://review.openstack.org/529380
15:46:30 openstackgerrit Matthew Booth proposed openstack/nova master: Remove redundant swap_volume tests https://review.openstack.org/531179
15:46:36 mriedem kashyap: https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/cinder-volume-multi-attach.html
15:46:39 kashyap mriedem: Will do
15:46:46 kashyap Thanks
15:47:27 openstackgerrit Andreas Karis proposed openstack/nova master: Add debug output for selected page size https://review.openstack.org/530662
15:48:13 coreycb mriedem: this is being tracked in https://bugs.launchpad.net/ubuntu/+source/qemu/+bug/1716028
15:48:14 openstack Launchpad bug 1716028 in libvirt (Ubuntu) "qemu 2.10 locks images with no feature flag" [Medium,Triaged]
15:48:57 coreycb mriedem: cpaelzer says that after bionic is done he'll take a look at a potential SRU, although he currently has the SRU to artful(pike) as a low priority.
15:49:28 mriedem coreycb: ok thanks
15:51:09 kashyap It sucks hard that one cannot access the URLs (https://review.openstack.org/#/c/267587/78/nova/virt/libvirt/driver.py) without Gerrit account
15:51:31 kashyap Can't pass in-progress patch URLs to people aren't Gerrit users. Surely there must be a way
15:51:44 kashyap Without bothering people to ask to make an account (much like mailing lists).
15:52:35 mriedem kashyap: i'm not signed in and i can view https://review.openstack.org/#/c/267587/78/nova/virt/libvirt/driver.py
15:52:50 mriedem you just can't comment or vote
15:52:55 kashyap mriedem: Err, sorry. The person was complaining about typing in a comment.
15:54:00 kashyap mriedem: Anyway, the quick point that Peter wanted to add was: " sharing disk image is possible even with current qemu/libvirt if the image is 'raw' and <shareable/> is used"
15:55:36 mriedem we must be using qcow2 images
15:56:35 mriedem another thing i can try
15:57:03 ildikov mdbooth: the service_uuid field was added here: https://review.openstack.org/#/c/519025/
15:57:41 mdbooth ildikov: Just found it. I think it's orthogonal to multi-attach, tbh.
15:57:51 mriedem kashyap: although it's a bit confusing,
15:57:57 mriedem we dump the disk config before trying to attach the device
15:58:00 mriedem and it says type="raw"
15:58:00 mriedem http://paste.openstack.org/show/638081/
15:58:51 mdbooth ildikov: Still a good idea. I could also remove my NFS locking stuff in Nova if we had that, although my NFS locking is finer grained but significantly more complex.
15:59:09 ildikov mdbooth: it was added along with a shared_targets field so we can use a lock in case the target exported by the back end is shared among volumes/attachments
15:59:35 mdbooth ildikov: I get it. It's a good idea, I just don't see the relationship to multi-attach.
15:59:51 kashyap mriedem: (Aside - we both wrote almost same comment 4 mins apart)
15:59:54 ildikov mdbooth: it is supposed to help to solve our detach problems
15:59:59 kashyap mriedem: Looking at your paste
16:00:00 mdbooth We hit this with or without multi-attach.
16:00:17 ildikov mdbooth: as if the target is shared and gets removed with the first attachment then the remaining attachments are screwed
16:00:30 mdbooth ildikov: Right, but you can can do that without multi-attach.
16:00:52 ildikov mdbooth: and the Nova patches using it are dependencies to multi-attach as it might be a problem otherwise as well
16:00:53 mdbooth Multi-attach doesn't even make the problem particularly worse.
16:01:35 ildikov mdbooth: well, I got it in referring to multi-attach, but I guess the path doesn't matter once you got where you wanted... :)
16:01:54 efried jaypipes mriedem cdent alex_xu Draft: http://paste.openstack.org/show/638080/ -- anything missing/incorrect/silly?
16:02:26 cdent efried: will look in a mo, thanks for doing that
16:02:36 mdbooth ildikov: Hehe, I hear you :)
16:04:39 mriedem mdbooth: is it just me or is libvirt.images_type, use_cow_images and force_raw_images set of options totally confusing?
16:04:40 stephenfin alex_xu: Done (https://review.openstack.org/#/c/530284/(
16:18:28 efried mriedem Log processing & coloring is handled in the openstack-infra/os-loganalyze project (but I think you knew that). Is there something in particular you're having trouble finding in there?
16:19:23 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Deprecate 'network_manager' https://review.openstack.org/530923
16:19:33 mriedem efried: what in infra actually calls os-loganalyze to format the logs
16:20:16 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Use new-style choice values https://review.openstack.org/530924
16:20:40 efried mriedem Ah - it's an apache plugin thingy. The files aren't actually modified - they get twiddled on the fly when you do your http request.
16:21:18 efried mriedem So you just need to muck with the filters to make sure they're being run on the files you're interested in.
16:22:21 efried mriedem If you have access to a log server, you can play by fiddling with the os-loganalyze source in place, restarting the apache server, and then reloading your browser.
16:27:51 mriedem i don't
16:44:52 cdent efried: information seems accurate at the detail level, but feels like it needs some kind of executive summary or something, a kind of "here's what we're trying to accomplish" which is then followed by the "and this is how it is being done" (which is what you've already got)
16:45:45 efried cdent Okay. I felt like it was already getting kinda long, but... tough :)
16:46:21 cdent I think length is useful in this case because there's been not enough in the way of spec, so this kind of stands in for that
16:50:17 clarkb mriedem: the test framework for it runs without apache
16:52:23 openstackgerrit Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: vSCSI https://review.openstack.org/526094
16:53:53 hrw https://marcin.juszkiewicz.com.pl/2018/01/04/today-i-was-fighting-with-nova-no-idea-who-won/
16:53:57 hrw kashyap: ^^
16:54:15 kashyap hrw: On a call, will read :-)
16:54:36 hrw kashyap: thx ;)
16:54:43 kashyap Damn, I already clicked
16:58:49 hrw kashyap: :D
16:58:50 kashyap hrw: It looks fine; there's an extra tab in there
16:59:00 kashyap "It" == the review
16:59:31 rybridges Hey guys, can anyone confirm for sure whether or not ephemeral GB is part of a VMs snapshot?
17:00:07 hrw kashyap: can you mark it in comment? I am unable to find it locally
17:00:19 kashyap hrw: Done.
17:00:22 kashyap It's extra tab
17:00:30 kashyap In the rel note file
17:00:55 hrw rught
17:01:04 hrw 4 spaces to be exact ;d
17:01:09 hrw that's why I did not ofund
17:01:37 openstackgerrit Marcin Juszkiewicz proposed openstack/nova master: libvirt: use 'host-passthrough' as default on AArch64 https://review.openstack.org/530965
17:01:50 hrw done
17:03:43 efried cdent (jaypipes) Howzat: http://paste.openstack.org/raw/638137/
17:21:12 mnaser is stable/pike ci broken?
17:21:16 mriedem yes

Earlier   Later