| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-12-13 | |||
| 17:30:19 | gibi | sean-k-mooney: https://review.opendev.org/c/openstack/nova/+/821423 | |
| 17:30:27 | sean-k-mooney | ok that was "fixed" for a different edgecase many releases ago | |
| 17:30:40 | sean-k-mooney | so im surpised tempest has not hit this already | |
| 17:33:06 | gibi | sean-k-mooney: what was actually fixed? | |
| 17:33:13 | gibi | sean-k-mooney: as this bug now has reproduction test | |
| 17:34:19 | gmann | yeah, we never added restriction for add/remove host having server | |
| 17:34:29 | gmann | that is what tempest tests is doing | |
| 17:35:09 | sean-k-mooney | is_safe_to_update_az | |
| 17:35:35 | sean-k-mooney | https://github.com/openstack/nova/commit/8e19ef4173906da0b7c761da4de0728a2fd71e24 | |
| 17:35:39 | sean-k-mooney | https://github.com/openstack/nova/commit/0ad5a64dc9ac4b1cfb8038f9171b6385fdf07f28 | |
| 17:35:46 | gmann | ohk, no two different AZ while adding | |
| 17:36:22 | sean-k-mooney | we currently block you form adding or removing the host to the host-aggreate that has the az metadata | |
| 17:37:12 | gmann | different AZ | |
| 17:37:14 | sean-k-mooney | well more when doign az update | |
| 17:37:33 | sean-k-mooney | but this code was ment to alos cater for this usecase | |
| 17:38:49 | gibi | hm the first one is aggregated update, that is different from add/remove host but yes that could race with instance boot | |
| 17:39:27 | gibi | the second is about one host can only be one AZ case | |
| 17:39:38 | gibi | that I think cannot race with instance boot | |
| 17:39:43 | sean-k-mooney | gibi: well yes and no. ading or removign a host form an aggreate is really an update too in the more general sense | |
| 17:40:14 | sean-k-mooney | my point was that https://review.opendev.org/c/openstack/nova/+/821423/1/nova/compute/api.py#6458 | |
| 17:40:27 | sean-k-mooney | should have been blocking this edgecacse already | |
| 17:40:58 | sean-k-mooney | but it does not hece the need for stephens change | |
| 17:42:24 | gibi | OK, I really need to drop | |
| 17:42:27 | gibi | for today | |
| 17:42:35 | gibi | I will think about it more tomorrow o/ | |
| #openstack-nova - 2021-12-14 | |||
| 00:03:42 | opendevreview | sean mooney proposed openstack/nova-specs master: Add spec for external cpu power managment https://review.opendev.org/c/openstack/nova-specs/+/821419 | |
| 00:19:13 | opendevreview | sean mooney proposed openstack/nova-specs master: add per process healthcheck spec https://review.opendev.org/c/openstack/nova-specs/+/821279 | |
| 01:09:30 | opendevreview | Steve Baker proposed openstack/nova master: Allow remaining in in DELETING for ironic cleaning https://review.opendev.org/c/openstack/nova/+/813729 | |
| 01:27:50 | opendevreview | Ghanshyam proposed openstack/nova master: Re-add python 3.6 functional testing https://review.opendev.org/c/openstack/nova/+/821636 | |
| 02:20:25 | opendevreview | Ghanshyam proposed openstack/os-vif master: Updating python testing classifier as per Yoga testing runtime https://review.opendev.org/c/openstack/os-vif/+/819204 | |
| 02:24:05 | opendevreview | Ghanshyam proposed openstack/placement master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/placement/+/819206 | |
| 02:26:24 | opendevreview | Ghanshyam proposed openstack/osc-placement master: Updating python testing as per Yoga testing runtime https://review.opendev.org/c/openstack/osc-placement/+/819203 | |
| 02:27:38 | opendevreview | Ghanshyam proposed openstack/os-traits master: Updating python testing classifier as per Yoga testing runtime https://review.opendev.org/c/openstack/os-traits/+/819205 | |
| 02:28:58 | opendevreview | Ghanshyam proposed openstack/python-novaclient master: Updating python testing classifier as per Yoga testing runtime https://review.opendev.org/c/openstack/python-novaclient/+/819208 | |
| 04:42:54 | gmann | gibi: sean-k-mooney this is what I was talking about, my test env is down so did not test that. I will check on morning if any error (feel free to update if you see any) | |
| 05:54:28 | opendevreview | Wenping Song proposed openstack/nova master: Fill the AcceleratorRequestBindingFailed exception msg info https://review.opendev.org/c/openstack/nova/+/817326 | |
| 08:27:41 | opendevreview | Wenping Song proposed openstack/nova master: Fill the AcceleratorRequestBindingFailed exception msg info https://review.opendev.org/c/openstack/nova/+/817326 | |
| 08:45:56 | opendevreview | Federico Ressi proposed openstack/nova master: Debug Nova APIs call failures https://review.opendev.org/c/openstack/nova/+/806683 | |
| 08:55:52 | bauzas | good spec review day, everyone | |
| 08:56:26 | bauzas | this starts with a core ping for my own easy repproval spec https://review.opendev.org/c/openstack/nova-specs/+/821376 | |
| 08:56:31 | bauzas | gibi : ^ | |
| 09:35:09 | gibi | on it | |
| 09:37:57 | gibi | bauzas: fast approved | |
| 09:55:32 | opendevreview | Merged openstack/nova-specs master: Repropose boot a vm with unaddressed port https://review.opendev.org/c/openstack/nova-specs/+/821376 | |
| 10:01:02 | gibi | sean-k-mooney: I've left questions in the external-cpu-management spec | |
| 10:04:38 | gibi | bauzas: I skip the update userdata spec for now as you had strong opposition about the use case. | |
| 10:05:54 | bauzas | gibi: I'll revise this spec given they provided a new PS | |
| 10:06:07 | gibi | bauzas: ack | |
| 10:18:25 | gibi | brinzhang0: left feedback in the OWNER_ spec | |
| 10:20:35 | brinzhang0 | gibi：uii:ack, we will review later, thanks | |
| 10:20:35 | brinzhang0 | gibi：uii:ack, we will review later, thanks | |
| 10:20:54 | brinzhang0 | s/uii;//g | |
| 10:30:57 | brinzhang0 | bauzas,gibi: do you have time to review the remove tenat_id series of patches? https://review.opendev.org/q/topic:%22bp%252Fremove-tenant-id%22+(status:open%20OR%20status:merged) | |
| 10:31:32 | bauzas | brinzhang0: yes, will do today | |
| 10:32:23 | brinzhang0 | thanks | |
| 11:26:49 | gibi | brinzhang0: I try to get to it tomorrow | |
| 11:28:23 | brinzhang0 | gibi: cool thanks | |
| 12:10:11 | gibi | bauzas: I'm done with the first round of spec reviews | |
| 12:10:19 | gibi | bauzas: could you please look at https://review.opendev.org/q/topic:any-traits-support today | |
| 12:10:25 | gibi | bauzas: I will try to ask melwitt to look at it too | |
| 12:20:46 | gtrevisan | Hello everyone, quick question. We have a bug currently in progress, which we've also evidenced in stable/ussuri. A fix was uploaded to it, which we believe should also apply to Ussuri. Should we suggest a cherry-pick of this fix to Ussuri as well? Is the Ussuri release still in extended maintenance? | |
| 12:25:54 | opendevreview | Balazs Gibizer proposed openstack/os-traits master: Updating python testing classifier as per Yoga testing runtime https://review.opendev.org/c/openstack/os-traits/+/819205 | |
| 12:26:42 | gibi | gtrevisan: you can propose backports to ussuri yes, but please note that our stable process requires to land the backport in order so first in the newer stable branches | |
| 12:55:23 | opendevreview | mitya-eremeev-2 proposed openstack/nova master: Delete bogus attachments. https://review.opendev.org/c/openstack/nova/+/820935 | |
| 12:58:38 | opendevreview | mitya-eremeev-2 proposed openstack/nova master: Delete bogus attachments. https://review.opendev.org/c/openstack/nova/+/820935 | |
| 14:15:40 | gibi | gmann, sean-k-mooney: do we have somewhere in our stack a inter process reader writer lock? I know fastener 0.16.0 has one but we are still on 0.14.1 https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L593 | |
| 14:18:42 | sean-k-mooney | not that im aware of we can proably get it from fastener yes that is proably the best option | |
| 14:18:58 | sean-k-mooney | but good point on it needing to be an inter process lock | |
| 14:19:02 | sean-k-mooney | i forgot about that | |
| 14:19:12 | gibi | we cannot get fastener :/ https://review.opendev.org/c/openstack/nova/+/813114 | |
| 14:19:20 | gibi | eventlet is not compatible | |
| 14:19:25 | sean-k-mooney | ah ok | |
| 14:19:39 | sean-k-mooney | asyncio has some lock | |
| 14:19:47 | sean-k-mooney | maybe the standard lib has one we could use | |
| 14:19:52 | gibi | I will check | |
| 14:20:06 | sean-k-mooney | although if it does not we might need to look at the taging approch instead | |
| 14:20:07 | gibi | worst case I can try to re-implement it based on olso's inter process lock | |
| 14:20:29 | sean-k-mooney | yes that is an option too | |
| 14:21:08 | sean-k-mooney | the semaintic i think we ould want is the writer would not start until all readers had finsihedn and when a writer is queued no new readers can start | |
| 14:30:29 | bauzas | gibi: +2 for both placement specs | |
| 14:30:36 | gibi | bauzas: awesome, thanks | |
| 14:31:40 | bauzas | gibi I guess I need to modify https://storyboard.openstack.org/#!/story/2005345 to tell that the task is merged :) | |
| 14:31:50 | bauzas | (of course once we +W the spec) | |
| 14:33:18 | gibi | I think the that part is automatic when the patch merges | |
| 15:29:15 | bauzas | last spec but not the least to review https://review.opendev.org/c/openstack/nova-specs/+/821279/2/specs/yoga/approved/per-process-healthchecks.rst | |
| 15:29:28 | bauzas | but I have our nova meeting to prepare :) | |
| 15:29:56 | bauzas | reminder : nova meeting in 30 mins, the last of year 2021 ;) | |
| 15:34:03 | gibi | \o/ | |
| 15:47:09 | bauzas | gibi: do you know if elodilles is around ? | |
| 15:47:20 | bauzas | or already in some kinda french-like vacations ? | |
| 15:47:21 | gibi | hm I think so | |
| 15:47:29 | gibi | I mean he should be around | |
| 15:47:35 | bauzas | ok | |
| 15:48:00 | elodilles | let me search for him | |
| 15:48:08 | elodilles | o:) | |
| 15:48:16 | bauzas | elodilles: if you have time and be around, please just update your stable branches topic ;) | |
| 15:48:34 | elodilles | oh, meeting time! | |
| 15:48:39 | bauzas | elodilles: this is fine, I'm also sometimes schizophrenic ;) | |
| 15:49:08 | elodilles | sorry, i completely lost in time :S (i was off yesterday and today felt like Monday to me :S) | |
| 15:49:48 | bauzas | not yet talking about me with the 3rd person | |