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