| opendevreview | shanyunfan33 proposed openstack/magnum master: Update python testing classifier https://review.opendev.org/c/openstack/magnum/+/826418 | 06:27 |
|---|---|---|
| jakeyip_ | I may not be able to join the meeting later, it's a public holiday today and I am away from computer | 07:52 |
| opendevreview | shanyunfan33 proposed openstack/magnum-tempest-plugin master: remove unicode from code https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/826426 | 08:21 |
| strigazi | jakeyip: ack, enjoy the holiday | 08:54 |
| strigazi | #startmeeting magnum | 09:00 |
| opendevmeet | Meeting started Wed Jan 26 09:00:33 2022 UTC and is due to finish in 60 minutes. The chair is strigazi. Information about MeetBot at http://wiki.debian.org/MeetBot. | 09:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 09:00 |
| opendevmeet | The meeting name has been set to 'magnum' | 09:00 |
| strigazi | #topic Roll Call | 09:00 |
| strigazi | o/ | 09:00 |
| oneswig | \o | 09:00 |
| parallax | o/ | 09:01 |
| gbialas | \o | 09:01 |
| strigazi | #topic Stories/Reviews | 09:03 |
| strigazi | Add Cluster API Kubernetes COE driver https://review.opendev.org/c/openstack/magnum-specs/+/824488 | 09:04 |
| oneswig | I've been discussing but not pushed any changes yet - apologies. | 09:04 |
| oneswig | I have it open here... | 09:04 |
| strigazi | oneswig: you want to discuss smth now or we wait next week? | 09:04 |
| oneswig | Probably best if I write things first - I think better that way. One thing this raises is how to do regression testing, to make sure we don't break things in the existing drivers | 09:06 |
| strigazi | oneswig: I think for the existing drivers, we won't have issues. The code will be pretty self contained. I know this part really well and even the the tempest tests we have (disable on CI) can cover us | 09:07 |
| oneswig | OK, that's good to hear. We think we can make a test plan for the new driver, although its external dependency on Cluster API adds a challenge | 09:08 |
| oneswig | Are the Tempest tests in working order? You mentioned they are disabled... | 09:09 |
| strigazi | oneswig: we can always blame the CAPI developers for issues with CAPI :) | 09:09 |
| strigazi | joking | 09:09 |
| strigazi | tempest tests should work, will give it a go today | 09:09 |
| oneswig | Great, thanks strigazi | 09:10 |
| oneswig | How much coverage do they provide? | 09:10 |
| strigazi | tempest creates a cluster and runs a pod, doesn't test much, but it covers if the old driver is working (being picked up properly) | 09:10 |
| tobias-urdin | o/ | 09:13 |
| oneswig | A lot of code paths will be covered to get to that point, I guess. But not the tricky ones | 09:13 |
| strigazi | tobias-urdin: \o | 09:13 |
| strigazi | oneswig: indeed not, but they cover us of the level of changes the CAPI driver will introduce | 09:14 |
| strigazi | oneswig: eg CAPI won't break calico or flannel or smth else inside the cluster | 09:14 |
| oneswig | sounds reasonable | 09:16 |
| strigazi | oneswig: let's continue in the SPEC then and discuss it next week again | 09:16 |
| oneswig | yes - will do | 09:17 |
| strigazi | moving on to open reviews, thanks to parallax we got the CI green again with that docs issue | 09:17 |
| parallax | np | 09:17 |
| strigazi | #topic Reviews | 09:18 |
| strigazi | Do you want to discuss smth from the list https://etherpad.opendev.org/p/magnum-weekly-meeting ? | 09:18 |
| strigazi | Other reviews that need attention? | 09:18 |
| strigazi | If not, I would like to ask some testers for: https://review.opendev.org/c/openstack/python-magnumclient/+/788689 | 09:19 |
| gbialas | strigazi: Yup, lot of them :) | 09:19 |
| gbialas | https://review.opendev.org/c/openstack/magnum/+/779378 this one, with dependencies | 09:20 |
| strigazi | gbialas: can you add them in the etherpad so that next week we can see what's left | 09:21 |
| gbialas | I would be very grateful if someone could take a look into this | 09:21 |
| gbialas | strigazi: Yes, will ddo. | 09:21 |
| strigazi | gbialas: calico is second in my list, first is the bump to f34/35 which is tiny | 09:22 |
| gbialas | That would be great, f33 is not supported anymore. | 09:23 |
| strigazi | gbialas: Do you want to bring up another one? | 09:26 |
| gbialas | I think this: https://review.opendev.org/c/openstack/magnum/+/775793/17 but it is in relation to previous one. | 09:29 |
| strigazi | gbialas: ack | 09:30 |
| strigazi | If there is nothing else to discuss, let's wrap up | 09:31 |
| strigazi | See you next week everyone | 09:33 |
| strigazi | #endmeeting | 09:33 |
| opendevmeet | Meeting ended Wed Jan 26 09:33:36 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:33 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-01-26-09.00.html | 09:33 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-01-26-09.00.txt | 09:33 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-01-26-09.00.log.html | 09:33 |
| opendevreview | Stig Telfer proposed openstack/magnum-specs master: Add Cluster API Kubernetes COE driver https://review.opendev.org/c/openstack/magnum-specs/+/824488 | 12:28 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!