| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-05-13 | |||
| 13:03:12 | gibi | stephenfin, sean-k-mooney: should we just rename the whitelist to device_list or we make the two config totally separate and use the device_list also to enable the new feature? | |
| 13:04:16 | gibi | if we make them separate then in the initial release the whitelist still needed for the neutron based sriov even if the device_list is used for the alias based PCI passthrough | |
| 13:05:15 | gibi | now I think that renaming is simpler and then I would add an extra config to enable the new placement based PCI tracking | |
| 13:05:39 | sean-k-mooney | ok | |
| 13:05:57 | sean-k-mooney | lets rename and add a hopefully tempoery extra option for placment tracking | |
| 13:06:04 | gibi | should the placement PCI tracking config be on libvirt virt driver level or on the compute level? | |
| 13:06:15 | gibi | compute level is future proof | |
| 13:06:26 | gibi | but a bit confusing as it will only be used by the libvirt driver now | |
| 13:06:38 | sean-k-mooney | good question | |
| 13:06:43 | sean-k-mooney | you could put it in the pci section | |
| 13:07:04 | gibi | that is a good middle ground | |
| 13:07:05 | gibi | thanks | |
| 13:07:08 | sean-k-mooney | [pci]/report_in_placement=True|False | |
| 13:08:06 | gibi | ack | |
| 13:08:19 | sean-k-mooney | stephenfin: any opipion ^ | |
| 13:09:32 | stephenfin | Agreed. Sounds like a '[pci]' option | |
| 13:09:34 | sean-k-mooney | gibi: i know there are some that want us to do the rename anyway for policital reasons so we proably should not make operators choose between fucntionality and politics | |
| 13:10:13 | gibi | yeah that also a + of this approach we just deprecate the old name, but the old name still get all the new tags for now | |
| 13:10:36 | gibi | so the deployer can choose when to rename it in their config | |
| 13:11:50 | sean-k-mooney | works for me | |
| 13:12:33 | sean-k-mooney | are those all the outstanding question in the pci spec adressed then? | |
| 13:13:02 | gibi | I feel like it, but I'm doing the update now so I might bump into qustions along the way | |
| 13:15:09 | bauzas | folks, limited connectivity here probably for the end of the day, my optic fiber is changed | |
| 13:20:29 | bauzas | I changed the provider, but it looks like the older provider made some mistakes | |
| 13:20:33 | opendevreview | sean mooney proposed openstack/nova master: trivial: fix deprecation warning in notification fixture https://review.opendev.org/c/openstack/nova/+/841756 | |
| 13:21:34 | sean-k-mooney | stephenfin: ok done ^ back to oslo dirver | |
| 13:21:54 | sean-k-mooney | bauzas: ack | |
| 13:22:52 | sean-k-mooney | bauzas: when i change my mobile provider recently my old provider blocked the number porting by mistake and i lost my old number that i have had for alomts 15-20 years | |
| 13:22:59 | sean-k-mooney | changing providers sucks | |
| 13:23:11 | sean-k-mooney | on the other hand no more spam calls | |
| 13:25:06 | bauzas | sean-k-mooney: the problem here is that my FTTH ONT number was wrote wrong | |
| 13:25:14 | bauzas | written* | |
| 13:25:26 | bauzas | the ONT number, not the phone number | |
| 13:26:00 | bauzas | so, the infrastructure operator thought it was a new fiber | |
| 13:26:04 | sean-k-mooney | ya apparently the same happened for me more or less the account number was apprently wrong but i got it from the bill and gave it directly to the new operator so im not sure how that could have been | |
| 13:26:20 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/wallaby: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/841760 | |
| 13:26:59 | sean-k-mooney | i actully no have 2 seperate FTTH lines and ONTs | |
| 13:27:27 | sean-k-mooney | i now have EIR FTTH i used to have SIRO form vodaphone | |
| 13:27:34 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/wallaby: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/841736 | |
| 13:28:09 | bauzas | but eventually, my new fiber tech found the right ONT number, so he went to the OLT | |
| 13:28:16 | sean-k-mooney | we have 2 semi state owned fiber networks in ireland | |
| 13:28:52 | bauzas | sean-k-mooney: I guess that's like in France | |
| 13:29:00 | sean-k-mooney | one is owned by what used to be the nationalise telecom company and the other is owned by the state owned electic network infra ownwer | |
| 13:29:19 | bauzas | we have different networks, depending on the town | |
| 13:29:25 | sean-k-mooney | both are then open to telecoms to offer servies over | |
| 13:29:36 | bauzas | like here, in Isere department, we have a specific network | |
| 13:29:57 | bauzas | for this network, we have an infra operator | |
| 13:30:48 | bauzas | that creates fibers for the whole town up to either a block of flats or a street | |
| 13:30:51 | sean-k-mooney | in ireland we are trying to avoid that and have 2 state owned/semi state owned open fiber networks | |
| 13:31:45 | bauzas | and then, every customer can ask a network operator to get a fiber until their either flat or house | |
| 13:32:19 | sean-k-mooney | ya that is similar here | |
| 13:32:21 | bauzas | like for me, this is changing from the infra operator to the network operator 5 meters off my house | |
| 13:32:29 | bauzas | in the street | |
| 14:51:14 | dansmith | yea/query sean-k-mooney | |
| 14:51:18 | dansmith | wow | |
| 15:15:05 | opendevreview | sean mooney proposed openstack/nova master: trivial: fix deprecation warning in notification fixture https://review.opendev.org/c/openstack/nova/+/841756 | |
| 15:15:23 | sean-k-mooney | stephenfin: fixed nits ^ | |
| 15:15:48 | opendevreview | Merged openstack/nova master: Add Python 3.10 functional jobs https://review.opendev.org/c/openstack/nova/+/839029 | |
| 15:33:35 | opendevreview | Balazs Gibizer proposed openstack/nova-specs master: PCI device tracking in Placement https://review.opendev.org/c/openstack/nova-specs/+/791047 | |
| 15:34:46 | gibi | sean-k-mooney, stephenfin, melwitt: fresh and simplified ^^ | |
| 15:37:33 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/wallaby: Add functional tests to reproduce bug #1960412 https://review.opendev.org/c/openstack/nova/+/841760 | |
| 15:37:51 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/wallaby: Clean up when queued live migration aborted https://review.opendev.org/c/openstack/nova/+/841736 | |
| 16:04:25 | sean-k-mooney | gibi: thanks im not sure i have the brain power left to read it fully at 5 pm on a friday | |
| 16:04:41 | sean-k-mooney | but ill defintly take a look on monday | |
| 16:04:54 | gibi | I totally understand :) | |
| 16:05:04 | gibi | have a nice weekend | |
| 16:05:28 | sean-k-mooney | you too o/ | |
| 16:09:15 | opendevreview | sean mooney proposed openstack/os-vif stable/xena: Use TCP keepalives for ovsdb connections https://review.opendev.org/c/openstack/os-vif/+/841771 | |
| 16:09:16 | opendevreview | sean mooney proposed openstack/os-vif stable/xena: only register tables used by os-vif https://review.opendev.org/c/openstack/os-vif/+/841772 | |
| 16:10:32 | opendevreview | sean mooney proposed openstack/os-vif stable/wallaby: Use TCP keepalives for ovsdb connections https://review.opendev.org/c/openstack/os-vif/+/841773 | |
| 16:10:33 | opendevreview | sean mooney proposed openstack/os-vif stable/wallaby: only register tables used by os-vif https://review.opendev.org/c/openstack/os-vif/+/841774 | |
| 16:13:09 | opendevreview | sean mooney proposed openstack/os-vif stable/victoria: Use TCP keepalives for ovsdb connections https://review.opendev.org/c/openstack/os-vif/+/841775 | |
| 16:13:10 | opendevreview | sean mooney proposed openstack/os-vif stable/victoria: only register tables used by os-vif https://review.opendev.org/c/openstack/os-vif/+/841776 | |
| 16:13:47 | opendevreview | sean mooney proposed openstack/os-vif stable/ussuri: Use TCP keepalives for ovsdb connections https://review.opendev.org/c/openstack/os-vif/+/841777 | |
| 16:13:48 | opendevreview | sean mooney proposed openstack/os-vif stable/ussuri: only register tables used by os-vif https://review.opendev.org/c/openstack/os-vif/+/841778 | |
| 16:14:21 | opendevreview | sean mooney proposed openstack/os-vif stable/train: Use TCP keepalives for ovsdb connections https://review.opendev.org/c/openstack/os-vif/+/841779 | |
| 16:14:22 | opendevreview | sean mooney proposed openstack/os-vif stable/train: only register tables used by os-vif https://review.opendev.org/c/openstack/os-vif/+/841780 | |
| 16:45:29 | opendevreview | Merged openstack/nova master: trivial: fix deprecation warning in notification fixture https://review.opendev.org/c/openstack/nova/+/841756 | |
| 18:53:57 | mnaser | happy friday -- https://review.opendev.org/c/openstack/nova/+/840985 + https://review.opendev.org/c/openstack/nova/+/840993 are trivial nice to have fixes that could use reviews =) | |
| 20:04:00 | opendevreview | melanie witt proposed openstack/placement stable/victoria: placement-status: check only consumers in allocation table https://review.opendev.org/c/openstack/placement/+/840702 | |
| 20:05:38 | opendevreview | melanie witt proposed openstack/placement stable/ussuri: placement-status: check only consumers in allocation table https://review.opendev.org/c/openstack/placement/+/840703 | |
| #openstack-nova - 2022-05-16 | |||
| 08:00:43 | bauzas | good morning Nova | |
| 08:11:01 | Uggla | bauzas, o/ | |
| 09:36:10 | opendevreview | Merged openstack/nova master: Switch libvirt event timeout message to warning https://review.opendev.org/c/openstack/nova/+/840985 | |
| 09:37:35 | opendevreview | ribaudr proposed openstack/nova master: Allow unshelve to a specific host https://review.opendev.org/c/openstack/nova/+/831507 | |
| 09:47:01 | opendevreview | ribaudr proposed openstack/python-novaclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-novaclient/+/831651 | |
| 09:49:45 | kashyap | Uggla: Hey, from a quick skim, you might want to split that "unshelve to a specific host" into a few patches. As it seems to be touching a lot of files in ine go. | |
| 09:52:32 | Uggla | kashyap, I'm gonna have a look. I need to think what could be the best split. | |
| 09:54:09 | kashyap | Uggla: One idea could be to split based on "one logical change per commit". It ensures that in case something needs to be reverted, it can be easily done | |
| 09:54:36 | kashyap | (And also many other reasons: https://wiki.openstack.org/wiki/GitCommitMessages#Structural_split_of_changes) | |
| 09:58:23 | sean-k-mooney | Uggla: i tend to split by subsystem | |
| 09:58:30 | sean-k-mooney | or binary that im touching | |
| 09:58:59 | sean-k-mooney | not always but its rare i will touch the api and compute node in one patch | |
| 09:59:03 | sean-k-mooney | or db and api | |
| 09:59:19 | Uggla | sean-k-mooney, yep but here it is only api. | |
| 09:59:44 | sean-k-mooney | ya for that patch the api microverison bump shoudl be the last patch | |
| 09:59:54 | sean-k-mooney | the valdiation code can be a sperat patch | |
| 09:59:59 | sean-k-mooney | not sure about the rest | |
| 10:00:34 | sean-k-mooney | i would not expect that chagne to be more the 2-5 patches honestly | |
| 10:00:51 | sean-k-mooney | its not partacalarly large, you could put all the docs changes in one | |