15:00:50 <tmazur> #startmeeting horizon
15:00:50 <opendevmeet> Meeting started Wed Jun  4 15:00:50 2025 UTC and is due to finish in 60 minutes.  The chair is tmazur. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:00:50 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:00:50 <opendevmeet> The meeting name has been set to 'horizon'
15:00:53 <jjasek> Hello everyone :-)
15:01:04 <tmazur> Hello everyone and welcome to the weekly meeting!
15:05:00 <tmazur> Let's start
15:05:02 <tmazur> #topic Notices
15:05:47 <tmazur> Today's agenda: https://etherpad.opendev.org/p/horizon-release-priorities#L40
15:05:55 <tmazur> This week is R-17
15:06:03 <tmazur> Flamingo Schedule: https://releases.openstack.org/flamingo/schedule.html
15:07:34 <tmazur> No other announcements, so moving forward
15:07:44 <tmazur> #topic Release priorities
15:07:55 <tmazur> https://etherpad.opendev.org/p/horizon-release-priorities#L44
15:08:11 <tmazur> There's two SDK patches on review:
15:08:16 <tmazur> 946269: SDK for Neutron Security Groups | https://review.opendev.org/c/openstack/horizon/+/946269
15:08:21 <tmazur> 949764: SDK for Neutron QoS | https://review.opendev.org/c/openstack/horizon/+/949764
15:09:10 <tmazur> And there's two patches on credentials:
15:09:15 <tmazur> 951409: pytest-based selenium tests for 2FA | https://review.opendev.org/c/openstack/horizon/+/951409
15:09:31 <anfimovir> Second SDK not ready, wait finish works from author.
15:09:47 <tmazur> Oh I see this one is already being merged, thanks rdopiera
15:10:38 <tmazur> anfimovir I thought the author removed WIP tag? Need to take a closer look
15:11:07 <tmazur> Anyway, one more on credentials: 951433: Add credential details with QR code for TOTP | https://review.opendev.org/c/openstack/horizon/+/951433
15:11:17 <tmazur> This one needs dependencies added
15:11:25 <jjasek> First SDK breaks instance creating so I reviewed it as -1 and I would not merge it until the changes in OpenstackSDK are done.
15:11:29 <tmazur> So team, please review 951580: Add qrcode | https://review.opendev.org/c/openstack/requirements/+/951580
15:11:50 <anfimovir> tmazur, ok, now run test again for received clean results
15:12:30 <tmazur> jjasek, thanks!
15:13:52 <opendevreview> Merged openstack/horizon master: pytest-based selenium tests for 2FA  https://review.opendev.org/c/openstack/horizon/+/951409
15:14:24 <tmazur> Perfect timing :D
15:14:28 <jjasek> :D
15:14:40 <tmazur> Next topic
15:14:51 <tmazur> #topic Bug deputy report
15:16:12 <anfimovir> Yes, one question from provios week... Buth MR for one problem, in one -1 in another +2
15:16:51 <tmazur> I had no time to look into them last week
15:17:16 <anfimovir> rdopiera, hello what do you think about?
15:18:09 <tmazur> They are identical
15:18:27 <anfimovir> Yes, this little strange ;)
15:18:42 <rdopiera> I think I'm missing something, I have no idea what you are talking about.
15:19:21 <anfimovir> 790504: Do not send full_listing=True to swift client | https://review.opendev.org/c/openstack/horizon/+/790504  927478: Set full_listing=False for swiftclient API calls | https://review.opendev.org/c/openstack/horizon/+/927478
15:19:38 <anfimovir> In first you +2, in second -1 =)
15:20:17 <tmazur> It was a discussion about implementing server-side pagination
15:20:23 <tmazur> To make a proper solution
15:21:09 <tmazur> However Swift code in angular based in Horizon, and pretty unique
15:21:49 <anfimovir> Yes, but the fix suggested there solves some of the problems and can be used.
15:23:28 <rdopiera> anfimovir: well, I agree with Akihiro's points, which I didn't think about when I first gave my +2
15:23:57 <rdopiera> anfimovir: more like masks than solves
15:24:16 <anfimovir> Yes, from this side you are right ;)
15:25:14 <rdopiera> as Dijsktra used to say, if correctness is not required, then any response will do
15:27:45 <anfimovir> can't argue with that =)
15:29:16 <tmazur> We will look into implementation of server side pagination when we come to rewriting swift code in python as part of our general effort
15:30:42 <tmazur> Let's move to the next topic
15:31:02 <tmazur> #topic Open reviews
15:31:16 <tmazur> http://etherpad.opendev.org/p/horizon-release-priorities#L58
15:31:16 <anfimovir> 951722: Fix security group renderring in port overview (4) | https://review.opendev.org/c/openstack/horizon/+/951722 951723: Fix security group renderring in port overview (4) | https://review.opendev.org/c/openstack/horizon/+/951723
15:31:31 <anfimovir> Please merge both, this clean backport.
15:31:38 <tmazur> Huge list of open reviews, thanks anfimovir
15:34:30 <anfimovir> tmazur, please find time for 920959: Support choice of volume type in instance create | https://review.opendev.org/c/openstack/horizon/+/920959
15:35:02 <anfimovir> And we need continue update documentation.
15:35:24 <tmazur> Will review it later today
15:36:22 <anfimovir> Thank you.
15:37:53 <tmazur> Cool.
15:38:04 <tmazur> Let's move to the next topic
15:38:16 <tmazur> #topic Open discussion
15:38:35 <tmazur> So we have this discussion in ML:
15:38:36 <tmazur> https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/W73J7ZHA6ZZJVYLOUV3FZQFTAKD6MMMQ/
15:39:34 <anfimovir> Yes, opened from our side.
15:40:46 <anfimovir> Ops....
15:40:51 <anfimovir> Sorry, I forgot...
15:41:21 <anfimovir> Radomir, can you look comments in https://review.opendev.org/c/openstack/requirements/+/950400 - mr. Jens said problem from another side...
15:42:55 <anfimovir> tmazur, what do you think about question with instances/servers? We wait PGT or...?)
15:44:39 <tmazur> It's not just about Horizon code. I'd like to organize a proper discussion with people from other projects. And it's not just about code, it's about documentation as well. Simple look through OpenStack pages finds lots of, say, Nova pages using Instance term
15:44:44 <rdopiera> anfimovir: yeah, I have no idea what to do there
15:45:12 <jjasek> My comment to the topic about naming of Instances in Horizon - I saw many very good arguments in the thread from people whose opinion is valuable in this topic and for me it looks like it will bring more mess than value to rename it. So I would stop this discussion at least for now and possibly open it during PTG.
15:45:29 <rdopiera> anfimovir: this is a dependency cycle but it's not a real dependency cycle for horizon, it's just a figment of how the upper constraints works
15:46:30 <anfimovir> @jjasek @tmazur, well, no problem, wait PTG.
15:47:08 <tmazur> I'll add to ML about this
15:47:43 <anfimovir> @tmazur, thank you, if you find time please look https://review.opendev.org/c/openstack/requirements/+/950400
15:48:40 <tmazur> Looking right now, no idea so far
15:49:49 <tmazur> Why it says it depends on the Horizon patch?
15:50:22 <rdopiera> I don't like the proposed solution to make it work with both versions, because that's just garbage in our code for no good reason, but maybe we could make it work with both versions temporarily, and when the upper constraints merges, remove that?
15:50:47 <rdopiera> tmazur: because otherwise the tests fail
15:50:58 <tmazur> I see, thanks
15:52:07 <omcgonag> my two cents on the naming of Instances topic in Horizon - I agree, wait until PTG
15:57:45 <omcgonag> I need to drop, cya
15:57:58 <tmazur> Do we want anything else to discuss today?
15:58:07 <tmazur> If not, let's wrap it up
15:58:22 <anfimovir> No, have a good week
15:58:28 <tmazur> Thanks everyone for joining and see you next week!
15:58:37 <tmazur> #endmeeting