| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-10-02 | |||
| 15:38:43 | bauzas | spotted* | |
| 15:38:53 | sean-k-mooney | but want to hear how people want that to be done | |
| 15:38:55 | bauzas | yeah, I thought it was API-related | |
| 15:39:05 | sean-k-mooney | the currnt patch is not api related | |
| 15:39:15 | bauzas | we have a solid experience on API stability while we remove backends | |
| 15:39:23 | sean-k-mooney | just the virt driver and remvoe teh virt dirver spcifric cofnig options and docs. | |
| 15:39:48 | sean-k-mooney | i belive gman has also already done hyperv and has the same open | |
| 15:39:58 | sean-k-mooney | i.e. waht to do with hyperv specific api aspect | |
| 15:40:08 | sean-k-mooney | so i wanted to group does disucssions togetehr | |
| 15:40:13 | sean-k-mooney | since hte approch should likely be the same | |
| 15:41:05 | bauzas | sounds a good idea to me | |
| 15:41:17 | bauzas | thanks for catching it | |
| 15:41:34 | sean-k-mooney | do we have the ptg etherpad yet | |
| 15:42:23 | bauzas | yep | |
| 15:42:33 | bauzas | oh but you weren't in the last meeting, sec | |
| 15:42:47 | sean-k-mooney | ya still catching up form PTO | |
| 15:42:54 | bauzas | np | |
| 15:43:25 | bauzas | https://etherpad.opendev.org/p/nova-caracal-ptg | |
| 15:43:42 | bauzas | (didn't made any huge effort for the naming) | |
| 15:44:23 | sean-k-mooney | cool ill add something to that then and you cna feel frre to rearragne things | |
| 15:45:09 | bauzas | cool | |
| 15:53:24 | opendevreview | Sylvain Bauza proposed openstack/nova master: doc: clarify that pre-commit is optional https://review.opendev.org/c/openstack/nova/+/897057 | |
| 16:08:16 | stephenfin | sean-k-mooney: I wonder if we could start setting and exposing ACPI index for network devices, as a consistent identifier that will be respected (unless PCI device addresses) https://libvirt.org/formatdomain.html#network-interfaces | |
| 16:08:44 | sean-k-mooney | hum let me read | |
| 16:08:48 | sean-k-mooney | but maybe | |
| 16:09:19 | sean-k-mooney | """Since 7.3.0, one can set the ACPI index against network interfaces. With some operating systems (eg Linux with systemd), the ACPI index is used to provide network interface device naming, that is stable across changes in PCI addresses assigned to the device. This value is required to be unique across all devices and be between 1 and (16*1024-1).""" | |
| 16:09:33 | sean-k-mooney | interesting so ya maybe | |
| 16:10:28 | sean-k-mooney | stephenfin: of the top of my head i dont see why not | |
| 16:11:01 | sean-k-mooney | well hum | |
| 16:11:06 | sean-k-mooney | there might be some issues with that | |
| 16:11:21 | sean-k-mooney | so the problem iw going to be how to corralate it with the port | |
| 16:11:31 | sean-k-mooney | so we willl likely need to store it in the virtural interfaces table | |
| 16:12:10 | sean-k-mooney | so that its accesable to the api or store it in the binding_profile on the neutorn port | |
| 16:52:03 | sean-k-mooney | ok so testing the acpi index theory <acpi index="4"/> is accepted in the xml | |
| 16:52:40 | sean-k-mooney | and the interface name becomes eno4 | |
| 16:52:44 | sean-k-mooney | so that is a good sigh | |
| 16:53:23 | sean-k-mooney | sean@upstream-devstack:~$ cat /sys/class/net/eno4/device/acpi_index | |
| 16:53:25 | sean-k-mooney | 4 | |
| 16:53:35 | sean-k-mooney | stephenfin: ^ so ya that is trivially discoverable | |
| 16:53:52 | stephenfin | \o/ | |
| 16:53:54 | stephenfin | noice | |
| 16:53:59 | sean-k-mooney | so if we were to add that to the metadata that would work | |
| 16:54:11 | sean-k-mooney | im going to make it 42 just to be sure it actully is working the way i think | |
| 16:55:26 | sean-k-mooney | sean@upstream-devstack:~$ cat /sys/class/net/eno42/device/acpi_index | |
| 16:55:28 | sean-k-mooney | 42 | |
| 16:55:47 | sean-k-mooney | yep so it will result in a stable device name as well as the docs suggest | |
| 18:22:05 | opendevreview | Dan Smith proposed openstack/nova master: Sanity check that new hosts have no instances https://review.opendev.org/c/openstack/nova/+/897076 | |
| 19:35:14 | opendevreview | Dan Smith proposed openstack/nova master: Sanity check that new hosts have no instances https://review.opendev.org/c/openstack/nova/+/897076 | |
| 20:38:09 | opendevreview | sean mooney proposed openstack/nova master: remove deprecated vmware virt driver https://review.opendev.org/c/openstack/nova/+/897017 | |
| 22:04:26 | opendevreview | sean mooney proposed openstack/nova master: fix sphinx-lint issues in releasenotes https://review.opendev.org/c/openstack/nova/+/897087 | |
| 22:04:27 | opendevreview | sean mooney proposed openstack/nova master: fix sphinx-lint issues in api guide https://review.opendev.org/c/openstack/nova/+/897088 | |
| 22:04:27 | opendevreview | sean mooney proposed openstack/nova master: fix sphinx-lint errors in docs and add ci https://review.opendev.org/c/openstack/nova/+/897089 | |
| 23:54:20 | opendevreview | sean mooney proposed openstack/nova master: [codespell] start fixing all the typos https://review.opendev.org/c/openstack/nova/+/897092 | |
| #openstack-nova - 2023-10-03 | |||
| 00:33:08 | opendevreview | sean mooney proposed openstack/nova master: [codespell] fix typos in api-ref https://review.opendev.org/c/openstack/nova/+/897094 | |
| 00:33:08 | opendevreview | sean mooney proposed openstack/nova master: [codespell] apply codespell to the releasenotes https://review.opendev.org/c/openstack/nova/+/897095 | |
| 00:33:09 | opendevreview | sean mooney proposed openstack/nova master: [codespell] doc,devstack and gate typos https://review.opendev.org/c/openstack/nova/+/897096 | |
| 07:36:49 | bauzas | cores, could you please quickly merge https://review.opendev.org/c/openstack/nova-specs/+/895815 before our Bobcat GA ? | |
| 08:12:45 | gibi | bauzas: Is it better to mark the ironic shard implemented with a note that it is reverted? Do we expect that shard spec will be reproposed in C? | |
| 08:13:06 | bauzas | gibi: have you seen the next change in the series ? | |
| 08:13:59 | bauzas | https://review.opendev.org/c/openstack/nova-specs/+/896679/1/specs/2023.2/implemented/ironic-shards.rst | |
| 08:14:11 | gibi | bauzas: I saw the note yes | |
| 08:14:32 | gibi | my question is will there be a new spec in C to merge the shard implementation again? | |
| 08:14:39 | bauzas | gibi: I hope so | |
| 08:14:50 | gibi | then we should not put the current shard spec to implemented | |
| 08:15:03 | bauzas | johnthetubaguy: do you know if you could be able to create a new spec in Caracal for ironic-shards ? | |
| 08:15:36 | gibi | we should only put the shard spec to implemented that was implemented and *released* | |
| 08:16:46 | bauzas | okay, then let me change the Launchpad blueprint | |
| 08:23:23 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Move Bobcat implemented specs https://review.opendev.org/c/openstack/nova-specs/+/895815 | |
| 08:23:24 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: explain why ironic-shards was reverted in the spec https://review.opendev.org/c/openstack/nova-specs/+/896679 | |
| 09:32:27 | gibi | thanks | |
| 09:37:53 | sean-k-mooney | sure ill review it now | |
| 09:38:21 | sean-k-mooney | done | |
| 09:40:02 | sean-k-mooney | back to codespell... | |
| 09:50:47 | bauzas | thanks | |
| 09:52:29 | opendevreview | Merged openstack/nova-specs master: Move Bobcat implemented specs https://review.opendev.org/c/openstack/nova-specs/+/895815 | |
| 09:52:59 | opendevreview | Merged openstack/nova-specs master: explain why ironic-shards was reverted in the spec https://review.opendev.org/c/openstack/nova-specs/+/896679 | |
| 10:18:18 | dvo-plv | sean-k-mooney, Hello, I had resolved your comment, Could you please review it again in a free time ? https://review.opendev.org/c/openstack/nova-specs/+/895924 | |
| 10:26:30 | sean-k-mooney | yep looks fine to me | |
| 10:26:54 | sean-k-mooney | easy reporposal ^ if anyone else wants to review that | |
| 10:27:07 | dvo-plv | Thank you | |
| 10:28:35 | dvo-plv | yes, bauzas told me yesterday, that its review week, so I would like to wait for it as far as it is in his agenda | |
| 10:30:54 | sean-k-mooney | well the bobcat release is mostly done now and we are getting ready for the ptg so its not really formal | |
| 10:31:05 | sean-k-mooney | but this week is when most of us are starting to focus more on the next cycle | |
| 10:32:00 | sean-k-mooney | thats why im getting all the liniting/spellecheking/style enhancment stuff i have wanted to do for the last year or so ready this week | |
| 10:32:43 | sean-k-mooney | i.e. before next cycle dev really picks up get the tooling to help use review and land code faster in place | |
| 11:02:40 | opendevreview | sean mooney proposed openstack/nova master: [codespell] fix typos in tests https://review.opendev.org/c/openstack/nova/+/897213 | |
| 11:02:40 | opendevreview | sean mooney proposed openstack/nova master: [codespell] fix final typos and enable ci https://review.opendev.org/c/openstack/nova/+/897214 | |
| 11:02:41 | opendevreview | sean mooney proposed openstack/nova master: [codespell] ignore codespell in git blame https://review.opendev.org/c/openstack/nova/+/897215 | |
| 11:08:38 | opendevreview | sean mooney proposed openstack/nova master: fix sphinx-lint errors in docs and add ci https://review.opendev.org/c/openstack/nova/+/897089 | |
| 11:11:51 | opendevreview | sean mooney proposed openstack/nova master: ignore sphinx-lint series in git blame https://review.opendev.org/c/openstack/nova/+/897218 | |
| 11:15:34 | opendevreview | sean mooney proposed openstack/nova master: remove deprecated vmware virt driver https://review.opendev.org/c/openstack/nova/+/897017 | |
| 11:17:17 | opendevreview | sean mooney proposed openstack/nova master: remove deprecated vmware virt driver https://review.opendev.org/c/openstack/nova/+/897017 | |
| 12:24:52 | opendevreview | sean mooney proposed openstack/nova-specs master: resubmit per-process-healthchecks spec for 2024.1 https://review.opendev.org/c/openstack/nova-specs/+/897225 | |
| 12:26:50 | opendevreview | sean mooney proposed openstack/nova master: [WIP] add initial healthcheck support https://review.opendev.org/c/openstack/nova/+/825015 | |
| 12:26:51 | opendevreview | sean mooney proposed openstack/nova master: [WIP] add healthcheck manager to manager base https://review.opendev.org/c/openstack/nova/+/827844 | |
| 12:26:51 | opendevreview | sean mooney proposed openstack/nova master: [WIP] add healthcheck tracker to nova context https://review.opendev.org/c/openstack/nova/+/829468 | |
| 12:26:52 | opendevreview | sean mooney proposed openstack/nova master: [WIP] add healthcheck utils and constants https://review.opendev.org/c/openstack/nova/+/829469 | |
| 12:26:52 | opendevreview | sean mooney proposed openstack/nova master: add healthcheck endpoint to proxy commands https://review.opendev.org/c/openstack/nova/+/830703 | |
| 12:35:10 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds device tagging functional tests https://review.opendev.org/c/openstack/nova/+/895162 | |
| 12:35:11 | opendevreview | Amit Uniyal proposed openstack/nova master: Device tags: don't pass pf_interface=True to get_mac_by_pci_address https://review.opendev.org/c/openstack/nova/+/670593 | |