| opendevreview | Merged openstack/magnum master: docs: Remove references to wiki https://review.opendev.org/c/openstack/magnum/+/897221 | 06:48 |
|---|---|---|
| opendevreview | Merged openstack/magnum master: docs: Change Storyboard links to Launchpad https://review.opendev.org/c/openstack/magnum/+/897223 | 06:48 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 07:06 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 08:09 |
| jakeyip | hi all meeting in a couple of minutes. feel free to add to agenda https://etherpad.opendev.org/p/magnum-weekly-meeting | 08:58 |
| jakeyip | #startmeeting magnum | 09:00 |
| opendevmeet | Meeting started Wed Oct 4 09:00:33 2023 UTC and is due to finish in 60 minutes. The chair is jakeyip. Information about MeetBot at http://wiki.debian.org/MeetBot. | 09:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 09:00 |
| opendevmeet | The meeting name has been set to 'magnum' | 09:00 |
| jakeyip | Agenda: | 09:00 |
| jakeyip | #link https://etherpad.opendev.org/p/magnum-weekly-meetingy | 09:00 |
| jakeyip | #topic Roll Call | 09:01 |
| dalees | o/ | 09:01 |
| jakeyip | o/ | 09:01 |
| mnasiadka | o/ | 09:01 |
| jakeyip | let's wait 5 mins | 09:02 |
| dalees | hi mkjpryor , currently rollcall for meeting if you're here to join/listen in. | 09:08 |
| dalees | (we're pausing 5min for others to join) | 09:08 |
| mkjpryor | :) | 09:09 |
| jakeyip | hi mkjpryor | 09:09 |
| jakeyip | alright let's have a quick CAPI update | 09:10 |
| mkjpryor | from our side, we are now working mostly in our out-of-tree driver | 09:11 |
| jakeyip | #topic CAPI | 09:11 |
| mkjpryor | I think JohnG is attempting to keep the upstream patches in sync | 09:11 |
| dalees | mkjpryor: thanks from us, we're using and testing with it. It builds and upgrades clusters :) | 09:12 |
| mkjpryor | That's good! It didn't for a while during the refactor :) | 09:12 |
| jakeyip | mkjpryor: did I break it? :P | 09:12 |
| dalees | yup, i noticed. John fixed things before I got the PRs created :) | 09:12 |
| mkjpryor | I have also been making changes to the underlying Helm charts to improve the CI | 09:13 |
| jakeyip | nice | 09:13 |
| mkjpryor | So we now test upgrading between Kubernetes versions and chart versions in the CI | 09:13 |
| mkjpryor | As of yesterday, we have added an explicit approval step that means we are now more comfortable running the CI on external PRs as well | 09:14 |
| dalees | mkjpryor: we'd be keen to keep the flatcar support PR moving, it's our primary target for deployment. I'll add a patch for the driver that will go alongside (and be updated as the chart changes to a better model from your suggestions) | 09:14 |
| mkjpryor | dalees: There are some changes about to merge that change the way we construct the kubeadmConfigSpec | 09:14 |
| mkjpryor | Basically, the spec can now be built in layers that are merged together instead of having to pull out bits and render them later | 09:15 |
| mkjpryor | It should make the Flatcar and Keystone webhook patches much simpler | 09:16 |
| dalees | great, we'll keep an eye out and look to rebase the PRs onto that. | 09:16 |
| mkjpryor | :) | 09:17 |
| mkjpryor | We also have another external contributor adding Manila support | 09:18 |
| mkjpryor | And we support 1.28 :) | 09:18 |
| mkjpryor | I have also been working on a patch for CAPO to allow us to have additional data volumes for machines. The primary use case for this is to put etcd on a different disk, for sites that only have a small pool of SSD. | 09:19 |
| mkjpryor | There will be a follow-up patch to allow making use of the ephemeral disk for a flavor, so etcd can be put on that while the root volume comes from Cinder. | 09:20 |
| dalees | cool - I have a patch that will go up soon to set the root volume type and size within Magnum (just sending through values that Magnum config already has settings for) | 09:21 |
| mkjpryor | Sounds good | 09:22 |
| jakeyip | is that from the current labels like docker_volume_size, ? | 09:23 |
| dalees | no, config file - CONF.cinder.default_boot_volume_size and CONF.cinder.default_boot_volume_type | 09:23 |
| dalees | there is more detail there my patch misses, i suspect. Especially with multiple volumes | 09:24 |
| jakeyip | ah ok. | 09:25 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: devstack: Install sonobuoy and kubectl https://review.opendev.org/c/openstack/magnum/+/893823 | 09:26 |
| jakeyip | keep chatting... :D | 09:28 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: Add kubernetes fedora coreos v1 jobs https://review.opendev.org/c/openstack/magnum/+/896244 | 09:29 |
| dalees | mkjpryor: got a question for you about the cluster-api-addon-provider - we're keen to get it supporting `oci://` hosted charts, which might mean a library change to something that supports helm3. any thoughts before we look closer? | 09:30 |
| jakeyip | So we met with mnaser. We talked about supporting VEXXHOST as an out of tree driver and how CI will work. He made good points about how Kubernetes development is now in Github and it's easier to attract devs there. | 09:32 |
| jakeyip | I think the plan may be to shrink Magnum into an API shell with the ability to support multiple drivers. So we do away with all the issues that has been crippling Magnum. | 09:34 |
| jakeyip | whether we will have an in tree reference CAPI driver or not is still unclear. first we have to find an elegant solution to support multiple drivers. | 09:36 |
| jakeyip | mnasiadka think this will require a spec and I agree. | 09:39 |
| jakeyip | anything else for CAPI? | 09:41 |
| dalees | for the capi helm driver, all the manifests live in the chart repo, and the magnum template can specify the version. It may be that with this, an old Magnum can support a very new k8s with no release or code changes. | 09:42 |
| dalees | ^ referring to the driver model (whether it's in tree or out, i suppose) | 09:42 |
| dalees | nothing else on CAPI from me. | 09:43 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 09:43 |
| jakeyip | nice, thanks | 09:44 |
| jakeyip | #topic CI | 09:44 |
| mnasiadka | Yay | 09:44 |
| jakeyip | as we can all see mnasiadka is hard at work doing CI, and it's coming together. very exciting | 09:44 |
| mnasiadka | I've been working on a patch | 09:45 |
| mnasiadka | #link https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 09:45 |
| mnasiadka | there's a depends-on that downloads kubectl and sonobuoy | 09:45 |
| mnasiadka | #link https://review.opendev.org/c/openstack/magnum/+/893823 | 09:45 |
| mnasiadka | and then finally there's a patch to magnum repo to use those jobs with different kube_tags | 09:45 |
| mnasiadka | #link https://review.opendev.org/c/openstack/magnum/+/896244 | 09:45 |
| mnasiadka | For now those jobs don't use a load balancer - but will follow up after it gets merged with using OVN LBs to save on memory in the runner nodes (amphora VMs are quite big) | 09:46 |
| mnasiadka | And then add sonobuoy, and add multinode | 09:46 |
| mnasiadka | And then we'll get into hell of maintaining those jobs and catching out things that fail :) | 09:47 |
| dalees | nice work mnasiadka | 09:47 |
| jakeyip | great work :) | 09:47 |
| mnasiadka | I was thinking for starters we could have two jobs - one for 1.26 and one for 1.27 (since that's in our support matrix for B) | 09:48 |
| mnasiadka | and each of those versions with a flannel and calico variant | 09:48 |
| mnasiadka | any comments? ideas? | 09:49 |
| jakeyip | sounds good. I'm anxious about carbon footprint of our tests; they take so long | 09:50 |
| mnasiadka | You haven't seen other projects testing :) | 09:52 |
| jakeyip | if only devstack can be containerised... | 09:52 |
| mkjpryor | I would like a devstack that runs in a standard GitHub runner :) | 09:53 |
| jakeyip | I'd be happy to stick with calico only and expand if there is a need. compared to what we had which is nothing, it's already great | 09:54 |
| mkjpryor | But there would be no space left for test workloads unfortunately | 09:54 |
| mnasiadka | mkjpryor: vexxhost is using paid github actions nodes for that... | 09:54 |
| jakeyip | hahaha | 09:54 |
| mnasiadka | so just do a Calico version of the CI jobs? fine by me | 09:54 |
| mnasiadka | I never liked flannel :) | 09:55 |
| dalees | +1 Calico | 09:55 |
| jakeyip | we use fannel but +1 calico :P | 09:55 |
| * dalees eyes up Cilium... one day. | 09:55 | |
| jakeyip | yeah I am interested too | 09:56 |
| jakeyip | CAPI can do it? | 09:56 |
| dalees | CAPI can do anything | 09:56 |
| dalees | yeah, it's just a CNI so it *can*; not sure if it's in the helm charts yet though. Vexxhost driver is calico only for now. | 09:57 |
| jakeyip | goody | 09:57 |
| jakeyip | ok. think we are at the end. anyone has anything else? | 09:57 |
| jakeyip | #topic Open Discussion | 09:57 |
| jakeyip | I think matt let so nothing else, let's end the meeting before the hour :) | 09:58 |
| dalees | all good, thanks | 09:58 |
| jakeyip | #endmeeting | 09:58 |
| opendevmeet | Meeting ended Wed Oct 4 09:58:48 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:58 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/magnum/2023/magnum.2023-10-04-09.00.html | 09:58 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/magnum/2023/magnum.2023-10-04-09.00.txt | 09:58 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/magnum/2023/magnum.2023-10-04-09.00.log.html | 09:58 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: Add kubernetes fedora coreos v1 jobs https://review.opendev.org/c/openstack/magnum/+/896244 | 11:16 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: Add kubernetes fedora coreos v1 jobs https://review.opendev.org/c/openstack/magnum/+/896244 | 11:18 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: WIP: Enable SQLAlchemy 2.x warnings https://review.opendev.org/c/openstack/magnum/+/886452 | 11:43 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: WIP: Enable SQLAlchemy 2.x warnings https://review.opendev.org/c/openstack/magnum/+/886452 | 12:09 |
| opendevreview | Alfredo Moralejo proposed openstack/magnum stable/2023.2: sqlalchemy: Import String from sqlalchemy directly https://review.opendev.org/c/openstack/magnum/+/897286 | 13:01 |
| opendevreview | Jake Yip proposed openstack/magnum master: Add kubernetes fedora coreos v1 jobs https://review.opendev.org/c/openstack/magnum/+/896244 | 13:10 |
| opendevreview | Jake Yip proposed openstack/magnum master: Add kubernetes fedora coreos v1 jobs https://review.opendev.org/c/openstack/magnum/+/896244 | 13:37 |
| opendevreview | Merged openstack/magnum stable/2023.2: sqlalchemy: Import String from sqlalchemy directly https://review.opendev.org/c/openstack/magnum/+/897286 | 14:21 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 14:52 |
| opendevreview | Michal Nasiadka proposed openstack/magnum master: Add kubernetes fedora coreos v1 jobs https://review.opendev.org/c/openstack/magnum/+/896244 | 14:53 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 16:25 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 18:38 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 19:23 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 19:24 |
| opendevreview | Michal Nasiadka proposed openstack/magnum-tempest-plugin master: Add job for cluster tests https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/893131 | 19:30 |
| opendevreview | Felipe Reyes proposed openstack/magnum master: Validate fixed_network and fixed_subnet https://review.opendev.org/c/openstack/magnum/+/897362 | 20:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!