| opendevreview | Jake Yip proposed openstack/magnum master: WIP: Deprecated fedora_atomic driver https://review.opendev.org/c/openstack/magnum/+/833949 | 04:02 |
|---|---|---|
| strigazi | #startmeeting magnum | 08:59 |
| opendevmeet | Meeting started Wed Mar 16 08:59:47 2022 UTC and is due to finish in 60 minutes. The chair is strigazi. Information about MeetBot at http://wiki.debian.org/MeetBot. | 08:59 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 08:59 |
| opendevmeet | The meeting name has been set to 'magnum' | 08:59 |
| strigazi | #topic Roll Call | 08:59 |
| jakeyip | o/ | 09:00 |
| strigazi | o/ | 09:00 |
| dale` | o/ | 09:01 |
| gbialas | o/ | 09:01 |
| strigazi | #topic Past Action Items | 09:03 |
| strigazi | #link https://etherpad.opendev.org/p/magnum-weekly-meeting | 09:03 |
| strigazi | jakeyip to change magnum/api/validation.py#L259-L260 to not allow mesos as a coe option | 09:04 |
| strigazi | jakeyip: we have a patch for this? | 09:04 |
| jakeyip | yes, merged https://review.opendev.org/c/openstack/magnum/+/830594 | 09:06 |
| strigazi | cool, thanks | 09:06 |
| strigazi | I'm relaying the following to the next: | 09:07 |
| strigazi | #action strigazi to comment on 821213: Drop mesos driver | https://review.opendev.org/c/openstack/magnum/+/821213 , 803780: Drop bay and baymodel from controllers | https://review.opendev.org/c/openstack/magnum/+/803780 , 803629: Drop bay and baymodel | https://review.opendev.org/c/openstack/python-magnumclient/+/803629 | 09:07 |
| strigazi | next is: | 09:07 |
| strigazi | strigazi to review Octavia Patches https://review.opendev.org/c/openstack/magnum/+/764444 https://review.opendev.org/c/openstack/magnum/+/765309 https://review.opendev.org/c/openstack/magnum/+/767119 https://review.opendev.org/c/openstack/magnum/+/773923 | 09:07 |
| strigazi | I just finished with these three, thanks mnasiadka | 09:08 |
| strigazi | I don't who is using devstack for testing, but master doesn | 09:08 |
| strigazi | I don't who is using devstack for testing, but master doesn't work for me since a couple of weeks | 09:08 |
| strigazi | I had to use wallaby | 09:08 |
| jakeyip | master of? | 09:08 |
| strigazi | jakeyip: master branch of all components including devstack | 09:09 |
| jakeyip | I opened a bug with ovn-octavia-provider and patched another thing | 09:09 |
| strigazi | jakeyip: I deployed with wallaby and then just bumped magnum to master | 09:09 |
| jakeyip | it worked for me for ovn-octavia-provider master | 09:09 |
| jakeyip | their devstack wasn't working properly | 09:10 |
| strigazi | jakeyip: But for the magnum patches we can merge right? | 09:10 |
| strigazi | i think it was fine | 09:10 |
| jakeyip | yeah it's working for me now. a couple of rough edges. one thing I am thinking of is if we should squash those changes | 09:11 |
| jakeyip | it's not really working until you have all three changes, does this warrant a squash? | 09:11 |
| strigazi | we could, gbialas what do you think? | 09:11 |
| strigazi | jakeyip: to not hold things we can squash in case of a cherry-pick | 09:12 |
| strigazi | they're hanging for some time, so let's take them | 09:12 |
| jakeyip | sure. before you merge, we might also need https://review.opendev.org/c/openstack/magnum/+/832877 else it is going to fail gate | 09:13 |
| gbialas | What is the question. ? | 09:13 |
| strigazi | I'll add it as an action again to make it's in by the next meeting | 09:13 |
| strigazi | gbialas: to squash the three OVN patches into one | 09:14 |
| strigazi | jakeyip: ack for https://review.opendev.org/c/openstack/magnum/+/832877 | 09:14 |
| gbialas | I think it is ok. They are covering one story, and are dependent. | 09:15 |
| strigazi | #action strigazi to follow Octavia Patches https://review.opendev.org/c/openstack/magnum/+/764444 https://review.opendev.org/c/openstack/magnum/+/765309 https://review.opendev.org/c/openstack/magnum/+/767119 https://review.opendev.org/c/openstack/magnum/+/773923 | 09:15 |
| strigazi | gbialas: ok | 09:15 |
| strigazi | #action strigazi to follow: 832877: Remove the deprecated argument tenant from RequestContext | https://review.opendev.org/c/openstack/magnum/+/832877 | 09:16 |
| strigazi | jakeyip: we need this in the yoga branch? ^^ | 09:16 |
| jakeyip | not sure I didn't test yoga | 09:16 |
| strigazi | ok | 09:17 |
| strigazi | next item is: | 09:17 |
| strigazi | mnasiadka to follow up kubernetes conformance for Yoga https://www.cncf.io/certification/software-conformance/ | 09:17 |
| strigazi | I'll relay since mnasiadka didn't join today | 09:17 |
| strigazi | #action mnasiadka to follow up kubernetes conformance for Yoga https://www.cncf.io/certification/software-conformance/ | 09:18 |
| strigazi | next: | 09:18 |
| strigazi | strigazi to review https://review.opendev.org/c/openstack/magnum/+/764254 https://review.opendev.org/c/openstack/magnum/+/764397 DONE and in time for yoga | 09:18 |
| mnasiadka | still need to do that, sorry - $job overtook my life :) | 09:18 |
| strigazi | mnasiadka: no worries | 09:18 |
| strigazi | and last one, | 09:18 |
| strigazi | jakeyip to deprecate fedora-atomic driver(s) | 09:18 |
| strigazi | pushed: https://review.opendev.org/c/openstack/magnum/+/833949 | 09:19 |
| jakeyip | I have a WIP in https://review.opendev.org/c/openstack/magnum/+/833949/1 , following feilong's and others deprecation strategy. can you let me know if that is OK I'll do reno and rest | 09:19 |
| strigazi | #action strigazi, jakeyip to follow 833949: WIP: Deprecated fedora_atomic driver | https://review.opendev.org/c/openstack/magnum/+/833949 | 09:19 |
| strigazi | jakeyip: we can do it like that, looks good | 09:20 |
| jakeyip | good | 09:20 |
| strigazi | #topic Open Discussion | 09:21 |
| mnasiadka | What about PTG slots? I think Magnum is not on the list of projects that will have PTG sessions ;-) | 09:21 |
| strigazi | I just wanted to check with gbialas and mnasiadka if there are any news for the CAPI driver | 09:21 |
| mnasiadka | I don't think so, these are other people from our company that wanted to do this - but I'm afraid they are busy with other tasks. | 09:22 |
| mnasiadka | But I'll check after the meeting. | 09:22 |
| strigazi | mnasiadka: cool, let us know | 09:22 |
| oneswig_ | hi strigazi - apologies. mnasiadka is right - the problem is other deadlines for end of March | 09:23 |
| strigazi | mnasiadka: for the PTG, I don't think another two hour slot would make a difference. If we make good progress with the weekly meetings we can do another session in the next PTG | 09:24 |
| mnasiadka | strigazi: sure, fine by me :) | 09:24 |
| strigazi | oneswig_: no worries, we are eager to move with it. When you manage to allocate ping us at CERN, others are for sure interested too | 09:25 |
| oneswig_ | thanks strigazi, will do | 09:25 |
| strigazi | mnasiadka: But it's a good point, I'll send an email to tell people and remind them for the meeting | 09:25 |
| strigazi | dale`: do you want to bring something up? | 09:26 |
| dale` | thanks, just listening in today | 09:26 |
| strigazi | cool | 09:27 |
| strigazi | all, any other business | 09:27 |
| strigazi | all, any other business? | 09:27 |
| jakeyip | one deprecation topic, there are a few stuffs marked as deprecated and can be removed, namely overlay (not overlay2) and in-tree cinder csi. should these move ahead? | 09:27 |
| strigazi | we can drop them | 09:28 |
| strigazi | less is more :) | 09:28 |
| jakeyip | good | 09:28 |
| strigazi | #action jakeyip to drop "overlay" storage driver | 09:28 |
| strigazi | #action jakeyip to drop cinder in-tree provider | 09:29 |
| jakeyip | there's a question a few days earlier about admission-control that I don't have the answer to | 09:29 |
| strigazi | #action strigazi to notify the ML for not having a PTG session | 09:30 |
| strigazi | jakeyip: what was it? | 09:30 |
| jakeyip | > dhrp | The specific problem that I have is; which feels super dumb. Is that that when using the CLI to create the template, I cannot pass a comma separated list. with the admission_control_list. This probably just a parsing issue. | 09:30 |
| jakeyip | I think the issue is with label values being a comma separated list | 09:31 |
| strigazi | --labes "the comma separated list" --labels kube_tag=v1.23.1 --labels .. | 09:31 |
| strigazi | you can pass labels many times | 09:32 |
| strigazi | Indeed the client probably parses the list incorrectly | 09:32 |
| gbialas | label=value,label=value,label=value... is the way to go in cli | 09:33 |
| strigazi | yes, you can split it in many --labels arguments | 09:34 |
| jakeyip | that works gbialas thanks | 09:34 |
| jakeyip | strigazi: one other review for attention - dropping baymodel in client https://review.opendev.org/c/openstack/python-magnumclient/+/803629 | 09:35 |
| gbialas | strigazi you are right, this also works | 09:35 |
| strigazi | #action strigazi to follow 803629: Drop bay and baymodel | https://review.opendev.org/c/openstack/python-magnumclient/+/803629 | 09:35 |
| strigazi | all, any other business? | 09:40 |
| strigazi | thanks for joining everyone | 09:43 |
| strigazi | #endmeeting | 09:43 |
| opendevmeet | Meeting ended Wed Mar 16 09:43:27 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 09:43 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-03-16-08.59.html | 09:43 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-03-16-08.59.txt | 09:43 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/magnum/2022/magnum.2022-03-16-08.59.log.html | 09:43 |
| jakeyip | thanks! | 09:43 |
| Gues__________________________ | jakeyip re: there's a question a few days earlier about admission-control that I don't have the answer to | 09:44 |
| *** Gues__________________________ is now known as dhrp | 09:44 | |
| jakeyip | hi dhrp :) | 09:44 |
| dhrp | I've actually found that if I use terraform adding this label with comma's works. | 09:45 |
| dhrp | And sorry for missing this meeting. I wanted to join but had another meeting at the same tiem. | 09:45 |
| dhrp | I'm having one other question, that I hope someone can help me with.. | 09:46 |
| dhrp | the 'post_install_manifest_url' doesn't appear to work, is this a known issue, or a way I can debug this? | 09:47 |
| jakeyip | not sure I don't use it; I'll give it a go since we need this too and our prod cloud just got to wallaby | 09:50 |
| strigazi | not using it either | 09:50 |
| dhrp | strigazi > you can pass labels many times -- Ok, that would help. thanks | 09:51 |
| dale` | dhrp: we have post_install_manifest_url configured at Catalyst Cloud, and it does seem to apply. it looks like it's used in kube_cluster_config script, so it runs once at cluster create time on a master | 09:52 |
| dhrp | ok | 09:54 |
| dale` | (running wallaby) | 09:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!