| *** ykarel_ is now known as ykarel | 06:39 | |
| opendevreview | Merged openstack/magnum master: Fix POD to POD networking with ML2/OVN https://review.opendev.org/c/openstack/magnum/+/773923 | 08:09 |
|---|---|---|
| opendevreview | Merged openstack/magnum master: Remove temporal workaround to increase quota in Glance https://review.opendev.org/c/openstack/magnum/+/802541 | 08:09 |
| opendevreview | Merged openstack/magnum master: Quota deletion bug fix https://review.opendev.org/c/openstack/magnum/+/764397 | 08:09 |
| opendevreview | Merged openstack/magnum master: CoreDNS support EndpointSlices https://review.opendev.org/c/openstack/magnum/+/830603 | 08:09 |
| strigazi | #startmeeting magnum | 08:59 |
| opendevmeet | Meeting started Wed Mar 2 08:59:26 2022 UTC and is due to finish in 60 minutes. The chair is strigazi. Information about MeetBot at http://wiki.debian.org/MeetBot. | 08:59 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:59 |
| opendevmeet | The meeting name has been set to 'magnum' | 08:59 |
| strigazi | #topic Roll Call | 08:59 |
| strigazi | o/ | 08:59 |
| jakeyip | o/ | 09:00 |
| mnasiadka | o/ | 09:02 |
| bbezak | o/ | 09:02 |
| gbialas | o/ | 09:04 |
| strigazi | #topic Previous Action Items | 09:04 |
| strigazi | #link https://etherpad.opendev.org/p/magnum-weekly-meeting | 09:04 |
| strigazi | jakeyip: did you manage to change the validation for mesos? | 09:05 |
| jakeyip | yeah I've sent up a very simple patch | 09:06 |
| jakeyip | https://review.opendev.org/c/openstack/magnum/+/830594 - works and just complains that `Requested COE type %s is not supported` if we use mesos | 09:07 |
| strigazi | cool, maybe add a release note too | 09:08 |
| jakeyip | ok | 09:09 |
| strigazi | #action jakeyip, strigazi follow 830594: Remove mesos API validation | https://review.opendev.org/c/openstack/magnum/+/830594 | 09:11 |
| strigazi | next was to comment on mesos, baymodel removal for Z. I will add it today. | 09:12 |
| strigazi | #action trigazi to comment on 821213: Drop mesos driver | https://review.opendev.org/c/openstack/magnum/+/821213 , 803780: Drop bay and baymodel from controllers | https://review.opendev.org/c/openstack/magnum/+/803780 , 803629: Drop bay and baymodel | https://review.opendev.org/c/openstack/python-magnumclient/+/803629 | 09:12 |
| strigazi | Next two items are from mnasiadka's for reviews | 09:13 |
| strigazi | mnasiadka: I managed to test and merge the OVN pod-to-pod patch https://review.opendev.org/c/openstack/magnum/+/773923 | 09:13 |
| mnasiadka | yes, thanks for that :) | 09:13 |
| strigazi | for the three octavia patches. the code lgtm but my devstack was not cooperating. I think today or tomorrow we should merge them | 09:14 |
| mnasiadka | We're using that downstream in production for some time, so those should be working :) | 09:14 |
| strigazi | jakeyip: did you have time to look as well? it is this series: https://review.opendev.org/c/openstack/magnum/+/767119/ | 09:15 |
| strigazi | mnasiadka: ok :) | 09:15 |
| jakeyip | same, I am having devstack issues, so still trying to get it | 09:15 |
| jakeyip | mnasiadka: do your clusters use octavia ingress? does it work with OVN? | 09:15 |
| mnasiadka | No, we use nginx ingress, octavia ingress controller does not support OVN LB | 09:16 |
| mnasiadka | We weren't very happy around octavia ingress controller - but I don't remember why | 09:16 |
| mnasiadka | bbezak: do you remember? | 09:16 |
| mnasiadka | basically, OVN LB supports only TCP and UDP mode balancing, so we can't use octavia-ingress-controller for this | 09:17 |
| strigazi | mnasiadka: you also want proxy? | 09:17 |
| mnasiadka | strigazi: proxy? as in an Octavia/OVN load balancer in front on the nginx ingress controller? | 09:18 |
| jakeyip | mnasiadka: i see. so is ovn octavia only for the api / etcd ? | 09:19 |
| strigazi | mnasiadka PROXY protocol | 09:20 |
| mnasiadka | jakeyip: you can use loadbalancer service via occm | 09:20 |
| strigazi | mnasiadka: ah, actually its HTTP(S) that is missing too :) | 09:21 |
| jakeyip | ah ok | 09:21 |
| mnasiadka | yes, https needs to be terminated on the service itself (OVN is only TCP/UDP load balancer, no HTTP) | 09:21 |
| mnasiadka | no PROXY, no fancy features | 09:21 |
| mnasiadka | but also no Amphora VM :) | 09:22 |
| strigazi | mnasiadka: we use TungstenFabric that has no VMs either, it manages haproxy processes though | 09:22 |
| mnasiadka | I'm familiar with TungstenFabric, OVN LBs are purely OpenFlow rules | 09:23 |
| mnasiadka | like an iptables load balancer | 09:23 |
| mnasiadka | but it has healthchecks now (if you use OVN 21.06), so a bit better than iptables | 09:23 |
| strigazi | yeah yeah, for many things that's more optimal, does it keep the client-ip? | 09:24 |
| jakeyip | strigazi: does CERN use OVN? | 09:24 |
| mnasiadka | strigazi: yes, client ip is seen on the connection target, no fancy headers needed | 09:25 |
| strigazi | jakeyip: no, just Tungsten for LBs at the moment | 09:25 |
| jakeyip | ok. we are migrating to OVN so this patches will be helpful for us, thanks :) | 09:25 |
| strigazi | mnasiadka: cool, because with iptable (eg kube-proxy) you lose it | 09:25 |
| strigazi | Let's move on, next was: | 09:26 |
| opendevreview | Merged openstack/magnum master: Support quota hard_limit values of zero https://review.opendev.org/c/openstack/magnum/+/764254 | 09:26 |
| strigazi | mnasiadka to follow up kubernetes conformance for Yoga https://www.cncf.io/certification/software-conformance/ | 09:26 |
| strigazi | did you have time to look at it last week? | 09:27 |
| mnasiadka | started to set up an env for this, so should file a PR in the regular repo in the coming days | 09:27 |
| strigazi | mnasiadka: awesome! So it passes right? | 09:27 |
| mnasiadka | yes, sonobuoy has no errors, so just need to gather the logs and fire off a pull request in github | 09:28 |
| strigazi | #action strigazi to review Octavia Patches https://review.opendev.org/c/openstack/magnum/+/764444 https://review.opendev.org/c/openstack/magnum/+/765309 https://review.opendev.org/c/openstack/magnum/+/767119 | 09:28 |
| strigazi | mnasiadka: That's great, thanks | 09:28 |
| strigazi | #action mnasiadka to follow up kubernetes conformance for Yoga https://www.cncf.io/certification/software-conformance/ | 09:29 |
| strigazi | next, was the quota patches, both merged, one of them seconds ago | 09:29 |
| strigazi | and last is jakeyip to deprecate fedora-atomic driver(s) | 09:29 |
| strigazi | jakeyip: ^^ you managed? | 09:30 |
| jakeyip | yeah will do it similarly to mesos | 09:31 |
| jakeyip | haven't got started | 09:31 |
| strigazi | #action jakeyip to deprecate fedora-atomic driver(s) | 09:31 |
| strigazi | thanks | 09:31 |
| strigazi | I'm adding one more patch to have it ready for next week: | 09:31 |
| strigazi | #action strigazi, jakeyip to review 775793: Support extra_network and extra_subnet labels | https://review.opendev.org/c/openstack/magnum/+/775793 | 09:32 |
| strigazi | #topic Install calico via helm | 09:32 |
| strigazi | Recently, we updated calico, to the update more managable helm would help, as anyone looke at the tigera operator recently? | 09:33 |
| strigazi | I think i discussed this with someone from stackHPC | 09:33 |
| bbezak | mnasiadka: Octavia ingress controller was a bumpy road last time I've looked into it, and not much maintainers on it. Ingress-nginx is a out of the box experience. | 09:33 |
| strigazi | #link https://github.com/tigera/operator | 09:34 |
| bbezak | It looks like we can bump coredns to 1.8.* versions after this merged in apparently - https://review.opendev.org/c/openstack/magnum/+/830603 | 09:34 |
| strigazi | bbezak: yeap | 09:35 |
| strigazi | For calico, do you think it makes sense to have our own helm-chart? | 09:38 |
| strigazi | jakeyip: mnasiadka: you use calico, flannel or smth else? | 09:39 |
| mnasiadka | we use mainly calico | 09:40 |
| mnasiadka | but we haven't tried the tigera operator | 09:40 |
| strigazi | mnasiadka: the main issue right now is that we can configure it to use our registry | 09:41 |
| mnasiadka | but I guess it would make sense to try it out, but I don't know if we can commit to doing that before Yoga release | 09:41 |
| jakeyip | we use flannel - there is a problem with using calico with our current SDN. will evaluate calico again when we migrate to OVN | 09:41 |
| mnasiadka | strigazi: right | 09:41 |
| strigazi | let's leave it for Z then | 09:42 |
| strigazi | #topic Open Discussion | 09:42 |
| strigazi | Any other business? | 09:43 |
| strigazi | See you next week then! | 09:46 |
| strigazi | #endmeeting | 09:46 |
| opendevmeet | Meeting ended Wed Mar 2 09:46:03 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:46 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-03-02-08.59.html | 09:46 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-03-02-08.59.txt | 09:46 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-03-02-08.59.log.html | 09:46 |
| mnasiadka | thanks! | 09:51 |
| jakeyip | thanks! | 09:54 |
| *** ykarel_ is now known as ykarel | 09:54 | |
| opendevreview | Jake Yip proposed openstack/magnum master: Remove mesos API validation https://review.opendev.org/c/openstack/magnum/+/830594 | 10:02 |
| *** ykarel_ is now known as ykarel | 14:17 | |
| *** ykarel is now known as ykarel|away | 15:40 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!