| jakeyip | hi all, strigazi and mnasiadka are you around? | 09:01 |
|---|---|---|
| strigazi | ο/ | 09:02 |
| jakeyip | strigazi: you want me to chair this time? | 09:03 |
| strigazi | yes, if possibe | 09:03 |
| jakeyip | ok | 09:04 |
| jakeyip | #startmeeting magnum | 09:04 |
| opendevmeet | Meeting started Wed May 11 09:04:18 2022 UTC and is due to finish in 60 minutes. The chair is jakeyip. Information about MeetBot at http://wiki.debian.org/MeetBot. | 09:04 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 09:04 |
| opendevmeet | The meeting name has been set to 'magnum' | 09:04 |
| jakeyip | #topic Roll Call | 09:04 |
| jakeyip | o/ | 09:04 |
| strigazi | o/ | 09:04 |
| jakeyip | Thanks for joining the meeting strigazi | 09:06 |
| jakeyip | #topic Past action items | 09:06 |
| jakeyip | Drop mesos docs https://review.opendev.org/c/openstack/magnum/+/839556 , strigazi to comment | 09:07 |
| strigazi | DONE, just now | 09:08 |
| jakeyip | ok next is "Drop bay and baymodel from controllers". there is a couple of patches | 09:09 |
| jakeyip | first in magnum code https://review.opendev.org/c/openstack/magnum/+/803779 - this never passed zuul. | 09:09 |
| strigazi | Are we taking them now in the end? | 09:09 |
| jakeyip | I wonder if Diogo is still interested in working on this? or should we take over | 09:10 |
| jakeyip | sorry I don't understand, can you clarify | 09:10 |
| strigazi | I mean, we can merge this in the cycle right? | 09:11 |
| jakeyip | this meaning mesos? | 09:11 |
| strigazi | for bay/baymodel | 09:12 |
| jakeyip | yes | 09:13 |
| strigazi | cool | 09:13 |
| strigazi | I left comment in the patch | 09:14 |
| jakeyip | ok, there's another one - https://review.opendev.org/c/openstack/magnum/+/803780 - this one diogo marked as WIP | 09:14 |
| strigazi | same comment | 09:15 |
| jakeyip | same topic, on the client side - https://review.opendev.org/c/openstack/python-magnumclient/+/803629 | 09:15 |
| strigazi | commented | 09:17 |
| jakeyip | should I add this as action strigazi ? | 09:17 |
| strigazi | sure | 09:17 |
| jakeyip | #action jakeyip strigazi follow up with diogo on bay and baymodel patches | 09:18 |
| jakeyip | next - "Add support for choosing Octavia provider" | https://review.opendev.org/c/openstack/magnum/+/839709 | 09:19 |
| strigazi | approved | 09:20 |
| jakeyip | thanks | 09:20 |
| jakeyip | mnasiadka isn't around so we'll keep his items | 09:20 |
| jakeyip | #action mnasiadka: to follow up kubernetes conformance for Yoga https://www.cncf.io/certification/software-conformance/ | 09:20 |
| jakeyip | #action mnasiadka to check out implications of https://review.opendev.org/c/openstack/python-keystoneclient/+/675870 and possibly endpoint type configurable | 09:20 |
| jakeyip | #action gbialas, mnasiadka to follow CAPI driver | 09:20 |
| jakeyip | next something new - "Adapt Cinder CSI to upstream manifest" | https://review.opendev.org/c/openstack/magnum/+/833354 | 09:22 |
| jakeyip | was hoping to ask mnasiadka if they want to push this, I'm not too familar with this, strigazi you know what to do? | 09:23 |
| strigazi | better to move with Mohammed's proposal | 09:24 |
| strigazi | but this is the issue with managing manifest in code | 09:24 |
| strigazi | hence the much bigger idea to move all this to a meta helmchart containing everything | 09:25 |
| jakeyip | yeah agree. do we ever have an action item tracking it? | 09:25 |
| strigazi | I don't think so | 09:25 |
| jakeyip | ok it's a big thing and I don't know where to start. let's have this discussion when more people are around, or at PTG | 09:26 |
| jakeyip | what do you think strigazi ? | 09:26 |
| strigazi | sounds good | 09:27 |
| opendevreview | Merged openstack/magnum master: Drop mesos documentation https://review.opendev.org/c/openstack/magnum/+/839556 | 09:28 |
| jakeyip | ok, still on previous items. for wiki I cleaned up https://wiki.openstack.org/wiki/Magnum. I left the table there until I have some time to move it to magnum src | 09:29 |
| jakeyip | any comments? | 09:29 |
| strigazi | no, counds good | 09:30 |
| strigazi | s/counds/sounds/ | 09:30 |
| jakeyip | ok I think that's it for past action items. anything I missed out? | 09:30 |
| strigazi | I think that was all | 09:31 |
| jakeyip | I would like a topic for New Reviews that I need help with | 09:31 |
| strigazi | sure | 09:31 |
| jakeyip | #topic New Reviews | 09:32 |
| jakeyip | below are some in magnumclient | 09:32 |
| jakeyip | first - 'Drop lower-constraints.txt and its testing' | https://review.opendev.org/c/openstack/python-magnumclient/+/840078 | 09:32 |
| strigazi | done | 09:34 |
| strigazi | I mean, approved | 09:34 |
| jakeyip | thanks | 09:35 |
| jakeyip | next - 'Remove translation sections from setup.cfg' | https://review.opendev.org/c/openstack/python-magnumclient/+/750281 | 09:35 |
| strigazi | approved | 09:36 |
| jakeyip | I think this is done everywhere so should be fine | 09:36 |
| jakeyip | thanks | 09:36 |
| jakeyip | next up is reviews in magnum-ui | 09:36 |
| jakeyip | Fixed Unit test for magnum-ui | https://review.opendev.org/c/openstack/magnum-ui/+/835963 | 09:37 |
| jakeyip | how do we test this? or if Zuul passes it's generally ok? | 09:37 |
| strigazi | I think zuul is good enough for this | 09:39 |
| jakeyip | ok. good to hear. I'll +2 that and rebase the lower-constraint patch in the same repo https://review.opendev.org/c/openstack/magnum-ui/+/840077 | 09:40 |
| strigazi | cool | 09:40 |
| jakeyip | that's all for reviews, thanks | 09:40 |
| jakeyip | any reviews you want to bring up? | 09:41 |
| jakeyip | strigazi ^ | 09:41 |
| strigazi | I'm good | 09:41 |
| jakeyip | #topic Open Discussion | 09:42 |
| jakeyip | there's something from storyboard | 09:42 |
| jakeyip | K8s 1.24+ : Remove dockershim kubelet options | https://storyboard.openstack.org/#!/story/2010028 | 09:42 |
| jakeyip | strigazi: have you tried k8s 1.24? | 09:43 |
| strigazi | no | 09:43 |
| strigazi | has he proposed a change? | 09:43 |
| jakeyip | I don't think so | 09:43 |
| strigazi | Maybe we tell the person who open the propose one :) | 09:44 |
| strigazi | Maybe we tell the person who open the story to propose one :) | 09:44 |
| opendevreview | Merged openstack/python-magnumclient master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/python-magnumclient/+/840078 | 09:44 |
| jakeyip | ok team work :) | 09:44 |
| opendevreview | Merged openstack/python-magnumclient master: Remove translation sections from setup.cfg https://review.opendev.org/c/openstack/python-magnumclient/+/750281 | 09:44 |
| jakeyip | another story I don't understand | https://storyboard.openstack.org/#!/story/2010023 | 09:45 |
| jakeyip | have you seen this? | 09:45 |
| strigazi | no, first time I see it | 09:46 |
| strigazi | the issue I mean. The story I have seen | 09:46 |
| jakeyip | ok I just wanted to check, I can go digging if I find some time | 09:46 |
| jakeyip | that's all from me | 09:47 |
| jakeyip | anything you want to discuss? | 09:47 |
| strigazi | I'm good, thank you for chairing Jake! | 09:48 |
| jakeyip | you're welcome. Thanks! see you next week! | 09:48 |
| jakeyip | #endmeeting | 09:48 |
| opendevmeet | Meeting ended Wed May 11 09:48:51 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:48 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-05-11-09.04.html | 09:48 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-05-11-09.04.txt | 09:48 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-05-11-09.04.log.html | 09:48 |
| opendevreview | Jake Yip proposed openstack/magnum-ui master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/magnum-ui/+/840077 | 09:55 |
| opendevreview | Merged openstack/magnum master: Add support for choosing Octavia provider https://review.opendev.org/c/openstack/magnum/+/839709 | 10:12 |
| opendevreview | Merged openstack/magnum-ui master: Fixed Unit test for magnum-ui https://review.opendev.org/c/openstack/magnum-ui/+/835963 | 10:36 |
| opendevreview | Merged openstack/magnum-ui master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/magnum-ui/+/840077 | 10:37 |
| opendevreview | Vishal Manchanda proposed openstack/magnum-ui master: Use Horizon project template for nodejs jobs https://review.opendev.org/c/openstack/magnum-ui/+/841322 | 12:00 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!