| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-07 | |||
| 21:31:10 | mriedem | maybe if you guys didn't break the api contract | |
| 21:31:12 | mriedem | ... | |
| 21:31:29 | dansmith | nobody uses simple_tenant_usage | |
| 21:31:30 | mriedem | dansmith: comments are in the nova-status change also | |
| 21:31:34 | dansmith | yep, I saw | |
| 21:34:15 | openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Add live-migration-force-after-timeout spec https://review.openstack.org/438467 | |
| 21:38:35 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | |
| 21:38:36 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | |
| 21:38:38 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | |
| 21:38:39 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | |
| 21:38:41 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | |
| 21:38:41 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | |
| 21:38:44 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | |
| 21:47:13 | mriedem | dansmith: questions about sort order and cell0 in here https://review.openstack.org/#/c/441388/ | |
| 21:47:21 | mriedem | i think cell0 is fine but wanted to ask it out loud | |
| 21:52:52 | openstackgerrit | melanie witt proposed openstack/nova master: Count resources to check quota for cells https://review.openstack.org/416521 | |
| 21:52:53 | openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | |
| 21:52:53 | openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | |
| 21:58:26 | mtreinish | mriedem: good waffles do stick together :) | |
| 22:07:44 | openstackgerrit | Sujitha proposed openstack/nova master: [WIP] Reduce calls to load_tags() to 0 https://review.openstack.org/435146 | |
| 22:08:10 | openstackgerrit | Sujitha proposed openstack/nova master: Reduce calls to load_tags() to 0 https://review.openstack.org/435146 | |
| 22:12:20 | inc0 | hey guys, so we've run into serious issue with new libvirt 2.5 | |
| 22:12:55 | inc0 | issue being nova, when creating files for vms (like console.log), uses creates directory with permissions 744 | |
| 22:13:12 | inc0 | so if you run nova-compute with different user than qemu, what we do in Kolla | |
| 22:13:30 | inc0 | qemu can't access console.log which makes spawning vm impossible | |
| 22:14:02 | inc0 | and with libvirt 2.5 for reasons unknown to me, dynamic permissiosn seems to be failing to chown these files | |
| 22:14:55 | inc0 | anyone seen that happening? can we specify permissions for vm files (i.e. change them to 774)? | |
| 22:32:17 | openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | |
| 22:33:10 | mriedem | inc0: you'll probably need to talk to kashyap and/or sgordon on that one, maybe mdbooth - some of the red hat people, | |
| 22:33:22 | mriedem | given the time of day kashyap and mdbooth are probably no longer around, not sure about sgordon | |
| 22:33:48 | mriedem | we're nowhere near testing with libvirt 2.5 in the gate | |
| 22:34:18 | mriedem | we test with 1.3.1 which is what's in xenial | |
| 22:34:43 | inc0 | yes...but not in cloud archive | |
| 22:34:49 | openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Add live-migration-force-after-timeout spec https://review.openstack.org/438467 | |
| 22:34:56 | inc0 | we will probably fall back to 1.3 ourselves | |
| 22:35:13 | openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | |
| 22:35:38 | mriedem | inc0: we == kolla? | |
| 22:35:46 | inc0 | mriedem: correct | |
| 22:36:22 | inc0 | I'm not sure when canonical bumped libvirt version this high..but here we are | |
| 22:36:24 | mriedem | ok. i'm not sure how you guys pick what your use or support, but definitely using anything beyond what we test with in the integrated gate using xenial opens you up to risk for stuff like this | |
| 22:36:45 | mriedem | inc0: might as well report a bug regardless | |
| 22:37:00 | inc0 | well yeah, but cloud archive is where you're supposed to pick it up if you install openstack from apt | |
| 22:37:07 | inc0 | and there is libvirt out there too | |
| 22:37:15 | inc0 | not sure if it's libvirt issue or nova issue | |
| 22:37:21 | mriedem | kashyap is good at figuring out what changed in upstream libvirt and why they made such changes, in case they are not backward compatible | |
| 22:37:32 | inc0 | seems more like libvirt tbh..dynamic ownership used to work wel | |
| 22:37:34 | inc0 | l | |
| 22:37:44 | mriedem | i'm assuming it was a conscious decision | |
| 22:38:03 | mriedem | dynamic ownership sounds like a security exposure that they'd close, but i don't know much about it | |
| 22:38:05 | inc0 | kashyap lives in which timezone? | |
| 22:38:19 | mriedem | i think he's in bulgaria? | |
| 22:38:22 | mriedem | dansmith: ^? | |
| 22:38:27 | dansmith | netherlands | |
| 22:38:32 | mriedem | or that | |
| 22:38:37 | mriedem | idk, somewhere in europe :) | |
| 22:38:40 | mriedem | it's all the same right?! | |
| 22:38:49 | inc0 | well, Europe, thanks, I'll try to poke him tomorrow morning | |
| 22:38:55 | mriedem | yw | |
| 22:38:56 | inc0 | no, not the same:) | |
| 22:38:59 | inc0 | but close enough:) | |
| 22:39:32 | mriedem | i asked a russian at the ptg if they vacation in crimea, which apparently was a faux pas...so par for the course | |
| 22:40:18 | inc0 | don't worry, it's getting confusing, in Poland for example time difference to UK changes from 1hr to 2hrs every 6 months | |
| 22:40:43 | mriedem | that's the same in the US | |
| 22:40:46 | mriedem | except for arizona | |
| 22:40:51 | inc0 | daylight savings \o/ | |
| 22:40:59 | inc0 | saves no daylight since 1800s | |
| 22:41:00 | mriedem | yeah which is here in another week or so | |
| 22:41:15 | mriedem | it saves me from being able to sleep properly | |
| 22:42:38 | sdague | that reminds me, I really need to find another venue for my "when will then be now" talk about time in software :) | |
| 22:43:45 | openstackgerrit | Eric Brown proposed openstack/python-novaclient master: Remove py34 tox env and pypi classifier https://review.openstack.org/442847 | |
| 22:43:57 | mriedem | sdague: as in not AZ? | |
| 22:44:34 | sdague | it included all kinds of fun about dates, timezones, dst, the leapsecond | |
| 22:45:51 | inc0 | duh running qemu with nova user fixes it | |
| 22:46:00 | inc0 | but it's no bueno security | |
| 22:47:04 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: List instances using Searchlight (spec) https://review.openstack.org/441692 | |
| 22:47:10 | inc0 | anyway guys, please check out libvirt 2.5.0, it potentially breaks lots of stuff | |
| 22:48:14 | inc0 | I mean it's not horrible if only we could make vm files being group-writable | |
| 22:50:35 | inc0 | I think when nova creates dirs for instance it uses default umask which would be 722, if we could make it 772, that would fix it | |
| 23:11:07 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Fix paths when moving specs https://review.openstack.org/432470 | |
| 23:11:07 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Fix a couple of spec names https://review.openstack.org/432469 | |
| 23:11:08 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Move implemented specs for Ocata https://review.openstack.org/432471 | |
| 23:17:11 | dansmith | mriedem: did you see my question about the sort order thing? | |
| 23:18:06 | dansmith | mriedem: I think the sort order is just there to ensure a stable marker search within a single query, not for the overall sort of the results | |
| 23:20:41 | mriedem | looking | |
| 23:22:29 | dansmith | ah yeah diana just confirmed | |
| 23:23:43 | mriedem | me too | |
| 23:23:47 | mriedem | the marker is the uuid | |
| 23:23:52 | mriedem | and i sort of remember talking about this in the spec, | |
| 23:24:09 | mriedem | that uuid being a marker is silly because you can create a new instance and it's random uuid could mess up your marker/sort order | |
| 23:24:31 | dansmith | meh, pagination is lossy anyway | |
| 23:24:39 | dansmith | that doesn't bother me | |
| 23:24:59 | dansmith | so I'm good to push right? | |
| 23:25:07 | mriedem | umm | |
| 23:25:12 | mriedem | so isn't she saying we should be sorting? | |
| 23:25:17 | mriedem | at least by project_id? | |
| 23:25:25 | dansmith | no? | |
| 23:25:37 | mriedem | i'm going to read https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/paginate-simple-tenant-usage.html quick | |
| 23:25:47 | mriedem | https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/paginate-simple-tenant-usage.html#data-model-impact | |
| 23:26:13 | dansmith | the only reason we'd need to sort the results is if you care about the sort order as a user right? | |
| 23:26:47 | mriedem | then why was it added to https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/paginate-simple-tenant-usage.html#data-model-impact? | |