| *** tetsuro has joined #openstack-nova | 00:35 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix deprecated base64.decodestring warning https://review.openstack.org/610401 | 00:40 |
|---|---|---|
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix deprecated base64.decodestring warning https://review.openstack.org/610401 | 00:41 |
| *** erlon has joined #openstack-nova | 00:43 | |
| openstackgerrit | Merged openstack/python-novaclient master: Enable split logging for cinder-novaclient interaction https://review.openstack.org/581218 | 00:45 |
| *** erlon has quit IRC | 00:47 | |
| *** brinzhang has joined #openstack-nova | 00:48 | |
| *** dave-mccowan has joined #openstack-nova | 01:04 | |
| *** Dinesh_Bhor has joined #openstack-nova | 01:21 | |
| *** cfriesen has quit IRC | 01:23 | |
| *** zhanglong has joined #openstack-nova | 01:30 | |
| *** mrsoul has quit IRC | 01:30 | |
| *** yikun has joined #openstack-nova | 01:47 | |
| *** mhen has quit IRC | 01:51 | |
| *** dave-mccowan has quit IRC | 01:53 | |
| *** mhen has joined #openstack-nova | 01:54 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova stable/queens: Remove unnecessary redirect https://review.openstack.org/607402 | 02:23 |
| *** hongbin has joined #openstack-nova | 02:24 | |
| openstackgerrit | Tushar Patil proposed openstack/nova-specs master: Support filtering of allocation_candidates by forbidden aggregates https://review.openstack.org/609960 | 02:28 |
| openstackgerrit | Tao Li proposed openstack/nova master: Cleanup the instance when MessageDeliveryFailure exception https://review.openstack.org/608500 | 02:43 |
| *** hshiina has joined #openstack-nova | 02:46 | |
| *** psachin has joined #openstack-nova | 02:56 | |
| *** TuanDA has joined #openstack-nova | 02:59 | |
| *** TuanDA has quit IRC | 02:59 | |
| *** TuanDA has joined #openstack-nova | 03:01 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Migrate nova v2.0 legacy job to zuulv3 https://review.openstack.org/610403 | 03:02 |
| *** psachin has quit IRC | 03:11 | |
| *** hoangcx has joined #openstack-nova | 03:20 | |
| *** whoami-rajat has joined #openstack-nova | 03:34 | |
| *** med_ has quit IRC | 03:48 | |
| openstackgerrit | Mike Chen proposed openstack/nova master: doc: update metadata service doc https://review.openstack.org/602593 | 03:54 |
| *** udesale has joined #openstack-nova | 03:57 | |
| *** tetsuro has quit IRC | 03:57 | |
| *** kevinbenton has quit IRC | 04:00 | |
| *** kevinbenton has joined #openstack-nova | 04:00 | |
| *** hshiina has quit IRC | 04:04 | |
| *** hshiina has joined #openstack-nova | 04:05 | |
| *** udesale has quit IRC | 04:07 | |
| *** hshiina has quit IRC | 04:12 | |
| *** hshiina_ has joined #openstack-nova | 04:12 | |
| *** udesale has joined #openstack-nova | 04:20 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add descriptions of error cases https://review.openstack.org/610429 | 04:24 |
| *** Dinesh_Bhor has quit IRC | 04:25 | |
| *** hongbin has quit IRC | 04:28 | |
| *** psachin has joined #openstack-nova | 04:29 | |
| *** hemna has quit IRC | 04:49 | |
| *** hemna has joined #openstack-nova | 04:52 | |
| *** Dinesh_Bhor has joined #openstack-nova | 05:01 | |
| *** TuanDA has quit IRC | 05:06 | |
| *** Dinesh_Bhor has quit IRC | 05:11 | |
| *** hoonetorg has quit IRC | 05:11 | |
| *** Dinesh_Bhor has joined #openstack-nova | 05:16 | |
| *** hshiina_ has quit IRC | 05:24 | |
| *** ratailor has joined #openstack-nova | 05:26 | |
| *** hoonetorg has joined #openstack-nova | 05:29 | |
| *** ccamacho has joined #openstack-nova | 05:36 | |
| *** rmart04 has joined #openstack-nova | 05:46 | |
| *** tetsuro has joined #openstack-nova | 05:50 | |
| *** rmart04 has quit IRC | 06:03 | |
| *** rmart04 has joined #openstack-nova | 06:03 | |
| *** Luzi has joined #openstack-nova | 06:10 | |
| *** dpawlik has joined #openstack-nova | 06:20 | |
| *** rmart04 has left #openstack-nova | 06:23 | |
| *** skatsaounis has joined #openstack-nova | 06:27 | |
| *** adrianc has joined #openstack-nova | 06:29 | |
| *** pooja_jadhav has joined #openstack-nova | 06:31 | |
| *** pcaruana has quit IRC | 06:37 | |
| *** bhagyashris has joined #openstack-nova | 06:41 | |
| *** slaweq has joined #openstack-nova | 06:43 | |
| *** Bhujay has joined #openstack-nova | 06:47 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova-specs master: Support initial allocation ratios https://review.openstack.org/552105 | 06:48 |
| *** Bhujay has quit IRC | 06:49 | |
| *** mtreinish has quit IRC | 06:55 | |
| *** elod is now known as elod_off | 07:01 | |
| *** tetsuro has quit IRC | 07:01 | |
| *** tetsuro has joined #openstack-nova | 07:01 | |
| *** pcaruana has joined #openstack-nova | 07:02 | |
| *** tetsuro has quit IRC | 07:02 | |
| *** rcernin has quit IRC | 07:04 | |
| *** moshele has joined #openstack-nova | 07:06 | |
| *** rcernin has joined #openstack-nova | 07:07 | |
| *** adrianc_ has joined #openstack-nova | 07:09 | |
| *** maciejjozefczyk has joined #openstack-nova | 07:10 | |
| *** leakypipes has quit IRC | 07:10 | |
| *** leakypipes has joined #openstack-nova | 07:11 | |
| *** adrianc has quit IRC | 07:13 | |
| *** ralonsoh has joined #openstack-nova | 07:17 | |
| *** helenafm has joined #openstack-nova | 07:18 | |
| *** hshiina has joined #openstack-nova | 07:19 | |
| *** adrianc__ has joined #openstack-nova | 07:20 | |
| *** threestrands has quit IRC | 07:20 | |
| *** rcernin has quit IRC | 07:21 | |
| *** adrianc_ has quit IRC | 07:24 | |
| *** rpittau has joined #openstack-nova | 07:29 | |
| *** giblet is now known as gibi | 07:30 | |
| *** jangutter has joined #openstack-nova | 07:35 | |
| *** alexchadin has joined #openstack-nova | 07:41 | |
| *** xek has joined #openstack-nova | 07:55 | |
| *** brinzhang has quit IRC | 07:57 | |
| *** brinzhang has joined #openstack-nova | 07:57 | |
| *** takashin has left #openstack-nova | 08:00 | |
| *** ttsiouts has joined #openstack-nova | 08:04 | |
| *** xek has quit IRC | 08:05 | |
| *** ttsiouts has quit IRC | 08:06 | |
| bauzas | good morning Nova | 08:13 |
| gibi | good morning bauzas :) | 08:14 |
| *** ttsiouts has joined #openstack-nova | 08:17 | |
| bauzas | :) | 08:18 |
| *** tssurya has joined #openstack-nova | 08:19 | |
| *** k_mouza has joined #openstack-nova | 08:20 | |
| *** ttsiouts has quit IRC | 08:22 | |
| *** ttsiouts has joined #openstack-nova | 08:24 | |
| BlackDex | Hello there. How can i force a specific flavor to be only used on specific compute nodes? I have 2 compute nodes which have ssd local storage and i want to have a flavor which will use that local storage so no volumes, and only run on those two hypervisors. So if i select this flavor, it will only start on those two hypervisors. | 08:25 |
| BlackDex | Can i do this, and how? | 08:25 |
| *** tssurya has quit IRC | 08:33 | |
| *** Bhujay has joined #openstack-nova | 08:36 | |
| *** Bhujay has quit IRC | 08:37 | |
| *** Bhujay has joined #openstack-nova | 08:37 | |
| *** derekh has joined #openstack-nova | 08:41 | |
| gibi | BlackDex: make sure that filter scheduler configured to use AggregateInstanceExtraSpecsFilter, create a host aggregate, put the two hosts in the aggregate, add the same metadata to the aggregate and the flavor extra_spec. See https://blog.russellbryant.net/2013/05/21/availability-zones-and-host-aggregates-in-openstack-compute-nova/ (ignore the availability zone as that is not needed for you now) | 08:42 |
| BlackDex | gibi: Oke, thx i will check that out :) | 08:43 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Consider nested allocations during allocation cleanup https://review.openstack.org/606050 | 08:44 |
| *** hshiina has quit IRC | 08:52 | |
| *** tetsuro has joined #openstack-nova | 09:02 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Rewrite the console doc https://review.openstack.org/606148 | 09:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add minimal documentation for RDP consoles https://review.openstack.org/606992 | 09:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Add minimal documentation for MKS consoles https://review.openstack.org/606993 | 09:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Deprecate the nova-console service https://review.openstack.org/610075 | 09:04 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Deprecate the nova-xvpvncproxy service https://review.openstack.org/610076 | 09:04 |
| *** TuanDA has joined #openstack-nova | 09:06 | |
| *** ttsiouts has quit IRC | 09:11 | |
| *** ttsiouts has joined #openstack-nova | 09:12 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Consider nested allocations during allocation cleanup https://review.openstack.org/606050 | 09:13 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject forced move with nested source allocation https://review.openstack.org/605785 | 09:13 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125 | 09:13 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Handle allocations consuming only from the child RPs https://review.openstack.org/608298 | 09:13 |
| *** ttsiouts has quit IRC | 09:16 | |
| *** ttsiouts has joined #openstack-nova | 09:17 | |
| *** k_mouza has quit IRC | 09:20 | |
| *** helenafm has quit IRC | 09:30 | |
| BlackDex | An other question. Is it possible to have multiple local storage disks per instance? | 09:30 |
| BlackDex | kind of multiple local ephemeral disks on one instance | 09:33 |
| *** adrianc__ has quit IRC | 09:35 | |
| *** lpetrut has joined #openstack-nova | 09:35 | |
| *** adrianc_ has joined #openstack-nova | 09:35 | |
| *** k_mouza has joined #openstack-nova | 09:39 | |
| gibi | BlackDex: in https://developer.openstack.org/api-ref/compute/#create-server look at the block_device_mapping_v2.source_type attribute. You can do a source_type=blank destination_type=local bdm to add additional local sotrage | 09:45 |
| gibi | storage | 09:45 |
| BlackDex | indeed ;) | 09:45 |
| *** Bhujay has quit IRC | 09:47 | |
| *** bhagyashris has quit IRC | 09:50 | |
| *** s10 has joined #openstack-nova | 09:54 | |
| *** imacdonn has quit IRC | 09:54 | |
| *** Bhujay has joined #openstack-nova | 09:54 | |
| *** imacdonn has joined #openstack-nova | 09:54 | |
| *** tssurya has joined #openstack-nova | 09:55 | |
| *** Bhujay has quit IRC | 09:55 | |
| *** Bhujay has joined #openstack-nova | 09:56 | |
| *** TuanDA has quit IRC | 10:00 | |
| *** adrianc_ has quit IRC | 10:03 | |
| *** erlon has joined #openstack-nova | 10:03 | |
| *** adrianc_ has joined #openstack-nova | 10:06 | |
| *** s1061123 has quit IRC | 10:18 | |
| *** s1061123_ has joined #openstack-nova | 10:19 | |
| *** ttsiouts has quit IRC | 10:19 | |
| *** brinzh has joined #openstack-nova | 10:22 | |
| *** brinzhang has quit IRC | 10:25 | |
| *** leakypipes is now known as jaypipes | 10:26 | |
| jaypipes | fried_rice: hey, I'll be boarding a plane during the scheduler meeting today, just FYI | 10:26 |
| *** zhanglong has quit IRC | 10:26 | |
| *** psachin has quit IRC | 10:32 | |
| s10 | Should https://review.openstack.org/#/c/603044/ be backported to the stable/pike? | 10:38 |
| *** psachin has joined #openstack-nova | 10:56 | |
| *** dave-mccowan has joined #openstack-nova | 11:01 | |
| *** psachin has quit IRC | 11:10 | |
| *** brinzh has quit IRC | 11:14 | |
| *** jamesdenton has joined #openstack-nova | 11:24 | |
| *** udesale has quit IRC | 11:24 | |
| *** dtantsur is now known as dtantsur|brb | 11:30 | |
| *** ratailor has quit IRC | 11:32 | |
| *** mvkr has quit IRC | 11:35 | |
| *** ttsiouts has joined #openstack-nova | 11:40 | |
| *** Dinesh_Bhor has quit IRC | 11:48 | |
| *** fnordahl has quit IRC | 11:49 | |
| *** fnordahl has joined #openstack-nova | 11:49 | |
| *** artom has joined #openstack-nova | 11:51 | |
| *** dpawlik has quit IRC | 12:07 | |
| *** mvkr has joined #openstack-nova | 12:09 | |
| *** moshele has quit IRC | 12:11 | |
| *** k_mouza has quit IRC | 12:13 | |
| *** dpawlik has joined #openstack-nova | 12:14 | |
| *** ttsiouts has quit IRC | 12:25 | |
| *** tetsuro has quit IRC | 12:32 | |
| *** ttsiouts has joined #openstack-nova | 12:35 | |
| *** jroll has quit IRC | 12:38 | |
| *** jroll has joined #openstack-nova | 12:39 | |
| *** derekh has quit IRC | 12:46 | |
| *** k_mouza has joined #openstack-nova | 12:46 | |
| *** k_mouza_ has joined #openstack-nova | 12:47 | |
| *** k_mouza has quit IRC | 12:50 | |
| *** jaypipes has quit IRC | 12:55 | |
| *** dtantsur|brb is now known as dtantsur | 12:59 | |
| openstackgerrit | Ivaylo Mitev proposed openstack/nova master: VMware: OVA and StrOpt images as VM templates https://review.openstack.org/609736 | 13:00 |
| *** awaugama has joined #openstack-nova | 13:01 | |
| *** fried_rice is now known as efried | 13:02 | |
| openstackgerrit | Merged openstack/nova-specs master: Remove force flag from live-migrate and evacuate https://review.openstack.org/609330 | 13:04 |
| *** helenafm has joined #openstack-nova | 13:15 | |
| *** mvkr has quit IRC | 13:15 | |
| *** mriedem has joined #openstack-nova | 13:19 | |
| *** lbragstad has joined #openstack-nova | 13:20 | |
| *** takashin has joined #openstack-nova | 13:20 | |
| *** k_mouza has joined #openstack-nova | 13:23 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add microversion 2.67 to rest api version history https://review.openstack.org/610349 | 13:25 |
| *** k_mouza_ has quit IRC | 13:26 | |
| *** SteelyDan is now known as dansmith | 13:28 | |
| *** tetsuro has joined #openstack-nova | 13:28 | |
| *** dave-mccowan has quit IRC | 13:42 | |
| *** eharney has joined #openstack-nova | 13:47 | |
| openstackgerrit | Sean McGinnis proposed openstack/nova master: Increment versioning with pbr instruction https://review.openstack.org/610534 | 13:47 |
| *** mvkr has joined #openstack-nova | 13:48 | |
| efried | n-sch meeting in 7 minutes in #openstack-meeting-alt | 13:52 |
| *** whoami-rajat has quit IRC | 13:53 | |
| *** cdent has joined #openstack-nova | 13:56 | |
| *** Luzi has quit IRC | 13:58 | |
| mriedem | dansmith: i didn't add a unit test to https://review.openstack.org/#/c/610098/ because the functional test covers it, so a unit test would be redundant. i can add one if you think it's necessary. | 14:02 |
| dansmith | mriedem: well, I think the functional is a looong way away from the actual thing and it'd be really hard to connect the test with that change in even a month's time | 14:04 |
| mriedem | alright | 14:04 |
| dansmith | those kinds of special behaviors seem important to unit test if for no other reason than to document that we have specific behaviors here for a reason | 14:04 |
| *** Bhujay has quit IRC | 14:06 | |
| *** whoami-rajat has joined #openstack-nova | 14:07 | |
| *** alexchadin has quit IRC | 14:08 | |
| *** munimeha1 has joined #openstack-nova | 14:12 | |
| *** tetsuro has quit IRC | 14:12 | |
| artom | mriedem, could I ask you to add https://review.openstack.org/#/c/587439/ to your list of volume/live migration things to look at? | 14:13 |
| artom | (Along with https://review.openstack.org/#/c/609517/ - not sure how stringent you're feeling about the splitting-out-the-method thing) | 14:14 |
| mriedem | i would at least split it out in a follow up | 14:15 |
| mriedem | b/c those live migration methods are a gd mess | 14:15 |
| *** mlavalle has joined #openstack-nova | 14:16 | |
| artom | Oh I definitely agree about that second part | 14:17 |
| artom | It's like the Energizer bunny - keeps going and going and going | 14:18 |
| artom | (Disclaimer: I am not sponsored by Energizer) | 14:18 |
| *** k_mouza has quit IRC | 14:23 | |
| *** k_mouza has joined #openstack-nova | 14:43 | |
| s10 | Should https://review.openstack.org/#/c/603044/ be backported to the stable/pike? | 14:46 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't persist RequestSpec.requested_destination https://review.openstack.org/610098 | 14:47 |
| efried | s10: Did you add those reviewers on there? | 14:47 |
| efried | You probably meant to add the stable team. | 14:47 |
| efried | https://review.openstack.org/#/admin/groups/540,members | 14:48 |
| *** spatel has joined #openstack-nova | 14:48 | |
| *** hshiina has joined #openstack-nova | 14:49 | |
| openstackgerrit | Corey Bryant proposed openstack/os-vif master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610068 | 14:50 |
| s10 | efried: I don't remember, I definitely added myself. Thank you. | 14:50 |
| openstackgerrit | Corey Bryant proposed openstack/osc-placement master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610074 | 14:51 |
| mriedem | when you cherry pick using the gerrit UI it adds the original reviewers | 14:52 |
| mriedem | i think | 14:52 |
| mriedem | i might be thinking of revert | 14:52 |
| efried | yeah, cherry pick doesn't do that, but revert does. | 14:52 |
| mriedem | you definitely don't need to explicitly add the entire stable core team to a change | 14:52 |
| mriedem | ^ is kind of obnoxious | 14:52 |
| dansmith | extremely | 14:52 |
| s10 | efried: probably i did that. nova-stable-maint adds 17 people. | 14:52 |
| s10 | efried: cherry pick adds only the author of patchset. | 14:53 |
| *** dpawlik has quit IRC | 14:54 | |
| openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances https://review.openstack.org/609779 | 15:01 |
| *** macza has joined #openstack-nova | 15:01 | |
| *** beekneemech is now known as bnemec | 15:01 | |
| *** Bhujay has joined #openstack-nova | 15:02 | |
| *** efried has quit IRC | 15:05 | |
| *** takashin has left #openstack-nova | 15:06 | |
| aspiers | artom: thanks again for the feedback - I think patch set 5 of this ^^^^ is much better than 4 was | 15:09 |
| mriedem | dansmith: don't forget https://review.openstack.org/#/c/610088/ :) | 15:11 |
| *** efried has joined #openstack-nova | 15:13 | |
| dansmith | mriedem: oh sorry | 15:13 |
| dansmith | thought I had already done that one last week | 15:14 |
| *** Bhujay has quit IRC | 15:14 | |
| mriedem | i need some nova core reviews for more upgrade checker docs for people working on the community wide goal https://review.openstack.org/#/c/604486/ | 15:16 |
| mriedem | hopefully easy | 15:16 |
| mriedem | at some point these probably move over to the upgrade check library docs | 15:16 |
| stephenfin | mriedem: Docs? I can take a look | 15:17 |
| *** macza has quit IRC | 15:18 | |
| *** macza has joined #openstack-nova | 15:19 | |
| *** s10 has quit IRC | 15:20 | |
| *** ttsiouts has quit IRC | 15:21 | |
| openstackgerrit | Adam Spiers proposed openstack/nova-specs master: Add spec for libvirt driver launching AMD SEV-encrypted instances https://review.openstack.org/609779 | 15:22 |
| *** dpawlik has joined #openstack-nova | 15:23 | |
| *** dpawlik has quit IRC | 15:27 | |
| dansmith | mriedem: are you going to fix stephenfin's literal nits/ | 15:30 |
| dansmith | if so I'll wait and hit it when you do that | 15:30 |
| dansmith | or i can fix it for you and then +W if you're in the middle of something else | 15:30 |
| *** Bhujay has joined #openstack-nova | 15:32 | |
| artom | aspiers, you're welcome - I'll read it over again soon (no firm date promises, sorry!) | 15:32 |
| *** munimeha1 has quit IRC | 15:33 | |
| aspiers | artom: thanks! IIUC it would need to be merged by Oct 25 (s-1 milestone) in order to make it in for Stein, right? | 15:35 |
| aspiers | really hoping that's possible - but I know it's tight ... | 15:36 |
| *** melwitt has joined #openstack-nova | 15:36 | |
| artom | aspiers, it's doable - you basically have to harass the spec-cores ;) | 15:37 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add some more docs for upgrade checkers https://review.openstack.org/604486 | 15:37 |
| aspiers | artom: ah OK ;-) and who are those lucky people? | 15:37 |
| mriedem | dansmith: done | 15:37 |
| melwitt | 15:37 | |
| *** jmlowe has quit IRC | 15:37 | |
| *** tbachman has joined #openstack-nova | 15:37 | |
| artom | aspiers, https://review.openstack.org/#/admin/groups/302,members | 15:37 |
| aspiers | yeah, I just realised Gerrit would probably tell me ... | 15:37 |
| artom | For the traits stuff jaypipes would be your man, along with efried, though he's not specs-core (yet?) | 15:38 |
| aspiers | OK cool, well jay, dansmith, johnthetubaguy, and mriedem are already on the reviewer list | 15:39 |
| aspiers | I'm not sure whether it would be fair to spam the *whole* specs-core team ;-) | 15:40 |
| *** ttsiouts has joined #openstack-nova | 15:40 | |
| artom | aspiers, it's a fine line between persistent and pest ;) | 15:41 |
| aspiers | indeed X-D | 15:41 |
| artom | Honestly though, it's hard | 15:42 |
| efried | artom, aspiers: What are we talking about? | 15:42 |
| aspiers | efried: https://review.openstack.org/#/c/609779/ | 15:42 |
| efried | ... | 15:42 |
| artom | efried, aspiers has a spec that deals with traits, I figures you might want to look | 15:42 |
| efried | Just looking at the title, is this a thing like: | 15:43 |
| efried | - virt driver sets a "I_CAN_HAZ_SEV" trait | 15:43 |
| efried | - op sets trait:I_CAN_HAZ_SEV=required | 15:43 |
| efried | - scheduler schedules to a host that has I_CAN_HAZ_SEV | 15:43 |
| efried | - virt driver sees the trait and flips a switch in the VM | 15:43 |
| efried | ? | 15:43 |
| *** hshiina has quit IRC | 15:43 | |
| artom | efried, pretty much | 15:44 |
| aspiers | yes | 15:44 |
| efried | I see mriedem referred you to the HPET thread. If it's a similar deal, then structure it like that --^ and you should be able to get a reasonably fast approval. | 15:44 |
| aspiers | exactly, except the switch to flip is *slightly* involved due to the extra libvirt XML required | 15:45 |
| aspiers | and also it's not as simple as just looking for a flag in /proc/cpuinfo (but it's not that much more complicated either) | 15:45 |
| mriedem | i haven't reviewed the hpet spec fwiw | 15:46 |
| aspiers | efried: yeah, the recent patch sets of my spec already link to that HPET thread | 15:46 |
| aspiers | and point out that the proposal aligns with the HPET consensus | 15:46 |
| mriedem | speaking of, | 15:46 |
| mriedem | why do we have two HPET specs? | 15:46 |
| mriedem | https://review.openstack.org/#/c/571111/ https://review.openstack.org/#/c/607989/ | 15:46 |
| *** openstackgerrit has quit IRC | 15:47 | |
| efried | mriedem: TPM != HPET? | 15:47 |
| aspiers | is https://wiki.openstack.org/wiki/Nova/Stein_Release_Schedule authoritative? | 15:47 |
| mriedem | sob | 15:47 |
| mriedem | aspiers: yes | 15:47 |
| aspiers | OK cool | 15:47 |
| *** openstackgerrit has joined #openstack-nova | 15:47 | |
| aspiers | thought I heard somewhere else that s-1 milestone was Oct 22 | 15:47 |
| aspiers | but Oct 25 gives me 3 extra days of precious breathing space ;-) | 15:48 |
| efried | Spec freeze is 1/10 | 15:48 |
| efried | at s-2 | 15:48 |
| aspiers | oh, so it doesn't even need to be merged until Jan 10? | 15:49 |
| mriedem | yes we have several months to ignore your spec | 15:49 |
| aspiers | LOL | 15:50 |
| aspiers | can't wait to enjoy those peaceful months! | 15:50 |
| artom | You can't say that to new people! At least put an /s or ;) or something | 15:50 |
| artom | /s ;) | 15:50 |
| aspiers | hahahahaha | 15:50 |
| aspiers | hrm, I might even be able to code the thing before Jan 10 | 15:51 |
| mriedem | yes we have several months to ignore your spec >:( | 15:51 |
| aspiers | I don't expect it's gonna be that hard | 15:51 |
| aspiers | so maybe you'll even be able to ignore not just my spec, but my patches too! | 15:51 |
| mriedem | that was supposed to be angry eyebrows | 15:52 |
| aspiers | and if you need help with that, I remember corvus pointing out that gertty makes ignoring reviews easier than ever ;-) | 15:52 |
| melwitt | speaking of ignoring specs, I've been thinking whether we should organize a spec review day for next week ahead of s-1 (even though s-1 is not the spec freeze) | 15:53 |
| aspiers | maybe he can even add a -3 hotkey specially for jaypipes ... | 15:53 |
| *** munimeha1 has joined #openstack-nova | 15:53 | |
| *** phuongnh has joined #openstack-nova | 15:54 | |
| mriedem | melwitt: yes we likely need a spec review day; i know i haven't been doing a good job of reviewing specs for stein. | 15:59 |
| *** helenafm has quit IRC | 16:04 | |
| melwitt | we usually do them on tuesday. are y'all game for a spec review day next tuesday oct 23? alex_xu dansmith efried johnthetubaguy stephenfin bauzas | 16:05 |
| efried | wfm | 16:05 |
| *** mrch_ has joined #openstack-nova | 16:05 | |
| *** gyee has joined #openstack-nova | 16:09 | |
| *** ttsiouts has quit IRC | 16:10 | |
| *** ttsiouts has joined #openstack-nova | 16:11 | |
| *** manjeets has joined #openstack-nova | 16:14 | |
| *** ttsiouts has quit IRC | 16:15 | |
| *** jmlowe has joined #openstack-nova | 16:17 | |
| openstackgerrit | Corey Bryant proposed openstack/osc-placement master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610074 | 16:18 |
| openstackgerrit | Corey Bryant proposed openstack/os-vif master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610068 | 16:19 |
| openstackgerrit | Jan Gutter proposed openstack/os-vif master: Extend host_info to cover port profiles https://review.openstack.org/610636 | 16:21 |
| *** cfriesen has joined #openstack-nova | 16:24 | |
| openstackgerrit | Corey Bryant proposed openstack/os-traits master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610065 | 16:24 |
| *** cdent has left #openstack-nova | 16:32 | |
| *** lpetrut has quit IRC | 16:37 | |
| *** tssurya has quit IRC | 16:39 | |
| mriedem | tuesday the 23rd works for me | 16:40 |
| *** k_mouza has quit IRC | 16:40 | |
| *** Bhujay has quit IRC | 16:46 | |
| *** dpawlik has joined #openstack-nova | 16:48 | |
| *** dpawlik has quit IRC | 16:50 | |
| *** dpawlik has joined #openstack-nova | 16:50 | |
| *** moshele has joined #openstack-nova | 16:50 | |
| melwitt | I'll send a mail to the ML proposing next tuesday to get some async responses | 16:54 |
| *** mvkr has quit IRC | 17:04 | |
| *** lpetrut has joined #openstack-nova | 17:04 | |
| *** k_mouza has joined #openstack-nova | 17:06 | |
| *** k_mouza has quit IRC | 17:10 | |
| *** itlinux has joined #openstack-nova | 17:12 | |
| *** dpawlik has quit IRC | 17:18 | |
| *** dpawlik_ has joined #openstack-nova | 17:18 | |
| *** phuongnh has quit IRC | 17:18 | |
| *** moshele has quit IRC | 17:19 | |
| *** moshele has joined #openstack-nova | 17:36 | |
| openstackgerrit | Merged openstack/nova master: Use nova-consoleauth only if workaround enabled https://review.openstack.org/607068 | 17:36 |
| *** panda is now known as panda|off | 17:37 | |
| *** dtantsur is now known as dtantsur|afk | 17:38 | |
| openstackgerrit | melanie witt proposed openstack/nova stable/rocky: Use nova-consoleauth only if workaround enabled https://review.openstack.org/610673 | 17:38 |
| *** moshele has quit IRC | 17:39 | |
| openstackgerrit | Merged openstack/nova master: Add microversion 2.67 to rest api version history https://review.openstack.org/610349 | 17:42 |
| *** mrch_ has quit IRC | 17:45 | |
| *** mvkr has joined #openstack-nova | 17:47 | |
| *** moshele has joined #openstack-nova | 17:50 | |
| *** moshele has quit IRC | 17:54 | |
| *** erlon has quit IRC | 17:56 | |
| *** erlon_ has joined #openstack-nova | 17:56 | |
| mriedem | efried: jackding: a few things in https://review.openstack.org/#/c/607989/ | 17:57 |
| mriedem | mostly to just tighten up that spec | 17:57 |
| jackding | mriedem, will have a look thanks | 18:00 |
| *** moshele has joined #openstack-nova | 18:03 | |
| *** adrianc_ has quit IRC | 18:04 | |
| *** dpawlik_ has quit IRC | 18:06 | |
| efried | mriedem, jackding: Responded to a few of them. | 18:07 |
| openstackgerrit | Corey Bryant proposed openstack/osc-placement master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610074 | 18:10 |
| openstackgerrit | Corey Bryant proposed openstack/os-traits master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610065 | 18:10 |
| openstackgerrit | Corey Bryant proposed openstack/os-vif master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610068 | 18:11 |
| coreycb | melwitt: hi, you may have noticed we started proposing patches to enable py37 unit tests and disable py35 unit tests. just wanted to give you a heads up to ensure it's not a surprise. just a few more to go. | 18:17 |
| melwitt | coreycb: I have noticed and thank you for the heads up :) | 18:18 |
| openstackgerrit | Corey Bryant proposed openstack/nova master: Change python3.5 job to python3.7 job on Stein+ https://review.openstack.org/610694 | 18:19 |
| mriedem | efried: it wasn't clear to me the compute code was going to be setting the trait | 18:21 |
| efried | mriedem: Guess that's a thing to be called out in the spec, then :p | 18:21 |
| *** dpawlik has joined #openstack-nova | 18:21 | |
| mriedem | yeah | 18:21 |
| mriedem | otherwise this is all straight-forward | 18:21 |
| efried | I took it for granted based on the code, but that shouldn't be a prereq for understanding the spec of course | 18:22 |
| *** dpawlik has quit IRC | 18:26 | |
| mriedem | artom: a few small things in https://review.openstack.org/#/c/609517/ | 18:40 |
| artom | mriedem, much thanks, will address now! | 18:41 |
| *** lpetrut has quit IRC | 18:43 | |
| *** jmlowe has quit IRC | 18:44 | |
| *** moshele has quit IRC | 18:45 | |
| mriedem | artom: also, something is a false positive in the tests | 18:46 |
| artom | mriedem, how so? | 18:46 |
| *** ralonsoh has quit IRC | 18:46 | |
| mriedem | i moved the try/except around only the terminate_connection call and both tests passed | 18:46 |
| mriedem | but the attachment_delete one should have failed | 18:46 |
| artom | Dammi :( OK, I'll look into that | 18:47 |
| artom | (I *did* do that with a previous version, I swear!) | 18:47 |
| openstackgerrit | Merged openstack/nova master: Add some more docs for upgrade checkers https://review.openstack.org/604486 | 18:49 |
| mriedem | Kevin_Zheng: i replied to your -1 in https://review.openstack.org/#/c/608777/ - are you still -1 on that or can it be dropped? | 18:57 |
| *** awaugama has quit IRC | 19:00 | |
| mriedem | this seems...excessive: http://logstash.openstack.org/#dashboard/file/logstash.json?query=message%3A%5C%22detaching%20network%20adapter%20failed.%3A%20libvirtError%3A%20internal%20error%3A%20End%20of%20file%20from%20qemu%20monitor%5C%22%20AND%20tags%3A%5C%22screen-n-cpu.txt%5C%22&from=7d | 19:04 |
| *** mdrabe has joined #openstack-nova | 19:05 | |
| *** c06 has joined #openstack-nova | 19:05 | |
| *** c06 has left #openstack-nova | 19:05 | |
| mriedem | oh i see, | 19:07 |
| mriedem | we're tearing down, we get a network-vif-deleted event right before the guest is deleted in the hypervisor | 19:07 |
| mriedem | doesn't "End of file from qemu monitor" usually indicate the guest is gone? | 19:08 |
| *** erlon_ has quit IRC | 19:12 | |
| *** erlon has joined #openstack-nova | 19:12 | |
| openstackgerrit | Jack Ding proposed openstack/nova-specs master: High Precision Event Timer (HPET) on x86 guests https://review.openstack.org/607989 | 19:13 |
| *** awaugama has joined #openstack-nova | 19:16 | |
| *** awaugama has quit IRC | 19:18 | |
| *** awaugama has joined #openstack-nova | 19:19 | |
| *** jmlowe has joined #openstack-nova | 19:19 | |
| *** sapd1_ has joined #openstack-nova | 19:27 | |
| artom | mriedem, ah, it's because my poor man's mock checks for volume_id, but is called with an attachment_id. | 19:29 |
| artom | So it doesn't actually raise | 19:29 |
| mriedem | yeah... | 19:30 |
| mriedem | so track call count and raise count? | 19:31 |
| *** sapd1 has quit IRC | 19:31 | |
| * mriedem moves | 19:32 | |
| *** mriedem has quit IRC | 19:32 | |
| *** mriedem has joined #openstack-nova | 19:35 | |
| *** moshele has joined #openstack-nova | 19:35 | |
| mriedem | jackding: just a couple of small updates in that spec | 19:38 |
| mriedem | for hpet | 19:38 |
| *** jobewan has joined #openstack-nova | 19:38 | |
| mriedem | btw, how does the libvirt driver know it can report the hpet trait? | 19:38 |
| mriedem | something on the host capabilities? | 19:39 |
| *** moshele has quit IRC | 19:39 | |
| dansmith | I think so yeah | 19:40 |
| imacdonn | dansmith mriedem efried: please re-review this - would like to get it wrapped up, so I can look at applying the fix to cinder too.... https://review.openstack.org/608091 | 19:47 |
| efried | imacdonn: My hands are off that one. It's down to dansmith and mriedem. | 19:47 |
| imacdonn | efried: OK. The logic did change a bit, so thought you might want to look again | 19:48 |
| dansmith | I've had it open in the queue | 19:48 |
| dansmith | I keep coming up with excuses :) | 19:48 |
| imacdonn | :) | 19:48 |
| efried | imacdonn: I would, if I weren't completely underwater, or if there weren't already two cores on it who are way more familiar with the domain. | 19:48 |
| imacdonn | efried: understood - thanks! | 19:49 |
| jackding | mriedem: I'm not familiar with traits, but I thought that's something virt driver could query the host from. | 19:51 |
| mriedem | jackding: efried: well efried said the virt driver would report the hpet trait, | 19:53 |
| mriedem | but looking at the nova code change, i don't see it doing anything like that | 19:54 |
| efried | mriedem: The code change hasn't caught up yet. | 19:54 |
| efried | What I said earlier was based on the fact that the virt driver code is doing the error checking. | 19:54 |
| mriedem | so...we need to configure the guest with a timer element https://libvirt.org/formatdomain.html#elementsTime | 19:56 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Handle volume API failure in _post_live_migration https://review.openstack.org/609517 | 19:56 |
| mriedem | i guess it already does, but always sets it to false | 19:56 |
| dansmith | imacdonn: you're going to love me | 19:58 |
| imacdonn | uh oh | 19:58 |
| dansmith | imacdonn: but I -1d for a single character change | 19:58 |
| dansmith | imacdonn: however, fix it fast and I shall stand by for +2ing | 19:59 |
| imacdonn | dang .. ok | 19:59 |
| mriedem | jackding: i guess it's not clear to me how the libvirt driver will determine that the underlying hypervisor supports hpet | 19:59 |
| mriedem | is it just assumed that all x86 hosts support hpet? | 20:00 |
| openstackgerrit | iain MacDonnell proposed openstack/nova master: Handle online_data_migrations exceptions https://review.openstack.org/608091 | 20:00 |
| artom | Back when I was compiling my own kernels this was an option you could disable, so I don't think we should assume that | 20:00 |
| imacdonn | dansmith: how fast is fast? :) | 20:01 |
| dansmith | imacdonn: that fast | 20:01 |
| imacdonn | cool | 20:01 |
| dansmith | artom: correct | 20:01 |
| jackding | in our implementation we assume x86 support hpet | 20:01 |
| mriedem | blech | 20:02 |
| mriedem | this is when i need kashyap | 20:02 |
| mriedem | i realize windriver can be super opinionated on the stuff they deploy, but not so much upstream | 20:02 |
| dansmith | mriedem: if all x86 could be assumed to have it, you wouldn't need this trait, just require arch=x86 | 20:04 |
| mriedem | good point | 20:05 |
| artom | So, hardware-wise, it's apparently as easy as [artom@jayne ~]$ cat /sys/devices/system/clocksource/clocksource0/available_clocksource | 20:08 |
| artom | tsc hpet acpi_pm | 20:08 |
| artom | But. Do we need anything from libvirt? | 20:08 |
| mriedem | i was hoping something would be in the host capabilities API | 20:09 |
| mriedem | http://logs.openstack.org/39/587439/4/check/tempest-full/fdb1cf9/controller/logs/screen-n-cpu.txt.gz#_Sep_11_14_44_27_658449 | 20:09 |
| mriedem | but i don't see that | 20:09 |
| mriedem | or maybe hpet just doesn't show up there | 20:10 |
| mriedem | ? | 20:10 |
| mriedem | because it's nested virt? | 20:10 |
| dansmith | mriedem: I don't see it on my physical box either | 20:10 |
| mriedem | physical box is kvm or qemu? | 20:11 |
| mriedem | not sure if that matters | 20:11 |
| artom | The SEV guys had to dig into a commit to see how libvirt reports SEV | 20:11 |
| artom | (https://libvirt.org/git/?p=libvirt.git;a=commit;h=6688393c6b222b5d7cba238f21d55134611ede9c) | 20:11 |
| dansmith | mriedem: physical box meaning baremetal | 20:12 |
| artom | Their domain capabilities docs aren't exchaustive :/ https://libvirt.org/html/libvirt-libvirt-domain.html#virConnectGetDomainCapabilities | 20:12 |
| *** tbachman_ has joined #openstack-nova | 20:14 | |
| *** tbachman has quit IRC | 20:14 | |
| *** tbachman_ is now known as tbachman | 20:14 | |
| cfriesen | mriedem: I think HPET is supported on x86 since like 2005. | 20:14 |
| artom | So, on a physical machine with an HPET, virsh capabilities doesn't include any mention of it | 20:14 |
| cfriesen | on qemu at least, my understanding is that HPET is always emulated using other clocksources | 20:15 |
| dansmith | cfriesen: is that's the case, then what is the point of the trait? | 20:15 |
| dansmith | if there aren't any x86-capable hypervisors that don't support HPET we shouldn't need it at all right? | 20:16 |
| dansmith | presumably if you don't have HPET enabled on your kernel and you're using libvirt for lxc, then just keying off of "is x86" isn't enough | 20:17 |
| mriedem | artom: yeah this was close to the same thing https://www.reddit.com/r/VFIO/comments/5g6rxb/a_fix_for_windows_vm_performancestuttering_when/ | 20:17 |
| cfriesen | dansmith: I'm told that some operators might not want to enable HPET for all guests for security reasons. | 20:17 |
| cfriesen | dansmith: so I was kind of assuming that we'd have a hypervisor specific config option that when turned off would prevent the trait from being advertised | 20:17 |
| dansmith | cfriesen: so what's the key? | 20:17 |
| dansmith | dear god | 20:17 |
| dansmith | I think we're maybe uncovering that this is a super tiny line item on one person's wish list :) | 20:18 |
| mriedem | so we're back to my original question in the spec, which is some external thing can set hpet on the compute node resource provider | 20:18 |
| dansmith | mriedem: I don't remember where we landed with efried on being able to set custom traits on compute-managed providers | 20:19 |
| mriedem | this is a standard trait | 20:19 |
| dansmith | I thought the current thinking was that those would get wiped | 20:19 |
| mriedem | or being proposed as one | 20:19 |
| dansmith | not what I meant | 20:19 |
| mriedem | we'll merge them | 20:19 |
| dansmith | I don't mean custom I mean random | 20:19 |
| cfriesen | dansmith: the note at the PTG was literally "some way to request an HPET in the guest for x86 guests" :) | 20:19 |
| mriedem | things that the virt driver reports + custom set external things get merged | 20:19 |
| dansmith | meaning set by the operator, unknown to the compute service | 20:19 |
| efried | It's the issue of "trait ownership" | 20:19 |
| dansmith | efried: right | 20:20 |
| efried | It sucks, but basically we need to decide whether the HPET trait is owned by compute or not. | 20:20 |
| mriedem | also discussed at some length related to https://review.openstack.org/#/c/538498/ | 20:20 |
| efried | If it is, then the user *can't* set it. | 20:20 |
| efried | if they do, and compute disagrees, compute will wipe it. | 20:20 |
| efried | and vice versa. | 20:20 |
| dansmith | right, so did we end up being able to declare some as compute-manage and some not, on a given provider? | 20:21 |
| mriedem | it sounds like the only way the libvirt driver can know is if it execs "cat /sys/devices/system/clocksource/clocksource0/available_clocksource" on startup | 20:21 |
| efried | unless we figure some override in the (as yet vaporware) provider customization file... | 20:21 |
| mriedem | and greps for hpet | 20:21 |
| efried | dansmith: Otherwise, it's basically just hardcoded, and we would have to document it somewhere. | 20:21 |
| dansmith | mriedem: even still cfriesen is arguing that it's an operator decision, not a property of the host | 20:21 |
| cfriesen | mriedem: that's whether the host kernel has the hpet clocksource available. I don't think that impacts whether it can provide an emulated HPET to the guest. | 20:21 |
| efried | except for custom traits, which we can namespace like we talked about in denver. | 20:21 |
| openstackgerrit | Matthew Edmonds proposed openstack/nova master: Use tempfile for powervm config drive https://review.openstack.org/610174 | 20:22 |
| mriedem | namespace custom traits? | 20:22 |
| mriedem | i thought CUSTOM_* was it? | 20:22 |
| *** jobewan has quit IRC | 20:22 | |
| mriedem | dansmith: yeah b/c some ops want to disable it... | 20:22 |
| dansmith | yeah, so I definitely thought this was going to be an actual capability, not just a "is this turned on" sort of thing when we last discussed it | 20:23 |
| dansmith | not even is it turned on, but is it in policy for this host | 20:23 |
| efried | mriedem: namespaced to signify the owner. Like if the trait.startswith('CUSTOM_NOVA_') then the op isn't allowed to touch it. Otherwise, fair game. | 20:24 |
| efried | ...until CUSTOM_CYBORG_ is a thing, etc. | 20:24 |
| efried | but at least then we have to have a short hardcoded list of possible owners and their namespaces, rather than a long hardcoded list of individual traits (or trait regexes) that are system-owned. | 20:24 |
| mriedem | are there any cases where we'll have CUSTOM_NOVA_* traits that aren't just standard traits in os-traits? | 20:24 |
| mriedem | CUSTOM_NOVA_* sounds pretty not custom to me | 20:25 |
| efried | mriedem: An example would be a trait signifying the PCI address of a device. | 20:25 |
| mriedem | is that a trait? | 20:26 |
| efried | it's not a *capability* | 20:26 |
| efried | But it is a *trait*. | 20:26 |
| efried | Dammit. | 20:26 |
| mriedem | it's a....thing | 20:26 |
| mriedem | what's the point of that being a trait? so we can schedule to a specific PCI address? | 20:27 |
| cfriesen | is it schedulable? | 20:27 |
| efried | yes | 20:27 |
| efried | pets, baby. | 20:27 |
| mriedem | huh | 20:27 |
| efried | more realistically, trait for vendor/product ID. | 20:27 |
| cfriesen | but can't the IOMMU redirect to whatever virtual address is desired? | 20:27 |
| mriedem | i seem to remember people saying we weren't going to allow scheduling to that level of granularity, like down to host numa node and such | 20:28 |
| efried | I do this weekly now, look up 'trait' in the dictionary to assure myself that 'capability' is nowhere in the definition. Also not in the thesaurus. | 20:28 |
| cfriesen | wouldn't "pci devices with a given vendor/product-id" be a countable resource ? | 20:28 |
| efried | cfriesen: "PCI device" is the countable resource. "Vendor ID" and "Product ID" are traits. (But not capabilities) | 20:29 |
| efried | I mean, arguably they're metadata (key/value). | 20:29 |
| efried | But since we don't have that... | 20:29 |
| cfriesen | I'd suggest that a whole set of capabilities are implied by the vendor/product | 20:29 |
| efried | Completely agree. | 20:29 |
| efried | But that overlaps with a set of capabilities implied by a different vendor/product. | 20:30 |
| efried | In a perfect world, we would have a zillion individual traits for each of those capabilities | 20:30 |
| efried | and I would schedule based on those capabilities | 20:30 |
| efried | but | 20:30 |
| efried | it is not a perfect world | 20:30 |
| efried | where vendors are going to supply us handy dandy tables of those capabilities | 20:30 |
| cfriesen | ick...then you'd need to specify which drivers are in your image, and suport all of them. | 20:31 |
| efried | so we have to make do with what we've got. | 20:31 |
| mriedem | so this is the thing where because we don't have metadata on resource providers, we're going to have super-encoded traits right? | 20:31 |
| efried | yeah, something like that. | 20:31 |
| mriedem | CUSTOM_NOVA_PCI_VENDOR_FOO_PRODUCT_BAR? | 20:31 |
| efried | because then I can schedule based on required=in:{buncha vendor/product IDs that have the capabilities I need}, whereas if it was encoded in the resource class, that would not be possible. | 20:31 |
| efried | mriedem: CUSTOM_NOVA_PCI_VENDORID_FOO and CUSTOM_NOVA_PCI_PRODUCTID_BAR, separate traits, but yeah. | 20:32 |
| cfriesen | efried: is there a logical AND in there? since different vendors can have the same product ID | 20:32 |
| efried | cfriesen: The boolean logic that's possible with `required` is not complete, but should be rich enough to get where you need to be. | 20:33 |
| cfriesen | efried: I think you'd need something like ((A AND B) OR (C AND D)) | 20:34 |
| efried | btw, this idea is far from having any kind of consensus behind it. There are certain people who will need to be cold underground before it becomes reality. | 20:34 |
| efried | cfriesen: True, and I don't think you can quiiite get there actually. What are the odds of ((A or C) AND (B OR D)) winding up meaningfully different in such a way that it actually hits a wrong device? | 20:35 |
| efried | i.e. ((vendorA or vendorC) AND (productB or productD)) when what you really want is ((vendorA and productB) or (vendorC and productD)) => what are the chances that (vendorA+productD) or (vendorB+productC) actually exist? | 20:36 |
| cfriesen | well, each vendor ID is unique, but I think product IDs are only unique within a given vendor | 20:39 |
| cfriesen | but we may be lucky and not have overlap for commonly-used devices. | 20:40 |
| *** spatel has quit IRC | 20:47 | |
| edleafe | efried: don't put too much weight on the name 'trait'. That itself was a compromise betwee those who wanted them named 'capability' and those who wanted 'tag' | 20:48 |
| *** eharney has quit IRC | 20:48 | |
| edleafe | efried: It was always supposed to represent capabilities | 20:48 |
| efried | I'll certainly admit that if they were called 'capabilities' I would not argue as strongly to put stuff like product IDs in them. | 20:51 |
| efried | But they ain't. | 20:51 |
| efried | Juliet had something to say about this. And look how that turned out. | 20:52 |
| edleafe | "A capability by any other name would still generate semantic arguments..." | 20:53 |
| cfriesen | so where are we at for HPET (and vTPM)? I'd suggest that for vTPM it's actually a capability since it depends on hypervisor version and such. For HPET it's two things: 1) asking for it where currently it's turned off unconditionally, and 2) maybe allowing operators to refuse to provide it. | 20:53 |
| mriedem | cfriesen: my current question on the hpet spec is who sets the trait | 20:54 |
| mriedem | i assumed it was set externally but was told the virt driver would set it, so started pulling the thread of how the virt driver would set it since that wasn't in the spec | 20:55 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: don't log error if guest gone during interface detach https://review.openstack.org/610727 | 20:56 |
| cfriesen | mriedem: so as far as I know the LibvirtDriver when used with qemu/kvm could unconditionally set it, unless we want to add a config option to allow operators to explicitly enable it. | 20:57 |
| cfriesen | mriedem: with qemu it seems to be purely emulated, unrelated to the host capabilities | 20:57 |
| cfriesen | with other hypervisors I have no idea | 20:58 |
| cfriesen | presumably with LXC or Xen it might use the real HPET | 20:58 |
| mriedem | i don't really care about lxc honestly, we don't test it | 20:58 |
| mriedem | lyarwood: methinks you have duplicate forum sessions https://www.openstack.org/summit/berlin-2018/summit-schedule/global-search?t=yarwood | 21:04 |
| cfriesen | mriedem: dansmith: looks like this was the original reason why HPET was removed unconditionally from x86 guests: https://blueprints.launchpad.net/nova/+spec/add-no-hpet-option-into-guest-clock | 21:10 |
| *** erlon has quit IRC | 21:23 | |
| *** awaugama has quit IRC | 21:33 | |
| lbragstad | is anyone here planning on going to https://www.openstack.org/summit/berlin-2018/summit-schedule/#day=2018-11-13 ? | 21:41 |
| lbragstad | sorry - wrong link https://www.openstack.org/summit/berlin-2018/summit-schedule/events/22785/change-of-ownership-of-resources | 21:42 |
| lbragstad | i noticed it's at the same time as the project update | 21:42 |
| efried | which project update? | 21:45 |
| lbragstad | nova's | 21:46 |
| mriedem | artom: and i've left comments in https://review.openstack.org/#/c/587439/ | 21:47 |
| mriedem | as requested | 21:47 |
| mriedem | lbragstad: yes i plan on going to that, and assume dansmith will also | 21:52 |
| dansmith | yarp | 21:52 |
| lbragstad | ok - so that over the project update i assume? | 21:52 |
| lbragstad | nova's project update* | 21:53 |
| mriedem | ask melwitt | 21:54 |
| mriedem | i'd like to heckle at both | 21:54 |
| mriedem | if possible | 21:54 |
| lbragstad | same - minus the heckling | 21:55 |
| *** openstackgerrit has quit IRC | 21:56 | |
| mriedem | stephenfin has a presentation at the same time too | 21:56 |
| mriedem | https://www.openstack.org/summit/berlin-2018/summit-schedule/events/22582/how-to-double-the-performance-of-vswitch-based-deployments | 21:56 |
| *** openstackgerrit has joined #openstack-nova | 21:57 | |
| mriedem | hmm, for some reason my schedule isn't showing me all the things i've marked either.. | 21:57 |
| lbragstad | ? | 21:57 |
| mriedem | oh nvm | 21:58 |
| lbragstad | did you turn it off and back on again? | 21:58 |
| mriedem | switching between days and tracks | 21:58 |
| mriedem | i jiggled the handle | 21:58 |
| *** openstackgerrit has quit IRC | 21:58 | |
| *** openstackgerrit has joined #openstack-nova | 21:59 | |
| *** slaweq has quit IRC | 22:05 | |
| artom | mriedem, much thanks! Supper time for me now, but I'll try to address both patches before sleep | 22:06 |
| *** slaweq has joined #openstack-nova | 22:11 | |
| *** openstackgerrit has quit IRC | 22:12 | |
| *** openstackgerrit has joined #openstack-nova | 22:13 | |
| *** slaweq has quit IRC | 22:15 | |
| *** mlavalle has quit IRC | 22:16 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Move live_migration.pre.start to the start of the method https://review.openstack.org/610739 | 22:20 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Reject forced move with nested source allocation https://review.openstack.org/605785 | 22:25 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Run negative server moving tests with nested RPs https://review.openstack.org/604125 | 22:25 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Handle allocations consuming only from the child RPs https://review.openstack.org/608298 | 22:26 |
| mriedem | dansmith: you probably want to get these pike backports of that host aggregate case name revert thing https://review.openstack.org/#/q/topic:bug/1709260+(status:open) | 22:30 |
| *** munimeha1 has quit IRC | 22:34 | |
| *** rcernin has joined #openstack-nova | 22:41 | |
| *** efried has quit IRC | 22:43 | |
| *** openstackgerrit has quit IRC | 22:43 | |
| *** itlinux has quit IRC | 22:54 | |
| *** mriedem has quit IRC | 23:00 | |
| *** dklyle has quit IRC | 23:04 | |
| *** mchlumsky has quit IRC | 23:10 | |
| *** slaweq has joined #openstack-nova | 23:11 | |
| *** slaweq has quit IRC | 23:16 | |
| *** macza_ has joined #openstack-nova | 23:18 | |
| *** macza has quit IRC | 23:22 | |
| *** tbachman has quit IRC | 23:22 | |
| *** macza_ has quit IRC | 23:23 | |
| alex_xu | melwitt: I'm good at Oct 23 | 23:24 |
| *** tbachman has joined #openstack-nova | 23:24 | |
| *** macza has joined #openstack-nova | 23:27 | |
| *** dklyle has joined #openstack-nova | 23:30 | |
| *** tbachman has quit IRC | 23:31 | |
| *** macza has quit IRC | 23:32 | |
| *** macza has joined #openstack-nova | 23:44 | |
| *** macza has quit IRC | 23:45 | |
| *** macza has joined #openstack-nova | 23:46 | |
| *** openstackgerrit has joined #openstack-nova | 23:48 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Handle volume API failure in _post_live_migration https://review.openstack.org/609517 | 23:48 |
| *** macza has quit IRC | 23:50 | |
| *** gyee has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!