| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-10-25 | |||
| 18:30:29 | artom | Really rough initial first idea | |
| 18:33:09 | TheJulia | artom: I guess the question I would ahve is what spured the thought to think to move it | |
| 18:35:48 | artom | TheJulia, well the current problem is that in your proposed patch we'd be modifying instance.host and potentially doing it while something else is accessing it. Except we can't use the lock that everything else uses because we have no guarantee that we're on the same compute host | |
| 18:35:58 | artom | And the lock is host-local | |
| 18:37:12 | TheJulia | Well, it is a single field update on a row, the db should lock it for the transaction unless sqlachemy's save is update all fields or insert/delete | |
| 18:37:29 | TheJulia | and *if* something should happen where there is a split perception, it should reconcile | |
| 18:37:34 | TheJulia | on the net run | |
| 18:37:49 | TheJulia | s/net/next | |
| 18:39:19 | artom | I... guess? In any case, there be dragons | |
| 18:39:32 | TheJulia | there are dragons today :) | |
| 18:39:40 | TheJulia | and they think we are tasty with ketchup | |
| 19:10:35 | opendevreview | mitya-eremeev-2 proposed openstack/nova master: Close Glance image if downloading failed. https://review.opendev.org/c/openstack/nova/+/815347 | |
| 19:37:20 | opendevreview | Gustavo Santos proposed openstack/nova master: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/815373 | |
| #openstack-nova - 2021-10-26 | |||
| 08:27:06 | bauzas | folks, the PTG summary email was sent yesterday, if I made any mistake, lemme know | |
| 08:40:11 | gibi | bauzas: I left a question you might be interested in in https://review.opendev.org/c/openstack/nova/+/810220/9/nova/compute/manager.py#1436 | |
| 08:40:44 | gibi | bauzas: about the PTG mail, did we agreed changing the OWNER trait name from OWNER_CYBORG to OWNED_BY_CYBORG? I don't remember' | |
| 08:41:12 | bauzas | gibi: ah, no worries | |
| 08:41:20 | bauzas | I don't want to bikeshed about the naming | |
| 08:41:46 | bauzas | if we want to use 'OWNER_project', OK, that said I'll provide a comment for it | |
| 08:42:46 | gibi | bauzas: I think the name was OWNER_project before the PTG but your mail now suggest otherwise and I don't know that this change is intentional or not | |
| 08:45:19 | bauzas | gibi: the etherpad says "OWNER_CYBORG" | |
| 08:45:56 | bauzas | gibi: my summary was for operations explaining about what we discussed, but the name is not an issue | |
| 08:45:58 | jkulik | bauzas: thank you very much for the summary. very helpful | |
| 08:47:18 | gibi | bauzas: OK, so that was just an honest mistak in your mail. I will reply sortly to fix that | |
| 08:47:31 | bauzas | k | |
| 09:00:58 | gibi | bauzas: can I get a quick approval on the re proposed pps spec? https://review.opendev.org/c/openstack/nova-specs/+/814531 I also refreshed the remaining implementation patch which is only the nova-manage support https://review.opendev.org/c/openstack/nova/+/802060/17 | |
| 09:01:36 | gibi | and I have a related bugfix https://review.opendev.org/c/openstack/nova/+/811396 | |
| 09:02:32 | bauzas | gibi: I could diff but I trust, what's the diff between the xena and the yoga specs ? | |
| 09:02:43 | bauzas | trust you* | |
| 09:05:25 | gibi | bauzas: the main change is that from upgrade perspective we dont need to care about the older nova newer neutron case | |
| 09:05:48 | gibi | as nova merged the support for the new api extension in xena while neutron did not merged the api extension in yoga | |
| 09:06:02 | gibi | so the nova xena - neutron yoga case does not need special handling any more | |
| 09:06:12 | gibi | but it really only impacts the neutron impl | |
| 09:06:30 | gibi | as now they don't need to make the extension optional | |
| 09:06:39 | bauzas | yup, that's what I see | |
| 09:07:35 | gibi | that what I remember :) | |
| 09:08:21 | gibi | since xena M3 I had also time to propose a set of tempest test for the pps feature so I'm pretty confident it works | |
| 09:09:20 | gibi | if you are interested https://review.opendev.org/q/project:openstack/tempest+status:open+topic:bp/qos-minimum-guaranteed-packet-rate | |
| 09:12:06 | gibi | gmann: when you are up, to be able to move forward with the pps feature in neutron, we need to land the tempest test adaptation for the new API extension. So this is ready https://review.opendev.org/c/openstack/neutron/+/805637 but it needs https://review.opendev.org/c/openstack/tempest/+/809168/ first. If you have time I appreaciate some review on the tempest side | |
| 09:45:38 | opendevreview | Merged openstack/nova-specs master: Re-propose QoS minimum guaranteed packet rate https://review.opendev.org/c/openstack/nova-specs/+/814531 | |
| 10:23:18 | opendevreview | Takashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service https://review.opendev.org/c/openstack/nova/+/778696 | |
| 10:26:52 | opendevreview | Takashi Kajinami proposed openstack/nova master: Clean up allocations left by evacuation when deleting service https://review.opendev.org/c/openstack/nova/+/778696 | |
| 13:12:40 | opendevreview | Wenping Song proposed openstack/nova master: Log error msg when delete resource provider failed https://review.opendev.org/c/openstack/nova/+/815498 | |
| 13:13:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add a WA flag waiting for vif-plugged event during reboot https://review.opendev.org/c/openstack/nova/+/813419 | |
| 13:46:12 | gmann | gibi: sure, will review after my breakfast. | |
| 13:47:51 | gibi | gmann: thanks! | |
| 14:00:29 | bauzas | what a joy, 30 new bugs | |
| 14:00:37 | bauzas | +17 | |
| 14:01:03 | gibi | somebody was busy reporting bugs why we had the PTG | |
| 15:16:30 | bauzas | gibi: can you please help me on this one ? https://bugs.launchpad.net/nova/+bug/1948776 | |
| 15:17:32 | bauzas | looks to me legit | |
| 15:18:45 | sean-k-mooney | im not sure that is a bug so much as an oversite | |
| 15:19:05 | sean-k-mooney | it will be a notificaiton object chnage if we start sending it right | |
| 15:19:11 | sean-k-mooney | so it wont be backporatable | |
| 15:19:30 | sean-k-mooney | but the locked state could be included going forward | |
| 15:20:04 | sean-k-mooney | it cannot be added to queens or even xena at this point however | |
| 15:22:57 | sean-k-mooney | bauzas: gibi i triaged that as whishlist and remove the nonstandard tags does that work for ye https://bugs.launchpad.net/nova/+bug/1948776 | |
| 15:23:26 | bauzas | sean-k-mooney: cool, thanks | |
| 15:25:53 | gibi | bauzas, sean-k-mooney: thank I agree with your assesments | |
| 15:26:41 | opendevreview | Gustavo Santos proposed openstack/nova master: Reattach mdevs to guest on resume https://review.opendev.org/c/openstack/nova/+/815373 | |
| 15:50:13 | bauzas | reminder : nova meeting in 10 mins here | |
| 16:00:02 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:02 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:02 | opendevmeet | Meeting started Tue Oct 26 16:00:02 2021 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:02 | bauzas | #startmeeting nova | |
| 16:00:15 | gibi | o/ | |
| 16:00:19 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:00:35 | bauzas | hola everyone, nice to see your virtual faces ;) | |
| 16:01:26 | elodilles | o/ | |
| 16:02:08 | bauzas | wow, not everyone at the same time :) | |
| 16:03:15 | gibi | maybe the rest of us had enough last week D: | |
| 16:03:30 | bauzas | :) | |
| 16:03:36 | bauzas | ok, then let's start :) | |
| 16:04:01 | bauzas | hopefully, it would be a small meeting :) | |
| 16:04:08 | bauzas | #topic Bugs (stuck/critical) | |
| 16:04:12 | bauzas | One Critical bug | |
| 16:04:20 | bauzas | https://bugs.launchpad.net/nova/+bug/1947687 | |
| 16:04:41 | gibi | it is a gate bug | |
| 16:04:50 | gibi | but it does not hit us in every run | |
| 16:04:51 | bauzas | maybe because it was a gate bug, yes | |
| 16:04:58 | gibi | I agree to decrease this to High | |
| 16:05:03 | bauzas | yeah | |
| 16:05:07 | gibi | I have not seen it recently | |
| 16:05:16 | bauzas | ok done | |
| 16:05:32 | bauzas | gibi: maybe we could see whether logstash still works for this one | |
| 16:05:49 | gibi | bauzas: yeas a signture would be nice | |
| 16:06:07 | bauzas | gibi: can you test this and provide the logstash link ? | |
| 16:06:10 | gibi | sure | |
| 16:06:16 | bauzas | thanks++ | |
| 16:06:35 | bauzas | okay, moved to High | |
| 16:06:42 | bauzas | anything to discuss about it now ? | |
| 16:06:52 | gibi | - | |
| 16:06:56 | bauzas | k | |
| 16:07:01 | bauzas | #link 24 new untriaged bugs (+6 since the last meeting): #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New | |
| 16:07:10 | bauzas | I triaged 6 or 7 | |
| 16:07:25 | bauzas | (already, I mean) | |
| 16:07:35 | bauzas | I'll continue to triage a few of them tomorrow | |
| 16:08:00 | bauzas | but in case some contributors want to triage too, I'd be happy | |
| 16:08:11 | bauzas | for the moment, that's fine | |
| 16:08:17 | gibi | yeah I feel the pressure but still have other pressors | |