| *** nel1x has joined #openstack-nova | 00:19 | |
| *** jmlowe has joined #openstack-nova | 01:08 | |
| *** jmlowe has quit IRC | 01:08 | |
| *** hoonetorg has quit IRC | 01:38 | |
| *** hoonetorg has joined #openstack-nova | 01:51 | |
| *** damien_r has joined #openstack-nova | 01:54 | |
| *** openstackgerrit has quit IRC | 02:04 | |
| *** ircuser-1 has quit IRC | 02:10 | |
| *** davee__ has quit IRC | 02:16 | |
| *** davee__ has joined #openstack-nova | 02:16 | |
| *** ociuhandu has joined #openstack-nova | 02:33 | |
| *** ociuhandu has quit IRC | 02:33 | |
| *** ociuhandu has joined #openstack-nova | 02:34 | |
| *** ociuhandu has quit IRC | 02:40 | |
| *** damien_r has quit IRC | 03:28 | |
| *** psachin has joined #openstack-nova | 03:35 | |
| *** jhesketh has quit IRC | 03:56 | |
| *** davee___ has joined #openstack-nova | 04:03 | |
| *** davee__ has quit IRC | 04:04 | |
| *** davee_ has quit IRC | 04:05 | |
| *** davee__ has joined #openstack-nova | 04:05 | |
| *** udesale has joined #openstack-nova | 04:25 | |
| *** ratailor has joined #openstack-nova | 04:55 | |
| *** ratailor_ has joined #openstack-nova | 04:56 | |
| *** ratailor has quit IRC | 04:59 | |
| *** ociuhandu has joined #openstack-nova | 05:30 | |
| *** ociuhandu has quit IRC | 05:36 | |
| *** Luzi has joined #openstack-nova | 05:45 | |
| *** links has joined #openstack-nova | 05:50 | |
| *** slaweq__ has joined #openstack-nova | 06:03 | |
| *** tbachman has joined #openstack-nova | 06:07 | |
| *** tbachman_ has joined #openstack-nova | 06:11 | |
| *** tbachman has quit IRC | 06:12 | |
| *** tbachman_ is now known as tbachman | 06:12 | |
| *** Liang__ has joined #openstack-nova | 06:45 | |
| *** tkajinam has quit IRC | 07:02 | |
| *** tkajinam has joined #openstack-nova | 07:04 | |
| *** tbachman has quit IRC | 07:19 | |
| *** ralonsoh has joined #openstack-nova | 07:24 | |
| *** dpawlik has joined #openstack-nova | 07:26 | |
| *** luksky has joined #openstack-nova | 07:29 | |
| *** damien_r has joined #openstack-nova | 07:51 | |
| *** dpawlik has quit IRC | 07:52 | |
| *** tkajinam_ has joined #openstack-nova | 07:52 | |
| *** tkajinam has quit IRC | 07:55 | |
| *** tosky has joined #openstack-nova | 07:56 | |
| *** dpawlik has joined #openstack-nova | 07:59 | |
| *** iurygregory has joined #openstack-nova | 08:01 | |
| *** tesseract has joined #openstack-nova | 08:13 | |
| *** lbragstad has quit IRC | 08:15 | |
| *** lbragstad has joined #openstack-nova | 08:15 | |
| *** tkajinam_ has quit IRC | 08:18 | |
| *** rpittau|afk is now known as rpittau | 08:18 | |
| *** pcaruana has joined #openstack-nova | 08:25 | |
| *** ratailor_ has quit IRC | 08:37 | |
| *** ociuhandu has joined #openstack-nova | 08:47 | |
| *** ratailor has joined #openstack-nova | 08:51 | |
| *** vesper11 has quit IRC | 09:01 | |
| *** vesper11 has joined #openstack-nova | 09:05 | |
| *** martinkennelly has joined #openstack-nova | 09:21 | |
| *** ccamacho has joined #openstack-nova | 09:25 | |
| *** mrch has joined #openstack-nova | 09:30 | |
| *** derekh has joined #openstack-nova | 09:37 | |
| *** dpawlik has quit IRC | 09:40 | |
| *** xek has joined #openstack-nova | 09:42 | |
| stephenfin | bauzas: Today's a spec review day for me. Just reviewed https://review.opendev.org/#/c/702943/ | 09:45 |
|---|---|---|
| *** openstackgerrit has joined #openstack-nova | 09:48 | |
| openstackgerrit | John Garbutt proposed openstack/nova-specs master: Small fixes to unified limits spec https://review.opendev.org/703773 | 09:49 |
| *** ociuhandu has quit IRC | 09:50 | |
| bauzas | stephenfin: ack, thanks | 09:59 |
| bauzas | FWIW, I should do the same... | 10:00 |
| *** dtantsur|afk is now known as dtantsur | 10:23 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova-specs master: Re-proposes multiple vGPU types in libvirt https://review.opendev.org/702943 | 10:38 |
| stephenfin | bauzas, efried: Think you folks could look at https://review.opendev.org/#/c/682655/ today? It's just a reproposal and I hope to have the mostly trivial code up later this week | 10:50 |
| *** udesale has quit IRC | 10:58 | |
| *** etp has quit IRC | 10:58 | |
| *** etp has joined #openstack-nova | 11:00 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Additional upgrade clarifications for cpu-resources https://review.opendev.org/666032 | 11:17 |
| *** rpittau is now known as rpittau|bbl | 11:21 | |
| openstackgerrit | Merged openstack/nova-specs master: spec update: virtual persistent memory https://review.opendev.org/695868 | 11:30 |
| *** lpetrut has joined #openstack-nova | 11:32 | |
| *** kashyap has joined #openstack-nova | 11:52 | |
| stephenfin | kashyap: Loads of comments left on https://review.opendev.org/#/c/693844/, when you have time | 11:53 |
| * kashyap waves | 11:53 | |
| kashyap | stephenfin: Will check, on the phone; thanks! | 11:54 |
| *** luksky has quit IRC | 11:55 | |
| kashyap | stephenfin: Thanks for taking time to review; reasonable questions, will respond there. | 12:07 |
| *** ociuhandu has joined #openstack-nova | 12:11 | |
| stephenfin | huaqiang: Is it you that's working on bp/use-pcpu-and-vcpu-in-one-instance ? If so, just hit the spec at https://review.opendev.org/#/c/668656/ | 12:15 |
| *** ociuhandu has quit IRC | 12:19 | |
| *** damien_r has quit IRC | 12:28 | |
| *** ociuhandu has joined #openstack-nova | 12:30 | |
| openstackgerrit | Merged openstack/nova master: nova-net: Remove remaining nova-network quotas https://review.opendev.org/697152 | 12:33 |
| *** johnthetubaguy has joined #openstack-nova | 12:34 | |
| *** beekneemech has joined #openstack-nova | 12:51 | |
| *** bnemec has quit IRC | 12:51 | |
| *** rpittau|bbl is now known as rpittau | 12:55 | |
| *** ratailor has quit IRC | 13:00 | |
| *** damien_r has joined #openstack-nova | 13:01 | |
| *** ociuhandu has quit IRC | 13:14 | |
| *** psachin has quit IRC | 13:29 | |
| *** ociuhandu has joined #openstack-nova | 13:34 | |
| gibi | stephenfin: replied in https://review.opendev.org/#/c/701763 | 13:35 |
| *** smcginnis has quit IRC | 13:50 | |
| *** smcginnis has joined #openstack-nova | 13:51 | |
| *** smcginnis is now known as smcginnis|FOSDEM | 13:53 | |
| gibi | stephenfin: you were the second +2 on https://review.opendev.org/#/c/699669/ but did not +A it. Is it intentional? | 14:00 |
| *** udesale has joined #openstack-nova | 14:03 | |
| *** links has quit IRC | 14:11 | |
| openstackgerrit | Merged openstack/nova-specs master: Re-proposes multiple vGPU types in libvirt https://review.opendev.org/702943 | 14:11 |
| *** Luzi has quit IRC | 14:13 | |
| *** haleyb has joined #openstack-nova | 14:14 | |
| *** luksky has joined #openstack-nova | 14:16 | |
| efried | stephenfin: Would you please make a blueprint for that? | 14:18 |
| openstackgerrit | Vladyslav Drok proposed openstack/nova master: Make RBD imagebackend flatten method idempotent https://review.opendev.org/704330 | 14:35 |
| openstackgerrit | Vladyslav Drok proposed openstack/nova master: Make RBD imagebackend flatten method idempotent https://review.opendev.org/704330 | 14:36 |
| *** tbachman has joined #openstack-nova | 14:49 | |
| efried | lyarwood: I've been procrastinating on https://review.opendev.org/#/c/669121/ -- at some point would you mind directing me to the piece of the CI that shows it's doing what's expected? | 14:50 |
| efried | oh, I guess it's not waiting on me, since there's an open dep. That makes me feel less slackerish. | 14:51 |
| *** nweinber_ has joined #openstack-nova | 14:54 | |
| lyarwood | efried: yup I've been pinging away in openstack-qa about this for a while, I'll try to get it over the line today. The CI you're looking for is actually against that change as the dep is for an openstack/grenade change. | 14:55 |
| * lyarwood adds a comment with log links | 14:56 | |
| efried | thanks | 14:57 |
| lyarwood | gah .gz files, has zuul regressed recently in terms of being able to display these .gz log files as plain text via the web ui? | 14:58 |
| lyarwood | or were these just plain text before and are now .gz? | 14:58 |
| lyarwood | either way it's now impossible to share URLs to specific log lines etc | 14:59 |
| *** tbachman has quit IRC | 15:01 | |
| *** damien_r has quit IRC | 15:01 | |
| *** damien_r has joined #openstack-nova | 15:01 | |
| efried | lyarwood: Yeah, infra changed a thing, something about swift, and now a few jobs, including legacy ones, are double-compressing log files. It's a serious PITA. | 15:01 |
| efried | Apparently we can fix it somewhere in the job definition itself, but so far nobody has gone and done that. Not sure how much of that is because nobody knows how (that's certainly *my* excuse) or just haven't had the time. | 15:02 |
| efried | Another hesitation with that is that we *should* just go convert those jobs to zv3, which would make the problem go away, and which we're supposed to be doing anyway. | 15:03 |
| efried | that also has been not done, and for similar reasons I imagine. | 15:03 |
| efried | smarter people than me have made attempts, but nothing has closed | 15:03 |
| lyarwood | efried: ack thanks, I did start looking at switching the nova-lvm job over last week but didn't post anything. | 15:04 |
| lyarwood | sean-k-mooney: have you started converting any of the grenade jobs to zuulv3? | 15:04 |
| lyarwood | iirc you were looking at this right? | 15:05 |
| efried | I will personally buy a cookie for whoever makes the .gz problem go away, I don't care how. | 15:05 |
| * efried bbiab | 15:05 | |
| *** dviroel has joined #openstack-nova | 15:11 | |
| gmann | lyarwood: grenade job zuulv3 work https://review.opendev.org/#/q/topic:grenade_zuulv3+status:open | 15:21 |
| lyarwood | gmann: awesome thank you, I'll take a look shortly | 15:22 |
| *** tbachman has joined #openstack-nova | 15:28 | |
| *** artom has joined #openstack-nova | 15:31 | |
| *** damien_r has quit IRC | 15:46 | |
| *** damien_r has joined #openstack-nova | 15:47 | |
| lyarwood | melwitt: https://review.opendev.org/#/c/703005/ - Did you have another openstack/nova change using https://review.opendev.org/#/c/548936/ btw? | 15:50 |
| * lyarwood can't find it if you did | 15:50 | |
| melwitt | lyarwood: yeah, I was dual purposing this https://review.opendev.org/701478 | 16:02 |
| lyarwood | melwitt: ah cool thanks | 16:03 |
| melwitt | I got motivated to test out the grenade change while trying to get some info on that cell timeout gate failure in grenade jobs (bc the log files are all gzipped and annoying to have to download). zuul v3 conversion solves that problem | 16:04 |
| *** efried has quit IRC | 16:08 | |
| *** efried has joined #openstack-nova | 16:10 | |
| dansmith | efried: just want to make sure it's clear that I'm waiting for you to ack this before I really do much with it: https://review.opendev.org/#/c/631243/52 | 16:13 |
| efried | dansmith: okay, thanks for the heads up. | 16:14 |
| dansmith | efried: seems like you haven't been through that set lately, are you planning to go through it again soon? | 16:15 |
| efried | I can do that, yes. | 16:15 |
| efried | dansmith: I was actually mostly waiting for you to be happy with it. | 16:15 |
| dansmith | efried: last time I did I got to like patch #4 and found yet many more sections of code where there wasn't any testing, and/or tests that asserted things were called, but not with any specific parameters and got annoyed | 16:15 |
| efried | okay. | 16:16 |
| dansmith | and so I'm feeling a little like I'm the only guy digging deep on those things, which is exhausting | 16:16 |
| *** ociuhandu has quit IRC | 16:16 | |
| *** udesale has quit IRC | 16:19 | |
| efried | I'll take a thorough swipe this week. You should take some time off to prevent burnout. Go do some sledding or something. | 16:24 |
| dansmith | well, I'm just back from time off for general burn-out reasons, | 16:25 |
| dansmith | but I also don | 16:25 |
| dansmith | I also don't want to get too myopic on this set, so I think it would be good to have someone else take a pass through and see if they find things I'm not seeing | 16:25 |
| dansmith | like, a little more than just my comments get changed each time and I don't want to settle into an itemized list of "yes, my comment was addressed" without looking close enough at the whole change | 16:26 |
| efried | ack | 16:26 |
| *** gyee has joined #openstack-nova | 16:29 | |
| *** ccamacho has quit IRC | 16:34 | |
| *** Liang__ has quit IRC | 16:35 | |
| lyarwood | melwitt: sorry got slightly distracted, did you want to use that change to switch things over to the new zuulv3 job or should we create a fresh one? | 16:37 |
| melwitt | lyarwood: I'd create a fresh one. I don't know why I didn't make a fresh one, sorry | 16:38 |
| lyarwood | melwitt: ack np I can take a swing at that now | 16:40 |
| melwitt | lyarwood: yeah sure, go ahead. thanks | 16:40 |
| melwitt | just have to remove the -py3 from the name, I think that's all | 16:41 |
| melwitt | (and depends-on tosky's grenade change) | 16:41 |
| lyarwood | melwitt: yeah, doesn't our grenade job also run some live migration tests? | 16:42 |
| * lyarwood checks | 16:42 | |
| melwitt | lyarwood: I ... don't know tbh | 16:43 |
| lyarwood | https://github.com/openstack/nova/blob/80539a5e849bf8ec1239f0ed01433a7f8a54015b/playbooks/legacy/nova-grenade-multinode/run.yaml#L57-L59 - yeah it does, I don't think the new job includes this yet. | 16:44 |
| melwitt | note that I was referencing only the single node grenade job earlier, we also have a multinode grenade job that I did _not_ try to convert to yet using tosky's change | 16:44 |
| lyarwood | ah okay I see | 16:44 |
| melwitt | so that might be the one | 16:44 |
| melwitt | and you already linked it so yes | 16:44 |
| *** TxGirlGeek has joined #openstack-nova | 16:46 | |
| tosky | and tosky wasn't able to find a proper workaround for the failures when using grenade plugins (shame on him) | 16:46 |
| *** mrch has quit IRC | 16:47 | |
| melwitt | tosky: do you have a change where you are working on getting plugins to work? is that here? https://review.opendev.org/639774 is there any way we could help? | 16:54 |
| tosky | melwitt: I don't have a change; the problem is that I'm not able to figure out where the py2 openstackclient is installed and make sure its CLI used over its py3 counterpart | 16:56 |
| *** dtantsur is now known as dtantsur|afk | 16:57 | |
| melwitt | oh, interesting | 16:57 |
| tosky | I would expect that starting from a py3 deployment, even if the py2 version is installed afterwards, the py3 version installed first would still be the one used for the CLI | 16:57 |
| *** iurygregory has quit IRC | 16:58 | |
| *** martinkennelly has quit IRC | 16:58 | |
| melwitt | I would expect that as well | 17:00 |
| stephenfin | efried: create a blueprint for what? | 17:01 |
| lyarwood | stephenfin: https://review.opendev.org/702943 - I think he was talking about that spec that had just merged. | 17:02 |
| stephenfin | ah, makes sense | 17:03 |
| stephenfin | will do | 17:03 |
| tosky | talking about jobs, a few days ago the nova-live-migration job was failing in stable/pike; do you know if it has been solved? | 17:03 |
| melwitt | tosky: it has not been solved yet. sean-k-mooney said they would try to dig into it more soon. it's not 100% failure rate but seems like it's close to that | 17:04 |
| *** jdillaman has joined #openstack-nova | 17:04 | |
| efried | stephenfin: I was talking about https://review.opendev.org/#/c/682655/ | 17:06 |
| efried | It looks like you're using a new bp name for this round, but there's no bp by that name. | 17:07 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Switch to the Zuulv3 grenade job https://review.opendev.org/704364 | 17:09 |
| stephenfin | efried: https://blueprints.launchpad.net/nova/+spec/flavor-extra-spec-validators | 17:10 |
| efried | thanks | 17:11 |
| *** mlavalle has joined #openstack-nova | 17:12 | |
| *** ociuhandu has joined #openstack-nova | 17:14 | |
| sean-k-mooney | tosky: its actully the other way around. which ever version is installed last get used but only if its the first installation | 17:15 |
| *** gregwork has joined #openstack-nova | 17:15 | |
| sean-k-mooney | if you upgrade an existing install it does not replace the console script i think | 17:15 |
| sean-k-mooney | so ify its py2 then py3 on n-1 1 and py3 only on n i thing the console scrript will point to py2 | 17:16 |
| sean-k-mooney | anyway its a bit of a pain. we likely should make the py2 install in py3 mode optional in train | 17:17 |
| sean-k-mooney | so we can disable it in the greade job | 17:17 |
| sean-k-mooney | but yes i will try to look at nova-live-migration this week. i think i have got a downstream backport working so i should have more time to look at it this week | 17:18 |
| sean-k-mooney | i might try to set aside tomorow to look at it | 17:18 |
| *** ircuser-1 has joined #openstack-nova | 17:20 | |
| stephenfin | gibi: RE: https://review.opendev.org/#/c/701763/4/specs/ussuri/approved/non-admin-filter-instance-by-az.rst@83, are you saying it's not possible to use policy to allow any user access to the API? | 17:26 |
| *** luksky has quit IRC | 17:27 | |
| gibi | stephenfin: I think the policy is not per field | 17:28 |
| gibi | stephenfin: so if we want to make some fields available for filtering to the normal user then we need to change code | 17:30 |
| stephenfin | Oh, so even with this spec there are fields that the admin will be able to filter on but not a normal user? | 17:31 |
| stephenfin | I missed that, if so | 17:31 |
| *** ociuhandu_ has joined #openstack-nova | 17:32 | |
| gibi | stephenfin: yeah I think so | 17:33 |
| stephenfin | riiight, okay then, my bad | 17:33 |
| gibi | admin can filter on admin only fields too | 17:33 |
| stephenfin | lemme double that check and then I'll hit it again | 17:33 |
| gibi | I mean those fields that are not returned in the GET /servers/ to non-admins | 17:33 |
| stephenfin | yeah, makes sense in hindsight | 17:33 |
| *** ociuhandu has quit IRC | 17:36 | |
| tosky | sean-k-mooney: thanks for both :) | 17:37 |
| *** mriedem has joined #openstack-nova | 17:37 | |
| *** ociuhandu_ has quit IRC | 17:37 | |
| *** tesseract has quit IRC | 17:38 | |
| *** beekneemech is now known as bnemec-ooo | 17:39 | |
| *** slaweq__ is now known as slaweq | 17:45 | |
| *** jobewan has joined #openstack-nova | 17:53 | |
| *** TxGirlGeek has quit IRC | 18:00 | |
| *** TxGirlGeek has joined #openstack-nova | 18:01 | |
| *** martinkennelly has joined #openstack-nova | 18:01 | |
| *** rpittau is now known as rpittau|afk | 18:18 | |
| *** TxGirlGeek has quit IRC | 18:30 | |
| *** martinkennelly has quit IRC | 18:36 | |
| *** stephenfin has left #openstack-nova | 18:47 | |
| *** stephenfin has joined #openstack-nova | 18:47 | |
| *** mgariepy has quit IRC | 18:52 | |
| *** lpetrut has quit IRC | 18:55 | |
| *** TxGirlGeek has joined #openstack-nova | 18:55 | |
| *** mgariepy has joined #openstack-nova | 18:57 | |
| *** xek has quit IRC | 19:02 | |
| *** xek has joined #openstack-nova | 19:02 | |
| *** TxGirlGeek has quit IRC | 19:04 | |
| *** TxGirlGeek has joined #openstack-nova | 19:09 | |
| *** jobewan has quit IRC | 19:10 | |
| openstackgerrit | Merged openstack/nova stable/pike: Avoid redundant initialize_connection on source post live migration https://review.opendev.org/683008 | 19:18 |
| *** ralonsoh has quit IRC | 19:22 | |
| *** gregwork has quit IRC | 19:25 | |
| *** KeithMnemonic has joined #openstack-nova | 19:26 | |
| KeithMnemonic | Eureka, that BDM patch on stable/pike finally merged. Thanks a ton for all of your endless help mriedem: , melwitt: and of course hemna: | 19:27 |
| melwitt | cool KeithMnemonic | 19:39 |
| *** luksky has joined #openstack-nova | 19:47 | |
| *** bbowen has quit IRC | 20:06 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: zuul: Add Fedora based jobs to the experimental queue https://review.opendev.org/701231 | 20:13 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP libvirt: Use virDomainBlockCopy to swap volumes with >= 5.10.0 https://review.opendev.org/696834 | 20:14 |
| *** gyee has quit IRC | 20:27 | |
| *** gyee has joined #openstack-nova | 20:28 | |
| *** openstackstatus has joined #openstack-nova | 20:28 | |
| *** ChanServ sets mode: +v openstackstatus | 20:28 | |
| *** artom has quit IRC | 20:31 | |
| *** lbragstad has quit IRC | 20:40 | |
| *** lbragstad has joined #openstack-nova | 20:41 | |
| *** lbragsta_ has joined #openstack-nova | 20:47 | |
| *** lbragstad has quit IRC | 20:50 | |
| *** lbragsta_ has quit IRC | 20:50 | |
| *** lbragstad has joined #openstack-nova | 20:50 | |
| *** mgariepy has quit IRC | 20:55 | |
| *** efried has quit IRC | 21:18 | |
| *** efried1 has joined #openstack-nova | 21:18 | |
| *** martinkennelly has joined #openstack-nova | 21:19 | |
| *** efried1 is now known as efried | 21:21 | |
| *** bbowen has joined #openstack-nova | 21:21 | |
| *** xek has quit IRC | 21:29 | |
| *** ociuhandu has joined #openstack-nova | 21:50 | |
| *** lpetrut has joined #openstack-nova | 21:51 | |
| *** lpetrut has quit IRC | 21:55 | |
| *** nweinber_ has quit IRC | 21:56 | |
| *** rcernin has joined #openstack-nova | 22:10 | |
| *** slaweq has quit IRC | 22:17 | |
| *** martinkennelly has quit IRC | 22:22 | |
| *** slaweq has joined #openstack-nova | 22:34 | |
| *** nweinber_ has joined #openstack-nova | 22:35 | |
| *** nweinber_ has quit IRC | 22:35 | |
| *** slaweq has quit IRC | 22:38 | |
| *** tkajinam has joined #openstack-nova | 22:55 | |
| *** artom has joined #openstack-nova | 23:13 | |
| *** damien_r has quit IRC | 23:15 | |
| efried | dansmith: remind me (again) which direction we're allowed to have mismatched versions: controller newer than compute or vice versa? | 23:18 |
| *** mriedem has quit IRC | 23:22 | |
| sean-k-mooney | controler newer then compute i think | 23:22 |
| sean-k-mooney | we expect the contol plane to be upgraded first | 23:22 |
| efried | k | 23:25 |
| sean-k-mooney | in either case the sender is ment to back level to the max version supported by the reciver if memory serves. greade upgrade the contol plane first so that is what we test. | 23:26 |
| efried | I'm trying to understand the upgrade implications of the extra spec validation thing. | 23:27 |
| sean-k-mooney | ah ok | 23:27 |
| sean-k-mooney | that does rais an intersting question. if the extra specs were ovo should you allow the flaovr to be back leveled if a unsupporte extra spec was requested. | 23:28 |
| sean-k-mooney | i have not review that proposal by the way so that is jsut me specualting but i would lean towrads no | 23:29 |
| efried | I think it's simpler than that. | 23:29 |
| efried | Ultimately an extra spec that's used on/by the compute (which is many of them) is being validated at the API. | 23:30 |
| *** ociuhandu has quit IRC | 23:30 | |
| sean-k-mooney | yes at least initally | 23:30 |
| *** ociuhandu has joined #openstack-nova | 23:30 | |
| efried | and if the versions are mismatched, we could have an extra spec that the control plane (because newer) understands, but the compute won't. If the developer of that extra spec is doing their job right, they'll have tied it to a request filter so you would never land on that compute, so probably no big deal. | 23:31 |
| efried | But the other way around is also possible. If we deprecate/remove support for an extra spec, the control plane thinks it's invalid while the compute still supports it. | 23:31 |
| efried | I guess that means you have to wait *two* cycles before you get to add the validator that forbids the removed extra spec. | 23:32 |
| efried | or something. | 23:32 |
| sean-k-mooney | ya althoguh im not sure we have ever remvoed an extra spec | 23:33 |
| sean-k-mooney | i can think of one or two i woudl consider removing but i dont think we have ever actully done it | 23:33 |
| sean-k-mooney | and actully i have reviewd https://review.opendev.org/#/c/682655/1/specs/ussuri/approved/flavor-extra-spec-validators.rst but it was last cycle | 23:33 |
| *** rchurch has quit IRC | 23:34 | |
| *** rchurch has joined #openstack-nova | 23:36 | |
| *** ociuhandu has quit IRC | 23:39 | |
| *** tosky has quit IRC | 23:45 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!