| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-28 | |||
| 13:17:29 | bauzas | because again, some operators opt-out the AZfilter | |
| 13:17:31 | gibi | and such we can break old buggy behaviro | |
| 13:17:45 | bauzas | gibi: it's not a bug, it's a 40x | |
| 13:17:56 | gibi | bauzas: the db inconsistency is the bug | |
| 13:17:59 | bauzas | no | |
| 13:18:00 | sean-k-mooney | gibi: stephenfin if we add this check we shoudl also move the az exits check to the api also | |
| 13:18:09 | bauzas | you asked for a target you can't succeed | |
| 13:18:23 | sean-k-mooney | stephenfin: you added that to your check but did you remove the check later | |
| 13:18:29 | bauzas | gibi: you can create an instance on AZ1, then force migrate to AZ2 | |
| 13:18:30 | gibi | bauzas: I still think that if nova creates an inconsistent db record then we should fix that | |
| 13:18:41 | bauzas | gibi: and then, good luck with resizing the instance | |
| 13:19:08 | sean-k-mooney | bauzas: well a resize in that case will resize back to AZ1 | |
| 13:19:08 | gibi | so I can accept any fix that result in a consistent db data. | |
| 13:19:31 | bauzas | again, this is a forced operation and we made a clear statement on the fact broken migrations are not nova's fault | |
| 13:20:28 | bauzas | gibi: if we really want to fix this thing | |
| 13:20:40 | bauzas | gibi: I'd then suggest two things | |
| 13:20:51 | bauzas | gibi: 1/ remove the call by a new microversion | |
| 13:21:24 | bauzas | 2/ change the az value to None or to the host AZ in the az hack method | |
| 13:21:38 | bauzas | the az value is meaningless when you use the force hosts | |
| 13:21:59 | bauzas | but I wouldn't hardstop on the call | |
| 13:22:06 | bauzas | eg. | |
| 13:22:36 | bauzas | nova boot --az az1:host_in_az2 would consist into getting the tuple (None, host, node) | |
| 13:22:36 | gibi | 1/ is totally OK to me. So remove the hack in future version. | |
| 13:22:39 | sean-k-mooney | setting it to none would be consitent with using --host | |
| 13:23:02 | bauzas | or actually (schedule_default_az, host, node) | |
| 13:23:18 | bauzas | I mean, setting the returned az to be the default AZ from the option | |
| 13:23:26 | bauzas | (which defaults to None) | |
| 13:23:33 | gibi | 2/ if we can simulate --host when --az was given with bad az name, and log a warning, then I can accep that as well | |
| 13:24:15 | gibi | so keep the existing bad (but used) behavior but avoid incosistent db data | |
| 13:24:22 | gibi | in old microversin | |
| 13:24:23 | bauzas | gibi: the crucial distinction between --host and the az hack is the fact we call out the scheduler on the former, not on the latter | |
| 13:25:11 | bauzas | gibi: honestly, again, ops are using the az hack not for the az, but for providing a target | |
| 13:25:49 | bauzas | gibi: so agreed, we should log a warning (after all, this is an op who did this) and just propose the default AZ as a returned AZ | |
| 13:26:39 | bauzas | if people really want to both force to a target *AND* stick on this AZ, then they can use --host and --az (without the az hack) | |
| 13:27:26 | gibi | yepp | |
| 13:27:42 | bauzas | I'll log my thoughts in the review | |
| 13:28:54 | gibi | bauzas: thanks | |
| 13:29:11 | gibi | let's see how stephenfin feels about it after his lunch | |
| 13:33:31 | bauzas | sure | |
| 13:34:23 | sean-k-mooney | gibi: here is a patch to update teh neutron doc by the way https://review.opendev.org/c/openstack/neutron/+/798302 | |
| 13:34:54 | sean-k-mooney | git distracted by the previous conversation | |
| 13:35:02 | bauzas | gosh, eavesdrop is soooo slow to update the | |
| 13:35:45 | sean-k-mooney | bauzas: i think its a cron job or similar | |
| 13:35:56 | bauzas | last updated bits are from more than 20 mins | |
| 13:35:58 | sean-k-mooney | it often pretty quick but sometimes its delayed | |
| 13:36:19 | sean-k-mooney | ya that sometimes happens | |
| 13:36:35 | sean-k-mooney | ususally its only a minute or so behind at most | |
| 13:37:37 | bauzas | still lagging | |
| 13:39:55 | ganso | lyarwood: hi! could you please take one quick look at https://review.opendev.org/c/openstack/nova/+/795432 ? The other reviewers said they are waiting for your feedback. Thanks in advance! | |
| 13:40:21 | gibi | sean-k-mooney: ups, I also pushed a doc patch https://review.opendev.org/c/openstack/neutron/+/798294 | |
| 13:41:36 | sean-k-mooney | oh ok lol | |
| 13:42:07 | bauzas | can someone confirm it's not PEBKAC if https://meetings.opendev.org/irclogs/%23openstack-nova/%23openstack-nova.2021-06-28.log.html is lagging ? | |
| 13:42:39 | sean-k-mooney | gibi: i dont mind going with yours we took slightly different approches but the message is similar | |
| 13:42:55 | artom | bauzas, https://meetings.opendev.org/irclogs/%23openstack-nova/latest.log.html | |
| 13:43:11 | sean-k-mooney | artom: its the same page | |
| 13:43:14 | artom | sean-k-mooney, I know | |
| 13:43:20 | artom | 13:15 is the latest timestamp there | |
| 13:43:28 | artom | That's in UTC I imagine | |
| 13:43:36 | artom | So about 30 minutes behind... | |
| 13:43:41 | sean-k-mooney | bauzas: i assume you just want to link to the point where the converstation started | |
| 13:43:45 | artom | Yeah, feels longer than normal | |
| 13:44:02 | bauzas | sean-k-mooney: I'd rather point to the written agrement | |
| 13:44:03 | sean-k-mooney | bauzas: it have updated by the time anyone actully reads it | |
| 13:44:09 | sean-k-mooney | ah ok | |
| 13:44:28 | sean-k-mooney | ill quickly check with infra | |
| 13:45:00 | artom | Also, "then your dog" | |
| 13:50:15 | bauzas | artom: did you like it ? did i used it correctly ? | |
| 13:50:38 | artom | bauzas, I actually have no idea what you were trying to say :P | |
| 13:50:52 | bauzas | artom: "then it's your problem" | |
| 13:51:23 | gibi | sean-k-mooney: ack, let me know if you think some part of your message should be incorporated to mine | |
| 13:51:26 | artom | Never heard it used like that... or at all, in fact | |
| 13:51:50 | bauzas | "mommy, I don't wanna take the dog out it's raining", "darling, you wanted it, so YOUR DOG" | |
| 13:53:05 | artom | bauzas, I think you just invented an expression. You're this century's Shakespeare | |
| 13:53:13 | gibi | :D | |
| 13:53:16 | bauzas | artom: Voltaire, please | |
| 14:18:00 | bauzas | so, I left my comment but i need to get my kids from school, ttyl | |
| 14:33:07 | opendevreview | sean mooney proposed openstack/os-vif master: [WIP] add configurable per port bridges https://review.opendev.org/c/openstack/os-vif/+/798055 | |
| 14:57:05 | gibi | bauzas, sean-k-mooney: you were +2 on the Placement RP re-parenting spec, could you also look at the implementation https://review.opendev.org/c/openstack/placement/+/784020 ? | |
| 14:57:21 | bauzas | surelyu | |
| 14:57:24 | sean-k-mooney | yes i can do that | |
| 14:57:36 | bauzas | I tho have a meeting in 2 mins... so tomorrow morning, your dog :p | |
| 14:58:15 | gibi | bauzas: thanks :) | |
| 14:58:17 | gibi | sean-k-mooney: thanks | |
| 14:58:56 | gibi | also on placement side there is a quick small patch to add pps resources to os-resource-classes https://review.opendev.org/c/openstack/os-resource-classes/+/796591 | |
| 15:29:44 | bauzas | +2d on the last one | |
| 15:33:15 | gibi | bauzas: thanks | |
| 15:36:04 | opendevreview | Elod Illes proposed openstack/nova stable/pike: Update pci stat pools based on PCI device changes https://review.opendev.org/c/openstack/nova/+/798345 | |
| 16:32:07 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: api: Validate host belongs to availability zone https://review.opendev.org/c/openstack/nova/+/798145 | |
| 19:19:35 | opendevreview | Merged openstack/nova stable/ussuri: [CI] Fix gate by using zuulv3 live migration and grenade jobs https://review.opendev.org/c/openstack/nova/+/795432 | |
| 19:23:53 | opendevreview | sean mooney proposed openstack/os-vif master: [WIP] add configurable per port bridges https://review.opendev.org/c/openstack/os-vif/+/798055 | |
| 19:30:38 | ganso | gibi, bauzas: if you have a spare minute to please look at this backport that is ready for an extra +2 and +W: https://review.opendev.org/c/openstack/nova/+/796719 | |
| 20:36:07 | tosky | melwitt, elodilles: yay, now time to update the train backport of the legacy cleanup and the CI should be (almost) fine | |
| #openstack-nova - 2021-06-29 | |||
| 02:14:52 | opendevreview | Merged openstack/nova stable/ussuri: Remove broken legacy zuul jobs https://review.opendev.org/c/openstack/nova/+/795374 | |
| 07:03:48 | gibi | ganso: I have no +2 rights on stable branches | |
| 08:03:02 | opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | |
| 08:03:02 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | |
| 08:03:04 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - create arqs https://review.opendev.org/c/openstack/nova/+/758944 | |
| 08:03:04 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - build instance with smartnic arqs https://review.opendev.org/c/openstack/nova/+/798249 | |
| 08:03:05 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - cleanup arqs https://review.opendev.org/c/openstack/nova/+/798054 | |
| 08:03:05 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |