13:01:01 #startmeeting barbican 13:01:01 Meeting started Tue Jun 9 13:01:01 2020 UTC and is due to finish in 60 minutes. The chair is redrobot. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:01:02 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 13:01:04 The meeting name has been set to 'barbican' 13:01:25 #topic Roll Call 13:01:28 o/ 13:01:32 Courtesy ping for ade_lee dave-mccowan hrybacki jamespage Luzi lxkong mhen moguimar raildo rm_work xek nearyo 13:01:54 o/ 13:01:55 As usual our (empty) agenda can be found here: 13:01:58 #link https://etherpad.opendev.org/p/barbican-weekly-meeting 13:03:33 We could almost have this in the work IRC moguimar raildo 😝 13:03:54 redrobot++ 13:04:30 xD 13:04:30 OK, let's get started! 13:04:52 #topic Liaison Updates 13:04:58 moguimar, any news from Oslo? 13:05:03 nope 13:05:17 we had a quick one yesterday 13:05:59 moguimar, anything notable from the PTG we should be aware of? 13:06:33 castellan wasn't a topic at the PTG 13:06:36 but policy was 13:06:45 so maybe raildo might have some updates 13:07:24 What do you say raildo? Anything we should be aware of from PTG? 13:07:30 hell yea :) well first of all, I believe that we had a great conversation on the barbican side :) 13:08:08 Yeah! I have an agenda item to recap real quick in a bit. 13:08:40 redrobot, cool, so let's wait for it 13:09:00 I'm not a fan of spoilers anyway hehe 13:09:19 haha, ok, let's do it right now (we'll come back to moguimar's wayward reviews) 13:09:24 #topic PTG Recap 13:09:45 #link https://etherpad.opendev.org/p/victoria-ptg-barbican 13:10:13 Like raildo said, we had a great PTG 13:10:17 notes are on the link above 13:10:26 and we did record all 4 hours of the session 13:10:39 and as soon as the OSF gives us the video, we'll post it. 13:12:47 #action redrobot to make Taiga cards out of the work we planned for Victoria 13:13:00 I was thinking exactly about that 13:13:09 an action item for the Taiga board 13:13:44 moguimar, great minds think alike! 🤓 13:17:13 raildo, anything from the policy discussions we need to mention? 13:18:10 well, we had some discussion on the oslo/keystone side most of it is summarized on the Keystone etherpad 13:18:51 #link https://etherpad.opendev.org/p/victoria-ptg-keystone (lines 20-67 and lines 146-187) 13:19:32 we basically got the nova's feedback on that work and planned some improvements to be done on victoria 13:20:12 but for Barbican itself, atm, focus on what we discussed on the PTG would be enough for now 13:20:44 awesome, thanks raildo 13:20:55 #topic Secret Consumers 13:20:57 also trying to understand a bit more about those new concepts, like system scope, we have some cool video like this one from Lance 13:20:59 #link https://www.youtube.com/watch?v=NI4MVMG7FGw 13:21:07 right on! 13:21:17 * redrobot adds video to his watchlist 13:21:47 OK, let's see ... we talked a little bit about Secret Consumers at the Image Encryption meeting yesterday 13:22:08 Looks like we should focus on adding client support to python-barbicanclient 13:24:42 I don't think the python-$PROJECTclient libraries are going away any time soon. 13:24:58 So maybe ade_lee is right that we should have two test suites for python-barbicanclient and openstacksdk 13:25:19 strike that maybe 13:25:41 moguimar++ 13:27:43 OK, moving on 13:27:50 #topic Wayward Reviews 13:27:59 #link https://tinyurl.com/yd3x48fl 13:28:43 I guess we could start with the top ones 13:28:43 that's a nice query, moguimar 13:28:59 that's ade_lee's work 13:29:31 #link https://review.opendev.org/#/c/720975/ 13:29:39 gotcha, the only thing I would change is adding the ansible-role-*-hsm and barbican-ui repos 13:29:57 can do 13:30:13 so, this one I think is pretty straightforward 13:30:23 replaces mock with unittest.mock 13:30:50 one less requirement and Zuul is happy about it 13:30:52 moguimar, cool, merged 13:31:29 you didn't even let me link the second one xD 13:32:06 I also have posted this one 13:32:09 #link https://review.opendev.org/#/c/734175/ 13:32:35 tiny fixes before I can get to the contributing goal 13:34:37 moguimar, lgtm 13:36:26 Andreas Jaeger proposed openstack/python-barbicanclient master: Update hacking for Python3 https://review.opendev.org/715622 13:36:33 cool, I'll poke ade_lee later 13:36:51 look, a wild review apperas xD 13:37:50 😆😆😆 13:38:43 ;) 13:38:53 already gave you my +1 AJaeger 13:39:04 lets wait on zuul for a +2 13:39:24 cool, any other quick reviews we could do right now moguimar? 13:39:30 sure, hope it passes now 13:39:37 well, we have 20 minutes, right? 13:39:44 I saw another from AJaeger 13:39:46 on pygments 13:39:54 with wrong commit message 13:39:59 how is that one comming AJaeger ? 13:40:23 moguimar: which one? 13:40:35 ah, you -1d it 13:40:39 https://review.opendev.org/#/c/729500/ 13:40:54 * AJaeger would love to have some guideance on updating hacking for barbican - see https://review.opendev.org/#/c/716730/ 13:41:12 they are 3 on that same line 13:41:15 https://review.opendev.org/#/c/729449/ 13:41:17 the tests fail, and I have two DNM on top of it. Could you advise me how to move forward and what you see as proper fix, please? 13:41:27 https://review.opendev.org/#/c/729501/ 13:41:47 moguimar: let me check 729501 13:42:18 729501 can be abandoned 13:42:29 I added myself to that one 13:43:20 AJaeger, I'll take a look at 716730 later today 13:43:26 moguimar: thanks! 13:44:06 reason to abandon 729501? 13:44:20 just wanna write something there 13:44:23 not a blank abandon 13:45:34 moguimar: I would write for all three - 449, 501, 500 - "another larger change already merged that includes this one, so this is a duplicate - abandon. Thanks for your change" 13:45:48 ah, I see 13:45:49 moguimar: agreed, blank abandon is bad 13:45:57 I remember seeing those changes somewhere already 13:46:08 moguimar: as part of my openstackdocstheme updates 13:46:16 yeah 13:46:38 ok, 3 less reviews in the list =D 13:46:48 yay! 13:46:52 Elod Illes proposed openstack/barbican stable/rocky: DNM: CI health check https://review.opendev.org/734560 13:48:22 done 13:49:32 since we're here 13:49:35 #link https://review.opendev.org/#/c/729001 13:50:15 \o/ 13:50:36 * AJaeger finally gets his backlog smaller... 13:50:36 lgtm 13:51:48 AJaeger, bring them on 13:51:57 any other you want us to have a look at? 13:52:13 for this last 8 minutes 13:52:18 of awesomeness 13:52:30 moguimar: sorry, no ;) 13:52:38 dang, ok 13:53:02 this wayward review felt a bit like our first Upstream Friday, right redrobot ? 13:53:11 moguimar, sure did! 13:53:24 thanks for bringing that up, moguimar 13:53:27 cool, lets keep it on the menu then =D 13:53:42 maybe we should do this on the regular meeting points 13:53:43 we have a similar slot at oslo meetings 13:53:45 jinx! 13:54:01 wait, do you review barbicanclient stable branches? ussuri is broken, https://review.opendev.org/734420 is the first t ofix it 13:55:05 AJaeger, lgtm 13:56:05 this is a backport of Takashi Kajinami's patch right? 13:56:13 moguimar: yes 13:56:18 cool 13:56:33 shipped 13:58:14 thx. Once that's in, I'll backport the next fix to bring that one back to life ;) 13:58:28 cool 13:58:50 allright, I guess that's all folks =D 13:59:08 I think this is the first time we've used the whole hour in a long while. 13:59:14 thanks for joining everyone! 13:59:16 #link https://www.youtube.com/watch?v=b9434BoGkNQ 13:59:35 😆😆😆 13:59:38 #endmeeting