| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-06 | |||
| 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 | |
| 16:16:57 | opendevreview | Rajat Dhasmana proposed openstack/nova-specs master: Clarify client changes in rebuild spec https://review.opendev.org/c/openstack/nova-specs/+/856164 | |
| 16:17:00 | bauzas | #link https://bugs.launchpad.net/nova/+bug/1988482 Proposed patch on the fly https://review.opendev.org/c/openstack/nova/+/855658 | |
| 16:17:13 | bauzas | as said, reviews are welcome ^ | |
| 16:17:23 | dansmith | sean-k-mooney: shoudn't the test be using an az? | |
| 16:17:25 | bauzas | I exceptionally relaxed my review needs | |
| 16:17:32 | dansmith | anyway, we can discuss elsewhere | |
| 16:17:42 | bauzas | so I gave +2 for a patch without testing | |
| 16:17:43 | sean-k-mooney | dansmith: i dont think so be we can follow up after ya | |
| 16:18:21 | bauzas | tl;dr: the problem is with PrettyTable having a changed behavior | |
| 16:18:43 | sean-k-mooney | yes so they have now fixed this by reverting the behaivor | |
| 16:18:49 | bauzas | gibi: sean-k-mooney: wants to address this issue now ? | |
| 16:18:56 | sean-k-mooney | i tested with the previous release broken release and new release with the revert | |
| 16:19:19 | sean-k-mooney | i woudl prefer to merge and backprot https://review.opendev.org/c/openstack/nova/+/855658 | |
| 16:19:22 | bauzas | so https://review.opendev.org/c/openstack/nova/+/855658 wouldn't be needed ? | |
| 16:19:25 | bauzas | ack | |
| 16:19:27 | sean-k-mooney | to yoga so we never need to thnk about htis again | |
| 16:19:51 | sean-k-mooney | its not needed unless the decided to reinstate the feature or change the default | |