| opendevreview | Jake Yip proposed openstack/magnum master: DNM: Test CI for v1.28.9 https://review.opendev.org/c/openstack/magnum/+/920725 | 07:57 |
|---|---|---|
| jakeyip | hi all, anyone around for meeting? | 08:40 |
| travisholton | Hi jakeyip, I'm around but I don't have anything for the agenda. | 08:45 |
| jakeyip | travisholton: cool :) | 08:46 |
| dalees | hi jakeyip - I've put a review into the agenda for some eyes, but nothing too much. | 08:56 |
| jakeyip | thanks dalees | 08:57 |
| jakeyip | #startmeeting magnum | 09:01 |
| opendevmeet | Meeting started Wed Jul 17 09:01:27 2024 UTC and is due to finish in 60 minutes. The chair is jakeyip. 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 |
| jakeyip | #link https://etherpad.opendev.org/p/magnum-weekly-meeting | 09:01 |
| jakeyip | Please put your topics into to Agenda | 09:01 |
| jakeyip | #topic Roll Call | 09:01 |
| jakeyip | o/ | 09:01 |
| dalees | o/ | 09:01 |
| travisholton | o/ | 09:01 |
| jakeyip | dalees: travisholton: thanks for coming! | 09:02 |
| jakeyip | #topic Review | 09:02 |
| jakeyip | https://review.opendev.org/c/openstack/magnum/+/906086 - control plane resize | 09:03 |
| jakeyip | I believe this is yours dalees :) | 09:03 |
| dalees | Yeah, pleased to get some feedback from mnaser on it. Awaiting his reply as to exactly what he means but otherwise it should be in reasonable shape for review | 09:04 |
| dalees | it works well with Heat (and does not permit cp resize), and helm driver (another patch to this is needed, once the above lands) | 09:05 |
| jakeyip | yeah I saw his notes, possibly he supports size >7? | 09:08 |
| jakeyip | I reviewed this previously, will take a look at the new PS | 09:08 |
| dalees | it's probably more the function signature - a driver can override that function and return any values it likes. | 09:09 |
| dalees | but yeah, i'm not sure. | 09:09 |
| jakeyip | ok | 09:10 |
| dalees | I'd appreciate the review to move it forward (without negating mnaser's feedback of course, until he has a chance to reply) | 09:11 |
| jakeyip | will do | 09:13 |
| jakeyip | anything else? | 09:13 |
| dalees | that's all today | 09:14 |
| jakeyip | ok my turn :P | 09:15 |
| jakeyip | https://review.opendev.org/c/openstack/magnum/+/920725 - need help with CI for v1.28.9 | 09:16 |
| jakeyip | oh hmm the test is passing now, wth | 09:16 |
| jakeyip | oh updated wrong thing | 09:17 |
| jakeyip | anyway there are two CI things I'm stuck with | 09:18 |
| jakeyip | one of them is getting heat driver passing for v1.28.9 | 09:18 |
| jakeyip | another one is getting capi working | 09:18 |
| jakeyip | anyone able to help with CAPI ? | 09:18 |
| dalees | was mnasiadka starting to work on CAPI at one point? it's quite a job to start up that CI with Zuul | 09:20 |
| jakeyip | I was hoping to catch up with him but he's been busy | 09:20 |
| mnasiadka | We have that task in our backlog, but don't count for it soon ;-) | 09:20 |
| jakeyip | hi mnasiadka :P | 09:20 |
| mnasiadka | hello | 09:20 |
| jakeyip | I am concentrating with getting heat v1.28.9 working | 09:21 |
| mnasiadka | I might have more time for Magnum in August | 09:21 |
| jakeyip | it's a Caracal promise | 09:21 |
| jakeyip | it's a D promise | 09:21 |
| jakeyip | it | 09:22 |
| jakeyip | it's working in our cloud but not in CI so a bit annoying | 09:22 |
| jakeyip | and the motivation to get heat working isn't great :D | 09:22 |
| jakeyip | just want to flag early to the team this might be missed, if I can't win CI | 09:24 |
| dalees | I've mostly cleaned Heat off my hands, here. | 09:25 |
| dalees | though the CI issue isn't really Heat specific, it's strange. | 09:26 |
| jakeyip | yeah it looks like issues talking to the API from inside VMs | 09:26 |
| jakeyip | we don't really test anything like create PVC in our test so that may never have worked | 09:27 |
| jakeyip | similarly we may run into it testing CAPI in CI too | 09:27 |
| jakeyip | let's go on | 09:29 |
| jakeyip | mnasiadka: have you seen this bug? https://bugs.launchpad.net/magnum/+bug/2067345 | 09:30 |
| jakeyip | seems like related to the changes for SQLA | 09:30 |
| mnasiadka | yeah, I thought it was too easy ;-) | 09:31 |
| mnasiadka | I can have a look in August if this doesn't get fixed by somebody else | 09:31 |
| opendevreview | Jake Yip proposed openstack/magnum master: DNM: Test CI for v1.28.9 https://review.opendev.org/c/openstack/magnum/+/920725 | 09:32 |
| jakeyip | mnasiadka: cool thanks | 09:33 |
| * dalees has to depart, apologies. | 09:33 | |
| jakeyip | I'll upgrade to Caracal for our cloud too and see if that turns up anything | 09:33 |
| jakeyip | bye dalees | 09:33 |
| jakeyip | I don't have anything else | 09:34 |
| jakeyip | anyone has anythign else? | 09:34 |
| travisholton | not me | 09:34 |
| jakeyip | #topic Open Discussion | 09:34 |
| mkjpryor | I only just managed to join due to another meeting, unfortunately | 09:34 |
| mkjpryor | Did I miss anything? | 09:34 |
| jakeyip | mkjpryor: I'm trying to persuade mnasiadka to do CAPI CI :P | 09:35 |
| mkjpryor | CI for the CAPI Helm driver is on our list of things to get to | 09:36 |
| jakeyip | mkjpryor: got a question - do you have patches to support AZ for servers and also maybe volumes | 09:36 |
| jakeyip | I realised instances don't have AZ set and also storageclass have wrong AZ set | 09:37 |
| mkjpryor | What exactly do you mean by volumes? Boot-from-volume? | 09:37 |
| mkjpryor | I see what you mean I think | 09:37 |
| mkjpryor | All those things are settable via the Helm charts. If you need them in the Magnum CAPI Helm driver, it should just be a case of a small patch that looks at the appropriate label or property on the Magnum cluster and updates the Helm values appropriately. | 09:38 |
| jakeyip | cool | 09:38 |
| jakeyip | TBH I haven't looked into this, it was just observed in our testing | 09:39 |
| jakeyip | will get to it in the next couple of weeks, sounds like it's easy | 09:39 |
| jakeyip | :) | 09:39 |
| mkjpryor | There are also some settings in the Helm charts to prevent CAPO from setting AZs at all | 09:39 |
| mkjpryor | Allowing placement to do its job | 09:40 |
| jakeyip | hm, setting az is still valid for placement filter | 09:44 |
| mkjpryor | Yes, but also not setting AZ is fine too | 09:44 |
| mkjpryor | I'm just saying both options are available | 09:44 |
| jakeyip | yeah, understand | 09:44 |
| jakeyip | cool I'll take a look | 09:45 |
| jakeyip | that's all from me | 09:46 |
| jakeyip | anything else? | 09:46 |
| mkjpryor | Just to say that we have started looking at filling out the paperwork to donate Azimuth and all its components to the CNCF sandbox | 09:48 |
| mkjpryor | This will include the addon provider, the janitor and the CAPI Helm charts from the stackhpc namespace | 09:49 |
| jakeyip | nice | 09:49 |
| travisholton | cool | 09:49 |
| mkjpryor | I know the CAPI Helm charts have already been forked into an opendev repo, but I was sort of hoping that this would give us a suitable vendor-agnostic home for the CAPI Helm charts? | 09:49 |
| mkjpryor | One where we don't have to rewrite all the automation that we have | 09:50 |
| mkjpryor | :D | 09:50 |
| mkjpryor | How would people feel about Magnum consuming the CAPI Helm charts from a CNCF project as the default? It feels OK to me. | 09:50 |
| mkjpryor | But I am biased | 09:51 |
| mkjpryor | :D | 09:51 |
| jakeyip | yeah this will prob help your case ;) | 09:51 |
| mkjpryor | The repos will be being moved into a new GitHub org this week | 09:52 |
| jakeyip | have you talked to the folks who maintain OCCM ? | 09:52 |
| mkjpryor | azimuth-cloud | 09:52 |
| mkjpryor | Not specifically about CNCF membership. Our membership would be slightly different as OCCM belongs to the Kubernetes project whereas we want a new project. | 09:53 |
| jakeyip | I see | 09:53 |
| travisholton | I think it sounds good. | 09:55 |
| mkjpryor | Just a heads up anyway | 09:55 |
| mkjpryor | We are hoping to do the paperwork over the next few weeks | 09:56 |
| mkjpryor | Then we'll see whether they accept us | 09:56 |
| travisholton | so after next week we should clone/fork from the new repo | 09:57 |
| jakeyip | do keep us updated :) | 09:58 |
| jakeyip | I'll end the meeting now, since we are almost at time | 09:58 |
| jakeyip | #endmeeting | 09:58 |
| opendevmeet | Meeting ended Wed Jul 17 09:58:38 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:58 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/magnum/2024/magnum.2024-07-17-09.01.html | 09:58 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/magnum/2024/magnum.2024-07-17-09.01.txt | 09:58 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/magnum/2024/magnum.2024-07-17-09.01.log.html | 09:58 |
| jakeyip | feel free to stay around | 09:58 |
| mkjpryor | travisholton - yes. I think GitHub is quite good at setting up redirects. | 10:00 |
| * travisholton departs. See you all later! | 10:02 | |
| jakeyip | bye all! | 10:15 |
| opendevreview | Jake Yip proposed openstack/magnum master: DNM: Test CI for v1.28.9 https://review.opendev.org/c/openstack/magnum/+/920725 | 10:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!