| *** osmanlicilegi is now known as Guest0 | 01:48 | |
| opendevreview | Tyler Stachecki proposed openstack/nova master: libvirt: Check VIF MTU matches network MTU https://review.opendev.org/c/openstack/nova/+/852367 | 03:07 |
|---|---|---|
| opendevreview | Tyler Stachecki proposed openstack/nova master: libvirt: Check if VIF MTU matches network MTU https://review.opendev.org/c/openstack/nova/+/852367 | 03:26 |
| opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/851337 | 04:01 |
| gibi | good morning | 07:33 |
| gibi | folks I've created the Zed bp tracking etherpad https://etherpad.opendev.org/p/nova-zed-blueprint-status please check it out, sign up for review, update status, note if I missed someting. We will look at it again on the todays meeting | 08:43 |
| gibi | bauzas: you own me a beer for that ^^ ;) | 08:44 |
| gibi | elodilles: I've updated the today's agenda on the wiki, feel free to add your stuff there | 08:44 |
| elodilles | gibi: ack, thanks! | 09:05 |
| stephenfin | gibi: Looking | 09:19 |
| stephenfin | artom: Boo | 09:19 |
| noonedeadpunk | hey there! Does anybody know any reason why https://specs.openstack.org/openstack/nova-specs/specs/ocata/approved/detach-boot-volume.html has not been implemented? As from what I see the only thing left is kind of changing condition here https://opendev.org/openstack/nova/blame/branch/master/nova/api/openstack/compute/volumes.py#L560 | 09:46 |
| noonedeadpunk | by adding `instance.vm_state not in (vm_states.SHELVED, vm_states.SHELVED_OFFLOADED)` ? | 09:47 |
| noonedeadpunk | As detaching from shelved instance should never hurt from my understanding? | 09:48 |
| opendevreview | Merged openstack/nova stable/ussuri: [stable-only] Allow Train compute hosts to register with Ussuri https://review.opendev.org/c/openstack/nova/+/851732 | 10:32 |
| sean-k-mooney[m] | i think it kind of stalled out because the person working on it stopped | 10:33 |
| sean-k-mooney[m] | im not sure if there was any technical reason that blocked it | 10:34 |
| sean-k-mooney[m] | noonedeadpunk: dansmith might remember more or gibi i think they might have been invovled with that at some point | 10:34 |
| opendevreview | chenwei proposed openstack/nova master: Remove unicode literal from code https://review.opendev.org/c/openstack/nova/+/852538 | 10:35 |
| sean-k-mooney[m] | ^ is such a waste fo ci and review bandwidth | 10:36 |
| sean-k-mooney[m] | also there is someone else arready changing that | 10:36 |
| noonedeadpunk | for some reason I was quite sure it's implemented :( | 10:38 |
| noonedeadpunk | the only tricky thing I see on how to re-attach volume to be same device name and how to disallow unshelve when root device is detached | 10:39 |
| sean-k-mooney[m] | one thing that would have to be done in addtion to the detach is likely clearing the delete on terminate info form teh volume too | 10:39 |
| sean-k-mooney[m] | well the name is a hint | 10:39 |
| sean-k-mooney[m] | so you cant rely on that | 10:40 |
| sean-k-mooney[m] | what you really need to be able to do is atach it and set the boot order on the volume | 10:40 |
| sean-k-mooney[m] | so that it will have precidence | 10:40 |
| noonedeadpunk | so basically preserve the order somehow | 10:40 |
| sean-k-mooney[m] | not really | 10:41 |
| sean-k-mooney[m] | the main usecause for detach was you detach the volmue and then delete the vm | 10:41 |
| sean-k-mooney[m] | and use the volume for a differnt vm | 10:41 |
| sean-k-mooney[m] | but if we wanted ot supprot reattaching it we would need a way to specify the boot order on attach as part of the bdms | 10:41 |
| noonedeadpunk | well that doesn't make much sense then? As for me main usecase would be to workaround not working `openstack server recover` for bfv | 10:42 |
| sean-k-mooney[m] | you can do that when you create a vm but currently you cant update it after the vm is created | 10:42 |
| sean-k-mooney[m] | that is being worked on currently | 10:42 |
| sean-k-mooney[m] | rescue now works | 10:42 |
| sean-k-mooney[m] | and rebuild is in flight | 10:42 |
| noonedeadpunk | as right now if you made a mistake in networking, and was not clever enough to set a password for the user - you need to delete VM, attach volume to another VM, then create third VM that will have same set of ports and volumes in correct order | 10:43 |
| sean-k-mooney[m] | rescuse for volume backed instances is aviable in wallaby maybe before | 10:43 |
| noonedeadpunk | oh, rly? as in Xena rescue is not working | 10:43 |
| sean-k-mooney[m] | rescue shoudl work in xena | 10:44 |
| noonedeadpunk | well, I just got `Instance 19d5f71e-3dde-474f-9802-7e3d2f512f3b cannot be rescued: Cannot rescue a volume-backed instance (HTTP 400)` | 10:45 |
| sean-k-mooney[m] | https://specs.openstack.org/openstack/nova-specs/specs/ussuri/implemented/virt-bfv-instance-rescue.html | 10:45 |
| noonedeadpunk | maybe it's client that does not provide api version... | 10:45 |
| sean-k-mooney[m] | it was implemted in ussuri | 10:45 |
| sean-k-mooney[m] | did you use osc or nova | 10:45 |
| noonedeadpunk | osc ofc | 10:45 |
| sean-k-mooney[m] | i dont think there was a micor version for this | 10:46 |
| noonedeadpunk | shouldn't python-novaclient be deprecated now :p | 10:46 |
| *** tosky is now known as Guest71 | 10:46 | |
| *** tosky_ is now known as tosky | 10:46 | |
| sean-k-mooney[m] | oh there is | 10:46 |
| sean-k-mooney[m] | https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#maximum-in-ussuri-and-victoria | 10:47 |
| sean-k-mooney[m] | 2.87 | 10:47 |
| sean-k-mooney[m] | noonedeadpunk: the shell is the python bindings currently are not | 10:47 |
| noonedeadpunk | ok, let me check on traits then | 10:47 |
| noonedeadpunk | well, there's a typo at very list in trait name in microversion history | 10:52 |
| sean-k-mooney[m] | https://github.com/openstack/os-traits/blob/master/os_traits/compute/__init__.py#L32 | 10:53 |
| sean-k-mooney[m] | ya its RESCUE_BFV not BFV_RESCUE | 10:53 |
| sean-k-mooney[m] | https://github.com/openstack/nova/commit/5b6f44efff5ad721779fd9a7e80d6a5330fad7d4 | 10:54 |
| sean-k-mooney[m] | the code uses the correct value | 10:55 |
| sean-k-mooney[m] | and its correct in the spec | 10:55 |
| sean-k-mooney[m] | just reversed in the docs | 10:55 |
| sean-k-mooney[m] | https://specs.openstack.org/openstack/nova-specs/specs/ussuri/implemented/virt-bfv-instance-rescue.html#proposed-change | 10:55 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/nova master: [doc] Fix BFV Rescue trait name in microversion https://review.opendev.org/c/openstack/nova/+/852543 | 10:56 |
| noonedeadpunk | Yeah and computes does have this trait | 10:56 |
| gibi | I dont remember much about detach root volume feature but what you discussed above make sense | 11:03 |
| noonedeadpunk | yeah, setting --os-compute-api-version 2.87 works | 11:03 |
| noonedeadpunk | funny/sad thing though is that `python-openstackclient 5.8.0` throws `osc_lib.exceptions.CommandError: versions supported by client: 2.1 - 2.87` | 11:04 |
| noonedeadpunk | oh, well, it didn't work at the end as instance moved to the error state with `Driver Error: internal error: qemu unexpectedly closed the monitor: 2022-08-09T11:04:55.719928Z qemu-system-x86_64: -blockdev {"driver":"rbd","pool":"vms","image":"c76df07c-02b6-4a28-964c-d7'}` | 11:09 |
| gibi | please file a bug report on these issues | 11:10 |
| sean-k-mooney[m] | that sound like a qemu crash | 11:10 |
| opendevreview | Merged openstack/nova stable/yoga: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/852238 | 11:13 |
| noonedeadpunk | ok, so real issue was qemu crash because of `error reading header from c76df07c-02b6-4a28-964c-d703555152b8_disk: No such file or directory` | 11:15 |
| noonedeadpunk | soundsl ike it tried to rescue but clean forgot to create ephemeral first | 11:16 |
| noonedeadpunk | because it called `c76df07c-02b6-4a28-964c-d703555152b8_disk.rescue` :D | 11:18 |
| noonedeadpunk | or well, both was provided. So seems that logic just ignores that it should not have ephemeral | 11:19 |
| sean-k-mooney[m] | maybe ya | 11:20 |
| sean-k-mooney[m] | are you also using rbd as the normal backend? | 11:21 |
| noonedeadpunk | yup | 11:21 |
| sean-k-mooney[m] | perhaps it does not like ceph images_backend adn ceph cinder | 11:21 |
| sean-k-mooney[m] | is it the same ceph cluster | 11:21 |
| noonedeadpunk | but different pools anyway | 11:22 |
| sean-k-mooney[m] | ack | 11:22 |
| noonedeadpunk | and different image names at very least... | 11:22 |
| sean-k-mooney[m] | well rescue uses the same image by default if you dont spcify one | 11:22 |
| sean-k-mooney[m] | but it would be a differnt volume | 11:23 |
| noonedeadpunk | so nova basically generated xml that does contain ephemeral device that was never there since it's bfv | 11:23 |
| sean-k-mooney[m] | i have not personally tried this but we do have tempest testign for this i belive | 11:23 |
| sean-k-mooney[m] | ya that sound like a bug | 11:23 |
| sean-k-mooney[m] | and perhaps one that is fixed already | 11:23 |
| sean-k-mooney[m] | it sound vagly familar | 11:24 |
| noonedeadpunk | well, it's Xena so I assume it can be... | 11:24 |
| noonedeadpunk | I don't really have time on hands to test it on master right now :( | 11:24 |
| noonedeadpunk | will write down to play with it later on | 11:25 |
| noonedeadpunk | also interesting, that xml does not have volumes at all. | 11:25 |
| sean-k-mooney[m] | https://bugs.launchpad.net/nova/+bug/1978958 | 11:25 |
| noonedeadpunk | yup, sounds familiar | 11:26 |
| sean-k-mooney[m] | https://bugs.launchpad.net/nova/+bug/1926601 | 11:27 |
| sean-k-mooney[m] | its reported twice | 11:27 |
| noonedeadpunk | first one was assigned 58m ago - what a co-incidence :D | 11:27 |
| sean-k-mooney[m] | basically you need to see hw_rescue_bus and hw_rescue_device | 11:27 |
| noonedeadpunk | um, should I set this on flavors then? | 11:28 |
| sean-k-mooney[m] | the image | 11:28 |
| noonedeadpunk | but this also means that I would need to update all existing instance specs in DB to allow them to be rescued as well? | 11:29 |
| sean-k-mooney[m] | its part of this spec which was implmeented at the same time https://specs.openstack.org/openstack/nova-specs/specs/ussuri/implemented/virt-rescue-stable-disk-devices.html | 11:29 |
| sean-k-mooney[m] | no | 11:29 |
| sean-k-mooney[m] | when we rescue we use the image metadta form the rescue image i belive | 11:29 |
| sean-k-mooney[m] | so just add it to the image | 11:29 |
| noonedeadpunk | ah, ok, gotcha now | 11:29 |
| noonedeadpunk | makes sense | 11:30 |
| sean-k-mooney[m] | generally for reasonse we recommend hw_rescue_bus=usb hw_rescue_device=disk | 11:30 |
| sean-k-mooney[m] | basically if you use a usb rescue bus/device then it does not change the /dev/sda naming | 11:31 |
| sean-k-mooney[m] | you can just use the same bus as the root disk normally uses too | 11:32 |
| sean-k-mooney[m] | https://specs.openstack.org/openstack/nova-specs/specs/ussuri/implemented/virt-rescue-stable-disk-devices.html#proposed-change | 11:32 |
| *** bhagyashris is now known as bhagyashris|afk | 11:44 | |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Heal PCI allocation during resize https://review.opendev.org/c/openstack/nova/+/852396 | 11:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Handle PCI dev reconf with allocations https://review.opendev.org/c/openstack/nova/+/852397 | 11:54 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Allow enabling PCI tracking in Placement https://review.opendev.org/c/openstack/nova/+/850468 | 11:54 |
| *** tosky is now known as Guest77 | 12:01 | |
| *** tosky__ is now known as tosky | 12:01 | |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Add more test coverage for devname base dev spec https://review.opendev.org/c/openstack/nova/+/844625 | 12:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Extra tests for remote managed dev spec https://review.opendev.org/c/openstack/nova/+/844626 | 12:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Unparent PciDeviceSpec from PciAddressSpec https://review.opendev.org/c/openstack/nova/+/844491 | 12:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Fix PciAddressSpec descendants to call super.__init__ https://review.opendev.org/c/openstack/nova/+/844565 | 12:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Remove dead code from PhysicalPciAddress https://review.opendev.org/c/openstack/nova/+/844628 | 12:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Clean up mapping input to address spec types https://review.opendev.org/c/openstack/nova/+/845765 | 12:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Remove unused PF checking from get_function_by_ifname https://review.opendev.org/c/openstack/nova/+/845775 | 12:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Fix type annotation of pci.Whitelist class https://review.opendev.org/c/openstack/nova/+/845780 | 12:18 |
| opendevreview | Balazs Gibizer proposed openstack/nova master: Move __str__ to the PciAddressSpec base class https://review.opendev.org/c/openstack/nova/+/845781 | 12:18 |
| *** tosky is now known as Guest78 | 12:30 | |
| *** tosky_ is now known as tosky | 12:30 | |
| *** bhagyashris|afk is now known as bhagyashris | 12:50 | |
| opendevreview | Merged openstack/nova stable/yoga: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/852235 | 12:56 |
| *** tosky is now known as Guest85 | 13:13 | |
| *** tosky__ is now known as tosky | 13:13 | |
| *** dasm|off is now known as dasm | 13:30 | |
| sfinucan | sean-k-mooney[m]: I +W'd this on the assumption that you're happy with it, but feel free to re-review if you'd like https://review.opendev.org/c/openstack/nova/+/826526/4..6 | 15:12 |
| sfinucan | gibi: melwitt: How are we going to unblock that ephemeral encryption series without bauzas around? I assume that's still happening this cycle? | 15:12 |
| sean-k-mooney[m] | sure ill take a look now but i generaly trus your judgement | 15:13 |
| sfinucan | sean-k-mooney[m]: Can you give me a summary of what you concluded on the ephemeral vs. disk debate? | 15:14 |
| sfinucan | At risk of rehashing old arguments, doesn't 'hw_disk_encryption' suggest cinder-backed "disks" (i.e. volumes) will be encrypted also? | 15:15 |
| sean-k-mooney[m] | the term ephemeral is ambigious but its not worth changing as it has mindshare | 15:15 |
| sfinucan | context is https://review.opendev.org/c/openstack/nova/+/760454 | 15:15 |
| sean-k-mooney[m] | so i was ment to drop my -1 and proceed with the current names | 15:15 |
| sean-k-mooney[m] | but escalation | 15:15 |
| sean-k-mooney[m] | so ill do that now | 15:15 |
| sfinucan | Ah, no worries | 15:15 |
| *** sfinucan is now known as stephenfin | 15:16 | |
| stephenfin | order is restored (IRC nick) | 15:16 |
| gibi | sfinucan: good question. I hope we can have enough cores to land it. I put together a tracking pad to get cores signing up for reviewing certain series (thanks sean-k-mooney[m] for the idea) https://etherpad.opendev.org/p/nova-zed-blueprint-status | 15:16 |
| gibi | we will discuss this on the today's meeting | 15:17 |
| sean-k-mooney[m] | sfinucan: https://review.opendev.org/c/openstack/nova/+/760454/11#message-391f61808b977003be0f33c48e768e37bec8697e | 15:18 |
| stephenfin | sean-k-mooney[m]: thanks | 15:19 |
| sean-k-mooney[m] | hehe i was just going to say i just +w'd the windows enlightements patch | 15:19 |
| sean-k-mooney[m] | so that should be on its way | 15:19 |
| gibi | one less to care about | 15:26 |
| gibi | thanks | 15:26 |
| stephenfin | sean-k-mooney[m]: another one to drop your -1 from, I think https://review.opendev.org/c/openstack/nova/+/764486 | 15:40 |
| stephenfin | sean-k-mooney[m]: and https://review.opendev.org/c/openstack/nova/+/760455 | 15:45 |
| sean-k-mooney[m] | stephenfin: done | 15:49 |
| gibi | fyi nova meeting starts in 10 minutes here in the channel | 15:50 |
| *** dasm is now known as dasm|off | 15:54 | |
| elodilles | o/ | 16:02 |
| elodilles | gibi: meeting? :) | 16:02 |
| gibi | hups | 16:02 |
| gibi | #startmeeting nova | 16:02 |
| opendevmeet | Meeting started Tue Aug 9 16:02:33 2022 UTC and is due to finish in 60 minutes. The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:02 |
| opendevmeet | The meeting name has been set to 'nova' | 16:02 |
| dansmith | o/ | 16:02 |
| elodilles | o/ | 16:02 |
| gibi | this is typical that I remember up until 10 minutes before the actual time :) | 16:03 |
| sean-k-mooney[m] | o/ | 16:03 |
| gibi | then I forget | 16:03 |
| gibi | anyhow | 16:03 |
| elodilles | :D | 16:03 |
| gibi | #topic Bugs (stuck/critical) | 16:04 |
| gibi | #info No Critical bug | 16:04 |
| gibi | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 11 new untriaged bugs (+0 since the last meeting) | 16:04 |
| gibi | #link https://storyboard.openstack.org/#!/project/openstack/placement 27 open stories (+0 since the last meeting) in Storyboard for Placement | 16:04 |
| gibi | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:05 |
| gibi | thanks elodilles for the bug triage this week | 16:05 |
| gibi | #info bug baton is being passed to gibi | 16:05 |
| gibi | is there any bugs we need to discuss today? | 16:06 |
| elodilles | i had little time, but triaged 3 bugs | 16:06 |
| elodilles | this one seems interesting: Shelved (offloaded) instance still have port bound to host | 16:06 |
| gibi | yep that sort of a known issue | 16:07 |
| elodilles | oh, i see | 16:07 |
| gmann | o/ | 16:07 |
| gibi | probably it does not break anything but I agree that it is valid | 16:07 |
| elodilles | then maybe that's it from my side :) | 16:08 |
| gibi | any other bugs from anybody? | 16:08 |
| *** bauzas is now known as bauzas_pto | 16:09 | |
| gibi | #topic Gate status | 16:09 |
| *** bauzas_pto is now known as bauzas_away | 16:09 | |
| gibi | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:09 |
| gibi | I dont see any new bug there | 16:09 |
| gibi | #link https://zuul.openstack.org/builds?project=openstack%2Fplacement&pipeline=periodic-weekly Placement periodic job status | 16:09 |
| gibi | it is green | 16:09 |
| gibi | #link https://zuul.openstack.org/builds?job_name=tempest-integrated-compute-centos-9-stream&project=openstack%2Fnova&pipeline=periodic-weekly&skip=0 Centos 9 Stream periodic job status | 16:10 |
| gibi | green too | 16:10 |
| gibi | #link https://zuul.opendev.org/t/openstack/builds?job_name=nova-emulation&pipeline=periodic-weekly&skip=0 Emulation periodic job runs | 16:10 |
| gibi | green as well | 16:10 |
| gibi | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:10 |
| gibi | #info STOP DOING BLIND RECHECKS aka. 'recheck' https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures | 16:10 |
| gibi | any gate issue we need to discuss? | 16:10 |
| gibi | I have one thing | 16:11 |
| gibi | we merged the switch to unittest.mock that also showed us that in py310 the unittest.mock is more strict | 16:12 |
| gibi | it does not allow mocking the same function twice | 16:12 |
| gibi | this made our py310 non-voting jobs red | 16:12 |
| gibi | we fixed it once but broke it later again | 16:13 |
| gibi | so I'm fixing it again https://review.opendev.org/c/openstack/nova/+/852464 and making py310 jobs voting for nova | 16:13 |
| gibi | to pervent breaking this again | 16:13 |
| gibi | I know that officially we don't support py310 yet | 16:13 |
| gibi | but I don't really want to fix double mocking in AA or BB again | 16:13 |
| gibi | so any disagreement switchin the unittest and functional test jobs with py310 voting? | 16:14 |
| sean-k-mooney[m] | if we can fix the final issues to make it green again | 16:15 |
| sean-k-mooney[m] | then i have no issue with graduating it early | 16:15 |
| sean-k-mooney[m] | it has been stable form what i have seen | 16:15 |
| gibi | I think I made it green again | 16:15 |
| gmann | +1, many project have it voting and in next cycle we need to run it voting. so +1 on making it voting now | 16:15 |
| gibi | with https://review.opendev.org/c/openstack/nova/+/852464 | 16:15 |
| sean-k-mooney[m] | ack ya i saw you mentioned that we misses a case with fixtures | 16:16 |
| sean-k-mooney[m] | so once that is landed im +1 on making it voting | 16:16 |
| gibi | ack | 16:16 |
| gibi | I don't see disagreement so moving on :) | 16:16 |
| sean-k-mooney[m] | actully that does that curretnly | 16:17 |
| gibi | #topic Release Planning | 16:17 |
| sean-k-mooney[m] | https://review.opendev.org/c/openstack/nova/+/852464/1/.zuul.yaml | 16:17 |
| sean-k-mooney[m] | but yes lets move on | 16:17 |
| gibi | yes, I want to make the switch atomic :) | 16:17 |
| gibi | #link https://releases.openstack.org/zed/schedule.html | 16:17 |
| gibi | #info Zed-3 is in 3 weeks | 16:17 |
| gibi | #link Zed tracking etherpad: https://etherpad.opendev.org/p/nova-zed-blueprint-status | 16:17 |
| gibi | #link https://etherpad.opendev.org/p/nova-zed-microversions-plan | 16:17 |
| gibi | so we have 3 week left to feature freeze | 16:18 |
| gibi | and we have a bunch of features in flight as usual | 16:18 |
| gibi | so it would be good to see what has chance to land | 16:18 |
| gibi | so I ask you to update https://etherpad.opendev.org/p/nova-zed-blueprint-status with any status I missed | 16:18 |
| gibi | and if you feel you have review bandwidth for some of those features | 16:18 |
| gibi | then please add you name to the feature | 16:19 |
| gibi | so we see where we have enough reviewers | 16:19 |
| sean-k-mooney[m] | i need to add vdpa to that but its not ready for full review yet | 16:20 |
| sean-k-mooney[m] | so ill hold off until it is | 16:20 |
| gibi | sean-k-mooney[m]: hm, I don't know how I missed the vdpa bp | 16:20 |
| gibi | or is it just a bug? | 16:20 |
| sean-k-mooney[m] | bug for fist patch specless for final 3 | 16:21 |
| gibi | ack, I see the bug https://bugs.launchpad.net/nova/+bug/1970467 do you have a link to the bp? | 16:21 |
| sean-k-mooney[m] | https://blueprints.launchpad.net/nova/+spec/vdpa-suspend-detach-and-live-migrate | 16:21 |
| sean-k-mooney[m] | it did not have the serise goal or milestone so i added them now | 16:22 |
| gibi | ahh that is the reason I missed it. I added it to the pad now | 16:23 |
| gibi | cool | 16:23 |
| gibi | any other thing I missed from the pad? or any specific bp we need to discuss now? | 16:24 |
| gibi | OK, then we will look at this again next week, but please keep that status in the pad updated during the week | 16:26 |
| gibi | #topic Review priorities | 16:27 |
| gibi | #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+(label:Review-Priority%252B1+OR+label:Review-Priority%252B2) | 16:27 |
| gibi | ideally this should be a superset of the bp tracking etherpad commitments | 16:27 |
| gibi | but it isn't | 16:28 |
| gibi | not a big deal | 16:28 |
| gibi | moving on | 16:28 |
| gibi | #topic Stable Branches | 16:28 |
| gibi | mic -> elodilles | 16:28 |
| elodilles | yes, so | 16:29 |
| elodilles | unfortunately nothing news really :/ | 16:29 |
| elodilles | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:29 |
| elodilles | #info stable/train (and older) are blocked, see details in the etherpad ^^^ | 16:30 |
| elodilles | i did not get there to continue the grenade/live-migration debug on stable/train :/ | 16:30 |
| gibi | I have no spare cycle either | 16:31 |
| sean-k-mooney[m] | i think i agree with melwitt however that we shoudl stop rechecking the train use shell patch | 16:32 |
| sean-k-mooney[m] | until we figure out why its failing | 16:32 |
| elodilles | yes, that is completely worthless | 16:32 |
| sean-k-mooney[m] | but i dont have time to look in the near term | 16:32 |
| sean-k-mooney[m] | melwitt: maybe we coudl -w it to signal that | 16:33 |
| gibi | yepp that make sense | 16:35 |
| gibi | anything else about stable? | 16:35 |
| elodilles | nothing from me | 16:35 |
| gibi | #topic Open discussion | 16:36 |
| gibi | nothing on the agenda | 16:36 |
| gibi | is there anything else today/ | 16:37 |
| gibi | ? | 16:37 |
| gibi | then I will give you back 22 minutes | 16:38 |
| gibi | #endmeeting | 16:38 |
| opendevmeet | Meeting ended Tue Aug 9 16:38:35 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:38 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-08-09-16.02.html | 16:38 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-08-09-16.02.txt | 16:38 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-08-09-16.02.log.html | 16:38 |
| opendevreview | Merged openstack/nova master: Update libvirt enlightenments for Windows https://review.opendev.org/c/openstack/nova/+/847641 | 16:47 |
| *** dasm|off is now known as dasm | 18:23 | |
| *** lifeless_ is now known as lifeless | 18:24 | |
| melwitt | stephenfin: thanks for bringing up the ephemeral encryption stuff. I was wary to bother people about it, there's been so much else going on. we'll need bauzas_away to remove his -2 on the bottom patch. IIRC he should be back Aug 30 (2 days before FF), so hopefully he can remove it then cc sean-k-mooney[m] | 19:37 |
| melwitt | sean-k-mooney[m], stephenfin: wrt the naming, I wanted to give everyone a break before bringing it up again 😬 but ... I had thought more about it and thinking about it in the context of the other extra spec/image prop names, I thought "hw_disk_encryption" does seemingly fit better https://github.com/openstack/nova/blob/1220d53c2fd6231bb569a54f004a0e858536e970/nova/objects/image_meta.py#L347-L351 but as you pointed out, there's a | 19:47 |
| melwitt | chance that users might anyway think it could be referring to cinder volumes as well (?). none of the names are a slam dunk really :( | 19:47 |
| *** dasm is now known as dasm|off | 20:43 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!