13:00:01 #startmeeting barbican 13:00:02 Meeting started Tue Feb 9 13:00:01 2021 UTC and is due to finish in 60 minutes. The chair is redrobot. Information about MeetBot at http://wiki.debian.org/MeetBot. 13:00:03 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 13:00:05 The meeting name has been set to 'barbican' 13:00:10 #topic Roll Call 13:00:14 o/ 13:00:15 Courtesy ping for ade_lee dave-mccowan hrybacki jamespage Luzi lxkong mhen moguimar raildo rm_work xek nearyo 13:00:32 Hi Luzi 13:00:37 o/ 13:00:43 morning 13:00:48 Ohai raildo! 13:01:15 hi 13:01:39 Hi tosky! 13:01:52 Lots of folks here today! 😁 13:01:55 Let's get started 13:01:56 o/ 13:02:15 As usual the agenda is over here: 13:02:24 #link https://etherpad.opendev.org/p/barbican-weekly-meeting 13:02:52 #topic Past Meeting Action Items 13:02:58 #link http://eavesdrop.openstack.org/meetings/barbican/2021/barbican.2021-02-02-13.00.html 13:03:06 Looks like we didn't have any, so let's move right along 13:03:20 #topic Liaison Updates 13:03:24 moguimar any Oslo updates? 13:04:22 no much from me; I've seen the updates to barbican-tempest-plugin, tuned a review and added some comments (we can talk about them later if you want) 13:04:46 tosky we can talk about it now 13:04:54 just that feature freeze is comming 13:04:58 next week 13:05:02 #link https://review.opendev.org/q/project:openstack%252Fbarbican-tempest-plugin+status:open 13:05:54 tosky just saw your comment on this patch: 13:05:56 #link https://review.opendev.org/c/openstack/barbican-tempest-plugin/+/628499 13:06:19 as far as I can tell, the previous patch is failing because the gate is still running in Python 3 13:06:25 s/3/2.7 13:06:35 correct, https://review.opendev.org/c/openstack/barbican-tempest-plugin/+/628499 adds USE_PYTHON3=True but that value should be really inherited from the environment 13:06:39 looks like the pip script uses an F string that causes the gatesto fail 13:06:44 yes 13:07:13 but it seems the problem is that the older jobs use focal, while they should be pinned to bionic 13:07:50 I see 13:07:55 so I've updated https://review.opendev.org/c/openstack/barbican-tempest-plugin/+/756185 which adds the proper nodeset, by also adding victoria and removing stein 13:08:02 and also removing USE_PYTHON3 13:08:29 So we'll be good with those two patches merged then? 13:08:29 so either the two patches should be merged, or it's probably easier to merge them at the same time 13:08:57 it seems so, yes 13:09:14 Well, since we've got moguimar here let's just go ahead and merge them both now 13:09:29 thanks 13:09:35 I'm not sure about the failure here: https://review.opendev.org/c/openstack/barbican-tempest-plugin/+/678708 13:10:06 it seems that job started failing on all repositories yesterday, probably due to the new cryptography dependency 13:10:24 Yeah, that's an odd one 13:10:28 that job (test-release-openstack) still use bionic 13:10:42 done 13:10:42 I've asked on #openstack-infra earlier but I guess it was too early 13:11:44 k, I'll keep an eye on it and maybe go poke #opendev about it 13:13:18 I'm not sure how many of these older tempest patches are still relevant? 13:13:55 some of them may be 13:14:32 would you mind taking a look and rebasing stuff that is still needed? 13:14:34 on a related note: the current master was tagged as last-stein, but I'm not sure it's really working on stein without the first patch 13:14:44 oh, sure, I can take a look at the other patches 13:14:49 thanks tosky! 13:15:20 hmm... do we need to backport that patch to stable/stein? 13:15:44 the plugin is branchless 13:15:54 Oh ... hmm ... 13:16:00 not sure what we need to do then? 13:17:22 I guess it doesn't really matter too much, if something is really broken a new 1.3.1 release could be tagged from those patches 13:17:36 ok, cool 13:17:50 thanks for your help with the tempest patches, tosky! 😁 13:18:35 thanks for moving forward, I should have done it last week :/ 13:19:13 OK, let's move along 13:19:17 #topic Kanban Review 13:19:19 #link https://tree.taiga.io/project/dmend-openstack-barbican/kanban 13:19:24 No updates from me 13:19:29 neither 13:19:30 anything on your side moguimar? 13:19:35 beat me to it 13:19:36 haha 13:19:40 OK, moving on 13:19:48 #topic Bug Review 13:20:25 bah, getting a 500 trying to log into storyboard 13:21:01 Well, it looks like storyboard has not had its coffee this morning 13:21:29 #link https://bugs.launchpad.net/castellan/+bugs?orderby=-id&start=0 13:21:36 looks like no new bugs in Castellan 13:22:00 #link https://bugs.launchpad.net/cursive/+bugs?orderby=-id&start=0 13:22:08 and no new bugs on Cursive 13:22:30 I got added to some castellan review this week 13:24:10 https://review.opendev.org/c/openstack/castellan/+/769560 13:24:44 moguimar added to my review backlog 13:24:56 #topic Wayward Reviews 13:25:05 #link https://tinyurl.com/y3zto3ad 13:25:45 oh, the cursive patch is missing just one last review 13:26:23 #link https://review.opendev.org/c/x/cursive/+/746584 13:26:26 this one? 13:26:42 yes 13:26:54 done 13:27:10 🎉 13:27:18 thanks! 13:27:27 This one is ready to be merged too https://review.opendev.org/c/openstack/barbican/+/765690 13:28:56 done 13:29:22 Woohoo! It's offically a merge party! 13:29:42 Another easy merge: https://review.opendev.org/c/x/cursive/+/618168 13:31:06 ok, now we have only the castellan ones 13:31:19 I think they demand a bit more time to review 13:31:29 moguimar agreed 13:31:34 #topic Open Discussion 13:31:43 Anything elese y'all want to talk about? 13:31:43 that said, the cursive patch will fail thanks to test-release-openstack 13:31:52 but once that job is fixed I will recheck it 13:32:07 sounds good tosky 13:36:49 Thanks for joining, everyone! 13:36:54 See y'all online. 13:36:58 #endmeeting