| *** zbr has quit IRC | 00:09 | |
| *** slaweq has joined #openstack-nova | 00:11 | |
| *** slaweq has quit IRC | 00:16 | |
| *** maciejjozefczyk has quit IRC | 00:22 | |
| *** ociuhandu has joined #openstack-nova | 00:28 | |
| *** ociuhandu has quit IRC | 00:39 | |
| *** mriedem has quit IRC | 00:44 | |
| *** yoctozepto has quit IRC | 00:53 | |
| *** yoctozepto has joined #openstack-nova | 00:54 | |
| *** dviroel has joined #openstack-nova | 00:59 | |
| *** nanzha has joined #openstack-nova | 01:02 | |
| *** yoctozepto has quit IRC | 01:05 | |
| *** yoctozepto has joined #openstack-nova | 01:06 | |
| *** Liang__ has joined #openstack-nova | 01:10 | |
| *** ociuhandu has joined #openstack-nova | 01:14 | |
| *** ociuhandu has quit IRC | 01:18 | |
| *** lbragstad_ has joined #openstack-nova | 01:24 | |
| *** lbragstad has quit IRC | 01:26 | |
| *** ociuhandu has joined #openstack-nova | 01:57 | |
| *** nweinber__ has joined #openstack-nova | 01:59 | |
| *** ociuhandu has quit IRC | 02:08 | |
| *** slaweq has joined #openstack-nova | 02:11 | |
| *** nanzha has quit IRC | 02:15 | |
| *** slaweq has quit IRC | 02:16 | |
| *** nanzha has joined #openstack-nova | 02:16 | |
| *** mkrai has joined #openstack-nova | 02:32 | |
| openstackgerrit | Merged openstack/nova master: Plumb allow_cross_cell_resize into compute API resize() https://review.opendev.org/635684 | 02:42 |
|---|---|---|
| *** ociuhandu has joined #openstack-nova | 02:46 | |
| *** nanzha has quit IRC | 02:47 | |
| *** nanzha has joined #openstack-nova | 02:48 | |
| *** ociuhandu has quit IRC | 02:51 | |
| *** ileixe has joined #openstack-nova | 02:55 | |
| *** rcernin_ has joined #openstack-nova | 03:00 | |
| *** tbachman has joined #openstack-nova | 03:03 | |
| *** rcernin has quit IRC | 03:03 | |
| *** tbachman_ has joined #openstack-nova | 03:08 | |
| *** tbachman has quit IRC | 03:08 | |
| *** tbachman_ is now known as tbachman | 03:08 | |
| *** brault has quit IRC | 03:13 | |
| *** brault has joined #openstack-nova | 03:20 | |
| *** rcernin_ has quit IRC | 03:26 | |
| *** rcernin has joined #openstack-nova | 03:26 | |
| *** tbachman has quit IRC | 03:26 | |
| *** tbachman has joined #openstack-nova | 03:29 | |
| *** bhagyashris has joined #openstack-nova | 03:40 | |
| *** udesale has joined #openstack-nova | 03:41 | |
| *** udesale has quit IRC | 03:45 | |
| *** udesale has joined #openstack-nova | 03:46 | |
| *** ivve has joined #openstack-nova | 03:46 | |
| *** nanzha has quit IRC | 03:48 | |
| *** nanzha has joined #openstack-nova | 03:48 | |
| *** zhubx has quit IRC | 03:51 | |
| *** zhubx has joined #openstack-nova | 03:51 | |
| *** slaweq has joined #openstack-nova | 04:11 | |
| *** slaweq has quit IRC | 04:16 | |
| *** yaawang has quit IRC | 04:20 | |
| *** yaawang has joined #openstack-nova | 04:23 | |
| *** nweinber__ has quit IRC | 04:28 | |
| *** tkajinam_ has joined #openstack-nova | 04:29 | |
| *** tkajinam has quit IRC | 04:32 | |
| *** tetsuro has joined #openstack-nova | 04:33 | |
| *** ociuhandu has joined #openstack-nova | 04:34 | |
| *** dviroel has quit IRC | 04:36 | |
| *** ociuhandu has quit IRC | 04:41 | |
| *** tkajinam_ has quit IRC | 04:58 | |
| *** tkajinam has joined #openstack-nova | 04:59 | |
| *** links has joined #openstack-nova | 05:12 | |
| *** ygk_12345 has joined #openstack-nova | 05:16 | |
| *** ygk_12345 has quit IRC | 05:19 | |
| *** ygk_12345 has joined #openstack-nova | 05:19 | |
| *** brinzhang_ has joined #openstack-nova | 05:25 | |
| *** nanzha has quit IRC | 05:26 | |
| *** slaweq has joined #openstack-nova | 05:26 | |
| *** brinzhang has quit IRC | 05:28 | |
| *** ratailor has joined #openstack-nova | 05:30 | |
| *** ociuhandu has joined #openstack-nova | 05:30 | |
| *** ociuhandu has quit IRC | 05:35 | |
| *** jraju__ has joined #openstack-nova | 05:36 | |
| *** links has quit IRC | 05:36 | |
| *** jraju__ has quit IRC | 05:36 | |
| *** links has joined #openstack-nova | 05:40 | |
| *** Luzi has joined #openstack-nova | 05:42 | |
| *** udesale has quit IRC | 05:43 | |
| *** udesale has joined #openstack-nova | 05:43 | |
| *** udesale has quit IRC | 05:44 | |
| *** udesale has joined #openstack-nova | 05:44 | |
| *** tbachman has quit IRC | 05:48 | |
| *** zhubx has quit IRC | 05:48 | |
| *** boxiang has joined #openstack-nova | 05:48 | |
| *** ivve has quit IRC | 05:59 | |
| *** takashin has left #openstack-nova | 06:33 | |
| *** lpetrut has joined #openstack-nova | 06:33 | |
| *** lpetrut has quit IRC | 06:34 | |
| *** lpetrut has joined #openstack-nova | 06:35 | |
| *** TxGirlGeek has joined #openstack-nova | 06:38 | |
| *** dpawlik has joined #openstack-nova | 06:49 | |
| *** jraju__ has joined #openstack-nova | 06:54 | |
| *** links has quit IRC | 06:55 | |
| *** dpawlik has quit IRC | 06:55 | |
| *** ivve has joined #openstack-nova | 07:03 | |
| *** udesale has quit IRC | 07:03 | |
| *** udesale has joined #openstack-nova | 07:04 | |
| *** chenhaw has joined #openstack-nova | 07:06 | |
| *** ociuhandu has joined #openstack-nova | 07:10 | |
| *** ociuhandu has quit IRC | 07:10 | |
| *** ociuhandu has joined #openstack-nova | 07:11 | |
| *** dpawlik has joined #openstack-nova | 07:14 | |
| *** ociuhandu has quit IRC | 07:16 | |
| *** rcernin has quit IRC | 07:22 | |
| *** trident has quit IRC | 07:37 | |
| *** mkrai has quit IRC | 07:42 | |
| openstackgerrit | Luyao Zhong proposed openstack/nova master: support live migration with vpmems https://review.opendev.org/687856 | 07:48 |
| *** ociuhandu has joined #openstack-nova | 07:48 | |
| *** trident has joined #openstack-nova | 07:48 | |
| *** ociuhandu has quit IRC | 07:54 | |
| *** damien_r has joined #openstack-nova | 07:57 | |
| bauzas | good morning Nova | 08:02 |
| * bauzas is back after 2 weeks :) | 08:02 | |
| *** adriant has quit IRC | 08:08 | |
| *** maciejjozefczyk has joined #openstack-nova | 08:09 | |
| *** tesseract has joined #openstack-nova | 08:11 | |
| *** mkrai has joined #openstack-nova | 08:14 | |
| *** ralonsoh has joined #openstack-nova | 08:14 | |
| *** awalende has joined #openstack-nova | 08:23 | |
| gibi | bauzas: good morning | 08:27 |
| gibi | bauzas: how is your jetlag? | 08:28 |
| *** zbr has joined #openstack-nova | 08:30 | |
| *** dpawlik has quit IRC | 08:31 | |
| *** dpawlik has joined #openstack-nova | 08:34 | |
| *** zbr has quit IRC | 08:35 | |
| bauzas | gibi: quite good, just woke up today at 6.30am :) | 08:35 |
| bauzas | gibi: and you ? | 08:35 |
| * bauzas needs to write emails FWIW | 08:36 | |
| *** zbr has joined #openstack-nova | 08:37 | |
| *** tkajinam has quit IRC | 08:37 | |
| *** dtantsur|afk is now known as dtantsur | 08:48 | |
| *** do3meli has joined #openstack-nova | 08:48 | |
| *** ileixe has quit IRC | 08:50 | |
| *** ileixe has joined #openstack-nova | 08:52 | |
| *** TxGirlGeek has quit IRC | 08:54 | |
| *** priteau has joined #openstack-nova | 08:58 | |
| *** rpittau|afk is now known as rpittau | 08:59 | |
| *** ociuhandu has joined #openstack-nova | 09:01 | |
| *** Luzi has quit IRC | 09:08 | |
| *** xek_ has joined #openstack-nova | 09:09 | |
| kashyap | lyarwood: efried: Back today. I haven't read the KM-long scrollback, afraid (buried under several things). | 09:22 |
| *** Luzi has joined #openstack-nova | 09:23 | |
| *** brinzhang has joined #openstack-nova | 09:23 | |
| *** brinzhang_ has quit IRC | 09:27 | |
| *** ociuhandu has quit IRC | 09:30 | |
| *** mkrai has quit IRC | 09:31 | |
| *** ociuhandu has joined #openstack-nova | 09:31 | |
| *** irclogbot_0 has quit IRC | 09:39 | |
| *** irclogbot_1 has joined #openstack-nova | 09:41 | |
| *** ociuhandu has quit IRC | 09:42 | |
| *** ociuhandu has joined #openstack-nova | 09:44 | |
| *** mkrai has joined #openstack-nova | 09:44 | |
| *** rcernin has joined #openstack-nova | 09:45 | |
| *** ileixe has quit IRC | 09:46 | |
| *** ileixe has joined #openstack-nova | 09:49 | |
| *** jaosorior has joined #openstack-nova | 09:51 | |
| *** ociuhandu has quit IRC | 09:53 | |
| *** ociuhandu has joined #openstack-nova | 09:54 | |
| *** ociuhandu has quit IRC | 09:55 | |
| *** ociuhandu has joined #openstack-nova | 09:56 | |
| *** ociuhandu has quit IRC | 09:57 | |
| *** ociuhandu has joined #openstack-nova | 09:58 | |
| *** ociuhandu has quit IRC | 10:00 | |
| *** ociuhandu has joined #openstack-nova | 10:01 | |
| *** ociuhandu has quit IRC | 10:02 | |
| *** ociuhandu has joined #openstack-nova | 10:03 | |
| *** awalende has quit IRC | 10:06 | |
| *** awalende has joined #openstack-nova | 10:06 | |
| *** links has joined #openstack-nova | 10:06 | |
| openstackgerrit | Liang Fang proposed openstack/nova-specs master: Support volume local cache https://review.opendev.org/689070 | 10:07 |
| *** jraju__ has quit IRC | 10:07 | |
| *** ociuhandu has quit IRC | 10:08 | |
| *** ygk_12345 has left #openstack-nova | 10:10 | |
| *** awalende has quit IRC | 10:11 | |
| *** rcernin has quit IRC | 10:11 | |
| *** mdbooth has joined #openstack-nova | 10:15 | |
| *** brinzhang_ has joined #openstack-nova | 10:16 | |
| *** Liang__ has quit IRC | 10:19 | |
| *** brinzhang has quit IRC | 10:20 | |
| *** tetsuro has quit IRC | 10:21 | |
| *** awalende has joined #openstack-nova | 10:23 | |
| *** panda has quit IRC | 10:24 | |
| *** ratailor_ has joined #openstack-nova | 10:26 | |
| *** ratailor has quit IRC | 10:27 | |
| *** panda has joined #openstack-nova | 10:29 | |
| *** vesper has quit IRC | 10:30 | |
| *** vesper11 has joined #openstack-nova | 10:31 | |
| *** brinzhang_ has quit IRC | 10:37 | |
| *** chenhaw has quit IRC | 10:42 | |
| *** dpawlik has quit IRC | 10:47 | |
| kashyap | Does anyone recall top off their head, AMD SEV support in Nova doesn't support live migration yet, does it? | 10:48 |
| *** ociuhandu has joined #openstack-nova | 10:48 | |
| kashyap | (I vaguely recall it doesn't.) | 10:48 |
| kashyap | I'm sure it's somewhere in the spec | 10:49 |
| *** rcernin has joined #openstack-nova | 10:50 | |
| *** awalende has quit IRC | 10:51 | |
| *** awalende has joined #openstack-nova | 10:51 | |
| kashyap | (Yep, my guess is correct - checked in the spec. Also needs support for LM-with-SEV from lower layers.) | 10:51 |
| *** awalende has quit IRC | 10:54 | |
| *** awalende has joined #openstack-nova | 10:54 | |
| *** ociuhandu has quit IRC | 10:54 | |
| *** bhagyashris has quit IRC | 10:54 | |
| *** ratailor__ has joined #openstack-nova | 10:55 | |
| *** ratailor_ has quit IRC | 10:57 | |
| *** gshippey has joined #openstack-nova | 11:00 | |
| *** awalende has quit IRC | 11:01 | |
| *** awalende has joined #openstack-nova | 11:02 | |
| gibi | bauzas: today was OK for me, yesterday I start my day at 4:45 | 11:03 |
| *** awalende has quit IRC | 11:06 | |
| *** awalende has joined #openstack-nova | 11:08 | |
| *** jraju__ has joined #openstack-nova | 11:09 | |
| *** jaosorior has quit IRC | 11:10 | |
| *** links has quit IRC | 11:10 | |
| *** xek_ has quit IRC | 11:12 | |
| *** dpawlik has joined #openstack-nova | 11:15 | |
| openstackgerrit | Merged openstack/nova master: Pass RequestContext to oslo_policy https://review.opendev.org/674038 | 11:17 |
| *** udesale has quit IRC | 11:17 | |
| openstackgerrit | Merged openstack/nova master: Add func test for 'required' PCI NUMA policy https://review.opendev.org/682941 | 11:17 |
| *** dpawlik has quit IRC | 11:19 | |
| openstackgerrit | Merged openstack/nova master: Resolve TODO in _remove_host_allocations https://review.opendev.org/693636 | 11:21 |
| *** rcernin has quit IRC | 11:26 | |
| *** dpawlik has joined #openstack-nova | 11:27 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Functional reproduce for bug 1852207 https://review.opendev.org/693817 | 11:29 |
| openstack | bug 1852207 in OpenStack Compute (nova) pike "reschedule ignores that requested availability zone" [Medium,Triaged] https://launchpad.net/bugs/1852207 - Assigned to Balazs Gibizer (balazs-gibizer) | 11:29 |
| gibi | elod: ^^ | 11:30 |
| *** brinzhang has joined #openstack-nova | 11:35 | |
| *** shilpasd has joined #openstack-nova | 11:44 | |
| *** bbowen has quit IRC | 11:46 | |
| *** do3meli has quit IRC | 11:50 | |
| *** dviroel has joined #openstack-nova | 11:58 | |
| *** maciejjozefczyk has quit IRC | 12:04 | |
| *** boxiang has quit IRC | 12:08 | |
| *** boxiang has joined #openstack-nova | 12:09 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: WIP: Support re-configure the delete_on_termination in server https://review.opendev.org/693828 | 12:10 |
| *** dpawlik has quit IRC | 12:12 | |
| *** shilpasd has quit IRC | 12:18 | |
| *** dave-mccowan has joined #openstack-nova | 12:19 | |
| *** do3meli has joined #openstack-nova | 12:31 | |
| *** awalende has quit IRC | 12:33 | |
| *** awalende has joined #openstack-nova | 12:33 | |
| *** awalende has quit IRC | 12:34 | |
| *** awalende has joined #openstack-nova | 12:34 | |
| *** mdbooth has quit IRC | 12:36 | |
| *** mdbooth has joined #openstack-nova | 12:37 | |
| *** dswebb has joined #openstack-nova | 12:38 | |
| *** dpawlik has joined #openstack-nova | 12:39 | |
| *** ratailor__ has quit IRC | 12:39 | |
| *** openstackgerrit has quit IRC | 12:41 | |
| *** mdbooth has quit IRC | 12:41 | |
| *** efried has quit IRC | 12:42 | |
| *** mdbooth has joined #openstack-nova | 12:42 | |
| *** dpawlik has quit IRC | 12:44 | |
| *** ociuhandu has joined #openstack-nova | 12:46 | |
| *** dpawlik has joined #openstack-nova | 12:47 | |
| *** efried has joined #openstack-nova | 12:49 | |
| *** brinzhang has quit IRC | 12:50 | |
| *** ociuhandu has quit IRC | 12:50 | |
| *** brinzhang has joined #openstack-nova | 12:51 | |
| *** jaosorior has joined #openstack-nova | 12:57 | |
| *** bbowen has joined #openstack-nova | 13:00 | |
| *** liuyulong has joined #openstack-nova | 13:02 | |
| jroll | efried: thanks, will hit it today | 13:04 |
| *** eharney has quit IRC | 13:04 | |
| *** efried has quit IRC | 13:05 | |
| *** henriqueof has quit IRC | 13:05 | |
| *** awalende has quit IRC | 13:06 | |
| *** awalende has joined #openstack-nova | 13:06 | |
| *** henriqueof has joined #openstack-nova | 13:08 | |
| *** awalende has quit IRC | 13:11 | |
| *** eharney has joined #openstack-nova | 13:12 | |
| *** ociuhandu has joined #openstack-nova | 13:16 | |
| *** awalende has joined #openstack-nova | 13:22 | |
| *** awalende has quit IRC | 13:24 | |
| *** awalende has joined #openstack-nova | 13:25 | |
| *** awalende has quit IRC | 13:28 | |
| *** awalende has joined #openstack-nova | 13:28 | |
| kashyap | git fetch gerrit | 13:36 |
| *** mriedem has joined #openstack-nova | 13:37 | |
| kashyap | Err | 13:37 |
| *** openstackgerrit has joined #openstack-nova | 13:37 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" spec https://review.opendev.org/693844 | 13:37 |
| *** ociuhandu has quit IRC | 13:38 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: Re-propose "Secure Boot support for KVM & QEMU guests" for Ussuri https://review.opendev.org/693844 | 13:38 |
| *** ociuhandu has joined #openstack-nova | 13:39 | |
| *** maciejjozefczyk has joined #openstack-nova | 13:40 | |
| *** ociuhandu has quit IRC | 13:44 | |
| *** jangutter has quit IRC | 13:48 | |
| *** nweinber__ has joined #openstack-nova | 13:48 | |
| mriedem | gibi: before i get started on rebasing the cross cell resize series and adding a change to just cast from api to conductor always ( https://review.opendev.org/#/c/635684/53/nova/compute/api.py@3852 ) i wanted to see if you're ok with that plan? | 13:49 |
| *** efried has joined #openstack-nova | 13:50 | |
| efried | Fun morning. About every 15 minutes, the power goes out for a half second. Just long enough to kill my router. | 13:50 |
| efried | Texas: "Ohh, noooo, it's three degrees below freezing, I'm dyyyyyying!" | 13:51 |
| mriedem | thoughts and prayers | 13:52 |
| efried | kashyap: Let me tldr it for you: what's the minimum qemu needed for *encrypted* emulated TPM? | 13:52 |
| kashyap | efried: Hiya; need to do some sleuthing. Let me do that... | 13:52 |
| efried | unencrypted was 2.11 (according to cfriesen) | 13:53 |
| efried | thanks kashyap | 13:53 |
| mriedem | we got a replacement tree put in over the weekend and you're supposed to water new trees but what about when the ground is frozen... | 13:53 |
| efried | not a huge gurry. | 13:53 |
| efried | mriedem: hot water? | 13:53 |
| kashyap | I'll not gurry. | 13:53 |
| * efried goes to figure out what a gurry is | 13:53 | |
| efried | "fish or whale offal". Nice. | 13:53 |
| kashyap | "the entrails of fish or whales" | 13:54 |
| * efried gets entry for typo of the day in early. | 13:54 | |
| efried | but we haven't seen sean-k-mooney yet, so... | 13:54 |
| efried | :P | 13:54 |
| mriedem | efried: i'm pretty sure hot water just freezes too | 13:54 |
| kashyap | Speaking of "offals" ... the other week I was in Lyon, France, for KVM Forum. They are all super big into "offals", apparently. (/me is a vegetarian; only heard it through the grapevine) | 13:54 |
| efried | That's... awfful | 13:55 |
| efried | (see what I did there?) | 13:55 |
| kashyap | Very pun | 13:55 |
| efried | Many cultures have an equivalent. Boudin sausage in Louisiana. Hot dogs in baseball stadiums... | 13:56 |
| efried | haggis in scotland | 13:56 |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Virtual instance rescue with stable disk devices https://review.opendev.org/693849 | 13:56 |
| *** dpawlik has quit IRC | 13:57 | |
| *** dpawlik has joined #openstack-nova | 13:58 | |
| kashyap | efried: I see. (Yeah, heard the "haggis" term echo around when in Edinburgh last year.) | 13:59 |
| *** awalende_ has joined #openstack-nova | 13:59 | |
| luyao | efried: hi Eric, I have a question, do I need a bp and separate spec for vpmem live migration? | 13:59 |
| *** Luzi has quit IRC | 14:00 | |
| efried | It must be like, at some point in their history all these cultures were starving, so they had to make use of every calorie they could scrounge up, and find a way to make it palatable. And then for reasons of... tradition? it remained in the culture long after it ceased to be necessary. | 14:00 |
| efried | Except for the hot dog thing. That's just because baseball fans are stupid and crazy. | 14:00 |
| *** awalende has quit IRC | 14:02 | |
| *** ociuhandu has joined #openstack-nova | 14:02 | |
| *** priteau has quit IRC | 14:04 | |
| dswebb | Hi,have a bit of an odd issue with Livemigration on rocky. I've got a 3 node nova cluster using iscsi -> nimble as my storage. I can successfully migrate a VM from node 1 to node 2 (or node2 to node1) once. If I try and migrate the vm back to the node it started on (or any node it's been on prior) it fails silently to migrate and the only logs I see that suggest any issue are the following: https://pastebin.com/Y2ccf3MR | 14:05 |
| gibi | mriedem: sorry I have to get back to you tomorrow (my) morning | 14:05 |
| *** dpawlik has quit IRC | 14:06 | |
| mriedem | gibi: ok i'll just post the change since it'll be simple and i need to rebase the series anyway, if you have a problem with it you and dan can fight to the death on the review | 14:06 |
| *** efried has quit IRC | 14:07 | |
| gibi | mriedem: sure. works for me :) | 14:07 |
| *** xek_ has joined #openstack-nova | 14:07 | |
| kashyap | Ah, efried is gone ... just when I have his answer for him | 14:07 |
| *** tbachman has joined #openstack-nova | 14:08 | |
| *** dpawlik has joined #openstack-nova | 14:09 | |
| *** efried has joined #openstack-nova | 14:11 | |
| *** ociuhandu has quit IRC | 14:13 | |
| *** ociuhandu has joined #openstack-nova | 14:14 | |
| luyao | efried: hi Eric, in case of you not seeing the msg just now, I need you confirm if I need a blueprint and separate spec for vpmem live migration. Thanks in advance. | 14:15 |
| dswebb | also worth mentioning that the same live migration (back and forth) using ceph as the cinder block store works fine | 14:16 |
| efried | luyao: I don't know; let's put it on the agenda for the next meeting and ask the team. | 14:18 |
| luyao | efried: OK, thanks | 14:19 |
| efried | luyao: I think gibi had at least a blueprint and possibly a spec for the continuation of lifecycle ops with bandwidth providers, but I think that's a much bigger effort than for vpmem. | 14:19 |
| *** ociuhandu has quit IRC | 14:19 | |
| kashyap | efried: Hey, so on your encrypted emulated TPM question -- | 14:19 |
| kashyap | efried: The QEMU version does not matter for encryption. What matters is the 'swtpm'. And you need at least libvirt 5.6.0 for the 'encryption' element | 14:20 |
| kashyap | https://libvirt.org/formatdomain.html#elementsTpm | 14:20 |
| *** dpawlik has quit IRC | 14:20 | |
| efried | kashyap: sweet, thank you very much. I was aware of the libvirt side, but couldn't find the qemu side documented anywhere. | 14:21 |
| * bauzas is done with writing ML threads for the PTG \o/ | 14:21 | |
| *** dpawlik has joined #openstack-nova | 14:21 | |
| efried | bauzas, gibi: Thank you both for writing up those summaries! | 14:21 |
| bauzas | np | 14:22 |
| bauzas | HTH | 14:22 |
| *** ivve has quit IRC | 14:22 | |
| luyao | efried: yes, in terms of code size, the lm for vpmem is not big | 14:23 |
| *** eharney has quit IRC | 14:23 | |
| efried | luyao: was there mention of the details of lm implementation in the original spec? (Save me digging it up) | 14:24 |
| *** nweinber__ has quit IRC | 14:25 | |
| luyao | efried: no, and I need update the original spec, since our implementation is different from it | 14:25 |
| *** nweinber has joined #openstack-nova | 14:26 | |
| openstackgerrit | George proposed openstack/nova stable/ocata: Support qemu >= 2.10 https://review.opendev.org/693851 | 14:27 |
| *** tbachman has quit IRC | 14:27 | |
| efried | luyao: I think a short spec is probably a good idea, since (IIRC) there are RPC object changes, right? | 14:28 |
| luyao | efried: yes, there is | 14:29 |
| efried | So yeah, please put together a brief spec. Thanks. | 14:30 |
| luyao | efried: Get it, thanks | 14:31 |
| *** jraju__ has quit IRC | 14:33 | |
| *** efried has quit IRC | 14:43 | |
| *** ociuhandu has joined #openstack-nova | 14:56 | |
| *** efried has joined #openstack-nova | 15:00 | |
| *** Liang__ has joined #openstack-nova | 15:00 | |
| *** ociuhandu has quit IRC | 15:00 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Helper to start computes with different HostInfos https://review.opendev.org/686832 | 15:04 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595 | 15:04 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404 | 15:04 |
| openstack | bug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms) | 15:04 |
| artom | stephenfin, if you've recovered from PTG and still have all your organs, ^^ is still waiting :) | 15:05 |
| *** eharney has joined #openstack-nova | 15:05 | |
| *** tbachman has joined #openstack-nova | 15:05 | |
| *** dpawlik has quit IRC | 15:06 | |
| bauzas | artom: stephenfin is still somewhere in China | 15:07 |
| artom | bauzas, intentionally? ;) | 15:07 |
| dansmith | lyarwood: I reviewed your rescue spec.. do you have any idea what needs to happen and or what blockers there would be to making rescue work for BFV instances? | 15:08 |
| bauzas | artom: and I can't tell for his organs, but last time I saw him, his liver was working correctly | 15:08 |
| bauzas | :p | 15:08 |
| artom | bauzas, haha | 15:08 |
| bauzas | artom: well, he found that Shanghai wasn't that huge so he visited another city | 15:08 |
| bauzas | oh, and FWIW, I'll keep my wechat account | 15:09 |
| bauzas | living in China for one week makes you realize how life can be different without big G and a few other tools | 15:09 |
| lyarwood | dansmith: thanks for that, yeah I have a rough idea but I've not looked at this since failing to land this in Newton. I can't think of much that's changed since then with regards to the high level implementation. I also can't think of any additional blockers for BFV at the moment tbh. | 15:14 |
| *** mlavalle has joined #openstack-nova | 15:14 | |
| dansmith | lyarwood: okay, maybe I'll try removing the BFV check on a devstack and poke around with what is and isn't working in the current setup | 15:15 |
| dansmith | lyarwood: because BFV instances are some of the more important ones you'd want to rescue, ya'know | 15:15 |
| lyarwood | dansmith: right, so for rescue to work you need to switch things around so the rescue device is the final thing attached, not the first and then boot from that device. | 15:16 |
| dansmith | lyarwood: that's what you're proposing.... | 15:17 |
| *** eharney has quit IRC | 15:17 | |
| lyarwood | dansmith: right sorry I think I'm misunderstanding what you're actually asking here. | 15:19 |
| dansmith | lyarwood: right now we just categorically refuse to rescue a BFV instance, and I think we should change that since BFV instances are likely pets, and likely in need of rescue | 15:20 |
| *** lbragstad_ is now known as lbragstad | 15:20 | |
| dansmith | so, with your proposed changes, in addition to the ordering, also including cinder volumes, | 15:20 |
| dansmith | I'm wondering what we could do to follow up your changes with a change to allow rescue of BFV instances | 15:20 |
| smcginnis | I've seen cattle with BFV too. Some just prefer to keep their storage on Cinder volumes. FWIW. | 15:21 |
| dansmith | smcginnis: sure I know | 15:22 |
| dansmith | smcginnis: I said "likely" not "always" :) | 15:22 |
| lyarwood | dansmith: Ah! The changes would actually allow BFV and non-BFV but I can see that isn't made clear in the spec. I totally missed this when copying it over from an older change this morning but I'll clean this up and make it clear now. | 15:23 |
| dansmith | lyarwood: sweet | 15:24 |
| dansmith | lyarwood: do you have old crusty code of up for this at all? | 15:24 |
| lyarwood | dansmith: there's an old abandoned series but iirc that was missing the compute service check that I think I had somewhere locally | 15:24 |
| dansmith | lyarwood: ack | 15:25 |
| lyarwood | dansmith: https://review.opendev.org/#/q/status:abandoned+topic:bp/virt-rescue-stable-disk-devices | 15:25 |
| dansmith | lyarwood: so you think that after these changes, if the api let them, we could rescue BFV instances no problem? | 15:25 |
| lyarwood | dansmith: yes | 15:26 |
| dansmith | if so, we'll need a compute capability to indicate this support, and could use that in place of the version check | 15:26 |
| lyarwood | yup true, I also didn't cover microversions in the spec that I think came up in that series | 15:26 |
| *** pcaruana has joined #openstack-nova | 15:28 | |
| dansmith | lyarwood: no REST API changes, so why a microversion? | 15:29 |
| *** eharney has joined #openstack-nova | 15:30 | |
| *** Liang__ has quit IRC | 15:31 | |
| lyarwood | dansmith: https://review.opendev.org/#/c/270288/18/nova/compute/api.py@2777 - mriedem raised it here as something we might have to do, I'm also not sure if that's still the case tbh. | 15:31 |
| dansmith | lyarwood: oh, you even remove that in the code, I see, | 15:32 |
| dansmith | well, in that case it probably does need a microversion and a rest api impact section, but in the spec I just reviewed you said "none" :) | 15:32 |
| lyarwood | yeah apologies this slipped my mind when I was pushing it up for review earlier. I'll mark the spec as WIP for now while I get things back in order. | 15:34 |
| dansmith | lyarwood: you said you were going to work on this this cycle right? | 15:34 |
| lyarwood | dansmith: yup I should actually have time this cycle | 15:35 |
| dansmith | lyarwood: cool, I | 15:36 |
| dansmith | I would suggest getting this going sooner than later if you can schedule it | 15:36 |
| dansmith | at least rebase the code and get it restored so that others can comment/help | 15:36 |
| dansmith | even if before the spec, although I think the spec is easy to merge pretty soon if you fix up the issues | 15:37 |
| lyarwood | dansmith: sure, I should be able to get something rebased pretty quickly. | 15:37 |
| lyarwood | famous last words | 15:37 |
| * dansmith screenshots | 15:37 | |
| efried | jroll: randomly generated passphrase: do you think there should be a conf option for the length? If so/not, what should the default/number be? | 15:41 |
| *** ociuhandu has joined #openstack-nova | 15:44 | |
| *** ociuhandu has quit IRC | 15:44 | |
| *** ociuhandu has joined #openstack-nova | 15:46 | |
| jroll | efried: probably not worth a config until/unless someone asks. 32 or 64 characters seems reasonable | 15:49 |
| efried | Roger that. (Glad I asked, I was gonna go 256 :P ) | 15:50 |
| jroll | 256 is also reasonable :D | 15:50 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Wire up a force disconnect_volume flag https://review.opendev.org/584849 | 15:50 |
| jroll | efried: I guess if nobody ever touches this, maybe 256 is a better option | 15:51 |
| efried | it certainly won't be readable/typable. Like an ssh key. How big are those? | 15:51 |
| *** awalende has joined #openstack-nova | 15:52 | |
| *** TxGirlGeek has joined #openstack-nova | 15:52 | |
| jroll | $ wc ~/.ssh/id_rsa | 15:52 |
| jroll | 54 61 3326 /Users/jrollenhagen/.ssh/id_rsa | 15:52 |
| *** ociuhandu has quit IRC | 15:53 | |
| efried | yours is bigger than mine | 15:53 |
| dansmith | mriedem: wha-chu-think main? https://review.opendev.org/#/c/636224/50/nova/compute/api.py | 15:53 |
| jroll | heh | 15:53 |
| dansmith | efried: whoa, wrong channel | 15:54 |
| efried | that's number of chars, b64-encoded, plus header/footer | 15:54 |
| jroll | right, looks like 3072 bytes | 15:55 |
| mriedem | dansmith: you can't compare None values can you? | 15:55 |
| *** awalende_ has quit IRC | 15:56 | |
| dansmith | mriedem: you can | 15:56 |
| dansmith | mriedem: >>> (None, 1) < (None, 2) | 15:57 |
| dansmith | True | 15:57 |
| *** awalende has quit IRC | 15:57 | |
| mriedem | >>> datetime.datetime.today() < None | 15:57 |
| mriedem | Traceback (most recent call last): | 15:57 |
| mriedem | File "<stdin>", line 1, in <module> | 15:57 |
| mriedem | TypeError: can't compare datetime.datetime to NoneType | 15:57 |
| mriedem | i guess because of the tuples | 15:57 |
| bauzas | AFAIR, you were even able to say something like None = 2 | 15:57 |
| dansmith | oh you mean compare datetime to none | 15:57 |
| bauzas | in py2 something | 15:57 |
| mriedem | that's what these are | 15:57 |
| mriedem | >>> now1 = datetime.datetime.now() | 15:58 |
| mriedem | >>> now2 = datetime.datetime.now() | 15:58 |
| mriedem | >>> now1 < now2 | 15:58 |
| mriedem | True | 15:58 |
| mriedem | >>> now1 < None | 15:58 |
| mriedem | Traceback (most recent call last): | 15:58 |
| mriedem | File "<stdin>", line 1, in <module> | 15:58 |
| mriedem | TypeError: can't compare datetime.datetime to NoneType | 15:58 |
| dansmith | yeah, ack | 15:58 |
| efried | jroll: afaict those would map to original random passphrases of ~1200-2400 chars | 15:58 |
| Roamer` | mriedem, you'd get the same result if you tried to compare it to "whee" or to 17 | 15:59 |
| dansmith | mriedem: we have to find something better than what you have | 15:59 |
| * efried ==> mtg | 15:59 | |
| Roamer` | ah, but you're commenting on the patch in the review request, sorry, yeah | 15:59 |
| mriedem | dansmith: i looked for some builtin sorting/filtering type function but didn't find anything obvious | 16:00 |
| dansmith | okay I got it | 16:01 |
| mriedem | sorted with a key function? | 16:01 |
| mdbooth | lyarwood: https://review.opendev.org/#/c/584850/ | 16:02 |
| kashyap | Is this a common convention? "os-migrateLive" | 16:02 |
| dansmith | mriedem: https://pastebin.com/pHtM7iEf | 16:02 |
| * kashyap is reviewing a spec on "no_performance_impact" | 16:03 | |
| kashyap | (I'll ask there) | 16:03 |
| mdbooth | lyarwood: I know that's abandoned, but any idea where that came from? The concept of having un-flushed data after LM switchover is disturbing. | 16:03 |
| Roamer` | mriedem, dansmith, maybe something like (yeah, I know, quite unwieldy) `dtnull = datetime.datetime.utcfromtimestamp(0)` and then `(u1 or dtnull, a1 or dtnull) < (u1 or dtnull, a2 or dtnull)` | 16:03 |
| dansmith | Roamer`: yep, or that | 16:03 |
| Roamer` | (with the correct variable names, of course, pfheh) | 16:03 |
| dansmith | mine makes anything with an updated_at win and short circuit before comparing the updated_ats which means you won't ever compare non-nones | 16:05 |
| dansmith | er, Non-None to None | 16:05 |
| *** jhesketh has quit IRC | 16:05 | |
| lyarwood | mdbooth: os-brick had a number of bugs around multiple devices getting grouped under the same mpath device blocking the final flush/removal on disconnect | 16:06 |
| Roamer` | dansmith, TBH, I hadn't seen yours before writing mine :) it took me a while to test it and copy it here, yours might be better and less unwieldy :) | 16:06 |
| lyarwood | mdbooth: QEMU's own flush was enough to ensure everything made it to the backend before we forced anything in that case | 16:06 |
| *** belmoreira has joined #openstack-nova | 16:06 | |
| mdbooth | lyarwood: That flush was happening on the source node, after switchover? | 16:07 |
| lyarwood | mdbooth: the QEMU flush? | 16:07 |
| lyarwood | mdbooth: no that's before we converge iirc | 16:07 |
| lyarwood | or maybe during | 16:07 |
| mdbooth | lyarwood: os-brick's flush on disconnect | 16:07 |
| mdbooth | So the bug was that os-brick thought it had data to flush, but it didn't really? | 16:08 |
| lyarwood | mdbooth: that's after but it's just something you have to do before disconnecting mpath devices iirc, there's no data actually cached anywhere at that point | 16:08 |
| mdbooth | Right. It would just be really bad if there really was data to flush at that point. | 16:08 |
| lyarwood | yup | 16:09 |
| dansmith | jroll: didn't you add some ironic grouping construct so that one ironic can serve independent pools of nodes to dedicated nova computes? | 16:10 |
| dansmith | ah, conductor groups | 16:10 |
| *** lpetrut has quit IRC | 16:10 | |
| belmoreira | dansmith jroll how that would work? | 16:15 |
| dansmith | belmoreira: did you read the spec I just sent you? :) | 16:15 |
| belmoreira | :) maybe... which one? :) | 16:15 |
| *** gyee has joined #openstack-nova | 16:15 | |
| dansmith | belmoreira: https://docs.openstack.org/ironic/latest/admin/conductor-groups.html | 16:15 |
| dansmith | oops, | 16:16 |
| dansmith | belmoreira: https://specs.openstack.org/openstack/nova-specs/specs/stein/implemented/ironic-conductor-groups.html | 16:16 |
| belmoreira | this is train | 16:16 |
| dansmith | what is? | 16:17 |
| belmoreira | sorry. This is available in Stein | 16:17 |
| dansmith | right | 16:17 |
| *** jaosorior has quit IRC | 16:18 | |
| belmoreira | I forgot about it. sorry. This is great however means that we will need to scale nova-compute in the same way as the ironic-conductor | 16:21 |
| *** jhesketh has joined #openstack-nova | 16:21 | |
| dansmith | belmoreira: well, you probably should be doing that anyway given you're hitting nova-compute scale issues when you make ironic big :) | 16:21 |
| dansmith | I understand the desire to make the RT loop in nova-compute more efficient, and I'm sure we can without breaking things, but it's a change to complex crufty code that only (positively) impacts the one oddball driver that uses multiple nodes, | 16:22 |
| dansmith | so I'm less enthusiastic about that, especially when grouping nova-computes gets you a lot more gain too, | 16:23 |
| dansmith | like better fault tolerance, mapping your existing failure domains through to ironic, etc | 16:23 |
| *** bbobrov has quit IRC | 16:24 | |
| belmoreira | dansmith I understand that. That's why the initial plan was the sharding. And I wasn't aware (forgot) about the conductor groups | 16:24 |
| belmoreira | this can make it | 16:24 |
| dansmith | cool | 16:24 |
| *** bbobrov has joined #openstack-nova | 16:24 | |
| belmoreira | Thanks dansmith | 16:31 |
| belmoreira | I will try this and let you know how it goes | 16:31 |
| *** tesseract has quit IRC | 16:32 | |
| jroll | \o/ | 16:32 |
| *** ociuhandu has joined #openstack-nova | 16:33 | |
| *** ociuhandu has quit IRC | 16:33 | |
| kashyap | mdbooth: On that "no_performance_impact" thing for post-copy / auto-converge spec, I have to bike-shed: the name is too generic for my taste. I suggested being more clear WTF it is doing: "no_post_copy_and_auto_converge". | 16:33 |
| dansmith | belmoreira: sweet that'd be super | 16:34 |
| *** ociuhandu has joined #openstack-nova | 16:34 | |
| *** spatel has joined #openstack-nova | 16:35 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: WIP Virtual instance rescue with stable disk devices https://review.opendev.org/693849 | 16:38 |
| *** TxGirlGeek has quit IRC | 16:39 | |
| *** do3meli has quit IRC | 16:39 | |
| *** ociuhandu has quit IRC | 16:40 | |
| *** damien_r has quit IRC | 16:40 | |
| *** TxGirlGeek has joined #openstack-nova | 16:41 | |
| *** ociuhandu has joined #openstack-nova | 16:47 | |
| *** boxiang has quit IRC | 16:48 | |
| *** zhubx has joined #openstack-nova | 16:48 | |
| dansmith | mriedem: on lyarwood's spec to do stable disk rescue, | 16:48 |
| dansmith | do you have strong feelings about bundling the can-now-rescue-BFV feature in it, | 16:48 |
| dansmith | vs. a relatively tiny follow-on effort? | 16:49 |
| *** mkrai has quit IRC | 16:51 | |
| *** ociuhandu has quit IRC | 16:52 | |
| *** macz has joined #openstack-nova | 16:55 | |
| openstackgerrit | sean mooney proposed openstack/nova master: block rebuild when numa topology changed https://review.opendev.org/687957 | 16:56 |
| openstackgerrit | sean mooney proposed openstack/nova master: Disable NUMATopologyFilter on rebuild https://review.opendev.org/689861 | 16:56 |
| *** ivve has joined #openstack-nova | 16:57 | |
| *** rpittau is now known as rpittau|afk | 17:00 | |
| mriedem | dansmith: i haven't read it but what's the difference? you mean making the api capability separate which is what, a microversion and a min compute service version compat check? | 17:02 |
| dansmith | mriedem: not even a min version check I think, a compute capability check | 17:02 |
| *** dtantsur is now known as dtantsur|afk | 17:02 | |
| mriedem | so the api checking the resource provider traits? | 17:02 |
| dansmith | er, maybe I'm confused, I thought the api could check the compute capabilities, but maybe that's stupid | 17:03 |
| dansmith | so yeah maybe a version check and a cap flag in the compute manager | 17:03 |
| *** pcaruana has quit IRC | 17:03 | |
| dansmith | mriedem: point being, lyarwood had it in his now-abandoned first rev to remove that api check, and his current spec says no API impact, so wondering which way you want to go | 17:03 |
| mriedem | i've brought up that idea before about the api checking compute capabilities via traits, e.g. https://review.opendev.org/#/c/666604/ | 17:03 |
| mriedem | well there is obviously an api impact | 17:04 |
| dansmith | mriedem: ...only if he removes that check in his series again | 17:04 |
| dansmith | mriedem: and, you think that needs a microversion? just removing a check in some cases? | 17:04 |
| *** ociuhandu has joined #openstack-nova | 17:05 | |
| mriedem | it's a behavior change that a user opts into, i.e. i can rescue volume-backed servers on cloud A but not cloud B, | 17:05 |
| dansmith | if so, I understand why, but I don't have a good gut feeling for when we do and don't need to signal removing checks | 17:05 |
| mriedem | which is usually a microversion | 17:05 |
| dansmith | they can opt in but still refuse with the same error | 17:06 |
| dansmith | it won't work for all hypervisors, | 17:06 |
| dansmith | and unless the api documents that you can't do that with volume-backed instances, | 17:06 |
| mriedem | same idea for multiattach volume support, same for volume-backed rebuild (if someone finished that up) | 17:06 |
| dansmith | it seems like an obscure thing to me | 17:06 |
| mriedem | i think we've said in the past (and in our docs) that the docs are not a contract, the api behavior is | 17:06 |
| mriedem | and yes i know we fudge these lines all the time, e.g. gibi's changes to support moving servers with qos ports | 17:06 |
| dansmith | absolutely, but I'm not sure this is discoverable as the reason, which is why it doesn't seem clear-cut to me | 17:07 |
| mriedem | and yes i realize not all hypervisors are going to support this so just because a cloud has the microversion that allows the request it doesn't mean it's going to work | 17:07 |
| dansmith | if you view the current error as "You can't rescue that instance for reasons I will not divulge" then all users right now just try it and accept the fate if they can't, without knowing why" | 17:08 |
| mriedem | as i said in lee's original change about microversions, i said i don't know what others would think so it's likely a ML discussion | 17:08 |
| dansmith | in which case allowing it on more things seems fine | 17:08 |
| mriedem | it's not a 409 though right? it's a 400 | 17:08 |
| mriedem | yes it's a 400 today | 17:08 |
| dansmith | the distinction doesn't mean anything to me | 17:08 |
| dansmith | 409 is transient or something? | 17:09 |
| mriedem | if it were a 409 that'd be a bit fudgier to saying "you can' do this right now" | 17:09 |
| dansmith | okay, well, that's legit | 17:09 |
| dansmith | probably *should* have been a 409 then, but alas | 17:09 |
| mriedem | i don't think there was a lot of thought put into response code nuance back in the day | 17:09 |
| dansmith | no, clearly not | 17:10 |
| dansmith | does 400 mean it's a fatal error specifically or just not imply retry-ability one way or the other? I would assume the latter as a catch-all | 17:10 |
| openstackgerrit | sean mooney proposed openstack/nova-specs master: Add spec for VM-scoped SR-IOV NUMA affinity https://review.opendev.org/683174 | 17:11 |
| *** ociuhandu has quit IRC | 17:11 | |
| dansmith | anyway, if it's a microversion, then it probably makes sense to have that be a separate effort, maybe even before this stable rescue series to open up the possibility, even with no implementors, followed by his patch that actually allows it | 17:11 |
| sean-k-mooney | efried: alex_xu ^ fixed the typos but that is otherwise the same | 17:11 |
| mriedem | 400 means don't try this again | 17:12 |
| dansmith | "The client SHOULD NOT repeat the request without modifications." | 17:14 |
| sean-k-mooney | or at least dont try this again until the resouce is in a different state | 17:14 |
| dansmith | sean-k-mooney: that's not what the w3c says for 400, but does for others, so not sure you're right about that | 17:14 |
| mriedem | sean-k-mooney: you just described 409 | 17:14 |
| sean-k-mooney | 400 is bad request | 17:14 |
| dansmith | mriedem: that's an unfortunate implication for our default error code, but fair enough | 17:14 |
| *** ociuhandu has joined #openstack-nova | 17:15 | |
| sean-k-mooney | but it may be a bad request for the give state fo the resouce but not in general | 17:15 |
| dansmith | sean-k-mooney: no | 17:15 |
| sean-k-mooney | 409 conflcit is ment for concurrent modifcation | 17:15 |
| dansmith | sean-k-mooney: it says the reason is "malformed syntax" and "do NOT try this again" | 17:15 |
| dansmith | https://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html | 17:15 |
| dansmith | the reason is not "the thing is in the wrong state for the operation you tried" | 17:15 |
| sean-k-mooney | ya fair | 17:16 |
| sean-k-mooney | i normally use https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/400 | 17:16 |
| sean-k-mooney | but yes it also highlights "The client should not repeat this request without modification." | 17:16 |
| dansmith | none of those reasons imply retry | 17:16 |
| sean-k-mooney | i think the only time you woudl retry is for 5xx errors | 17:17 |
| dansmith | mriedem: so, spec because api change, and put the api change ahead of lyarwood's patch so we can actually test through it and hold merge until both are ready yeah? | 17:18 |
| dansmith | sean-k-mooney: no, some of the 4xx errors are mentioned as retryable with no modification | 17:18 |
| dansmith | sean-k-mooney: like 408 | 17:18 |
| sean-k-mooney | oh ok i normaly think of 503 https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/503 when i think retry | 17:18 |
| mriedem | dansmith: if you're saying single spec, multiple changes (one for compute, one for api), then yes agree | 17:19 |
| sean-k-mooney | oh ya request timeouts | 17:19 |
| dansmith | mriedem: no, I mean another spec for the api change specifically just to avoid mucking up lyarwood's clean understandable spec with the spec to change the api behavior | 17:19 |
| efried | oh sean-k-mooney, you lost your +1s from Vieri and Andriy :( | 17:20 |
| sean-k-mooney | :( your +2 would equal both there +1s right :) | 17:20 |
| dansmith | mriedem: I don't want to do extra work, but it seems like having lyarwood's stable rescue spec be readable in isolation without a bunch of api semantic changes would be beneficial, but maybe the api docs required for just a status change wouldn't be too big | 17:21 |
| efried | sean-k-mooney: re+2, thanks for the update. | 17:21 |
| sean-k-mooney | i still need to fix the typos you noted in the code but thats next on my list | 17:21 |
| sean-k-mooney | then i need to adress dansmith's feedback on another series :) | 17:22 |
| mriedem | dansmith: are you saying the api change would just be changing the status code from 400 to 409 rather than a microversion? | 17:23 |
| dansmith | mriedem: no | 17:23 |
| mriedem | then i don't think a secondary api-only spec is going to be that big of a deal | 17:24 |
| dansmith | cripes | 17:24 |
| mriedem | "depends on blueprint x. in x we make y work on the compute. this spec enables it in the api with a new microversion" | 17:24 |
| mriedem | just do whatever y'all think is best | 17:25 |
| mriedem | i haven't read the current spec | 17:25 |
| dansmith | mriedem: I'm trying to say I think we should separate the api change into its own small spec to keep lyarwood's focused on the disk changes | 17:25 |
| mriedem | and i just said i was ok with that right? | 17:25 |
| dansmith | mriedem: in isolation without his change, we can make the api change to remove the hard block and say "as of microversion X you may or may not be able to rescue BFVs" | 17:25 |
| dansmith | mriedem: it wasn't clear to me | 17:26 |
| dansmith | because that's what I've been suggesting for ten minutes and you seemed to be disagreeing, so I was trying to explain | 17:26 |
| mriedem | so without any compute side changes, we should be able to rescue volume-backed servers today? | 17:26 |
| * dansmith headdesks | 17:26 | |
| mriedem | i'm fine with whatever | 17:27 |
| mriedem | let's just stop talking about this | 17:27 |
| dansmith | I'm clearly the problem here, I'll just butt out | 17:27 |
| *** ociuhandu has quit IRC | 17:32 | |
| *** ociuhandu has joined #openstack-nova | 17:33 | |
| *** ociuhandu has quit IRC | 17:33 | |
| *** ociuhandu has joined #openstack-nova | 17:34 | |
| artom | donnyd, heya, what the difference between your multi-numa and multi-numa-expanded flavors? Or where can I look to find out? | 17:34 |
| artom | (Also, we should talk about enabling hugepages at some point, but that can wait) | 17:35 |
| sean-k-mooney | artom: the expanded one has 16G of ram instead of 8 | 17:35 |
| artom | sean-k-mooney, that's it? | 17:35 |
| sean-k-mooney | yep | 17:35 |
| sean-k-mooney | otherwise they both have the same about of disk and cpu | 17:35 |
| artom | And NUMA nodes | 17:35 |
| sean-k-mooney | but have hw:numa_nodes=2 | 17:36 |
| sean-k-mooney | and hw:cpu_socket=2 and hw:cpu_thread=2 | 17:36 |
| artom | So the non-expanded one is fine | 17:36 |
| *** ociuhandu has quit IRC | 17:36 | |
| sean-k-mooney | we shoudl try to use the non expanded ones if we can to save space but we might need the expanded one on the contoler | 17:37 |
| artom | Ohh | 17:37 |
| sean-k-mooney | ya i didnt want to have to super optimise the job to get it to fit intially which is one of the reason we started with the expanded lables but it should not be needed longterm | 17:37 |
| artom | Let's try with non-expanded I guess first | 17:38 |
| *** ociuhandu has joined #openstack-nova | 17:38 | |
| artom | And if we hit oom or something we can -expand :D | 17:38 |
| sean-k-mooney | yep i think i was trying to allocte a low amout of hugepage in my job | 17:39 |
| sean-k-mooney | well in the orginal version | 17:39 |
| sean-k-mooney | i think we are generaly around the 5-6 G mark on used memroy on the contoler so you dont want to exceen more then about 1G of hugepages total | 17:40 |
| artom | Ohh, you actually set hugepages in Ansible | 17:40 |
| sean-k-mooney | yes here https://review.opendev.org/#/c/679656/13/playbooks/nfv/nfv.yaml | 17:40 |
| artom | So I should probably do that instead of https://review.opendev.org/#/c/693690/ | 17:40 |
| sean-k-mooney | but its too much currently for the contoler | 17:41 |
| artom | Although that skip could still be useful in other envs | 17:41 |
| *** david-lyle is now known as dklyle | 17:41 | |
| sean-k-mooney | ya i would basically do this | 17:42 |
| sean-k-mooney | hosts: controller | 17:42 |
| sean-k-mooney | tasks: | 17:42 |
| sean-k-mooney | - name: allocate hugepages | 17:42 |
| sean-k-mooney | shell: echo 512 > /sys/kernel/mm/hugepages/hugepages-2048kB/nr_hugepages | 17:42 |
| sean-k-mooney | become: yes | 17:42 |
| sean-k-mooney | - hosts: subnode | 17:42 |
| sean-k-mooney | tasks: | 17:42 |
| sean-k-mooney | - name: allocate hugepages | 17:42 |
| sean-k-mooney | shell: echo 2048 > /sys/kernel/mm/hugepages/hugepages-2048kB/nr_hugepages | 17:42 |
| sean-k-mooney | become: yes | 17:42 |
| *** ociuhandu has quit IRC | 17:42 | |
| sean-k-mooney | that will alocate 1G on the contoler and 4 on the compute of 2MB hugepages | 17:43 |
| sean-k-mooney | that should be enough | 17:43 |
| artom | Ack | 17:43 |
| *** ociuhandu has joined #openstack-nova | 17:45 | |
| *** spatel has quit IRC | 17:45 | |
| *** henriqueof has quit IRC | 17:47 | |
| *** ociuhandu has quit IRC | 17:54 | |
| artom | sean-k-mooney, I think the problem with using https://zuul-ci.org/docs/zuul-jobs/general-roles.html#role-copy-build-sshkey is that it's 1-user only | 17:55 |
| artom | IOW, 'tempest' can ssh to 'tempest' | 17:55 |
| artom | We need 'tempest' to SSH to 'stack' or some other account that has limitless sudo powers | 17:56 |
| *** igordc has joined #openstack-nova | 17:59 | |
| sean-k-mooney | artom: the same key is already copied to root and stack | 17:59 |
| sean-k-mooney | root is done in the pre job | 17:59 |
| sean-k-mooney | stack in the muntinode playbook | 17:59 |
| sean-k-mooney | so if we use it for tempest all 3 users will have the same private and public key | 18:00 |
| sean-k-mooney | you wouldnt want to do that in production but it should be fine for ci | 18:00 |
| *** bbowen has quit IRC | 18:01 | |
| *** bbowen has joined #openstack-nova | 18:01 | |
| artom | *facepalm* | 18:01 |
| artom | I see | 18:01 |
| artom | The devstack orchestrate-devstack role will put the pubkey in stack's authorized_keys for us | 18:01 |
| artom | And we can put the private key in tempest's .ssh | 18:02 |
| artom | In theory, anyways | 18:02 |
| * artom is sure something will explode | 18:02 | |
| *** bbowen_ has joined #openstack-nova | 18:05 | |
| artom | ... do we have access to another project's roles? Not sure... | 18:05 |
| donnyd | artom: the expanded flavors have more memory | 18:06 |
| *** bbowen has quit IRC | 18:08 | |
| *** bbowen_ has quit IRC | 18:11 | |
| sean-k-mooney | artom: am we can get acess to them yes there are two ways to do i but your job inherits form the tempest job yes | 18:11 |
| sean-k-mooney | if so then it will do it already | 18:11 |
| sean-k-mooney | you you can just invoke the copy-build-key role twice once for stack and once for tempest | 18:12 |
| artom | https://review.opendev.org/#/c/691062/57 let's see how badly stuff explodes | 18:28 |
| *** ralonsoh has quit IRC | 18:34 | |
| sean-k-mooney | artom: you do not need the ssh keys in that anymore | 18:41 |
| sean-k-mooney | but i think it should be ok | 18:41 |
| sean-k-mooney | its runing in a sperate pool so it should be pretty fast | 18:41 |
| artom | sean-k-mooney, something borked, it's retrying now | 18:42 |
| artom | Heh, oh yeah, I should get rid of the keys | 18:42 |
| artom | I'll wait for it to work first | 18:42 |
| sean-k-mooney | the console for the job is here http://zuul.openstack.org/stream/11de9dc2d4fd4f7d931eedeea276af3e?logfile=console.log | 18:43 |
| sean-k-mooney | looks like tis doing the normal multinode setup | 18:43 |
| artom | That's the second attempt | 18:44 |
| sean-k-mooney | yep | 18:44 |
| sean-k-mooney | ill leave it open on one of my monitors and see what hapens | 18:44 |
| sean-k-mooney | haha | 18:45 |
| sean-k-mooney | you have a typo | 18:45 |
| sean-k-mooney | artom: http://paste.openstack.org/show/785994/ | 18:46 |
| sean-k-mooney | you did not include teh whitebox-compute role in the commit | 18:47 |
| sean-k-mooney | or you forgot to remove that line | 18:47 |
| artom | *facepalm* | 18:49 |
| artom | 'git commit -a' isn't magic | 18:49 |
| sean-k-mooney | it speciricly ignore untracted files | 18:49 |
| *** henriqueof has joined #openstack-nova | 18:50 | |
| artom | I know | 18:52 |
| sean-k-mooney | good becaue i apparently cant spell specifcally and im not sure you woudl figure it out if you didnt :P | 18:52 |
| sean-k-mooney | i need to try enabling spell check for irc again but the last time i tried that it was kind of depressing | 18:55 |
| artom | *shrug* I forgive you :) | 18:57 |
| artom | You have other uses besides spelling ;) | 18:57 |
| sean-k-mooney | well if you want a terible way to generate passwords just ask me to spell somthing | 18:58 |
| artom | lulz | 18:59 |
| sean-k-mooney | so looking at the job you are trying to allocate 8G of hugepages on all nodes | 19:04 |
| sean-k-mooney | that will not work with the non expanded flavors | 19:05 |
| sean-k-mooney | *lables | 19:05 |
| artom | derp | 19:07 |
| *** ociuhandu has joined #openstack-nova | 19:08 | |
| *** ociuhandu has quit IRC | 19:12 | |
| *** jaosorior has joined #openstack-nova | 19:39 | |
| mriedem | cool, the instance_actions_events table has a detail TEXT column that is never used, and when creating the InstanceActionEvent object we pass the exc_val but it gets passed down to the DB API in a 'message' kwarg rather than 'details' so that column is never populated | 19:46 |
| mriedem | so as a non-admin user you can see a particular event passed or failed but not why, e.g. NoValidHost | 19:48 |
| mriedem | and only admins can see the traceback (like a fault) by default | 19:48 |
| sean-k-mooney | hum ok that sound like the behavior we want. are you going to reuse the colume for something | 19:49 |
| sean-k-mooney | such as storging the traceback? | 19:50 |
| mriedem | the traceback is stored in the traceback column | 19:50 |
| *** lbragstad has quit IRC | 19:50 | |
| mriedem | the exc_val is not stored at all | 19:50 |
| mriedem | b/c the object is using the wrong column name | 19:50 |
| *** lbragstad has joined #openstack-nova | 19:51 | |
| sean-k-mooney | sure but i was wondering if you planned to chagne that | 19:51 |
| mriedem | i think it would be useful to at least expose to the non-admin owner of the server the exception type, like we do for a fault message | 19:52 |
| sean-k-mooney | ya maybe. as long as its just the type and not the body of the excption its proably safe. although im not sure all operator would be ok with that | 19:53 |
| sean-k-mooney | e.g. it might affect there sla depening on what the falut was | 19:53 |
| mriedem | if the instance goes to ERROR status we're already showing the exception type in the fault message | 19:54 |
| mriedem | if not the formatted message | 19:54 |
| sean-k-mooney | ya that is ture | 19:54 |
| sean-k-mooney | i normally am logged in as root but we get the message in that case at least | 19:54 |
| sean-k-mooney | * admin | 19:54 |
| sean-k-mooney | so im not sure what a non admin sees | 19:55 |
| *** brinzhang_ has joined #openstack-nova | 19:57 | |
| *** brinzhang has quit IRC | 20:00 | |
| sean-k-mooney | artom: :) looks like it worked https://openstack.fortnebula.com:13808/v1/AUTH_e8fd161dc34c421a979a9e6421f823e9/zuul_opendev_logs_3ae/691062/59/check/whitebox-multinode-devstack/3ae547c/testr_results.html.gz | 20:10 |
| sean-k-mooney | we still need to adress the skips | 20:11 |
| sean-k-mooney | but the ssh keys worked correctly and it looks liek the hugepages allocation did not cause memory issues | 20:12 |
| *** jaosorior has quit IRC | 20:16 | |
| *** panda has quit IRC | 20:18 | |
| *** lbragstad has quit IRC | 20:19 | |
| *** lbragstad has joined #openstack-nova | 20:20 | |
| *** munimeha1 has joined #openstack-nova | 20:20 | |
| *** panda has joined #openstack-nova | 20:21 | |
| mriedem | dansmith: your suggestions here to use the tuples for comparing times doesn't work in all cases https://review.opendev.org/#/c/636224/50/nova/compute/api.py@4956 | 20:23 |
| mriedem | and i don't really want to bake a bunch of conditional logic into unreadable tuples | 20:23 |
| dansmith | aight, well, do what you want | 20:25 |
| dansmith | inlining all that in a closure seems bad, | 20:25 |
| *** damien_r has joined #openstack-nova | 20:25 | |
| dansmith | especially for testability, which has clearly manifested itself already | 20:25 |
| dansmith | that could be a | 20:26 |
| dansmith | def newer_than(obj) | 20:26 |
| dansmith | method on NovaObject and be usable for other things, as well as easier to test | 20:26 |
| dansmith | is there not some way we could do this without times? | 20:27 |
| dansmith | because that's kinda smelly in general, | 20:27 |
| sean-k-mooney | i was going to suggest a before or after method or something but ya the tuple way would be nice if it was not for the value error | 20:27 |
| dansmith | assuming they're all time-synced, you don't have any delays, delayed transactions, etc | 20:27 |
| sean-k-mooney | *type error | 20:27 |
| sean-k-mooney | oh this is in get_newer_job already | 20:29 |
| sean-k-mooney | you could define a lessthan operator on the migration object | 20:29 |
| sean-k-mooney | although i guess that would have the same proably with None | 20:29 |
| dansmith | no, it wouldn't | 20:29 |
| dansmith | but also, less-than can mean lots of things other than time | 20:30 |
| sean-k-mooney | for a migration object | 20:30 |
| sean-k-mooney | and ya i could | 20:30 |
| mriedem | we could just say f it and do like get_all and filter out by uuid after we've already processed one | 20:30 |
| mriedem | this https://github.com/openstack/nova/blob/master/nova/compute/api.py#L2883 | 20:31 |
| dansmith | mriedem: yeah | 20:31 |
| mriedem | the migration from the first cell processed in the results is always the one that is returned, | 20:32 |
| mriedem | which could be wrong, but i don't really care at this point | 20:32 |
| dansmith | does it matter/ | 20:32 |
| dansmith | you just want to not return dupes right? | 20:32 |
| mriedem | yes, but also figure that since we can know which is "newer" we should return that, | 20:33 |
| mriedem | because the migration in the target cell is the one that's going to be getting updated as the resize progresses | 20:33 |
| mriedem | including when it goes to 'finished' status and the server is in VERIFY_RESIZE status | 20:33 |
| *** damien_r has quit IRC | 20:34 | |
| sean-k-mooney | mriedem ya i think the nestest makes sense | 20:34 |
| mriedem | which is kind of important since i think i have functional tests later in the series asserting the migratoins api returns 1 and it's the correct status | 20:34 |
| dansmith | mriedem: once you create the new one, couldn't you set the hidden flag on the old one or something/ | 20:34 |
| mriedem | sure, that's the alternative mentioned in the code and the commit message | 20:34 |
| mriedem | but it has implications for the api because that hidden field has never been set before | 20:34 |
| dansmith | the *newest* only makes sense if we never update the old migration | 20:35 |
| dansmith | mriedem: I don't understand what the problem with that is, as I said.. api ignores migrations with the hidden field already ... | 20:35 |
| mriedem | i don't think we do update the source cell migration unless we revert the resize | 20:35 |
| dansmith | we just leave it pending forever? | 20:36 |
| mriedem | no, when you confirm the resize in the target cell we hard destroy the source cell instance which will hard destroy it's related migratoin in the source cell db, | 20:36 |
| mriedem | same (in reverse) for revert | 20:36 |
| dansmith | what I meant is.. we leave it in the half-finished state until that time comes | 20:37 |
| mriedem | otherwise you can never resize back to that cell b/c of the duplicate instance uuid | 20:37 |
| mriedem | pretty sure yeah | 20:37 |
| sean-k-mooney | so in that case cant we have teh db return the latest version based on update or created time | 20:37 |
| mriedem | sean-k-mooney: different dbs | 20:37 |
| mriedem | so no | 20:37 |
| sean-k-mooney | but we only need to look at the one of the dbs if we only want the latest one right? i gues we would have too look at all of them during a migration | 20:38 |
| *** henriqueof1 has joined #openstack-nova | 20:38 | |
| sean-k-mooney | ya ok | 20:38 |
| dansmith | sean-k-mooney: maybe you should read the patch we're discussing? | 20:38 |
| sean-k-mooney | i am but im also about to head off for the night | 20:38 |
| sean-k-mooney | the reason i changed my mind half way true was the Verify_resize comment | 20:39 |
| *** henriqueof has quit IRC | 20:41 | |
| mriedem | so it's not just the active migration that is a duplicate either, it's all migrations linked to the instance - those get copied into the target cell db early in TargetDBSetupTask, because once we flip the instance mapping that's where the api is going to go to get migrations for that server, | 20:43 |
| mriedem | so if we started setting the hidden field on migrations, we'd have to do it on all of them until we're sure where the server is going to ultimately be | 20:43 |
| efried | If I initiate an operation as an admin user, does the context have is_admin set, or does that only happen when nova generates an admin context explicitly? | 20:44 |
| mriedem | is_admin is set | 20:44 |
| mriedem | the difference is one has a token/user/project and the one nova creates for some periodics and stuff doesnt | 20:45 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/context.py#L141 https://github.com/openstack/nova/blob/master/nova/policy.py#L197 https://github.com/openstack/nova/blob/master/nova/policies/base.py#L24 | 20:46 |
| *** slaweq has quit IRC | 20:47 | |
| sean-k-mooney | mriedem: without your patch do we ignore the cell maping and simple cast to all cells by the way or just the cell where the instnce is | 20:48 |
| mriedem | for what api? | 20:48 |
| mriedem | GET /os-migrations iterates all cells | 20:49 |
| efried | mriedem: perhaps the better question is: can I create an instance as an admin user? | 20:49 |
| sean-k-mooney | based on what you jsut said i would not need too right since we would be copying all the migrations to the new cell | 20:49 |
| artom | sean-k-mooney, yeah I am le much flabbergast | 20:50 |
| sean-k-mooney | efried: an admin user is just a normal user with extra privladtes so it can have a project and a quota | 20:51 |
| sean-k-mooney | efried: so the admin use that gets set up with devstack for example can | 20:52 |
| efried | or maybe I should just say what I'm actually trying to do: | 20:52 |
| efried | While generating libvirt xml, I need a context. The way the code is set up right now, sometimes the context comes from the user and sometimes it comes from get_admin_context(). I want to detect (and reject) the latter. | 20:52 |
| efried | So I think you're saying that I should be able to tell based on whether project_id is set? | 20:52 |
| sean-k-mooney | why do you want to reject the later | 20:53 |
| sean-k-mooney | admin users shoudl be able to create vms with vtpm | 20:53 |
| sean-k-mooney | just not on behalf of another user/project | 20:53 |
| mriedem | sean-k-mooney: we need to copy the migrations to the target cell db because, like i said above, if the user confirms the resize there we hard-destroy the instance and it's old migrations in the source cell db | 20:53 |
| sean-k-mooney | mriedem: yes im just wondering of we can optimese the /os_migration api to only look at the cell where the vm is currently | 20:54 |
| mriedem | efried: a context created by nova.context.get_admin_context won't have a project_id, correct | 20:54 |
| sean-k-mooney | that info should be in the instnace maping in the api db right? | 20:54 |
| efried | sean-k-mooney: Like we were talking about yesterday, I need to be able to bounce on code paths where $not_the_owner_of_the_instance is trying to generate xml with a vtpm secret in it. | 20:54 |
| mriedem | sean-k-mooney: what info? | 20:55 |
| efried | I guess I could just let the key retrieval fail. | 20:55 |
| sean-k-mooney | mriedem: which cell the vm is in | 20:55 |
| dansmith | mriedem: sean-k-mooney has not read the patch and doesn't realize we're listing all migrations, not just for a specific instance | 20:55 |
| mriedem | correct | 20:55 |
| sean-k-mooney | dansmith: yes i have | 20:55 |
| sean-k-mooney | oh i though you were listing all migrtion for the instace | 20:56 |
| mriedem | sean-k-mooney: but the instance mapping saying the instance is in cell X doesn't mean the migration records from cell Y are in cell X unless we copy them there | 20:56 |
| sean-k-mooney | not all migrtions in general | 20:56 |
| mriedem | GET /os-migrations is not instance specific | 20:56 |
| mriedem | it's like GET /servers | 20:56 |
| mriedem | GET /servers/{server_id}/migrations is instance and cell specific | 20:56 |
| sean-k-mooney | right sorry i was confusing it with the iserver one | 20:56 |
| *** eharney has quit IRC | 20:57 | |
| dansmith | efried: I would prefer you try and fail to get/list the key vs. inspect the context and try to detect that it isn't "real" | 20:57 |
| sean-k-mooney | efried: you dont need to reject it you can let barbican do that as it will reject your token when you try to get the secret | 20:57 |
| efried | dansmith: ack. I'm trying to verify whether there are situations where I want to generate the XML but am not trying to boot the instance. | 20:57 |
| efried | In such cases it would be okay to inject the secret_uuid but not define the secret itself to libvirt | 20:58 |
| dansmith | efried: we pass pieces of xml to libvirt in operations like attach volume, but not the whole thing I think, so probably not relevant | 20:58 |
| efried | relevant in that I would need to make sure I don't hit the key manager in such operations. | 20:59 |
| dansmith | efried: but still, I imagine you can't define the xml for the guest referencing a secret that you've since deleted (which you said you were going to do) right? | 20:59 |
| efried | I can, as long as I don't try to boot the thing. | 20:59 |
| dansmith | okay I would have expected that to fail at define time | 21:00 |
| efried | the guest XML only contains the UUID, which (if the instance was ever booted) I've stored in its sysmeta. | 21:00 |
| sean-k-mooney | ya i dont think you will hit that in novas usage | 21:00 |
| efried | I may be misunderstanding, but I think _get_guest_xml just returns an etree, doesn't actually try to libvirtify it. | 21:00 |
| dansmith | efried: sure | 21:00 |
| dansmith | efried: I guess I thought you meant generate the xml as in... update the definition in libvirt | 21:00 |
| sean-k-mooney | the only case i am aware off was the requet to "create a server but not boot it" but we dont supprot that currently | 21:01 |
| efried | I possibly need to rethink the flow here... | 21:01 |
| efried | because I *actually* only want to define the secret to libvirt on boot-ish operations anyway. And then delete it right after. | 21:01 |
| dansmith | right so in driver.spawn() | 21:02 |
| mriedem | dansmith: if moving this get_newer_obj method to NovaObjectBase makes it less gross for you i can do that | 21:02 |
| efried | also cold boot? | 21:02 |
| dansmith | efried: cold boot? :) | 21:02 |
| dansmith | efried: start/ | 21:02 |
| efried | power_on | 21:02 |
| mriedem | i thought about it but didn't since this was origially much more special case to this api method | 21:02 |
| sean-k-mooney | cold boot or start call power_on which for libvirt calls hard_reboot which calls spawn | 21:02 |
| dansmith | mriedem: honestly I just hate the way that logic is laid out, probably irrationally, but before even looking for test gaps I knew it would be annoying to validate | 21:03 |
| dansmith | mriedem: the fact that there was one just makes me scared, but I'm clearly the only one, so just do whatever you want, I say | 21:03 |
| mriedem | i expected something like this to already exist but didn't find anything | 21:04 |
| mriedem | if i'm left to do whatever i want, i'm going to address mdbooth's comments and keep the series going. if there are things around this to be done later, i say let's do them later. | 21:05 |
| dansmith | aight | 21:05 |
| sean-k-mooney | efried: for libivrt you might want to create and remove the securte here https://github.com/openstack/nova/blob/7aa88029bbf6311033457c32801963da01e88ecb/nova/virt/libvirt/driver.py#L6188 | 21:06 |
| *** aluria has quit IRC | 21:07 | |
| *** dosaboy has quit IRC | 21:07 | |
| *** ganso has quit IRC | 21:07 | |
| sean-k-mooney | in libvirt that is | 21:07 |
| sean-k-mooney | you would have to add the secret to the xml before | 21:07 |
| sean-k-mooney | anyway night o/ | 21:09 |
| *** ociuhandu has joined #openstack-nova | 21:09 | |
| *** ganso has joined #openstack-nova | 21:10 | |
| *** nweinber has quit IRC | 21:10 | |
| *** dosaboy has joined #openstack-nova | 21:13 | |
| *** ociuhandu has quit IRC | 21:13 | |
| *** ociuhandu has joined #openstack-nova | 21:15 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Filter duplicates from compute API get_migrations_sorted() https://review.opendev.org/636224 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Start functional testing for cross-cell resize https://review.opendev.org/636253 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Handle target host cross-cell cold migration in conductor https://review.opendev.org/642591 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Validate image/create during cross-cell resize functional testing https://review.opendev.org/642592 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add zones wrinkle to TestMultiCellMigrate https://review.opendev.org/643450 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add negative test for cross-cell finish_resize failing https://review.opendev.org/643451 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add negative test for prep_snapshot_based_resize_at_source failing https://review.opendev.org/669013 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize_at_source compute method https://review.opendev.org/637058 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add ConfirmResizeTask https://review.opendev.org/637070 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize conductor RPC method https://review.opendev.org/637075 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize from the API https://review.opendev.org/637316 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize_at_dest compute method https://review.opendev.org/637630 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Deal with cross-cell resize in _remove_deleted_instances_allocations https://review.opendev.org/639453 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add finish_revert_snapshot_based_resize_at_source compute method https://review.opendev.org/637647 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add RevertResizeTask https://review.opendev.org/638046 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize conductor RPC method https://review.opendev.org/638047 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Revert cross-cell resize from the API https://review.opendev.org/638048 | 21:17 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize while deleting a server https://review.opendev.org/638268 | 21:17 |
| *** rcernin has joined #openstack-nova | 21:17 | |
| *** ociuhandu has quit IRC | 21:19 | |
| *** ociuhandu has joined #openstack-nova | 21:23 | |
| *** dpawlik has joined #openstack-nova | 21:24 | |
| *** ociuhandu has quit IRC | 21:28 | |
| *** dpawlik has quit IRC | 21:28 | |
| *** xek_ has quit IRC | 21:31 | |
| *** ociuhandu has joined #openstack-nova | 21:44 | |
| *** ociuhandu has quit IRC | 21:49 | |
| *** eharney has joined #openstack-nova | 21:58 | |
| *** slaweq has joined #openstack-nova | 22:11 | |
| *** mriedem has quit IRC | 22:12 | |
| *** slaweq has quit IRC | 22:15 | |
| *** dviroel has quit IRC | 22:15 | |
| *** ociuhandu has joined #openstack-nova | 22:17 | |
| *** ociuhandu has quit IRC | 22:22 | |
| *** brinzhang has joined #openstack-nova | 22:26 | |
| *** brinzhang_ has quit IRC | 22:29 | |
| *** eharney has quit IRC | 22:31 | |
| *** nweinber has joined #openstack-nova | 22:31 | |
| *** adriant has joined #openstack-nova | 22:41 | |
| *** mdbooth has quit IRC | 22:42 | |
| *** slaweq has joined #openstack-nova | 22:46 | |
| *** ociuhandu has joined #openstack-nova | 22:51 | |
| *** ociuhandu has quit IRC | 22:58 | |
| *** igordc has quit IRC | 23:06 | |
| *** tkajinam has joined #openstack-nova | 23:08 | |
| *** nweinber has quit IRC | 23:18 | |
| *** mmethot has quit IRC | 23:21 | |
| *** mmethot has joined #openstack-nova | 23:22 | |
| *** awalende has joined #openstack-nova | 23:23 | |
| *** ivve has quit IRC | 23:25 | |
| *** awalende has quit IRC | 23:28 | |
| *** ociuhandu has joined #openstack-nova | 23:29 | |
| *** ociuhandu has quit IRC | 23:30 | |
| *** ociuhandu has joined #openstack-nova | 23:30 | |
| *** igordc has joined #openstack-nova | 23:31 | |
| *** ociuhandu has quit IRC | 23:36 | |
| *** maciejjozefczyk has quit IRC | 23:42 | |
| *** klindgren_ has quit IRC | 23:47 | |
| *** klindgren has joined #openstack-nova | 23:47 | |
| *** munimeha1 has quit IRC | 23:49 | |
| *** klindgren has quit IRC | 23:50 | |
| *** klindgren has joined #openstack-nova | 23:51 | |
| *** brinzhang_ has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!