| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-02-02 | |||
| 16:39:50 | sean-k-mooney | as the virt-type | |
| 16:40:06 | sean-k-mooney | since you cant use kvm and do emulation concurrenlty | |
| 16:40:07 | stephenfin | yes, understood. However, afaict that isn't even possible at the moment | |
| 16:40:25 | sean-k-mooney | its ment to be posiable if you use qemu as the virt-type | |
| 16:40:42 | sean-k-mooney | at least that is what we ahve said a few times in the PTG | |
| 16:40:55 | sean-k-mooney | we dotn have testing for it though and said fi emulation does not work its a bug | |
| 16:40:55 | stephenfin | Yup, but it's not. Belmiro has a couple of bugs open for it | |
| 16:41:02 | sean-k-mooney | yep | |
| 16:41:10 | stephenfin | I also saw a patch up for this recently | |
| 16:41:16 | sean-k-mooney | i think it work at some point in the distent past | |
| 16:41:25 | sean-k-mooney | as in pre placment era | |
| 17:09:57 | bauzas | sean-k-mooney: quick question, when a port is bound to an instance, which kind of allocation the port is having ? deffered, I guess ? | |
| 17:10:08 | bauzas | (when you don't create a port) | |
| 17:10:24 | bauzas | (just when asking for an instance related to a network) | |
| 17:11:41 | sean-k-mooney | am | |
| 17:11:57 | sean-k-mooney | the ip allocation policy wont change | |
| 17:12:15 | sean-k-mooney | ip_allocation=deferened means the ip is assigined when its bound to a host | |
| 17:12:20 | sean-k-mooney | if that binding succeds | |
| 17:12:41 | sean-k-mooney | so if it had deffered perviously it willl still be deffered if i understand correctly | |
| 17:12:56 | bauzas | ok, so the ip_allocation should be "deferred" but it will have fixed_ips | |
| 17:13:00 | sean-k-mooney | it just wont cahnge ip again in the future once its initally assigned on | |
| 17:13:15 | sean-k-mooney | yep it should get a fix ip at binding time | |
| 17:13:27 | bauzas | while when passing a deferred port for an instance, the fixed_ips value should be emptu | |
| 17:13:30 | bauzas | empyu | |
| 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? | |