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