*** mhen_ is now known as mhen | 02:48 | |
opendevreview | Grzegorz Grasza proposed openstack/barbican master: Avoid race condition with multiple secret stores enabled https://review.opendev.org/c/openstack/barbican/+/901532 | 10:06 |
---|---|---|
tkajinam | xek, hi. do you have a minute to review and merge https://review.opendev.org/c/openstack/barbican/+/895346 ? this is automated patch so I'd suggest single approval. | 11:42 |
xek | tkajinam: ok, approved it, thanks! | 11:44 |
tkajinam | xek, thanks ! | 11:45 |
tkajinam | https://review.opendev.org/c/openstack/keystonemiddleware/+/856799 | 11:45 |
tkajinam | https://review.opendev.org/c/openstack/keystonemiddleware/+/894342 | 11:45 |
tkajinam | ^^ xek these are for keystone but I see you can +2 in this repo. in case you have a few more minutes then please merge these two as well. | 11:45 |
xek | tkajinam: ack, looking | 11:46 |
tkajinam | xek, thanks | 11:50 |
tkajinam | I had to update the first one to resolve conflicts but hopefully zuul votes verified +1 soon on it | 11:50 |
xek | #startmeeting Barbican | 12:00 |
opendevmeet | Meeting started Tue Nov 21 12:00:38 2023 UTC and is due to finish in 60 minutes. The chair is xek. Information about MeetBot at http://wiki.debian.org/MeetBot. | 12:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 12:00 |
opendevmeet | The meeting name has been set to 'barbican' | 12:00 |
opendevreview | Merged openstack/barbican master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/barbican/+/895346 | 12:00 |
xek | #topic Roll Call | 12:00 |
lpiwowar | o/ | 12:01 |
xek | Courtesy ping for dmendiza[m] ade_lee d34dh0r53 Luzi tosky tobias-urdin jjung mharley lpiwowar | 12:01 |
xek | o/ | 12:01 |
xek | As usual our agenda can be found here: | 12:01 |
xek | #link https://etherpad.openstack.org/p/barbican-weekly-meeting | 12:01 |
mharley | o/ | 12:02 |
xek | morning :) | 12:03 |
xek | #topic Review Past Meeting Action Items | 12:03 |
xek | I prepared a patch for https://bugs.launchpad.net/barbican/+bug/2043457 | 12:04 |
xek | #link https://review.opendev.org/c/openstack/barbican/+/901532 | 12:04 |
lpiwowar | thank you xek! | 12:04 |
lpiwowar | I've seen your patch. | 12:04 |
xek | lpiwowar: did you get a chance to check if it fixes the issue? | 12:05 |
xek | (I know, I submitted it just earlier today :P ) | 12:05 |
lpiwowar | Not yet. But I think we can just wait for the srbac job to finish. If we see the same failure in the logs then we know that it is not working. | 12:06 |
lpiwowar | I will keep an eye on this:). | 12:06 |
xek | cool, let's do that | 12:06 |
xek | #topic Liaison Updates | 12:06 |
lpiwowar | Ok, so it seems that the issue is not resolved by the patch:(. I've just seen the logs. The jobs have just finished. | 12:07 |
xek | Nothing from me | 12:07 |
lpiwowar | QA: There are two patches for barbican-tempest-plugin from students who are helping us with some stuff in upstream (https://review.opendev.org/c/openstack/barbican-tempest-plugin/+/899274, https://review.opendev.org/c/openstack/barbican-tempest-plugin/+/899191) but they are currently blocked by this: | 12:07 |
lpiwowar | https://bugs.launchpad.net/barbican/+bug/2043457. | 12:07 |
lpiwowar | Just wanted to mention this but is not important to merge those immediately :). | 12:07 |
xek | Ack, I'll take a look once the RBAC job issue is resolved | 12:09 |
lpiwowar | thank you | 12:10 |
xek | #topic Open Discussion | 12:11 |
xek | I haven't seen ade_lee or dmendiza recently in the weekly meeting, maybe we should move it to a later time slot? | 12:12 |
lpiwowar | It is ok for me if we want to move the time. | 12:14 |
xek | I would be fine with moving it an hour later... I'll ask them offilne if that would make a significant difference | 12:15 |
mharley | Same for me. | 12:15 |
xek | #topic Bug Review | 12:15 |
mharley | One hour later. | 12:15 |
xek | ack | 12:16 |
xek | There are no new bugs reported on launchpad since our last meeting :) | 12:17 |
xek | I still have to get to reviewing the YubiHSM 2 as PKCS11 backend support, also asked dmendiza to take a look | 12:17 |
xek | Alright, that's it for today, see y'all next week! | 12:18 |
xek | #endmeeting | 12:18 |
opendevmeet | Meeting ended Tue Nov 21 12:18:52 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 12:18 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/barbican/2023/barbican.2023-11-21-12.00.html | 12:18 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/barbican/2023/barbican.2023-11-21-12.00.txt | 12:18 |
opendevmeet | Log: https://meetings.opendev.org/meetings/barbican/2023/barbican.2023-11-21-12.00.log.html | 12:18 |
lpiwowar | thanks xek | 12:20 |
tkajinam | oops. I didn't notice the meeting was happening today. probably I should have raised https://review.opendev.org/c/openstack/barbican/+/900335 to ask some thoughts on deprecating unmaintained plugins | 12:56 |
tkajinam | and also wondering if we can merge https://review.opendev.org/c/openstack/barbican/+/848011 which has been kept for a while, so that we can consider replacing own db implementations by oslo.db early | 12:56 |
tkajinam | xek, do you mind adding +A to https://review.opendev.org/c/openstack/barbican/+/900662 ? I think it's better to restore the test coverage before something breaks sqla2 support | 12:57 |
tkajinam | xek, thanks again for your quick responses :-) | 13:13 |
opendevreview | Merged openstack/barbican master: Revert "Temporarily make sqlalchemy master job no-voting" https://review.opendev.org/c/openstack/barbican/+/900662 | 13:38 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!