15:19:09 <tmazur> #startmeeting horizon
15:19:09 <opendevmeet> Meeting started Wed Jan 22 15:19:09 2025 UTC and is due to finish in 60 minutes.  The chair is tmazur. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:19:09 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:19:09 <opendevmeet> The meeting name has been set to 'horizon'
15:19:23 <omcgonag> hi
15:19:52 <tmazur> Hi everyone and welcome to the weekly meeting!
15:20:02 <tmazur> Today's agenda can be found here:
15:20:03 <tmazur> https://etherpad.opendev.org/p/horizon-release-priorities#L33
15:20:45 <tmazur> #topic Notices
15:20:55 <tmazur> This week is R-10
15:21:06 <tmazur> Schedule: https://releases.openstack.org/epoxy/schedule.html
15:22:26 <tmazur> I don't have announcements for today
15:22:31 <tmazur> So moving forward
15:23:07 <tmazur> #topic Release priorities
15:24:52 <tmazur> Just a quick update from my side. I am working on the two-factor auth configuration in Horizon, and I will submit an initial patch later this week.
15:25:54 <tmazur> Also, we have started our research on AngularJS removal
15:26:42 <tmazur> And for the SDK patches, let's move to the next topic
15:26:45 <tmazur> #topic Open reviews
15:27:21 <tmazur> Please check the list for this week here: https://etherpad.opendev.org/p/horizon-release-priorities#L42
15:27:45 <tmazur> 938488: SDK for Neutron Floating IPs | https://review.opendev.org/c/openstack/horizon/+/938488
15:28:12 <tmazur> I am reviewing it today, and jjasek_ please take a look when you have a minute
15:29:26 <jjasek_> Will take a look at it :)
15:29:33 <tmazur> Thanks!
15:29:39 <tmazur> 939607: Imported Translations from Zanata | https://review.opendev.org/c/openstack/horizon/+/939607
15:30:18 <tmazur> ^^^ this one here is made manually by the I18n team
15:31:06 <tmazur> because of infra issues
15:32:01 <tmazur> It's huge, but I checked it and see no problems merging it. However if we could have more votes from the team, it would be great
15:32:26 <tmazur> So please omcgonag, jjasek_ take a look
15:33:15 <jjasek_> I can see there is most of the changes just changing of the lines length, just make one line shorter and second one longer. Is there any specific reason? Like pep8 for python?
15:34:10 <tmazur> Yes. It looks like they ran into the same issue that we had with our doc tests
15:35:03 <tmazur> So yes, the patch looks all good for me, I just don't want to approve it without any other votes
15:35:31 <jjasek_> But sometimes they make shorter line that was already shorter before than some of the other lines :-D. I did not count number of chars but I am wandering what is the reason.
15:36:01 <jjasek_> wondering*
15:36:23 <jjasek_> But I will take a look at it too.
15:36:39 <tmazur> Maybe they used some assistance? I can't imaging doing all this manually >-<
15:36:54 <opendevreview> Ivan Anfimov proposed openstack/horizon master: Changing "Launch" action for "Create" instance  https://review.opendev.org/c/openstack/horizon/+/939779
15:37:24 <tmazur> Anyway, moving forward
15:37:43 <tmazur> Two more patches
15:37:45 <tmazur> 939389: Support new pytest 8.3.4 | https://review.opendev.org/c/openstack/horizon/+/939389
15:38:09 <tmazur> jjasek_, since you're a pytest expert, I would like to bug you again :D
15:38:16 <tmazur> Please take a look ^^^
15:39:01 <tmazur> And this one:
15:39:03 <tmazur> 939546: [DNM] non-existent timezone fix | https://review.opendev.org/c/openstack/horizon/+/939546
15:39:20 <tmazur> omcgonag please see my comment
15:40:23 <jjasek_> Ack, will try to review tomorrow all the patches that are waiting for reviews (y).
15:40:39 <tmazur> Great, thank you very much!
15:40:59 <tmazur> That's it from me for today
15:41:06 <tmazur> #topic Open discussion
15:41:13 <tmazur> Do we need to discuss anything else?
15:41:40 <jjasek_> Not from me, thanks.
15:42:19 <tmazur> Okay, let's wrap it up then
15:42:30 <tmazur> Thanks everyone for joining and see you next week!
15:42:37 <tmazur> #endmeeting