| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-08-09 | |||
| 11:29:04 | 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:14 | 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:26 | sean-k-mooney[m] | no | |
| 11:29:41 | sean-k-mooney[m] | when we rescue we use the image metadta form the rescue image i belive | |
| 11:29:44 | sean-k-mooney[m] | so just add it to the image | |
| 11:29:51 | noonedeadpunk | ah, ok, gotcha now | |
| 11:30:02 | noonedeadpunk | makes sense | |
| 11:30:41 | sean-k-mooney[m] | generally for reasonse we recommend hw_rescue_bus=usb hw_rescue_device=disk | |
| 11:31:26 | sean-k-mooney[m] | basically if you use a usb rescue bus/device then it does not change the /dev/sda naming | |
| 11:32:00 | sean-k-mooney[m] | you can just use the same bus as the root disk normally uses too | |
| 11:32:38 | sean-k-mooney[m] | https://specs.openstack.org/openstack/nova-specs/specs/ussuri/implemented/virt-rescue-stable-disk-devices.html#proposed-change | |
| 11:54:28 | opendevreview | Balazs Gibizer proposed openstack/nova master: Handle PCI dev reconf with allocations https://review.opendev.org/c/openstack/nova/+/852397 | |
| 11:54:28 | opendevreview | Balazs Gibizer proposed openstack/nova master: Heal PCI allocation during resize https://review.opendev.org/c/openstack/nova/+/852396 | |
| 11:54:29 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow enabling PCI tracking in Placement https://review.opendev.org/c/openstack/nova/+/850468 | |
| 12:18:42 | 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:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: Extra tests for remote managed dev spec https://review.opendev.org/c/openstack/nova/+/844626 | |
| 12:18:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Unparent PciDeviceSpec from PciAddressSpec https://review.opendev.org/c/openstack/nova/+/844491 | |
| 12:18:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix PciAddressSpec descendants to call super.__init__ https://review.opendev.org/c/openstack/nova/+/844565 | |
| 12:18:44 | 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:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove dead code from PhysicalPciAddress https://review.opendev.org/c/openstack/nova/+/844628 | |
| 12:18:45 | 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:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix type annotation of pci.Whitelist class https://review.opendev.org/c/openstack/nova/+/845780 | |
| 12:18:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move __str__ to the PciAddressSpec base class https://review.opendev.org/c/openstack/nova/+/845781 | |
| 12:56:49 | opendevreview | Merged openstack/nova stable/yoga: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/852235 | |
| 15:12:11 | 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:51 | 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:13:31 | sean-k-mooney[m] | sure ill take a look now but i generaly trus your judgement | |
| 15:14:13 | sfinucan | sean-k-mooney[m]: Can you give me a summary of what you concluded on the ephemeral vs. disk debate? | |
| 15:15:04 | 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:08 | sean-k-mooney[m] | the term ephemeral is ambigious but its not worth changing as it has mindshare | |
| 15:15:11 | sfinucan | context is https://review.opendev.org/c/openstack/nova/+/760454 | |
| 15:15:19 | sean-k-mooney[m] | so i was ment to drop my -1 and proceed with the current names | |
| 15:15:24 | sean-k-mooney[m] | but escalation | |
| 15:15:28 | sean-k-mooney[m] | so ill do that now | |
| 15:15:31 | sfinucan | Ah, no worries | |
| 15:16:44 | stephenfin | order is restored (IRC nick) | |
| 15:16:46 | 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:17:32 | gibi | we will discuss this on the today's meeting | |
| 15:18:13 | sean-k-mooney[m] | sfinucan: https://review.opendev.org/c/openstack/nova/+/760454/11#message-391f61808b977003be0f33c48e768e37bec8697e | |
| 15:19:09 | stephenfin | sean-k-mooney[m]: thanks | |
| 15:19:12 | sean-k-mooney[m] | hehe i was just going to say i just +w'd the windows enlightements patch | |
| 15:19:17 | sean-k-mooney[m] | so that should be on its way | |
| 15:26:35 | gibi | one less to care about | |
| 15:26:38 | gibi | thanks | |
| 15:40:54 | stephenfin | sean-k-mooney[m]: another one to drop your -1 from, I think https://review.opendev.org/c/openstack/nova/+/764486 | |
| 15:45:33 | stephenfin | sean-k-mooney[m]: and https://review.opendev.org/c/openstack/nova/+/760455 | |
| 15:49:48 | sean-k-mooney[m] | stephenfin: done | |
| 15:50:31 | gibi | fyi nova meeting starts in 10 minutes here in the channel | |
| 16:02:05 | elodilles | o/ | |
| 16:02:25 | elodilles | gibi: meeting? :) | |
| 16:02:29 | gibi | hups | |
| 16:02:33 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:02:33 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:02:33 | 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:33 | gibi | #startmeeting nova | |
| 16:02:41 | dansmith | o/ | |
| 16:02:44 | elodilles | o/ | |
| 16:03:38 | gibi | this is typical that I remember up until 10 minutes before the actual time :) | |
| 16:03:43 | sean-k-mooney[m] | o/ | |
| 16:03:44 | gibi | then I forget | |
| 16:03:46 | gibi | anyhow | |
| 16:03:47 | elodilles | :D | |
| 16:04:19 | gibi | #topic Bugs (stuck/critical) | |
| 16:04:27 | gibi | #info No Critical bug | |
| 16:04:46 | gibi | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 11 new untriaged bugs (+0 since the last meeting) | |
| 16:04:55 | gibi | #link https://storyboard.openstack.org/#!/project/openstack/placement 27 open stories (+0 since the last meeting) in Storyboard for Placement | |
| 16:05:05 | 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:16 | gibi | thanks elodilles for the bug triage this week | |
| 16:05:30 | gibi | #info bug baton is being passed to gibi | |
| 16:06:02 | gibi | is there any bugs we need to discuss today? | |
| 16:06:30 | elodilles | i had little time, but triaged 3 bugs | |
| 16:06:49 | elodilles | this one seems interesting: Shelved (offloaded) instance still have port bound to host | |
| 16:07:01 | gibi | yep that sort of a known issue | |
| 16:07:07 | elodilles | oh, i see | |
| 16:07:21 | gmann | o/ | |
| 16:07:27 | gibi | probably it does not break anything but I agree that it is valid | |
| 16:08:14 | elodilles | then maybe that's it from my side :) | |
| 16:08:23 | gibi | any other bugs from anybody? | |
| 16:09:25 | gibi | #topic Gate status | |
| 16:09:34 | gibi | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:09:44 | gibi | I dont see any new bug there | |
| 16:09:51 | gibi | #link https://zuul.openstack.org/builds?project=openstack%2Fplacement&pipeline=periodic-weekly Placement periodic job status | |
| 16:09:55 | gibi | it is green | |
| 16:10:00 | 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:04 | gibi | green too | |
| 16:10:13 | gibi | #link https://zuul.opendev.org/t/openstack/builds?job_name=nova-emulation&pipeline=periodic-weekly&skip=0 Emulation periodic job runs | |
| 16:10:17 | gibi | green as well | |
| 16:10:23 | gibi | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:10:28 | gibi | #info STOP DOING BLIND RECHECKS aka. 'recheck' https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures | |
| 16:10:38 | gibi | any gate issue we need to discuss? | |
| 16:11:26 | gibi | I have one thing | |
| 16:12:11 | gibi | we merged the switch to unittest.mock that also showed us that in py310 the unittest.mock is more strict | |
| 16:12:17 | gibi | it does not allow mocking the same function twice | |
| 16:12:38 | gibi | this made our py310 non-voting jobs red | |
| 16:13:04 | gibi | we fixed it once but broke it later again | |
| 16:13:15 | gibi | so I'm fixing it again https://review.opendev.org/c/openstack/nova/+/852464 and making py310 jobs voting for nova | |
| 16:13:25 | gibi | to pervent breaking this again | |
| 16:13:39 | gibi | I know that officially we don't support py310 yet | |
| 16:13:55 | gibi | but I don't really want to fix double mocking in AA or BB again | |
| 16:14:30 | gibi | so any disagreement switchin the unittest and functional test jobs with py310 voting? | |