| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-12-13 | |||
| 17:23:02 | gibi | sean-k-mooney: then we might need to invent someting inside tempest to order cases? | |
| 17:23:14 | sean-k-mooney | unless we add a new feature to tempest to tag things as "must be serial" | |
| 17:23:18 | gibi | yeah | |
| 17:23:20 | gibi | like that | |
| 17:23:34 | sean-k-mooney | if we could do that as a decorator that would be nice | |
| 17:23:50 | gibi | OK, so we have couple of options a) RW lock b) @serial decorator | |
| 17:23:51 | sean-k-mooney | i just dont know if we can. i suspect its posible | |
| 17:24:05 | sean-k-mooney | c) tox.ini for now | |
| 17:24:19 | gibi | yeah temporarily tox.ini but I don't really like that | |
| 17:24:39 | sean-k-mooney | its what we have done for senario tests for years | |
| 17:24:39 | gmann | yeah we should not do that as other tempest user still will face the error | |
| 17:24:45 | sean-k-mooney | it works but you have to know to do it | |
| 17:25:18 | gibi | I will sleep on this. I don't need to rush to land the nova bug fix so we have time | |
| 17:25:21 | gmann | sean-k-mooney: we did that only for env/timeout restriction they can be run in parallel like we do in tempest-parallel job | |
| 17:25:38 | gibi | thanks for the input | |
| 17:25:45 | sean-k-mooney | gmann: they can if you have enough resouces in the could yes | |
| 17:25:47 | gibi | and I will read back if you still continue discussiing this | |
| 17:25:48 | gibi | :) | |
| 17:25:50 | gmann | sean-k-mooney: here we are making tempest cannot be run in parallel so we have to fix test not the way we run those | |
| 17:26:00 | sean-k-mooney | we do it because with our default concurance tehy wont fit in the ci vms | |
| 17:26:14 | sean-k-mooney | gmann: yep i understand | |
| 17:26:31 | sean-k-mooney | however if this is a gate blocker it better to unblock the gate and then fix it properly | |
| 17:26:44 | gmann | gibi:sean-k-mooney let me add test modification with checks and try/except. and later we can see if tagging a tests to run serial can be done or not | |
| 17:26:55 | sean-k-mooney | ack | |
| 17:27:00 | gmann | sean-k-mooney: this is not gate blokcer but change need with nova fix | |
| 17:27:20 | gmann | new bug fix in nova and so does tempest tests need fix | |
| 17:29:43 | sean-k-mooney | this is for stephens bug fix to prevent you updating aggreate when there are isntance on the host? | |
| 17:30:00 | gibi | sean-k-mooney: yes | |
| 17:30:02 | sean-k-mooney | that was previously ment to be blocked but i assume there was an edge case we missed | |
| 17:30:04 | gmann | yeah | |
| 17:30:18 | gmann | https://bugs.launchpad.net/nova/+bug/1907775 | |
| 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 | |