| *** jakeyip_ is now known as jakeyip | 00:20 | |
| opendevreview | Takashi Kajinami proposed openstack/magnum-ui master: Drop environments for nose https://review.opendev.org/c/openstack/magnum-ui/+/950941 | 06:00 |
|---|---|---|
| opendevreview | Merged openstack/magnum-ui master: Drop environments for nose https://review.opendev.org/c/openstack/magnum-ui/+/950941 | 07:19 |
| dalees | Kia Ora, meeting scheduled to start about now. Let me know if you're about and we'll make a start | 08:00 |
| dalees | I hear from jakeyip, who may be late. | 08:00 |
| jakeyip | hi dalees I'm here | 08:10 |
| dalees | hi jakeyip ! | 08:10 |
| dalees | #startmeeting magnum | 08:11 |
| 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 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:11 |
| opendevmeet | The meeting name has been set to 'magnum' | 08:11 |
| dalees | #topic Roll Call | 08:11 |
| dalees | o/ | 08:11 |
| jakeyip | o/ | 08:11 |
| dalees | any others? feel free to say hello later on if you join in, also! | 08:12 |
| dalees | #topic Reviews | 08:12 |
| dalees | A couple in the agenda for discussion | 08:13 |
| dalees | shall we start with the master lb fip patchests? | 08:13 |
| jakeyip | ok | 08:13 |
| dalees | mine is duplicate, I'll abandon it as the other merges. | 08:14 |
| dalees | I posted a review to the other, needs a few small changes | 08:14 |
| dalees | https://review.opendev.org/c/openstack/magnum-capi-helm/+/933619 | 08:14 |
| jakeyip | yeap I saw that :) | 08:15 |
| 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:15 |
| 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 |
| jakeyip | they get credit with the co-author tag | 08:17 |
| 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:18 |
| dalees | the other review on the agenda is https://review.opendev.org/c/openstack/magnum-capi-helm/+/950806 | 08:19 |
| 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 |
| 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:20 |
| dalees | perhaps separate the compatibility table from the code change? | 08:21 |
| jakeyip | maybe, but the current table was really documenting what is the min version that works with alpha7 since that is the change | 08:22 |
| jakeyip | the final table should actually have min-max versions. and also lead to an upgrade doc. and that gets more complicated... | 08:23 |
| dalees | ah, yes. I'm fine with combined or separate then. | 08:25 |
| 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:26 |
| 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:34 |
| jakeyip | yeah I agree, it must be easy for users / operators! | 08:38 |
| dalees | any more on that topic? We can move to open discussion if there is anything else. | 08:39 |
| jakeyip | no | 08:39 |
| dalees | #topic Open Discussion | 08:39 |
| jakeyip | prob need to catch up at some point on how to get ci working with magnum-capi-helm | 08:40 |
| jakeyip | and also wondering if anyone has looked into eventlet chage | 08:42 |
| 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:44 |
| dalees | but on the topic of CI - we could progress the helm chart repo snapshot and publish perhaps. | 08:45 |
| jakeyip | how is the autoscaling values fit with the labels proposal ? | 08:47 |
| 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:48 |
| dalees | so with it working well, that accomplishes one of my feature goals, at least. | 08:49 |
| jakeyip | so is the change you want to make dependent on the label spec? | 08:49 |
| dalees | no | 08:50 |
| jakeyip | ok | 08:50 |
| 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:51 |
| opendevreview | Merged openstack/magnum-tempest-plugin master: Bump hacking https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/943244 | 08:52 |
| dalees | but I'll follow this discussion up with a patchset soon. | 08:52 |
| jakeyip | ok | 08:54 |
| dalees | any other topics? we can end if nothing else | 08:56 |
| jakeyip | nothing. thanks for holding the meeting | 08:56 |
| dalees | thanks for joining! It'd be real quiet without you! | 08:57 |
| dalees | #endmeeting | 08:57 |
| opendevmeet | Meeting ended Tue May 27 08:57:16 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 08:57 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/magnum/2025/magnum.2025-05-27-08.11.html | 08:57 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/magnum/2025/magnum.2025-05-27-08.11.txt | 08:57 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/magnum/2025/magnum.2025-05-27-08.11.log.html | 08:57 |
| opendevreview | Jake Yip proposed openstack/magnum-capi-helm master: Use openstackcluster v1alpha7 https://review.opendev.org/c/openstack/magnum-capi-helm/+/950806 | 12:23 |
| opendevreview | Dale Smith proposed openstack/magnum master: Allow CA certificate actions for project users, not only cluster creator. https://review.opendev.org/c/openstack/magnum/+/951048 | 23:55 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!