| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-02-02 | |||
| 17:13:37 | bauzas | empty* | |
| 17:15:54 | sean-k-mooney | no | |
| 17:16:14 | sean-k-mooney | if its not a deffered ip_allocation policy then it default to immeidate | |
| 17:16:25 | sean-k-mooney | meaning it will get the fixed ip when the port is created | |
| 17:16:58 | sean-k-mooney | bauzas: we currently dont allow port to not have a fixed ip unless its defereed | |
| 17:17:06 | sean-k-mooney | in which case it will eventually have a fixed ip | |
| 17:17:21 | sean-k-mooney | as such it should always have a fixed ip after binding | |
| 17:17:44 | sean-k-mooney | oh sorry miss read that | |
| 17:17:49 | sean-k-mooney | yes on the nova command line | |
| 17:18:00 | bauzas | yeah ok | |
| 17:18:03 | sean-k-mooney | when passing a deferred port its fix-ip should be empty | |
| 17:18:40 | openstackgerrit | Stephen Finucane proposed openstack/nova master: cmd: Remove 'nova-manage db ironic_flavor_migration' https://review.opendev.org/c/openstack/nova/+/773776 | |
| 17:18:40 | openstackgerrit | Stephen Finucane proposed openstack/nova master: cmd: Remove 'nova-manage db null_instance_uuid_scan' https://review.opendev.org/c/openstack/nova/+/773775 | |
| 17:18:59 | sean-k-mooney | i mean it technically could be set but we would treat that the same as ip_allocation=imidieate or a live migration | |
| 17:19:24 | sean-k-mooney | e.g. the prence of an ip means we have to restrict the hots to the aggreate corresponidnng to the subnet/segment | |
| 17:27:07 | stephenfin | gmann: I'm +2 on everything I can +2 in the placement RBAC series. You should have +2 rights too now so if you've time this week your attention would be appreciated :) | |
| 17:29:31 | stephenfin | lyarwood: fwiw, there are still OpenStack Release Bot patches to be merged in placement. Could you take a look? https://review.opendev.org/q/project:openstack/placement+is:open+NOT+branch:master | |
| 17:37:48 | sean-k-mooney | stephenfin: removing those nova manage command technially might break some installer scripts if they use them | |
| 17:38:04 | sean-k-mooney | stephenfin: but they should be run via the online migration too | |
| 17:38:25 | sean-k-mooney | bauzas: amend? append? | |
| 17:38:28 | stephenfin | possibly, though that's why they're mentioned in the release notes | |
| 17:38:45 | sean-k-mooney | stephenfin: do we need to go through a deprecation cycle | |
| 17:38:51 | sean-k-mooney | havent opened the patch yet | |
| 17:39:03 | bauzas | sean-k-mooney: context is https://review.opendev.org/c/openstack/nova/+/749068/6/nova/tests/fixtures.py#2114 | |
| 17:39:05 | stephenfin | I don't think so. Not for those. They're usless now | |
| 17:39:08 | stephenfin | *useless | |
| 17:39:20 | bauzas | we can't blindly provide a port without knowing which segment was used | |
| 17:39:39 | sean-k-mooney | bauzas: we can | |
| 17:39:50 | sean-k-mooney | bauzas: just create a port with a network | |
| 17:40:13 | bauzas | but then when moving, we won't have a segment | |
| 17:40:21 | sean-k-mooney | we will | |
| 17:40:31 | sean-k-mooney | it will be assocaite when we boot the vm | |
| 17:40:34 | sean-k-mooney | at port binding | |
| 17:40:39 | bauzas | lemme upload my new reviison | |
| 17:40:46 | sean-k-mooney | assuming its a routed network that is | |
| 17:40:53 | bauzas | sean-k-mooney: no with the fixture | |
| 17:41:00 | sean-k-mooney | bauzas: ah ok | |
| 17:41:08 | bauzas | so I need to mock this | |
| 17:41:22 | sean-k-mooney | well the fixture would need to model the ip deffered logic | |
| 17:41:31 | sean-k-mooney | bauzas: you proably need to add a new netork to the fixture | |
| 17:41:41 | sean-k-mooney | then add logic that looks at the network id | |
| 17:41:52 | sean-k-mooney | if tis the routed one do extra stpes in binding | |
| 17:47:26 | lyarwood | stephenfin: ack will do | |
| 17:47:56 | stephenfin | thanks | |
| 18:24:54 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Add requested_networks field to RequestSpec object https://review.opendev.org/c/openstack/nova/+/749977 | |
| 18:24:55 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Add a routed networks scheduler pre-filter https://review.opendev.org/c/openstack/nova/+/749068 | |
| 18:25:48 | bauzas | sean-k-mooney: huzzah \o/ ^ | |
| 18:26:13 | bauzas | I found a solution for making sure that we would use the right segment | |
| 18:26:41 | bauzas | sean-k-mooney: fwiw, now we check the existing segment when moving | |
| 18:27:40 | bauzas | or when passing a port with an immediate IP allocation | |
| 18:28:09 | sean-k-mooney | bauzas: cool am i may not get time to reviwe it this evening | |
| 18:28:19 | sean-k-mooney | but ill take a look tomorrow in anycase | |
| 18:28:32 | sean-k-mooney | ping me about it if i dont get to it by the after noon | |
| 18:29:48 | bauzas | <3 | |
| 18:29:49 | sean-k-mooney | you did https://review.opendev.org/c/openstack/nova/+/749068/7/nova/tests/fixtures.py#2163 right | |
| 18:29:56 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: WIP: Add a routed networks scheduler pre-filter https://review.opendev.org/c/openstack/nova/+/749068 | |
| 18:29:58 | sean-k-mooney | that is basically what i was suggesting | |
| 18:30:02 | sean-k-mooney | or similar | |
| 18:30:28 | sean-k-mooney | checking the network id to do the right thing for multi segment networks | |
| 18:30:55 | bauzas | yup but the question was to make sure that the segment was correct | |
| 18:31:12 | sean-k-mooney | oh and you now have the unit tests too | |
| 18:31:18 | sean-k-mooney | cool | |
| 18:31:20 | bauzas | yup, will do tomorrow | |
| 18:31:49 | sean-k-mooney | ah you just update the current ones ya ok ill look a this proply whwen i have tiem to fully load the context | |
| 18:32:03 | sean-k-mooney | it looks like your almost done however which is nice | |
| 18:32:26 | bauzas | no worries | |
| 18:32:55 | bauzas | fortunately, we only call create_port if the user passes a network | |
| 18:33:29 | bauzas | when passing a specific port (even a deferred), we don't call it, so that's why I was sure that the first segment was accepted | |
| 18:33:37 | sean-k-mooney | yep | |
| 18:34:29 | bauzas | anyway, /me goes off now | |
| 18:38:40 | gmann | stephenfin: ack, on it. | |
| 19:22:07 | gmann | stephenfin: lbragstad done. few which are from stephenfin need one more +2 | |
| 20:06:23 | openstackgerrit | melanie witt proposed openstack/placement master: policy: Remove the deprecated 'placement' rule https://review.opendev.org/c/openstack/placement/+/772333 | |
| 20:20:29 | openstackgerrit | sean mooney proposed openstack/nova master: [WIP] support per port numa policies with sriov https://review.opendev.org/c/openstack/nova/+/773792 | |
| 21:02:50 | openstackgerrit | Merged openstack/placement master: Add functional-py3[89] tox targets https://review.opendev.org/c/openstack/placement/+/760890 | |
| 21:36:37 | rm_work | augh, how do I use OSC to quickly see which hypervisors are in an AZ? | |
| 21:36:55 | rm_work | failing to find the right command to let me quickly see this | |
| 21:38:59 | openstackgerrit | Merged openstack/placement master: Pass context objects to oslo.policy directly https://review.opendev.org/c/openstack/placement/+/766785 | |
| 21:42:29 | rm_work | ok rather, failing to figure out how i can show the relationship between hypervisor and AZ/aggregate at all | |
| 21:42:47 | rm_work | I assume i'm missing something obvious | |
| 21:42:57 | rm_work | or fundamentally misunderstand how this relationship works | |
| 22:50:02 | openstackgerrit | Ghanshyam proposed openstack/placement master: DNM: testing direct l-c https://review.opendev.org/c/openstack/placement/+/773813 | |
| 23:32:32 | openstackgerrit | Merged openstack/nova master: Include removal of ephemeral backing files in the image cache manager https://review.opendev.org/c/openstack/nova/+/689422 | |
| #openstack-nova - 2021-02-03 | |||
| 02:06:00 | openstackgerrit | Merged openstack/nova master: zuul: Increase SWIFT_LOOPBACK_DISK_SIZE within nova-lvm job https://review.opendev.org/c/openstack/nova/+/772702 | |
| 02:06:26 | openstackgerrit | Merged openstack/nova master: zuul: Mark nova-lvm as voting https://review.opendev.org/c/openstack/nova/+/772871 | |
| 02:10:50 | openstackgerrit | Wenping Song proposed openstack/nova master: Replaces tenant_id with project_id from Flavor Access APIs https://review.opendev.org/c/openstack/nova/+/767704 | |
| 05:37:22 | xinranwang | Hi all, fyi, here is a test report for smartnic support. We have tested the code in local, you can also following the report's instruction to test our patches. https://wiki.openstack.org/wiki/Cyborg/TestReport/IntelNic | |
| 05:56:23 | openstackgerrit | melanie witt proposed openstack/nova master: Centralize sqlite FK constraint enforcement https://review.opendev.org/c/openstack/nova/+/773833 | |
| 05:56:24 | openstackgerrit | melanie witt proposed openstack/nova master: WIP Dynamically archive FK related records in archive_deleted_rows https://review.opendev.org/c/openstack/nova/+/773834 | |
| 09:16:55 | kashyap | lyarwood: Thanks for the rebase! | |
| 09:17:05 | kashyap | lyarwood: (On this one: https://review.opendev.org/c/openstack/nova/+/758763) | |
| 09:19:44 | kashyap | It passed, BTW :-) | |
| 09:25:25 | lyarwood | kashyap: \o morning, np and cool I'll review now, elod ^ you also if you have time, could you review this please? | |
| 09:27:41 | kashyap | Morning | |
| 09:29:15 | kashyap | lyarwood: The tricky thing is that patch is a band-aid for some scenarios -- but will not fix the more difficult CPU-based live migration problems fully. | |
| 09:29:34 | kashyap | lyarwood: I've been thinking enough that I had a CPU model bug walk into my dream last night! | |
| 09:29:43 | lyarwood | haha | |
| 09:29:52 | kashyap | No kidding; I felt strange waking up | |
| 09:30:22 | kashyap | lyarwood: Given the bug OSP upgrades folks found ... for the fuller solution, a potential process trouble: | |
| 09:30:26 | lyarwood | kashyap: so that isn't going to resolve the the noTSX part? | |