| opendevreview | Merged openstack/nova master: Fix cover env HTML testr results missing unit tests https://review.opendev.org/c/openstack/nova/+/993151 | 00:50 |
|---|---|---|
| opendevreview | Seyeong Kim proposed openstack/nova master: Avoid serializing orphaned instance in log https://review.opendev.org/c/openstack/nova/+/992574 | 00:59 |
| opendevreview | Brian Haley proposed openstack/nova master: Change neutron API calls to use project_id https://review.opendev.org/c/openstack/nova/+/994321 | 01:44 |
| opendevreview | Ghanshyam Maan proposed openstack/nova master: DNM: Test removal of enforce_scope https://review.opendev.org/c/openstack/nova/+/986479 | 01:45 |
| gmaan | nova cores, these are the changes for the nova to merge before oslo.policy enforce_scope flag is removed, please check https://review.opendev.org/q/hashtag:%22remove-enforce-scope-flag%22+AND+hashtag:%22nova%22+status:open | 01:50 |
| *** bauzas7 is now known as bauzas | 01:54 | |
| opendevreview | Joan Gilabert proposed openstack/nova master: move compile earlier https://review.opendev.org/c/openstack/nova/+/950516 | 09:17 |
| opendevreview | Joan Gilabert proposed openstack/nova master: Add mtty support for testing fake mdevs https://review.opendev.org/c/openstack/nova/+/898100 | 09:17 |
| opendevreview | Joan Gilabert proposed openstack/nova master: Rename vtpm job and add mtty support for vgpu test https://review.opendev.org/c/openstack/nova/+/922140 | 09:17 |
| gibi | sean-k-mooney: could we land https://review.opendev.org/c/openstack/nova/+/936093 ? There was a bit of back an forth on IRC and in the review and then we hit some deadline and then we forgot about the patch. | 10:13 |
| gibi | we got yet another downstream report around interface MTUs and live migration | 10:13 |
| gibi | I know this is not the full fix but it is a start | 10:14 |
| sean-k-mooney | gibi: o/ | 10:20 |
| sean-k-mooney | amy proiably i need to load context | 10:20 |
| sean-k-mooney | gibi: so the issue that is not adressed is you cant change the mtu on live migration and we need to handel that | 10:22 |
| sean-k-mooney | but ya we can likely proceed | 10:22 |
| sean-k-mooney | im trying to see if i had a specific concern in the past beyond that i.e. somethign that woudl be hard to fix later beyond it being a partial fix | 10:23 |
| sean-k-mooney | gibi: so looking at my old comment the main ask i had was to add a functional regression test | 10:25 |
| sean-k-mooney | gibi: so that would still be nice to have but i dont have an objection to the approch to fixign it | 10:26 |
| gibi | OK. Lets see If I get some time to add a functional reproducer. If so I will ping you that patch | 10:28 |
| gibi | thanks for confirming that the direction is OK | 10:28 |
| sean-k-mooney | if you want i can just have ai create one now and i can adress my own request | 10:28 |
| gibi | if you have the capacit I'm OK with that | 10:30 |
| opendevreview | Joan Gilabert proposed openstack/nova master: Report OWNER_NOVA trait on resource providers https://review.opendev.org/c/openstack/nova/+/994299 | 10:37 |
| opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Add Intel TDX host capability detection https://review.opendev.org/c/openstack/nova/+/994421 | 10:55 |
| sean-k-mooney | bauzas: while i think of it im now +2 on https://review.opendev.org/c/openstack/nova/+/988777 if you want to readd yours | 11:14 |
| opendevreview | Joan Gilabert proposed openstack/nova master: move compile earlier https://review.opendev.org/c/openstack/nova/+/950516 | 11:18 |
| opendevreview | Joan Gilabert proposed openstack/nova master: Add mtty support for testing fake mdevs https://review.opendev.org/c/openstack/nova/+/898100 | 11:18 |
| opendevreview | Joan Gilabert proposed openstack/nova master: Rename vtpm job and add mtty support for vgpu test https://review.opendev.org/c/openstack/nova/+/922140 | 11:18 |
| opendevreview | Anton Iacobaeus proposed openstack/nova master: libvirt: Add Intel TDX host capability detection https://review.opendev.org/c/openstack/nova/+/994421 | 11:29 |
| opendevreview | Masanori Ueno proposed openstack/nova master: Add regression test for bug #2153425 https://review.opendev.org/c/openstack/nova/+/988777 | 13:13 |
| opendevreview | Masanori Ueno proposed openstack/nova master: live-migration: Ensure limits are set before _call_livem_checks_on_host https://review.opendev.org/c/openstack/nova/+/990212 | 13:16 |
| *** ykarel_ is now known as ykarel | 13:26 | |
| opendevreview | sean mooney proposed openstack/nova master: Fix network mtu metadata disappears after attaching https://review.opendev.org/c/openstack/nova/+/936093 | 13:32 |
| opendevreview | sean mooney proposed openstack/nova master: Add regression tests for interface attach MTU bug https://review.opendev.org/c/openstack/nova/+/994437 | 13:32 |
| opendevreview | sean mooney proposed openstack/nova master: Fix network mtu metadata disappears after attaching https://review.opendev.org/c/openstack/nova/+/936093 | 13:33 |
| sean-k-mooney | gibi: that also needed a rebase but ^ should cover all my nits | 13:33 |
| gibi | sean-k-mooney: thanks | 13:35 |
| sean-k-mooney | i didnt quite have tim eto push that before my meeting as i was waiting for tox runs | 13:35 |
| opendevreview | Erlon R. Cruz proposed openstack/nova master: Adding regression test to reproduce bug 1741364 https://review.opendev.org/c/openstack/nova/+/993597 | 13:51 |
| opendevreview | Erlon R. Cruz proposed openstack/nova master: Add RBD XML update functionality for migration process https://review.opendev.org/c/openstack/nova/+/974032 | 13:51 |
| *** raukadah is now known as chandankumar | 14:02 | |
| opendevreview | Joan Gilabert proposed openstack/nova master: Report OWNER_NOVA trait on resource providers https://review.opendev.org/c/openstack/nova/+/994299 | 14:32 |
| frickler | nova-ceph-multistore is failing on unmaintained/2024.1, I don't really want to bug people about it much, but maybe if someone knows an easy fix we can apply that instead of just dropping the job https://zuul.opendev.org/t/openstack/build/570a4539d0f24452ab6c5f9d75e3450a | 16:09 |
| melwitt | hm, "oslo_config.cfg.NoSuchOptError: no such option replication in group [volume-feature-enabled]" must be something like, cinder-tempest-plugin is not branched but maybe tempest on unmaintained/2024.1 is pinned cc gmaan | 16:25 |
| gmaan | yes, i think it is pinned | 16:28 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: compute: Validate SEV/SEV-ES support at start up https://review.opendev.org/c/openstack/nova/+/994342 | 16:28 |
| gmaan | yes tempest 45.0.0 is used and it does not have 'replication ' config option https://github.com/openstack/tempest/blob/45.0.0/tempest/config.py | 16:29 |
| gmaan | so test should be skipped in that case | 16:29 |
| gmaan | we need to pin cinder-tempest-plugin also so that it will not pick the latest tests from it | 16:31 |
| melwitt | makes sense | 16:37 |
| gmaan | will propose fix after meeting | 16:39 |
| gmaan | melwitt: sean-k-mooney you might be interested in these. nova side work needed for oslo.policy enforce_scope flag removal https://review.opendev.org/q/hashtag:%22remove-enforce-scope-flag%22+AND+hashtag:%22nova%22+status:open | 16:39 |
| melwitt | gmaan: ack will look | 16:41 |
| gmaan | thanks | 16:41 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | 16:55 |
| sean-k-mooney | gmaan: oh cool | 17:01 |
| sean-k-mooney | looks pretty mechnaical | 17:02 |
| sean-k-mooney | while while somewhat tedious to review is at elast not overly compex | 17:02 |
| opendevreview | Lajos Katona proposed openstack/nova master: Add regression test to repoduce bug 1854844 https://review.opendev.org/c/openstack/nova/+/991772 | 17:03 |
| sean-k-mooney | im slightly suprised the nova change depens on the placment one but i guess that is down to the fixture? | 17:04 |
| gmaan | sean-k-mooney: I think yes, somewhere it was failing and I need to do placement one first | 17:08 |
| sean-k-mooney | i am sumwhat suprised that we need to update teh gabit tests | 17:14 |
| sean-k-mooney | specific for system scoped rp creation | 17:15 |
| sean-k-mooney | given i was not expecting system scoped roels to really be valid based on https://github.com/openstack/placement/blob/master/placement/policies/resource_provider.py | 17:15 |
| sean-k-mooney | given admin or service is "role:admin or role:service" | 17:16 |
| sean-k-mooney | but i guess when scope enforcment is disabeld we are merging system and doamin roels into the set of token roles? | 17:17 |
| sean-k-mooney | gmaan: basiclly im asking did we change the behvior or was gabbi just not emululatign this properly | 17:17 |
| sean-k-mooney | for this change https://review.opendev.org/c/openstack/placement/+/987568/4/placement/tests/functional/gabbits/aggregate-legacy-rbac.yaml#55 | 17:18 |
| gmaan | sean-k-mooney: yes, it is same as nova test change. the update is to the tests which were testing scope disable and system_admin can do the things | 17:19 |
| gmaan | sean-k-mooney: those are needed to create the resource for other tests like update test, for example $ENVIRON[[RP_UUID] | 17:20 |
| gmaan | previously system_admin was allowed in those tests and it created the resouce | 17:20 |
| gmaan | i thought of remove the legacy tests but till we have the legacy policy, we should keep them tested | 17:20 |
| gmaan | Basically, it remove the scope disable tests and update a few that were dependent on the test resource creation by scope disable tests | 17:23 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: compute: Validate SEV/SEV-ES support at start up https://review.opendev.org/c/openstack/nova/+/994342 | 17:36 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Ignore SEV-ES when SEV-SNP is detected https://review.opendev.org/c/openstack/nova/+/994343 | 17:36 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Fix wrong mock for properties https://review.opendev.org/c/openstack/nova/+/994505 | 17:36 |
| opendevreview | Takashi Kajinami proposed openstack/nova master: Drop dependency on [libvirt] num_memory_encrypted_guests https://review.opendev.org/c/openstack/nova/+/994506 | 17:36 |
| opendevreview | Sahana proposed openstack/nova master: compute: remove redundant exception args from LOG.exception calls https://review.opendev.org/c/openstack/nova/+/994522 | 18:37 |
| opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | 18:57 |
| opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | 19:12 |
| gmaan | frickler: melwitt this will fix the nova-ceph-multistore job on 2024.1, we cna see devstack-plugin-ceph-tempest-py3 passing there https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/994503 | 19:31 |
| gmaan | but it seems gate is broken for doc job there so I am leaving this until someonw fix the gate. cannot spend more time on those. That is one of the reason i try not to touch unmaintained branches as they need lot of gate fixes before one merge :) | 19:32 |
| gmaan | CC: elodilles just to make you aware about ceph job fix | 19:32 |
| frickler | gmaan: thanks, I can try to look at the docs issue. and yes I can understand your reluctance, thx for checking this anyhow | 19:49 |
| melwitt | gmaan: ah makes sense | 19:57 |
| opendevreview | Merged openstack/nova master: move compile earlier https://review.opendev.org/c/openstack/nova/+/950516 | 20:01 |
| opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | 20:08 |
| erlon | sean-k-mooney: hey Sean, can you take a look in the RBD fix we discuss the other day? See if thats how you wanted the regression tests, please let me know if I'm missing anything: https://review.opendev.org/c/openstack/nova/+/974032 | 20:12 |
| sean-k-mooney | erlon: yes i started lookign at that elaire today | 20:14 |
| sean-k-mooney | i have the regression test open i na tab somewhere | 20:14 |
| sean-k-mooney | erlon: lookign at ti brifly yes it looks more or less correct ill try and review it proeprly but probaly tomorrow | 20:16 |
| erlon | nice, thanks | 20:16 |
| sean-k-mooney | i see melwitt updated https://review.opendev.org/c/openstack/nova/+/984623/21 as well so we shoudl get som new reuslt of the ipv6 test in a few hours | 20:17 |
| melwitt | yeah fingers crossed it doesn't run into another issue. this is the first time I try it with depends-on patches proposed to the proper repos. last fail was for database host using ipv4 address. I'll keep iterating on it | 20:19 |
| sean-k-mooney | is there a reason you made the ipv6 connectivity opt in instead of opt out? | 20:25 |
| sean-k-mooney | we often have ipv6 in the jobs | 20:25 |
| sean-k-mooney | even if its not the ipv6 job | 20:25 |
| sean-k-mooney | we just dont use those adress for the services by default | 20:25 |
| melwitt | no reason other than to keep existing behavior the same | 20:25 |
| sean-k-mooney | ack | 20:26 |
| sean-k-mooney | im not sure which is the better default i just said i would ask | 20:26 |
| sean-k-mooney | incasue it was because you hit a bug or something i was not aware of | 20:26 |
| melwitt | oh yeah. not yet anyway haha | 20:26 |
| sean-k-mooney | so if you respin you might want to mention ipv6 in the readme for that roles https://opendev.org/zuul/zuul-jobs/src/commit/6c0c643b8cffa1f61cf7af34f29d8a7e4d1bb114/roles/multi-node-bridge/README.rst | 20:27 |
| sean-k-mooney | melwitt: they do not use molecule in that repo | 20:29 |
| sean-k-mooney | but you may also want to exted https://opendev.org/zuul/zuul-jobs/src/commit/6c0c643b8cffa1f61cf7af34f29d8a7e4d1bb114/test-playbooks/multinode/multi-node-bridge.yaml | 20:29 |
| sean-k-mooney | to test the ipv6 functionlatiy | 20:29 |
| sean-k-mooney | but i dont really review code in that repo oftent so im not sure what the normal expecation are | 20:30 |
| melwitt | good point, I didn't know about those | 20:30 |
| sean-k-mooney | fungi and clarkb are inovled so i knwo there woudl be tests but i am not very familar with how they are setup but i guess this is just usign zuul to run the test playbooks directly | 20:31 |
| sean-k-mooney | looks like tey are using tox and unittest to run them at a quick glance | 20:31 |
| sean-k-mooney | although that may just be for the inline moduels | 20:32 |
| fungi | yeah, clarkb can probably explain it better since he wrote a lot of it, but the idea initially was to have an abstract foundation for multi-node devstack jobs to build on | 20:32 |
| sean-k-mooney | fungi: for the most part it works quite well. fungi context is melaine is adding supprot for multi node ipv6 | 20:34 |
| fungi | in particular, avoiding dependence on the network topology available to the test nodes themselves and setting up an encapsulated layer 2 network between them that would provide consistency | 20:34 |
| sean-k-mooney | so we can test live mgiration with ipv6 ceph | 20:34 |
| fungi | just to be clear, you want to tunnel v6 over the bridge (that shouldn't be a problem but maybe we baked in assumptions) | 20:35 |
| sean-k-mooney | fungi: yep that the idea. melwitt has some devstack josb mostly workign with pre-playbooks in nova and is now tryign to do it properly in zuul-jobs and devstack | 20:36 |
| melwitt | with barely any knowledge 😩 it just failed again. obviously I must have lost something while splitting out from the pre-playbooks into zuul-jobs. will have to figure this out | 20:38 |
| fungi | right, afaik we didn't want that role to restrict what kind of traffic you can send over it, modulo obvious mtu adjustments | 20:39 |
| sean-k-mooney | you may need to set bridge_configure_ipv6_address: true in the subnodes as well | 20:39 |
| sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/984623/21/.zuul.yaml#416 | 20:39 |
| fungi | also i think there may be some limitations or at least limitations carrying broadcast and multicast traffic on it | 20:39 |
| sean-k-mooney | fungi: its using vxlan | 20:40 |
| fungi | er, or at least inefficiencies | 20:40 |
| sean-k-mooney | so it shoud be fine | 20:40 |
| sean-k-mooney | ya its a partial mesh using ovs | 20:40 |
| sean-k-mooney | so all multicast becoms bradcast which si copied intrenlly t o all spokes in the mesh | 20:40 |
| fungi | i think vxlan relies on multicast groups underneath to carry broadcasts though? and our cloud providers don't let the multicast traffic through | 20:40 |
| sean-k-mooney | so its a littele inefficnet but it shoudl work | 20:40 |
| fungi | ah okay | 20:41 |
| sean-k-mooney | fungi: not with ovs | 20:41 |
| sean-k-mooney | but with linux bridge yes | 20:41 |
| sean-k-mooney | ovs only supprot unicast tunnels | 20:41 |
| fungi | you are far more familiar with all of this than i am | 20:41 |
| sean-k-mooney | but that actully makes it more portable since you do not rely on the cloud provdier supproting multicast grouping at the fabic level | 20:41 |
| fungi | all i can really say is afaik we didn't intentionally omit support for sending ipv6 packets over the bridge, so if it's not working i'd consider that a bug worth fixing | 20:42 |
| sean-k-mooney | it comes form addign ovs-dpdk suppreot neutron/nova and maintianing it for years:) | 20:42 |
| melwitt | sean-k-mooney: oh yeah, that makes sense. in pre, I was doing it for hosts: all but didn't put under both | 20:42 |
| fungi | `hosts: all` should run the same task on every test node (but not the zuul executor, which needs `hosts: localhost` instead, not really relevant to this playbook anyway) | 20:43 |
| sean-k-mooney | fungi: i have also impletend my own version fo the role 2 before although the most recent version i got ai to do :) | 20:45 |
| sean-k-mooney | fungi: https://github.com/SeanMooney/ard/tree/master/ansible/roles/ard_multinode_bridge does basically the same thign but it uses linux bridge and gre tunnels to avoid installign ovs | 20:46 |
| sean-k-mooney | i used to just use the zuul jobs one | 20:46 |
| sean-k-mooney | im not entrily sure im happy with how the new version works but i was tryign to make it compatiabe with our operator based isntaller and i dint want to fight with its installation and management of ovs | 20:48 |
| sean-k-mooney | that fact that ovs survies reboot makes the ovs based approch we use upstream less complex in many ways | 20:49 |
| sean-k-mooney | my the way the mtu caluation does have to be adjuscted for ipv6 but the role aslready subtratc 70 bytes which si enough to have ipv6 geneve trafic go over the bridge | 20:56 |
| sean-k-mooney | melwitt: https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/multi-node-bridge/tasks/common.yaml#L103-L139 | 20:56 |
| melwitt | hm ok | 20:58 |
| sean-k-mooney | techinaly the 70 bytes of overad is the bare minium for geneve to work over the bridge | 20:59 |
| sean-k-mooney | and it shoudl work for neutron in general | 20:59 |
| clarkb | its not geneve its vxlan iirc | 20:59 |
| clarkb | and that is why there isn't ipv6 | 21:00 |
| sean-k-mooney | i use a 100 byte safty buffer in my verions | 21:00 |
| fungi | sean-k-mooney: yeah, we originally used gre i believe, but it isn't allowed in all out providers | 21:00 |
| clarkb | there is/was work to convert it to geneve and drop the use of the ansible bridge module | 21:00 |
| sean-k-mooney | clarkb: yes the overly is | 21:00 |
| fungi | at least in some openstack clouds, the kernel conntrack plugin for gre isn't running | 21:00 |
| sean-k-mooney | clarkb: actully that not the part the matters | 21:00 |
| sean-k-mooney | we have other code in devstack playbooks to calulate teh neutron mtu config | 21:00 |
| sean-k-mooney | that the one that need to accomidate geneve whwen ovn is used | 21:01 |
| clarkb | https://review.opendev.org/c/zuul/zuul-jobs/+/959393 | 21:01 |
| sean-k-mooney | fungi: ya gre is not an ip protocol so its often blockded | 21:01 |
| clarkb | yes the original implementation used gre and linux bridges | 21:01 |
| sean-k-mooney | i depebated about gre or vxlan and for 8 bytpes of mtu vxlan is definly more portabl since its just udp form the network layer | 21:02 |
| clarkb | that didn't work because we couldn't get gre to pass even with wide open security groups so we switched to vxlan as it is udp | 21:02 |
| sean-k-mooney | clarkb: :) yep i remember | 21:02 |
| clarkb | and now ansible has dropped supprot for much of the machinery that we relied on to setup vxlan and ovs so we're switching back to linux bridge and taking the opportunity to use geneve in that change | 21:02 |
| clarkb | that work has clearly stalled out and probably should be picked up again. I think it is very close if not ready | 21:03 |
| clarkb | and the reason we didn't support ipv6 is taht netiher vxlan nor gre could support it iirc | 21:04 |
| clarkb | or maybe it was just vxlan that couldn't | 21:04 |
| sean-k-mooney | both can | 21:04 |
| clarkb | I think vxlan can now | 21:04 |
| sean-k-mooney | at both the underlay and over the mesh | 21:05 |
| clarkb | but it was a v4 only protocol | 21:05 |
| sean-k-mooney | it dependied on the kernel version | 21:05 |
| sean-k-mooney | and ovs version | 21:05 |
| clarkb | in any case if you want to make changes like this getting 959393 in first is probably a prereq | 21:05 |
| sean-k-mooney | vxlan suppoted it but we could supprot it on all distos we supproted if i recll | 21:05 |
| sean-k-mooney | well it does not stricly ned the refacto to linux bridge | 21:05 |
| sean-k-mooney | but that cerntly viable | 21:06 |
| clarkb | it does not but I will nto review a chnge that creatse a merge conflict | 21:06 |
| clarkb | maybe other people are willing to review it and land it | 21:06 |
| opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | 21:06 |
| sean-k-mooney | that is one of my version of that https://github.com/SeanMooney/ard/tree/master/ansible/roles/ard_multinode_bridge | 21:06 |
| sean-k-mooney | i have a much older version too somehwere | 21:06 |
| clarkb | the current code only works because I vendored the entirety of the ansible module that is no longer supported into that role | 21:06 |
| sean-k-mooney | ya i replaced all of tha twith a systemd unit file that calls a bashcript | 21:07 |
| clarkb | we really need to stop using that stuff and leaving it as is is fine. But if you want to make changes then we should do the right thing to fix the underlying ansible stuff | 21:07 |
| sean-k-mooney | https://github.com/SeanMooney/ard/blob/master/ansible/roles/ard_multinode_bridge/templates/ard-bridge-setup.sh.j2 | 21:07 |
| clarkb | and that work is basically done so there isn't any reason not to | 21:07 |
| sean-k-mooney | clarkb: ya https://review.opendev.org/c/zuul/zuul-jobs/+/959393 cerntely cleans up a lot | 21:08 |
| clarkb | rereading things it stalled beacuse people using ovs as a hard dependency weren't properly installing/settting up that dep and relying on that role instead | 21:08 |
| clarkb | I think we can basically say sorry you need to fix that and move forward | 21:09 |
| clarkb | which is also what I basically said back when I was trying to move this forward | 21:09 |
| sean-k-mooney | ya. is ther a reason it swapt to geneve? | 21:09 |
| sean-k-mooney | i know it says "This is a more modern and flexible approach than the | 21:09 |
| clarkb | I think because geneve is the future and more flexible | 21:09 |
| sean-k-mooney | previous vxlan implementation." | 21:09 |
| clarkb | right vxlan had a bunch of gotchas like the ipv6 support or lack of it | 21:10 |
| clarkb | geneve exists to solve all those problems with a new protocol | 21:10 |
| clarkb | so we should use it since those problems hit us previously | 21:10 |
| sean-k-mooney | the main one that geneve solve is actully fragmentation | 21:10 |
| sean-k-mooney | vxlan explity does not allow fragmentation | 21:10 |
| sean-k-mooney | of the vxlan packet | 21:10 |
| sean-k-mooney | geneve i belvie does | 21:10 |
| sean-k-mooney | which makes it more flexible acorss l3 subnets | 21:11 |
| sean-k-mooney | clarkb: but i guess i agree building on that patch woudl be good for the reason above | 21:12 |
| sean-k-mooney | anyway dinner is ready o/ | 21:12 |
| clarkb | ya I think the next step is to followup with ykarel to do what we can to properly capture ovs dependencies then proceed with the switch in zuul-jobs then add ipv6 support | 21:13 |
| clarkb | probably don't even need to make ipv6 support optional just add it | 21:13 |
| clarkb | ykarel: can you followup on https://review.opendev.org/c/zuul/zuul-jobs/+/959393 and indicate if properly managing jobs' dependency on ovs si a possibility? That really shouldn't be something we rely on a side effect from a base job role to handle | 21:14 |
| opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | 22:00 |
| opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | 22:51 |
| opendevreview | melanie witt proposed openstack/nova master: Add non-voting IPv6 ceph live migration job https://review.opendev.org/c/openstack/nova/+/984623 | 23:52 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!