| *** ociuhandu has quit IRC | 00:00 | |
| *** avolkov has quit IRC | 00:06 | |
| *** brinzhang has joined #openstack-nova | 00:12 | |
| *** brinzhang_ has quit IRC | 00:14 | |
| *** artom has quit IRC | 00:18 | |
| *** artom has joined #openstack-nova | 00:19 | |
| *** artom has quit IRC | 00:26 | |
| *** artom has joined #openstack-nova | 00:27 | |
| *** ociuhandu has joined #openstack-nova | 00:28 | |
| *** abaindur has quit IRC | 00:33 | |
| *** abaindur has joined #openstack-nova | 00:34 | |
| *** abaindur has joined #openstack-nova | 00:35 | |
| *** ttsiouts has joined #openstack-nova | 00:38 | |
| *** brinzhang has quit IRC | 00:42 | |
| *** spatel has joined #openstack-nova | 00:43 | |
| *** spatel has quit IRC | 00:48 | |
| *** ociuhandu has quit IRC | 00:54 | |
| *** Liang__ has joined #openstack-nova | 01:10 | |
| *** ttsiouts has quit IRC | 01:11 | |
| *** spatel has joined #openstack-nova | 01:13 | |
| *** spatel has quit IRC | 01:18 | |
| *** ociuhandu has joined #openstack-nova | 01:25 | |
| openstackgerrit | Merged openstack/nova master: Pass the actual target in quota class policy https://review.opendev.org/720042 | 01:28 |
|---|---|---|
| *** zhanglong has joined #openstack-nova | 01:29 | |
| *** ociuhandu has quit IRC | 01:31 | |
| *** dlbewley has quit IRC | 01:33 | |
| *** ttsiouts has joined #openstack-nova | 01:43 | |
| *** ttsiouts has quit IRC | 01:48 | |
| gmann | \o/ finally merged | 01:52 |
| *** MrWatson has joined #openstack-nova | 02:05 | |
| *** NostawRm has quit IRC | 02:05 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add docs and releasenotes for BP policy-defaults-refresh https://review.opendev.org/720129 | 02:06 |
| *** ttsiouts has joined #openstack-nova | 02:17 | |
| *** spatel has joined #openstack-nova | 02:27 | |
| *** spatel has quit IRC | 02:29 | |
| *** spatel has joined #openstack-nova | 02:29 | |
| openstackgerrit | Jie Li proposed openstack/nova master: replace the "hide_hypervisor_id" to "hw:hide_hypervisor_id" https://review.opendev.org/722187 | 02:30 |
| *** spatel has quit IRC | 02:30 | |
| *** threestrands has joined #openstack-nova | 02:33 | |
| *** mordred has quit IRC | 02:43 | |
| *** mordred has joined #openstack-nova | 02:45 | |
| *** ttsiouts has quit IRC | 02:49 | |
| *** mkrai has joined #openstack-nova | 02:51 | |
| openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Update bp/destroy-instance-with-datavolume SPEC https://review.opendev.org/722194 | 02:58 |
| *** brinzhang has joined #openstack-nova | 02:59 | |
| brinzhang | gibi, dansmith, gmann: I was update the bp/destroy-instance-with-datavolume SPEC https://review.opendev.org/#/c/722194, please review if you have free time | 03:00 |
| gmann | melwitt: need your +2 again. fixed one link in reno - https://review.opendev.org/#/c/720129/16 | 03:01 |
| gmann | brinzhang: thanks. i will check tomorrow. this is delete-on-termination spec update right ? | 03:02 |
| brinzhang | gmann: thanks | 03:02 |
| gmann | i mean as per latest code we merged as PUT not PATCH | 03:02 |
| brinzhang | gmann: yes, it's the delete-on-termination spec update | 03:02 |
| brinzhang | maybe I should rename this patch | 03:03 |
| gmann | ok. got it. thanks | 03:08 |
| *** ociuhandu has joined #openstack-nova | 03:12 | |
| brinzhang | gmann: thanks too | 03:16 |
| *** tetsuro has joined #openstack-nova | 03:16 | |
| *** tetsuro has quit IRC | 03:16 | |
| *** ociuhandu has quit IRC | 03:17 | |
| *** lbragstad has quit IRC | 03:20 | |
| *** dlbewley has joined #openstack-nova | 03:31 | |
| *** psachin has joined #openstack-nova | 03:35 | |
| *** dlbewley has quit IRC | 03:40 | |
| *** abaindur has joined #openstack-nova | 03:44 | |
| *** abaindur has joined #openstack-nova | 03:45 | |
| *** abaindur has quit IRC | 03:45 | |
| *** abaindur has joined #openstack-nova | 03:46 | |
| *** ttsiouts has joined #openstack-nova | 03:55 | |
| openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Update SPEC: Using PUT instead of PATCH API to update delete_on_termination https://review.opendev.org/722194 | 04:08 |
| *** tetsuro has joined #openstack-nova | 04:11 | |
| *** ratailor has joined #openstack-nova | 04:11 | |
| *** udesale has joined #openstack-nova | 04:22 | |
| *** ttsiouts has quit IRC | 04:28 | |
| *** evrardjp has quit IRC | 04:35 | |
| *** evrardjp has joined #openstack-nova | 04:35 | |
| *** bbowen_ has joined #openstack-nova | 04:39 | |
| *** bbowen has quit IRC | 04:41 | |
| *** brinzhang_ has joined #openstack-nova | 04:44 | |
| *** bbowen has joined #openstack-nova | 04:47 | |
| *** bbowen_ has quit IRC | 04:47 | |
| *** brinzhang has quit IRC | 04:47 | |
| *** ttsiouts has joined #openstack-nova | 05:00 | |
| *** ttsiouts has quit IRC | 05:05 | |
| *** vishalmanchanda has joined #openstack-nova | 05:27 | |
| *** gyee has quit IRC | 05:29 | |
| *** ttsiouts has joined #openstack-nova | 05:34 | |
| *** dlbewley has joined #openstack-nova | 05:37 | |
| *** ircuser-1 has quit IRC | 05:40 | |
| *** yedongcan has joined #openstack-nova | 05:42 | |
| *** dlbewley has quit IRC | 05:43 | |
| *** slaweq has joined #openstack-nova | 05:48 | |
| *** francoisp has quit IRC | 05:48 | |
| *** ociuhandu has joined #openstack-nova | 05:54 | |
| *** ociuhandu has quit IRC | 05:59 | |
| *** dpawlik has joined #openstack-nova | 06:01 | |
| *** ociuhandu has joined #openstack-nova | 06:01 | |
| *** ociuhandu has quit IRC | 06:06 | |
| *** ttsiouts has quit IRC | 06:07 | |
| *** zhanglong has quit IRC | 06:21 | |
| *** zhanglong has joined #openstack-nova | 06:23 | |
| *** mkrai has quit IRC | 06:37 | |
| *** mkrai has joined #openstack-nova | 06:50 | |
| *** ociuhandu has joined #openstack-nova | 06:50 | |
| *** nightmare_unreal has joined #openstack-nova | 06:52 | |
| *** mkrai has quit IRC | 06:56 | |
| gibi | brinzhang_: ack, I will look at it probably next week | 06:58 |
| *** belmoreira has joined #openstack-nova | 06:58 | |
| brinzhang_ | gibi: ack, it's not in a hurry | 06:59 |
| brinzhang_ | gibi: thanks | 06:59 |
| *** adrianc has quit IRC | 06:59 | |
| *** mkrai has joined #openstack-nova | 07:00 | |
| *** zhanglong has quit IRC | 07:02 | |
| *** tesseract has joined #openstack-nova | 07:03 | |
| *** ttsiouts has joined #openstack-nova | 07:11 | |
| *** dklyle has quit IRC | 07:11 | |
| *** KeithMnemonic has quit IRC | 07:12 | |
| *** adrianc has joined #openstack-nova | 07:12 | |
| *** ttsiouts has quit IRC | 07:15 | |
| *** ttsiouts has joined #openstack-nova | 07:16 | |
| *** mkrai has quit IRC | 07:21 | |
| *** ociuhandu has quit IRC | 07:23 | |
| *** tosky has joined #openstack-nova | 07:24 | |
| *** ociuhandu has joined #openstack-nova | 07:24 | |
| *** zhanglong has joined #openstack-nova | 07:27 | |
| *** xek has joined #openstack-nova | 07:27 | |
| openstackgerrit | Merged openstack/nova master: Add docs and releasenotes for BP policy-defaults-refresh https://review.opendev.org/720129 | 07:30 |
| *** ociuhandu has quit IRC | 07:30 | |
| bauzas | gibi: gmann: I'll provide a FUP for ^ for the reno prelude https://review.opendev.org/#/c/721548/ | 07:34 |
| *** rpittau|afk is now known as rpittau | 07:39 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: FUP: Amend ussuri prelude to add docs for policy concepts https://review.opendev.org/722220 | 07:40 |
| *** sapd1 has joined #openstack-nova | 07:40 | |
| bauzas | gmann: gibi: p | 07:40 |
| *** dlbewley has joined #openstack-nova | 07:40 | |
| gibi | bauzas: cool | 07:40 |
| gibi | thanks. I'm +A on the prelude and +2 on the FUP | 07:41 |
| bauzas | nice | 07:41 |
| bauzas | thanks | 07:41 |
| bauzas | gibi: any urgent change review we should do before RC1 ? | 07:41 |
| bauzas | gibi: given the owner haven't provided a new revision for https://review.opendev.org/#/c/719163/, I'll remove the bug tag | 07:42 |
| bauzas | oh you already did :) | 07:43 |
| *** yaawang has quit IRC | 07:43 | |
| gibi | yepp, I pinged you during the week about it but figured you were busy so went ahead and removed it | 07:43 |
| *** ralonsoh has joined #openstack-nova | 07:44 | |
| gibi | I can pick that fixup later if the author did not show up | 07:44 |
| gibi | bauzas: there is a simple revert we need to merge before RC1 https://review.opendev.org/#/c/718630 | 07:44 |
| *** dlbewley has quit IRC | 07:44 | |
| gibi | and the ussuri RPC alias https://review.opendev.org/#/c/719315/ | 07:45 |
| *** yaawang has joined #openstack-nova | 07:45 | |
| gibi | I'm not tracking anyithing else for RC1 at the moment | 07:45 |
| *** psachin has quit IRC | 07:49 | |
| bauzas | cool | 07:58 |
| * gibi goes and works on today's meeting agenda | 08:00 | |
| *** ociuhandu has joined #openstack-nova | 08:01 | |
| *** mkrai has joined #openstack-nova | 08:04 | |
| *** ttsiouts has quit IRC | 08:05 | |
| *** ttsiouts has joined #openstack-nova | 08:05 | |
| *** psachin has joined #openstack-nova | 08:06 | |
| bauzas | gibi: fwiw, all the RPC APIs should have aliases... | 08:10 |
| *** links has joined #openstack-nova | 08:11 | |
| bauzas | gibi: but for example, see the last alias for the scheduler... https://github.com/openstack/nova/blob/master/nova/scheduler/rpcapi.py#L120 | 08:11 |
| *** ociuhandu has quit IRC | 08:11 | |
| *** ociuhandu has joined #openstack-nova | 08:12 | |
| *** igordc has quit IRC | 08:13 | |
| *** ociuhandu has quit IRC | 08:17 | |
| *** martinkennelly has joined #openstack-nova | 08:17 | |
| gibi | bauzas: I do the aliasing based on https://docs.openstack.org/nova/latest/contributor/ptl-guide.html | 08:18 |
| gibi | "In the past, we used to update all service version aliases (example: https://review.opendev.org/230132) but since we really only support compute being backlevel/old during a rolling upgrade, we only need to update the compute service alias, see related IRC discussion: http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2018-08-08.log.html#t2018-08-08T17:13:45" | 08:18 |
| *** kevinz has joined #openstack-nova | 08:19 | |
| *** martinkennelly has quit IRC | 08:23 | |
| *** mkrai has quit IRC | 08:27 | |
| *** mkrai_ has joined #openstack-nova | 08:27 | |
| *** avolkov has joined #openstack-nova | 08:30 | |
| *** ociuhandu has joined #openstack-nova | 08:32 | |
| *** dtantsur|afk is now known as dtantsur | 08:32 | |
| *** threestrands has quit IRC | 08:33 | |
| *** breizhkoala has joined #openstack-nova | 08:44 | |
| *** happyhemant has joined #openstack-nova | 08:46 | |
| *** derekh has joined #openstack-nova | 08:46 | |
| *** psachin has quit IRC | 08:50 | |
| *** abaindur has quit IRC | 08:51 | |
| bauzas | gibi: oh ok thanks | 08:55 |
| openstackgerrit | Merged openstack/nova master: Ussuri 21.0.0 prelude section https://review.opendev.org/721548 | 09:01 |
| *** abaindur has joined #openstack-nova | 09:03 | |
| *** tkajinam has quit IRC | 09:03 | |
| *** abaindur has quit IRC | 09:03 | |
| *** ociuhandu has quit IRC | 09:07 | |
| *** ociuhandu has joined #openstack-nova | 09:12 | |
| *** ociuhandu has quit IRC | 09:17 | |
| *** ccamacho has joined #openstack-nova | 09:23 | |
| *** tetsuro has quit IRC | 09:24 | |
| *** ociuhandu has joined #openstack-nova | 09:26 | |
| openstackgerrit | Merged openstack/nova master: Update compute rpc version alias for ussuri https://review.opendev.org/719315 | 09:59 |
| *** Liang__ has quit IRC | 10:02 | |
| *** rcernin has quit IRC | 10:03 | |
| *** zhanglong has quit IRC | 10:11 | |
| *** rpittau is now known as rpittau|bbl | 10:21 | |
| *** yedongcan has left #openstack-nova | 10:32 | |
| *** huaqiang has quit IRC | 10:50 | |
| *** links has quit IRC | 10:52 | |
| *** links has joined #openstack-nova | 11:02 | |
| *** sapd1_x has joined #openstack-nova | 11:09 | |
| kashyap | artom: I've responded to your feedback; thanks for the commentary. Hope that makes sense. | 11:09 |
| kashyap | artom: Also see the important Plan-B | 11:09 |
| *** alex_xu has quit IRC | 11:10 | |
| *** sapd1_y has quit IRC | 11:11 | |
| *** breizhkoala has quit IRC | 11:24 | |
| zigo | bauzas: Hi there! I suppose FUP doesn't mean what's in this urban dictionnary: https://www.urbandictionary.com/define.php?term=FUP :) | 11:27 |
| *** sapd1_y has joined #openstack-nova | 11:27 | |
| zigo | Fair Usage Policy ? | 11:27 |
| kashyap | zigo: Yeah, that's a ridiculous acronym that I keep saying people not to use. | 11:27 |
| kashyap | Just say 'fix-up'. | 11:27 |
| kashyap | Clear writing is a sign of clear thinking. Don't use mindless acronyms. | 11:28 |
| kashyap | But that requires hardwork, and it's easy to be lazy | 11:28 |
| kashyap | [Not saying it anyone in particular, BTW :D, just saying it out loud.] | 11:28 |
| zigo | DMT TLA ... :P | 11:28 |
| *** ociuhandu has quit IRC | 11:29 | |
| *** sapd1_x has quit IRC | 11:29 | |
| zigo | TMD TLA I mean. | 11:30 |
| *** ociuhandu has joined #openstack-nova | 11:31 | |
| *** ociuhandu has quit IRC | 11:35 | |
| *** bbowen_ has joined #openstack-nova | 11:37 | |
| *** bbowen has quit IRC | 11:38 | |
| bauzas | zigo: Follow-Up Patch ;) | 11:41 |
| zigo | :) | 11:46 |
| *** ttsiouts has quit IRC | 11:48 | |
| *** nweinber has joined #openstack-nova | 11:48 | |
| *** mkrai_ has quit IRC | 11:53 | |
| *** mkrai__ has joined #openstack-nova | 11:53 | |
| *** sapd1 has quit IRC | 11:58 | |
| *** ttsiouts has joined #openstack-nova | 12:00 | |
| *** rpittau|bbl is now known as rpittau | 12:02 | |
| *** udesale_ has joined #openstack-nova | 12:20 | |
| *** udesale has quit IRC | 12:22 | |
| lyarwood | stephenfin: random question, any idea where _get_vcpu_total was removed in the libvirt driver? I'm trying to rebase a series from mdbooth and I can't find where it was removed and what it was replaced with. | 12:29 |
| stephenfin | it would have been in Train | 12:29 |
| lyarwood | stephenfin: looks like you refactored things around here for the pcpu stuff | 12:29 |
| stephenfin | git log -S _get_vcpu_total | 12:29 |
| lyarwood | TIL | 12:29 |
| lyarwood | odd that tig couldn't find it | 12:31 |
| lyarwood | I thought the searching in tig did the same tbh | 12:31 |
| lyarwood | I guess it's just looking at the subject, meh. | 12:31 |
| *** jraju__ has joined #openstack-nova | 12:41 | |
| *** links has quit IRC | 12:41 | |
| zigo | Are you guys planning the RC1 for today or tomorrow? | 12:43 |
| *** mkrai__ has quit IRC | 12:48 | |
| *** mkrai has joined #openstack-nova | 12:48 | |
| *** mkrai has quit IRC | 12:52 | |
| *** mkrai_ has joined #openstack-nova | 12:52 | |
| *** ociuhandu has joined #openstack-nova | 12:53 | |
| *** ratailor has quit IRC | 12:54 | |
| gibi | zigo: today | 12:58 |
| gibi | zigo: do you have a critical thing? | 12:59 |
| zigo | gibi: No, just wonder when I'll be able to package it for Debian ! :P | 13:03 |
| gibi | zigo: cool. I will ping you when the release patch merges | 13:03 |
| gibi | zigo: and please let us know if somehing breaks for you | 13:04 |
| zigo | It's gona take some time until I can spawn a first VM with Ussuri. It's going to be maybe next week... | 13:04 |
| zigo | I guess some of my puppet stuff will break, etc. | 13:04 |
| *** igordc has joined #openstack-nova | 13:07 | |
| *** mkrai has joined #openstack-nova | 13:08 | |
| gibi | zigo: we have 2 weeks for additional RC if needed | 13:11 |
| *** mkrai_ has quit IRC | 13:11 | |
| zigo | Usually, I can install the release between the first rc and the final releases. | 13:12 |
| *** lbragstad has joined #openstack-nova | 13:12 | |
| gibi | cool | 13:12 |
| gmann | bauzas: thanks | 13:12 |
| *** martinkennelly has joined #openstack-nova | 13:13 | |
| *** udesale_ has quit IRC | 13:16 | |
| *** udesale has joined #openstack-nova | 13:16 | |
| *** udesale has quit IRC | 13:18 | |
| *** udesale has joined #openstack-nova | 13:19 | |
| artom | sean-k-mooney, re - your point about changing the default machine type downstream/in ooo only | 13:30 |
| artom | I'll be honest, I only trust upstream's CI with this :P | 13:30 |
| sean-k-mooney | artom: we are chaning nova-next to use q35 | 13:36 |
| sean-k-mooney | and we could change the default in devstack too if we want too | 13:37 |
| sean-k-mooney | but i just am not sure we should change it in nova | 13:37 |
| sean-k-mooney | if we change it in nova and you cold migrate or you live migrate and then hard reboot the machine type would change | 13:38 |
| sean-k-mooney | and that can break things | 13:38 |
| *** mkrai has quit IRC | 13:38 | |
| lyarwood | sean-k-mooney: if we don't stash and use the original somehow | 13:39 |
| sean-k-mooney | so without recording the machine type in the instance which we dont do today when its not set in the image its not really safe from an upgrades perspective to change the default. | 13:39 |
| sean-k-mooney | lyarwood: yes we dont | 13:39 |
| lyarwood | yup | 13:39 |
| sean-k-mooney | we also dont schdule on this today | 13:40 |
| sean-k-mooney | which is less then ideal | 13:40 |
| lyarwood | so this is less about setting defaults and more about proving support? | 13:40 |
| lyarwood | and allowing it to be used in new deployments | 13:40 |
| sean-k-mooney | no not really | 13:40 |
| sean-k-mooney | i think in our product we should make the switch for new deployments | 13:41 |
| sean-k-mooney | upstream im not sure if we can in nova | 13:41 |
| lyarwood | right but we can in devstack | 13:41 |
| lyarwood | I didn't mean that Nova would change behaviour | 13:41 |
| sean-k-mooney | yep we can in any install tool | 13:41 |
| lyarwood | it would just support deployment tooling that did | 13:41 |
| sean-k-mooney | because on upgrade they can just set the old default or not set it at all | 13:41 |
| *** francoisp has joined #openstack-nova | 13:42 | |
| sean-k-mooney | lyarwood: kashyap's spec is chaning the nova defaults | 13:42 |
| sean-k-mooney | lyarwood: which is what i was talking to artom about | 13:42 |
| sean-k-mooney | its why i -1'd it | 13:42 |
| sean-k-mooney | i would like to supprot have default be configureable via a new hw:profile extra spec | 13:43 |
| sean-k-mooney | so you can opt in to different defaults but not actully change them | 13:43 |
| sean-k-mooney | my understanding is kashyap was advocating for changing the default in the libvirt driver directly | 13:43 |
| sean-k-mooney | kind of like we did for the random number generator | 13:44 |
| sean-k-mooney | but i dont think we can really do that for machine_type safely | 13:44 |
| *** mkrai has joined #openstack-nova | 13:44 | |
| sean-k-mooney | well unless we recored the current machine type when an instance first boots | 13:44 |
| *** tkajinam has joined #openstack-nova | 13:44 | |
| *** priteau has joined #openstack-nova | 13:45 | |
| sean-k-mooney | which would rereuie an online data migration to backfile existing instnaces | 13:45 |
| *** ttsiouts has quit IRC | 13:46 | |
| * kashyap catches up after a long walk | 13:46 | |
| sean-k-mooney | welll im not sure online data migration is the correct term but init_host would have to be extended to iterate over the domain on the host and populate the value in the db for all instance on the host if it is not already set | 13:47 |
| sean-k-mooney | so we would have to start the virtdriver at a minium | 13:47 |
| sean-k-mooney | anyway i need to bring my furniture in form my drive so ill be back in a few minutes | 13:48 |
| kashyap | sean-k-mooney: As I said on my change, I'm very much open to the "important Plan-B" | 13:48 |
| kashyap | artom: lyarwood: ^ | 13:48 |
| kashyap | Where "Plan-B" being [/me goes to get copy/paste text from the change ...] | 13:49 |
| *** ttsiouts has joined #openstack-nova | 13:49 | |
| kashyap | - Have all major Nova CI jobs be 'q35' based and passing. | 13:49 |
| kashyap | - Make installer tools set the precise q35 versioned machine types on all Compute nodes —they anyway have to do this to live migration compatibility. | 13:49 |
| artom | sean-k-mooney, yeah, I bring this up on the spec, in upgrade impact - we're likely to "break" windows guests if we change the hardware under them | 13:50 |
| *** grandchild has joined #openstack-nova | 13:53 | |
| *** vishalmanchanda has quit IRC | 14:07 | |
| *** mkrai has quit IRC | 14:08 | |
| *** mkrai_ has joined #openstack-nova | 14:08 | |
| artom | Didn't we have something more recent for "force no local disk/force boot from volume" than https://review.opendev.org/#/c/267673/? | 14:10 |
| *** sapd1 has joined #openstack-nova | 14:12 | |
| lyarwood | artom: we've discussed it since but I didn't think anything actually landed | 14:20 |
| artom | lyarwood, I tracked down https://review.opendev.org/#/c/561284/ | 14:23 |
| artom | Which achieves the same thing | 14:23 |
| francoisp | lyarwood, stephenfin, when you get a chance: https://review.opendev.org/#/c/722142/ | 14:25 |
| lyarwood | ah right I think I missed that actually landing | 14:25 |
| lyarwood | francoisp: looking | 14:25 |
| stephenfin | francoisp: I'm not stable core so I can't help, unfortunately :( | 14:26 |
| sean-k-mooney | artom: we are likely to break linux guest too if they have set thing like hw_vif_model=e1000 | 14:30 |
| sean-k-mooney | or enabled any other pci only device model | 14:30 |
| sean-k-mooney | hw_disk_bus=ide would also break | 14:31 |
| sean-k-mooney | unless we did extra work to add an ide conttoler | 14:31 |
| sean-k-mooney | kashyap: well in also not sure about using the versionded q35 machine type in the installeer unelss the ones that support multiple distos have a different one from each | 14:32 |
| sean-k-mooney | for devstack we should use the unversioned one e.g. q35 | 14:32 |
| sean-k-mooney | so it just works on all distros | 14:33 |
| sean-k-mooney | osp could use the versioned one but ooo likely will want to not as it would have to be updated with ever rhel/centos minor verions | 14:33 |
| *** tkajinam has quit IRC | 14:37 | |
| kashyap | sean-k-mooney: I mean whatever installer tool a given distro uses | 14:37 |
| kashyap | sean-k-mooney: TripleO works on RPM-based distributions (CentOS, RHEL, Scientific) probably - they can set whatever they want | 14:37 |
| kashyap | sean-k-mooney: If Ubuntu-based distros use a different installer, they will use their own | 14:37 |
| kashyap | sean-k-mooney: For DevStack, of course, we should use unversioned - the plain alias | 14:38 |
| kashyap | The whole point of versioned machie types is, as you might already know, to guarantee live migration compatibility | 14:38 |
| kashyap | sean-k-mooney: TripleO upstream already does that | 14:38 |
| kashyap | sean-k-mooney: Also, note: even for CentOS, the machine types have 'rhel' in their name. | 14:39 |
| lyarwood | elod: could you hit https://review.opendev.org/#/c/722142/ as well please? | 14:39 |
| kashyap | sean-k-mooney: Because CentOS obviously does not have "CentOS QEMU developers" to make the needless effort of making CentOS-based machine types. :) | 14:40 |
| kashyap | sean-k-mooney: To summarize: | 14:40 |
| kashyap | - DevStack -- unversioned, plain 'q35' | 14:40 |
| kashyap | - Installers -- versioned 'q35', based on the QEMU versions your installer + Linux distro can handle | 14:41 |
| *** dlbewley has joined #openstack-nova | 14:42 | |
| *** mkrai_ has quit IRC | 14:42 | |
| sean-k-mooney | kashyap: i assume ooo is using the minium version supported by the os major version then in that case | 14:49 |
| sean-k-mooney | so it will have one version that is never updated for all fo rhel/centos 8/x | 14:49 |
| sean-k-mooney | * 8.X | 14:49 |
| *** dklyle has joined #openstack-nova | 14:49 | |
| kashyap | sean-k-mooney: Why do you "minimum version"? E.g. https://review.opendev.org/#/c/716526/3/deployment/nova/nova-compute-container-puppet.yaml | 14:50 |
| kashyap | sean-k-mooney: They update it to whatever all the Compute nodes were tested with | 14:51 |
| sean-k-mooney | kashyap: when train released on 8.0 ooo still need to be able to install that on 8.1 or 8.2 later | 14:51 |
| kashyap | sean-k-mooney: Sure, it will work; why would that be any problem? | 14:52 |
| elod | lyarwood: looks good to me. +W'd | 14:52 |
| sean-k-mooney | but if you used the max version for that majory version there would be issues with live migraion later if you scaled out a cludete ther started on 8.0 with new node on 8.1 | 14:52 |
| kashyap | If you set the machine type to 8.0; you can of course install it on 8.1 and 8.2 | 14:52 |
| *** ttsiouts has quit IRC | 14:52 | |
| lyarwood | elod: thanks | 14:52 |
| kashyap | sean-k-mooney: Yes, the version is chosen in a way that the operator must ensure the Compute nodes are capable of it | 14:53 |
| kashyap | sean-k-mooney: Recall, the version in the machien type matches CentOS|RHEL minor version - it is pretty clear to figure out the problem due to host OS version mismatch... | 14:53 |
| sean-k-mooney | yep wich i ugess is the max verion supported by the oldes hypervior | 14:53 |
| sean-k-mooney | which would be the verion form 8.0 in this case | 14:54 |
| kashyap | Right. | 14:54 |
| kashyap | We clear? :) | 14:54 |
| kashyap | I take it we are. | 14:57 |
| *** ttsiouts has joined #openstack-nova | 15:00 | |
| sean-k-mooney | kashyap: yep sorry was talking dowstream | 15:02 |
| sean-k-mooney | and ya we are on the same page regarding the requirements here | 15:03 |
| kashyap | No problem; thanks | 15:03 |
| lyarwood | kashyap / sean-k-mooney ; silly question but why wouldn't we use the q35 alias all the time downstream? | 15:10 |
| kashyap | lyarwood: Live migration compatibility | 15:10 |
| lyarwood | kashyap: but LM wouldn't switch between the two surely? | 15:10 |
| kashyap | lyarwood: I have an example written down, with a scenario, so you can see it plainly | 15:10 |
| kashyap | Let me post it somewhere, so you can see it, and possibly punch holes in it | 15:11 |
| lyarwood | ah wait so QEMU would just blindly use the alias on the dest as well? | 15:12 |
| lyarwood | even if it pointed to a newer version | 15:12 |
| lyarwood | sorry I assumed libvirt/QEMU would translate the alias into an actual versioned q35 machine type and stick with that for the life cycle of the domain | 15:12 |
| kashyap | lyarwood: Yes | 15:14 |
| kashyap | lyarwood: If you try to migrate a guest with RHEL-7.7 machine type (running on RHEL-7 COmpute node) to a RHEL-7.6 Compute node -- it fails spectacularly | 15:15 |
| kashyap | Because 7.6 host obviously doesn't know about 7.7 :) | 15:15 |
| lyarwood | right but my point is more that forward migration from 7.6 to 7.7 just using the q35 alias could also fail | 15:16 |
| lyarwood | as the domain would jump from the 7.6 to 7.7 version of the machine type | 15:17 |
| kashyap | lyarwood: https://kashyapc.fedorapeople.org/versioned-machine-types-and-live-migration-gotcha.txt | 15:17 |
| lyarwood | well maybe not fail but things could change | 15:17 |
| kashyap | lyarwood: Forward migration fomr 7.6 to 7.7 will _not_ fail | 15:17 |
| kashyap | Why? | 15:17 |
| kashyap | Because, 7.7 host will have all the older (7.6, 7.5, 7.4) machine types precisely for live migration compat | 15:17 |
| kashyap | lyarwood: I had the above text somewhere in the spec, but deleted it, for "brevity" | 15:18 |
| kashyap | artom: Since you also spent non-trivial time reviewing the spec, you might benefit from reading this: https://kashyapc.fedorapeople.org/versioned-machine-types-and-live-migration-gotcha.txt | 15:18 |
| lyarwood | It will now be _impossible_ to migrate a guest from a RHEL 7.6-based | 15:18 |
| lyarwood | compute node to a RHEL 7.7-based — because RHEL 7.6 won't know about | 15:18 |
| lyarwood | the RHEL 7.7's machine type. | 15:18 |
| lyarwood | kashyap: ^ that was my point sorry | 15:18 |
| *** zhanglong has joined #openstack-nova | 15:19 | |
| lyarwood | and the workaround appears to be to pin to the oldest version | 15:19 |
| kashyap | Yep | 15:19 |
| lyarwood | but I just don't get how this works downstream across multple releases | 15:19 |
| lyarwood | we'd basically need to pin to the first version used in the env | 15:19 |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Fix list rendering in the accelerator support doc https://review.opendev.org/721846 | 15:20 |
| lyarwood | surely we should work out and persist the versioned machine type during a migration? | 15:21 |
| lyarwood | so if the source is 8.1 and using the q35 alias ensure it uses that on the dest | 15:22 |
| lyarwood | but all new instances use the 8.2 q35 type? | 15:22 |
| *** kaisers_ has joined #openstack-nova | 15:22 | |
| kashyap | lyarwood: For downstream, yes: you need to pick a versioned machine type and set it across the nodes before deplyoying. | 15:22 |
| aarents | lyarwood: kashyap we hit this kind of issues.. but Im not the guy who work on it I will check how we dealt with that.. | 15:22 |
| lyarwood | kashyap: sorry I'm talking out loud here, could you add the LM section back into the spec and I'll add words there. | 15:23 |
| kashyap | lyarwood: Sure, let me add that bit in the spec in a separate section; it's important | 15:23 |
| lyarwood | kashyap: yeah the issue with that is you can never move from it | 15:23 |
| *** zhanglong has quit IRC | 15:23 | |
| kashyap | lyarwood: Hang on - you _can_ move; it's a step-by-step process: | 15:24 |
| *** zhanglong has joined #openstack-nova | 15:24 | |
| kashyap | lyarwood: Err, my "Here's the problem" has a bug! | 15:25 |
| kashyap | It should read the _other_ way round. | 15:25 |
| kashyap | Let me update it | 15:25 |
| lyarwood | kashyap: kk, lets get the spec updated and chat there :) | 15:25 |
| kashyap | You cannot migrte from 7.7 to 7.6 | 15:26 |
| kashyap | But you can jolly well migrate from 7.6 to 7.7 :) | 15:26 |
| kashyap | Yes | 15:26 |
| kashyap | A sec | 15:26 |
| artom | I feel like this whole mess could be solved if we tracked machine type on a per-instance basis | 15:30 |
| kashyap | aarents: Noted :) | 15:30 |
| artom | Though the problem that raises is live-migration from an N+1 to an N compute | 15:31 |
| artom | The dest N might now know the machine type, as it's newer | 15:31 |
| kashyap | Per-instance tracking is just too damn fragile and messy | 15:31 |
| *** belmoreira has quit IRC | 15:31 | |
| kashyap | I'd say per-Compute based is better for debugging, too. | 15:31 |
| artom | Maybe, but how/why? | 15:31 |
| kashyap | We should not forget the Operator's predicmanet | 15:31 |
| kashyap | s/predicmanet/predicament/ | 15:32 |
| artom | This whole discussion shows that per-compute isn't super clean either :) | 15:32 |
| kashyap | artom: The live migration from N+1 to N always remains a problem. | 15:32 |
| artom | True | 15:32 |
| artom | Could we not incorporate machine type into our LM checks? | 15:33 |
| kashyap | Can you tell more what do we gain more per-instance tracking solve? Just asking sincerely... :) | 15:33 |
| * kashyap first goes to update the spec, before he needs to "head out" shortly | 15:33 | |
| sean-k-mooney | kashyap: if we dont track the machine type and you upgreade the host the vm might not work after a hardreboot | 15:42 |
| sean-k-mooney | e.g. if it had a conflciting image peroperty like hw_vif_model=e1000 or hw_disk_bus=ide | 15:43 |
| sean-k-mooney | so if its not set in the image and is just being chosen by our default or the host config then we need to recored it to not break existing instnace on upgrade | 15:44 |
| sean-k-mooney | hopefuly that makes sense | 15:44 |
| *** ttsiouts has quit IRC | 15:46 | |
| *** mlavalle has joined #openstack-nova | 15:49 | |
| gibi | nova meeting starts in 10 minutes on #openstack-nova | 15:50 |
| gibi | nova meeting starts in 10 minutes on #openstack-meeting-3 | 15:51 |
| gibi | /o\ | 15:51 |
| lyarwood | elod / melwitt: https://review.opendev.org/#/c/720587/ - I think this is ready right? We aren't waiting on actual failures to show up first in stable/stein? | 15:52 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/stein: Make RBD imagebackend flatten method idempotent https://review.opendev.org/708615 | 15:53 |
| kashyap | sean-k-mooney: Will respond in a few | 15:53 |
| *** gyee has joined #openstack-nova | 15:53 | |
| sean-k-mooney | kashyap: no rush | 15:53 |
| kashyap | sean-k-mooney: lyarwood: So, I just double-confirmed with upstream QEMU: as I guessed, depsite KM-long threads, there's no agreement whether to not add new 'pc' variants or not -- it is strictly a RHEL-only decsion | 15:53 |
| kashyap | And also other long-term distributions (SUSE, et al) | 15:54 |
| kashyap | So, I'll add "suitable words" | 15:54 |
| kashyap | (In the end this spec might even not be needed ... but at least it allows to flesh out ideas) | 15:54 |
| sean-k-mooney | kashyap: yep. ill try to review it more properly on monday but at least it s place to collect and discuss the alternitives | 15:55 |
| * sean-k-mooney im of tommorw hench the "ill revew on monday" | 15:55 | |
| kashyap | sean-k-mooney: Sure, yeah. It at least keeps the ideas in one place | 15:56 |
| *** udesale_ has joined #openstack-nova | 15:57 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient master: [Community goal] Update contributor documentation https://review.opendev.org/722391 | 15:58 |
| openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: [Community goal] Update contributor documentation https://review.opendev.org/722392 | 15:58 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs master: [Community goal] Update contributor documentation https://review.opendev.org/722393 | 15:58 |
| *** udesale has quit IRC | 16:00 | |
| *** zhanglong has quit IRC | 16:01 | |
| elod | lyarwood: according to logstash the failure disappeared, so maybe we could merge that now | 16:02 |
| *** sapd1 has quit IRC | 16:03 | |
| melwitt | lyarwood: it is ready yes. tbh I'm surprised it's 100% gone from logstash bc I expected to see it on older branches (train and earlier). I wonder if logstash does only master branch only? I'm not sure | 16:04 |
| lyarwood | I thought it did all of the supported releases tbh | 16:05 |
| lyarwood | train and stein still are | 16:05 |
| melwitt | that's what I had thought too | 16:05 |
| lyarwood | but anyway lets merge that | 16:05 |
| melwitt | but I guess they get much less traffic | 16:05 |
| lyarwood | yup very true | 16:06 |
| *** tesseract has quit IRC | 16:10 | |
| *** ttsiouts has joined #openstack-nova | 16:11 | |
| *** sapd1 has joined #openstack-nova | 16:16 | |
| *** udesale_ has quit IRC | 16:23 | |
| *** yaawang_ has joined #openstack-nova | 16:32 | |
| *** yaawang has quit IRC | 16:33 | |
| *** evrardjp has quit IRC | 16:35 | |
| *** evrardjp has joined #openstack-nova | 16:35 | |
| *** ociuhandu has quit IRC | 16:39 | |
| *** ociuhandu has joined #openstack-nova | 16:40 | |
| lyarwood | melwitt / elod ; following up on the stable discussion in the meeting can you hit https://review.opendev.org/#/c/708578/ | 16:41 |
| lyarwood | melwitt: can you also hit https://review.opendev.org/#/c/708615/ | 16:41 |
| lyarwood | and I think we should be ready to cut a release once these have merged | 16:41 |
| melwitt | sure | 16:41 |
| lyarwood | many thanks | 16:41 |
| bauzas | lyarwood: I'll look at those stable/stein changes | 16:42 |
| lyarwood | thanks | 16:44 |
| * lyarwood -> afk for a few hours | 16:44 | |
| *** ociuhandu has quit IRC | 16:45 | |
| gibi | bauzas: I will check back in couple of hours to update the RC1 release patch | 16:46 |
| bauzas | gibi: want me to do it this evening ? | 16:46 |
| gibi | bauzas: I will cover this. you covered the novaclient | 16:46 |
| bauzas | cool | 16:46 |
| gibi | loadsharing :) | 16:46 |
| bauzas | well, I'm a bit depressed by something with my house, so I would just turn into work to forget it :p | 16:48 |
| *** rpittau is now known as rpittau|afk | 16:49 | |
| *** derekh has quit IRC | 16:49 | |
| openstackgerrit | Merged openstack/nova-specs master: move implemented spec for train https://review.opendev.org/706276 | 16:51 |
| *** maciejjozefczyk has quit IRC | 16:55 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/python-novaclient master: [Community goal] Update contributor documentation https://review.opendev.org/722391 | 16:56 |
| *** belmoreira has joined #openstack-nova | 17:09 | |
| *** jraju__ has quit IRC | 17:14 | |
| *** belmoreira has quit IRC | 17:16 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: Fix doc build job for wanring turn into error https://review.opendev.org/722407 | 17:16 |
| openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: [Community goal] Update contributor documentation https://review.opendev.org/722392 | 17:16 |
| *** ttsiouts has quit IRC | 17:16 | |
| *** priteau has quit IRC | 17:16 | |
| *** nightmare_unreal has quit IRC | 17:21 | |
| *** ociuhandu has joined #openstack-nova | 17:22 | |
| openstackgerrit | Merged openstack/nova master: FUP: Amend ussuri prelude to add docs for policy concepts https://review.opendev.org/722220 | 17:22 |
| *** dtantsur is now known as dtantsur|afk | 17:28 | |
| *** ociuhandu has quit IRC | 17:29 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova-specs master: Make Q35 machine type the default for x86 https://review.opendev.org/631154 | 17:30 |
| *** slaweq has quit IRC | 17:50 | |
| *** ttsiouts has joined #openstack-nova | 17:53 | |
| *** sapd1 has quit IRC | 17:56 | |
| *** ttsiouts has quit IRC | 17:58 | |
| *** francoisp has quit IRC | 18:03 | |
| *** stephenfin has quit IRC | 18:03 | |
| *** stephenfin has joined #openstack-nova | 18:04 | |
| openstackgerrit | Merged openstack/nova stable/train: Add retry to cinder API calls related to volume detach https://review.opendev.org/722142 | 18:06 |
| *** ralonsoh has quit IRC | 18:07 | |
| openstackgerrit | Elod Illes proposed openstack/nova stable/stein: Reject boot request for unsupported images https://review.opendev.org/708578 | 18:12 |
| *** ociuhandu has joined #openstack-nova | 18:12 | |
| *** stephenfin has quit IRC | 18:19 | |
| *** stephenfin has joined #openstack-nova | 18:19 | |
| *** ttsiouts has joined #openstack-nova | 18:29 | |
| * artom sucks at upstream triage | 18:40 | |
| artom | I've set one bug to fix released, because that's what it was, and another as wontfix, because while super impressive, it's just not a concern in practice. | 18:41 |
| artom | I've also looked at a dozen others, all of which have ended with "I have no idea" | 18:41 |
| *** KeithMnemonic has joined #openstack-nova | 18:46 | |
| *** ttsiouts has quit IRC | 18:47 | |
| *** ttsiouts has joined #openstack-nova | 18:49 | |
| *** maciejjozefczyk has joined #openstack-nova | 18:51 | |
| artom | What's vmware's driver status? | 18:55 |
| artom | IIRC dansmith was talking about deprecating it? Do we have anyone working on it? | 18:56 |
| dansmith | we marked it as deprecated this cycle | 18:57 |
| dansmith | it'll be a few before we can remove it | 18:57 |
| artom | OK, so for a bugin that driver, it'd be fair to say wontfix? | 18:57 |
| artom | In order to set realistic exptectations? | 18:57 |
| dansmith | that's really for the owners of the driver to say I guess | 18:57 |
| dansmith | I would maybe mark it as triaged or whatever, if it looks valid and tag it so it's clearly vmware-related | 18:58 |
| dansmith | and maybe comment that it's unlikely to get fixed, but marking it as wontfix is probably too aggressive | 18:58 |
| artom | It's already tagged, I have no idea if it's valid, I'll leave it alone then | 18:58 |
| * dansmith nods | 18:59 | |
| artom | And now you're making me doubt closing https://bugs.launchpad.net/nova/+bug/1800204 as wontfix | 18:59 |
| openstack | Launchpad bug 1800204 in OpenStack Compute (nova) "n-cpu.service consuming 100% of CPU indeterminately" [Undecided,Won't fix] | 18:59 |
| dansmith | I would argue that bugs that rely on intercepting the rpc message and changing it are not worth our time, yes | 19:02 |
| artom | \o/ | 19:03 |
| *** ttsiouts has quit IRC | 19:04 | |
| *** maciejjozefczyk has quit IRC | 19:14 | |
| *** ociuhandu has quit IRC | 19:19 | |
| openstackgerrit | Merged openstack/nova master: Revert "Temporarily skip TestNovaMigrationsMySQL" https://review.opendev.org/718630 | 19:33 |
| *** vesper has joined #openstack-nova | 19:37 | |
| *** vesper11 has quit IRC | 19:38 | |
| *** igordc has quit IRC | 19:39 | |
| *** ttsiouts has joined #openstack-nova | 19:41 | |
| *** ttsiouts has quit IRC | 19:46 | |
| *** mgariepy has quit IRC | 19:47 | |
| *** priteau has joined #openstack-nova | 19:57 | |
| *** ttsiouts has joined #openstack-nova | 20:10 | |
| gibi | bauzas: fyi updated the RC1 release patch | 20:21 |
| *** mloza has quit IRC | 20:27 | |
| *** xek has quit IRC | 20:30 | |
| *** igordc has joined #openstack-nova | 20:32 | |
| melwitt | gibi: is the prelude update in there too? sorry I don't see yet | 20:32 |
| melwitt | for the policy doc | 20:32 |
| *** slaweq has joined #openstack-nova | 20:33 | |
| melwitt | or wait, maybe I'm getting confused. cycle highlight updated with link... did we need an update to the prelude, I thought we were going to but maybe not really needed | 20:34 |
| melwitt | no I think I got confused with the cycle highlights. nevermind me | 20:41 |
| melwitt | we did update it https://review.opendev.org/722220 ok good | 20:43 |
| *** jdillaman has quit IRC | 20:44 | |
| *** jdillaman has joined #openstack-nova | 20:46 | |
| *** ccamacho has quit IRC | 20:51 | |
| *** dpawlik has quit IRC | 21:09 | |
| openstackgerrit | Merged openstack/nova stable/stein: Reset the cell cache for database access in Service https://review.opendev.org/720587 | 21:19 |
| *** priteau has quit IRC | 21:33 | |
| openstackgerrit | OpenStack Release Bot proposed openstack/nova stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/722518 | 21:37 |
| openstackgerrit | OpenStack Release Bot proposed openstack/nova stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/722520 | 21:37 |
| openstackgerrit | OpenStack Release Bot proposed openstack/nova master: Update master for stable/ussuri https://review.opendev.org/722521 | 21:37 |
| openstackgerrit | OpenStack Release Bot proposed openstack/nova master: Add Python3 victoria unit tests https://review.opendev.org/722523 | 21:37 |
| *** ociuhandu has joined #openstack-nova | 21:39 | |
| *** ociuhandu has quit IRC | 21:45 | |
| *** kaisers_ has quit IRC | 21:50 | |
| *** igordc has quit IRC | 21:51 | |
| gmann | melwitt: yeah both are updated. | 22:00 |
| *** rcernin has joined #openstack-nova | 22:10 | |
| *** slaweq has quit IRC | 22:14 | |
| *** rcernin has quit IRC | 22:15 | |
| *** rcernin has joined #openstack-nova | 22:15 | |
| *** igordc has joined #openstack-nova | 22:26 | |
| *** eharney has quit IRC | 22:38 | |
| *** tosky has quit IRC | 22:58 | |
| *** tkajinam has joined #openstack-nova | 22:58 | |
| *** avolkov has quit IRC | 23:00 | |
| openstackgerrit | Takashi Natsume proposed openstack/nova master: Add placeholder migrations for Ussuri backports https://review.opendev.org/722546 | 23:04 |
| *** igordc has quit IRC | 23:05 | |
| *** ttsiouts has quit IRC | 23:05 | |
| openstackgerrit | Merged openstack/python-novaclient master: [Community goal] Update contributor documentation https://review.opendev.org/722391 | 23:06 |
| *** rcernin has quit IRC | 23:09 | |
| *** martinkennelly has quit IRC | 23:16 | |
| *** rcernin has joined #openstack-nova | 23:16 | |
| *** grandchild has quit IRC | 23:19 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!