| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-06 | |||
| 16:02:10 | gibi | o/ | |
| 16:02:12 | sean-k-mooney | o/ | |
| 16:02:19 | bauzas | #info Three Critical bugs | |
| 16:02:24 | bauzas | that's a bummer | |
| 16:02:36 | bauzas | but we have a story against each | |
| 16:02:41 | bauzas | #link https://bugs.launchpad.net/nova/+bug/1988311 See https://lists.openstack.org/pipermail/openstack-discuss/2022-September/030325.html | |
| 16:02:49 | bauzas | seems we have a way forward | |
| 16:03:04 | bauzas | with oslo.concurrency patches | |
| 16:03:23 | gibi | yeah there is a new oslo.concurrency release | |
| 16:03:30 | bauzas | yup | |
| 16:03:44 | gibi | the next step is to get the global version bump | |
| 16:03:52 | bauzas | correct | |
| 16:04:00 | gibi | I will abandon the nova specific patches once we have the bump merged | |
| 16:04:28 | gibi | we have to make sure though that stable/yoga gets the new oslo version too | |
| 16:04:37 | bauzas | yeah | |
| 16:04:43 | sean-k-mooney | i also have patches to fix fastener and update oslo in flight | |
| 16:04:48 | bauzas | I don't see any u-c change yet | |
| 16:04:49 | sean-k-mooney | thos will be for A | |
| 16:04:51 | sean-k-mooney | a this point | |
| 16:05:07 | Uggla | o/ | |
| 16:05:23 | bauzas | but I assume this patch will be done sooner than later | |
| 16:06:07 | gibi | yes based on the mail thread Herve will propose the bump | |
| 16:06:26 | gibi | https://lists.openstack.org/pipermail/openstack-discuss/2022-September/030352.html | |
| 16:06:36 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (db) https://review.opendev.org/c/openstack/nova/+/831193 | |
| 16:06:37 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 16:06:37 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | |
| 16:06:38 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 16:06:38 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 16:06:39 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 16:06:39 | opendevreview | ribaudr proposed openstack/nova master: Bump compute version and check shares support https://review.opendev.org/c/openstack/nova/+/850499 | |
| 16:06:40 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 16:06:40 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 16:06:42 | opendevreview | ribaudr proposed openstack/nova master: Add instance.power_on_error notification https://review.opendev.org/c/openstack/nova/+/852084 | |
| 16:06:42 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 16:06:44 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/852085 | |
| 16:06:44 | opendevreview | ribaudr proposed openstack/nova master: Add instance.power_off_error notification https://review.opendev.org/c/openstack/nova/+/852278 | |
| 16:06:46 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 16:06:46 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 16:06:48 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute and API part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 16:06:48 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | |
| 16:06:50 | opendevreview | ribaudr proposed openstack/nova master: Change microversion to 2.94 https://review.opendev.org/c/openstack/nova/+/852088 | |
| 16:06:59 | Uggla | oops sorry for that ^ | |
| 16:07:24 | bauzas | gibi: would you propose a patch to nova's requirements.txt or do we automatically pull the latest ? | |
| 16:07:38 | sean-k-mooney | https://review.opendev.org/c/openstack/requirements/+/856044 | |
| 16:07:39 | gibi | we are not constrained in noca | |
| 16:07:43 | gibi | nova | |
| 16:07:50 | sean-k-mooney | i think that is the requiremtns bump | |
| 16:08:05 | bauzas | gibi: ok, so we'll benefit from it | |
| 16:08:17 | gibi | yep, automatically | |
| 16:08:20 | sean-k-mooney | we shoudl once that merges yep | |
| 16:08:27 | bauzas | https://github.com/openstack/nova/blob/master/requirements.txt#L34 | |
| 16:08:33 | bauzas | yup, we're not constrained | |
| 16:08:46 | bauzas | ok, let's wait for the req patch to be merged then | |
| 16:09:11 | bauzas | for stable/yoga, seems we don't cap too | |
| 16:09:12 | bauzas | https://github.com/openstack/nova/blob/stable/yoga/requirements.txt#L30 | |
| 16:09:26 | sean-k-mooney | correct | |
| 16:09:31 | gibi | l | |
| 16:09:31 | gibi | coo | |
| 16:09:41 | sean-k-mooney | we can either pull in the latest oslo release there or cap fasterners | |
| 16:09:42 | bauzas | ok, we're on the good way | |
| 16:09:55 | bauzas | moving on to the next bug then | |
| 16:10:09 | bauzas | #link https://bugs.launchpad.net/nova/+bug/1988316 Will be set to High now that the gate is skipping the test | |
| 16:10:17 | bauzas | people agree with the proposal ? | |
| 16:10:23 | gibi | sean-k-mooney: I would bump this https://github.com/openstack/requirements/blob/stable/yoga/upper-constraints.txt#L26 | |
| 16:10:24 | bauzas | => High | |
| 16:10:45 | bauzas | gibi: oh you're right about yoga's u-c | |
| 16:11:32 | gibi | on the unshelve stuff I think we need a nova fix to return a better error message, but I'm OK to drop it from Critical to High | |
| 16:11:40 | bauzas | yeah | |
| 16:11:45 | gibi | it is not blocking the gate at the moment | |
| 16:11:47 | bauzas | not saying there will be no bug to fix | |
| 16:11:51 | sean-k-mooney | ya its a latent bug | |
| 16:12:06 | bauzas | but this isn't longer critical as the gate is now working back | |
| 16:12:07 | dansmith | unshelve thing meaning the to-host cell one? | |
| 16:12:13 | sean-k-mooney | yes | |
| 16:12:15 | bauzas | yup | |
| 16:12:29 | bauzas | saying we have to restrict to the same cell | |
| 16:12:35 | dansmith | yeah, that's not new just recently exposed so doesn't seem like we should be too critical in the bug status there | |
| 16:12:55 | sean-k-mooney | im saying its latent as its the same error we would have got with unshelve to AZ we just dont have test coverage for that in tempest that was also corss cell | |
| 16:13:23 | dansmith | right, and I'm saying being latent it's not a regression, so really shouldn't be >high :) | |
| 16:13:31 | sean-k-mooney | +1 | |
| 16:13:45 | bauzas | honestly, I'm in favor of saying "sorry but we don't support *yet* cross-cell unshelve to host" that's it | |
| 16:13:59 | bauzas | so the fix could be just docs | |
| 16:14:06 | bauzas | and a better exception handling | |
| 16:14:09 | sean-k-mooney | yep we can do both | |
| 16:14:18 | bauzas | anyway, set to High, there is | |
| 16:15:13 | dansmith | well, the test needs fixing too | |
| 16:15:24 | dansmith | it's just being skipped there right now right? | |
| 16:15:29 | bauzas | yup | |
| 16:15:34 | sean-k-mooney | the test is not broken | |
| 16:15:40 | sean-k-mooney | it just should not be run in a cross cell env | |
| 16:15:41 | bauzas | yeah, the test shouldn't be assuming we could | |
| 16:15:57 | sean-k-mooney | no this is a job config issue | |
| 16:15:59 | bauzas | if we would want a test, this would have to be negative | |
| 16:16:05 | sean-k-mooney | you cant tell if its cross cell form the api | |
| 16:16:09 | bauzas | like "I know this can't work" | |
| 16:16:12 | sean-k-mooney | so tempest cannot detech that | |
| 16:16:20 | sean-k-mooney | so this has to be done via job config | |
| 16:16:24 | bauzas | ahah true | |
| 16:16:47 | bauzas | anyway, bug report is still open, feel free to comment it out for resolution | |
| 16:16:50 | bauzas | moving on | |