opendevreview | Stavros Moiras proposed openstack/python-magnumclient master: Cluster config update https://review.opendev.org/c/openstack/python-magnumclient/+/788689 | 02:17 |
---|---|---|
strigazi | #startmeeting magnum | 09:01 |
opendevmeet | Meeting started Wed Feb 23 09:01:23 2022 UTC and is due to finish in 60 minutes. The chair is strigazi. Information about MeetBot at http://wiki.debian.org/MeetBot. | 09:01 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 09:01 |
opendevmeet | The meeting name has been set to 'magnum' | 09:01 |
strigazi | #topic Roll Call | 09:01 |
strigazi | o/ | 09:01 |
jakeyip | o/ | 09:01 |
gbialas_ | O/ | 09:01 |
mnasiadka | o/ | 09:01 |
strigazi | #topic Release final python-magnumclient for Yoga | https://review.opendev.org/c/openstack/releases/+/830241 | 09:03 |
strigazi | This needs to be merged by the 25th. | 09:04 |
strigazi | I would like to ask you for some help to include 788689: Cluster config update | https://review.opendev.org/c/openstack/python-magnumclient/+/788689 | 09:04 |
strigazi | I'm trying to follow it up as well | 09:05 |
strigazi | #topic Review action items | 09:07 |
jakeyip | ok I tested PS57, it was working OK but was just failing CI | 09:07 |
strigazi | change the default hyperkube to the rancher build, done: 830479: [k8s-coreos] Default hyperkube_prefix to rancher | https://review.opendev.org/c/openstack/magnum/+/830479 | 09:07 |
strigazi | jakeyip: thanks, I will try to do another round of review as well. | 09:09 |
jakeyip | ok, kube_tag and master_kube_tag is only used for hyperkube and nothing else? | 09:10 |
strigazi | yes | 09:10 |
opendevreview | Spyros Trigazis proposed openstack/magnum master: [k8s-coreos] Default hyperkube_prefix to rancher https://review.opendev.org/c/openstack/magnum/+/830479 | 09:12 |
strigazi | jakeyip: here as well but of the hyperkube https://github.com/openstack/magnum/blob/master/magnum/drivers/common/templates/kubernetes/fragments/upgrade-kubernetes.sh | 09:13 |
jakeyip | I'm not super familiar with code, what about https://github.com/openstack/magnum/blob/master/magnum/drivers/common/templates/kubernetes/fragments/configure-kubernetes-minion.sh#L157 | 09:14 |
strigazi | jakeyip: for fedora-coreos this is not used | 09:14 |
strigazi | see https://github.com/openstack/magnum/blob/master/magnum/drivers/common/templates/kubernetes/fragments/configure-kubernetes-minion.sh#L65 | 09:15 |
strigazi | jakeyip: change magnum/api/validation.py#L259-L260 to not allow mesos as a coe option, can you push a patch? | 09:16 |
jakeyip | ok like I say I'm not super familiar, I trust you | 09:16 |
jakeyip | sure, I will do that | 09:16 |
strigazi | jakeyip: in fedora coreos, "atomic install" does not exist a | 09:17 |
strigazi | jakeyip: in fedora coreos, "atomic install" does not exist at all | 09:17 |
jakeyip | oh yeah right we used podman | 09:17 |
strigazi | #action jakeyip to change magnum/api/validation.py#L259-L260 to not allow mesos as a coe option | 09:18 |
strigazi | #action strigazi 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:19 |
strigazi | #topic Yoga release | 09:19 |
strigazi | mnasiadka: gbialas_: jakeyip: is something else we want to update for Yoga? | 09:20 |
gbialas_ | Not today :) | 09:20 |
strigazi | We still have a couple of weeks | 09:21 |
mnasiadka | let me check pending patches... | 09:21 |
mnasiadka | OVN LB related patches: | 09:21 |
mnasiadka | https://review.opendev.org/c/openstack/magnum/+/764444 | 09:21 |
mnasiadka | https://review.opendev.org/c/openstack/magnum/+/765309 | 09:21 |
mnasiadka | https://review.opendev.org/c/openstack/magnum/+/767119 | 09:21 |
mnasiadka | And a patch related to allowed_address_pairs bug in ML2/OVN - https://review.opendev.org/c/openstack/magnum/+/773923 | 09:22 |
strigazi | mnasiadka: ACK | 09:22 |
mnasiadka | would be nice to get these in :) | 09:22 |
strigazi | #action strigazi to review https://review.opendev.org/c/openstack/magnum/+/764444 https://review.opendev.org/c/openstack/magnum/+/765309 https://review.opendev.org/c/openstack/magnum/+/767119 https://review.opendev.org/c/openstack/magnum/+/773923 | 09:22 |
strigazi | #action jakeyip to review https://review.opendev.org/c/openstack/magnum/+/764444 https://review.opendev.org/c/openstack/magnum/+/765309 https://review.opendev.org/c/openstack/magnum/+/767119 https://review.opendev.org/c/openstack/magnum/+/773923 | 09:23 |
strigazi | ( jakeyip I see you are not added in the magnum-core group yet, I will ping the ML thread) | 09:23 |
strigazi | And for Yoga, | 09:24 |
strigazi | #topic Conformance for 1.24 in Yoga | 09:24 |
strigazi | mnasiadka: gbialas_: Do you want to have a look at it? | 09:24 |
strigazi | We need to submit a patch with the result of the conformance test to have magnum certified | 09:25 |
strigazi | #link https://www.cncf.io/certification/software-conformance/ | 09:25 |
strigazi | #action strigazi to follow up kubernetes conformance for Yoga https://www.cncf.io/certification/software-conformance/ | 09:29 |
jakeyip | strigazi: not in hurry, but if quota fixes can be in for Yoga that'll be good too | 09:29 |
mnasiadka | strigazi: sure, we can look into that | 09:30 |
strigazi | #undo | 09:30 |
opendevmeet | Removing item from minutes: #action strigazi to follow up kubernetes conformance for Yoga https://www.cncf.io/certification/software-conformance/ | 09:30 |
strigazi | #action mnasiadka to follow up kubernetes conformance for Yoga https://www.cncf.io/certification/software-conformance/ | 09:30 |
strigazi | #action strigazi to review https://review.opendev.org/c/openstack/magnum/+/764254 https://review.opendev.org/c/openstack/magnum/+/764397 | 09:30 |
strigazi | I think that's it for past items | 09:31 |
strigazi | #topic Open Discussion | 09:31 |
jakeyip | just off the top of my mind, is atomic something that should be deprecated? | 09:31 |
strigazi | jakeyip: yes | 09:32 |
strigazi | it's replaced by fedora-coreos | 09:32 |
strigazi | jakeyip: want to do it? similar to mesos? | 09:32 |
jakeyip | yeah I am in favour of removing code, too much legacy code gets confusing. | 09:33 |
jakeyip | might not be in time for yoga, not sure | 09:33 |
strigazi | #action jakeyip to deprecate fedora-atomic driver(s) | 09:33 |
strigazi | feel free to push a plaholder patch at least and we see | 09:34 |
jakeyip | ok | 09:34 |
strigazi | all, any other business? | 09:35 |
strigazi | let's close then, see you next week! | 09:39 |
strigazi | #endmeeting | 09:39 |
opendevmeet | Meeting ended Wed Feb 23 09:39:29 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:39 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-02-23-09.01.html | 09:39 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-02-23-09.01.txt | 09:39 |
opendevmeet | Log: https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-02-23-09.01.log.html | 09:39 |
jakeyip | thanks strigazi | 09:39 |
opendevreview | Grzegorz Bialas proposed openstack/magnum master: Fix disabled SMT on Fedora CoreOS https://review.opendev.org/c/openstack/magnum/+/829519 | 09:48 |
opendevreview | Michal Nasiadka proposed openstack/magnum master: [k8s-coreos] Default hyperkube_prefix to rancher https://review.opendev.org/c/openstack/magnum/+/830479 | 10:17 |
*** ramishra_ is now known as ramishra | 11:11 | |
opendevreview | Jake Yip proposed openstack/magnum master: Remove mesos API validation https://review.opendev.org/c/openstack/magnum/+/830594 | 11:24 |
opendevreview | Daniel Meyerholt proposed openstack/magnum master: CoreDNS support EndpointSlices https://review.opendev.org/c/openstack/magnum/+/830603 | 13:07 |
*** gmann is now known as gmann_afk | 15:59 | |
*** gmann_afk is now known as gmann | 16:17 | |
opendevreview | Stavros Moiras proposed openstack/python-magnumclient master: Cluster config update https://review.opendev.org/c/openstack/python-magnumclient/+/788689 | 17:09 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!