08:11:01 <dalees> #startmeeting magnum
08:11:01 <opendevmeet> Meeting started Tue May 27 08:11:01 2025 UTC and is due to finish in 60 minutes.  The chair is dalees. Information about MeetBot at http://wiki.debian.org/MeetBot.
08:11:01 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
08:11:01 <opendevmeet> The meeting name has been set to 'magnum'
08:11:15 <dalees> #topic Roll Call
08:11:17 <dalees> o/
08:11:29 <jakeyip> o/
08:12:28 <dalees> any others? feel free to say hello later on if you join in, also!
08:12:47 <dalees> #topic Reviews
08:13:21 <dalees> A couple in the agenda for discussion
08:13:40 <dalees> shall we start with the master lb fip patchests?
08:13:53 <jakeyip> ok
08:14:02 <dalees> mine is duplicate, I'll abandon it as the other merges.
08:14:25 <dalees> I posted a review to the other, needs a few small changes
08:14:35 <dalees> https://review.opendev.org/c/openstack/magnum-capi-helm/+/933619
08:15:26 <jakeyip> yeap I saw that :)
08:15:58 <dalees> I could add my test case in there, and make the changes? But I'll give Piotr a bit more time to update it
08:17:12 <jakeyip> yeap. if that times out then you can combine his and your patches in your change and add him as a co-author. I did that for the change to unblock our ci
08:17:34 <jakeyip> they get credit with the co-author tag
08:18:25 <dalees> ah, so make a new one so I don't stand on toes in that one, and co-author? Yeah I'll give it another week or so and do that.
08:19:24 <dalees> the other review on the agenda is https://review.opendev.org/c/openstack/magnum-capi-helm/+/950806
08:20:04 <dalees> I just posted a review there, but happy to discuss further. Thanks for adding the compatibility table, we need to write that knowledge down.
08:20:47 <jakeyip> yeap I saw your comments on that one. it's very raw I must admit, I was planning on getting something down first then modifying it, I still hasn't worked it all out in my head
08:21:37 <dalees> perhaps separate the compatibility table from the code change?
08:22:58 <jakeyip> maybe, but the current table was really documenting what is the min version that works with alpha7 since that is the change
08:23:38 <jakeyip> the final table should actually have min-max versions. and also lead to an upgrade doc. and that gets more complicated...
08:25:01 <dalees> ah, yes. I'm fine with combined or separate then.
08:26:47 <jakeyip> I am not sure how the final table should look like though. or should we work backwards, with an idea what is the upgrade path?
08:34:22 <dalees> min-max versions where we know them is a reasonable start. There is also Magnum versions and CAPI version, but I don't want to write a huge matrix. It will be hard to keep updated as it is
08:38:05 <jakeyip> yeah I agree, it must be easy for users / operators!
08:39:12 <dalees> any more on that topic?  We can move to open discussion if there is anything else.
08:39:19 <jakeyip> no
08:39:25 <dalees> #topic Open Discussion
08:40:30 <jakeyip> prob need to catch up at some point on how to get ci working with magnum-capi-helm
08:42:01 <jakeyip> and also wondering if anyone has looked into eventlet chage
08:44:43 <dalees> No, I've been working on updating autoscaling values though; will be submitting some changes soon for magnum-capi-helm, and possibly Magnum (but I need to make them driver independent).
08:45:31 <dalees> but on the topic of CI - we could progress the helm chart repo snapshot and publish perhaps.
08:47:31 <jakeyip> how is the autoscaling values fit with the labels proposal ?
08:48:54 <dalees> nodegroups have `min_node_count` and `max_node_count` fields on them. These are being used over the labels of the same name, and they can be mutable without the labels being so.
08:49:24 <dalees> so with it working well, that accomplishes one of my feature goals, at least.
08:49:50 <jakeyip> so is the change you want to make dependent on the label spec?
08:50:11 <dalees> no
08:50:22 <jakeyip> ok
08:51:36 <dalees> in the driver, it's already starting to use the fields, over the labels. https://github.com/openstack/magnum-capi-helm/blob/master/magnum_capi_helm/driver.py#L666
08:52:13 <opendevreview> Merged openstack/magnum-tempest-plugin master: Bump hacking  https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/943244
08:52:40 <dalees> but I'll follow this discussion up with a patchset soon.
08:54:26 <jakeyip> ok
08:56:16 <dalees> any other topics? we can end if nothing else
08:56:28 <jakeyip> nothing. thanks for holding the meeting
08:57:12 <dalees> thanks for joining! It'd be real quiet without you!
08:57:16 <dalees> #endmeeting