Wednesday, 2025-06-04

opendevreviewTakashi Kajinami proposed openstack/horizon master: Skip tests for plugin-registry.csv update  https://review.opendev.org/c/openstack/horizon/+/95110901:34
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases   https://review.opendev.org/c/openstack/horizon/+/95100007:30
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove line breaks to translation system  https://review.opendev.org/c/openstack/horizon/+/94940207:39
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove line breaks to translation system  https://review.opendev.org/c/openstack/horizon/+/94940207:42
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove line breaks to translation system  https://review.opendev.org/c/openstack/horizon/+/94940207:43
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove line breaks to translation system  https://review.opendev.org/c/openstack/horizon/+/94940207:46
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove line breaks to translation system  https://review.opendev.org/c/openstack/horizon/+/94940207:47
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove line breaks to translation system  https://review.opendev.org/c/openstack/horizon/+/94940207:48
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove line breaks to translation system  https://review.opendev.org/c/openstack/horizon/+/94940207:51
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove line breaks to translation system  https://review.opendev.org/c/openstack/horizon/+/94940207:57
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases   https://review.opendev.org/c/openstack/horizon/+/95100008:43
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases  https://review.opendev.org/c/openstack/horizon/+/95100008:43
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases  https://review.opendev.org/c/openstack/horizon/+/95100008:51
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases  https://review.opendev.org/c/openstack/horizon/+/95100010:24
opendevreviewIvan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases  https://review.opendev.org/c/openstack/horizon/+/95100010:28
opendevreviewRadomir Dopieralski proposed openstack/horizon master: Don't hide region and domain on login screen when using other auth methods  https://review.opendev.org/c/openstack/horizon/+/95175310:37
opendevreviewIvan Anfimov proposed openstack/horizon master: Don't hide region and domain on login screen when using other auth methods  https://review.opendev.org/c/openstack/horizon/+/95175312:03
*** ralonsoh_ is now known as ralonsoh12:36
opendevreviewIvan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1  https://review.opendev.org/c/openstack/horizon/+/95096613:03
opendevreviewIvan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1  https://review.opendev.org/c/openstack/horizon/+/95096613:06
opendevreviewIvan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1  https://review.opendev.org/c/openstack/horizon/+/95096613:06
opendevreviewIvan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1  https://review.opendev.org/c/openstack/horizon/+/95096613:18
opendevreviewIvan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1  https://review.opendev.org/c/openstack/horizon/+/95096613:18
opendevreviewIvan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1  https://review.opendev.org/c/openstack/horizon/+/95096613:20
opendevreviewTakashi Kajinami proposed openstack/horizon master: Allow adding a subnet without gateway ip to a router  https://review.opendev.org/c/openstack/horizon/+/79408414:13
anfimovirHello15:00
rdopierao/15:00
tmazur#startmeeting horizon15:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
opendevmeetThe meeting name has been set to 'horizon'15:00
jjasekHello everyone :-)15:00
tmazurHello everyone and welcome to the weekly meeting!15:01
tmazurLet's start15:05
tmazur#topic Notices15:05
tmazurToday's agenda: https://etherpad.opendev.org/p/horizon-release-priorities#L4015:05
tmazurThis week is R-1715:05
tmazurFlamingo Schedule: https://releases.openstack.org/flamingo/schedule.html 15:06
tmazurNo other announcements, so moving forward15:07
tmazur#topic Release priorities15:07
tmazurhttps://etherpad.opendev.org/p/horizon-release-priorities#L4415:07
tmazurThere's two SDK patches on review:15:08
tmazur946269: SDK for Neutron Security Groups | https://review.opendev.org/c/openstack/horizon/+/94626915:08
tmazur949764: SDK for Neutron QoS | https://review.opendev.org/c/openstack/horizon/+/94976415:08
tmazurAnd there's two patches on credentials:15:09
tmazur951409: pytest-based selenium tests for 2FA | https://review.opendev.org/c/openstack/horizon/+/95140915:09
anfimovirSecond SDK not ready, wait finish works from author.15:09
tmazurOh I see this one is already being merged, thanks rdopiera15:09
tmazuranfimovir I thought the author removed WIP tag? Need to take a closer look15:10
tmazurAnyway, one more on credentials: 951433: Add credential details with QR code for TOTP | https://review.opendev.org/c/openstack/horizon/+/95143315:11
tmazurThis one needs dependencies added15:11
jjasekFirst 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
tmazurSo team, please review 951580: Add qrcode | https://review.opendev.org/c/openstack/requirements/+/95158015:11
anfimovirtmazur, ok, now run test again for received clean results 15:11
tmazurjjasek, thanks!15:12
opendevreviewMerged openstack/horizon master: pytest-based selenium tests for 2FA  https://review.opendev.org/c/openstack/horizon/+/95140915:13
tmazurPerfect timing :D15:14
jjasek:D15:14
tmazurNext topic15:14
tmazur#topic Bug deputy report15:14
anfimovirYes, one question from provios week... Buth MR for one problem, in one -1 in another +215:16
tmazurI had no time to look into them last week15:16
anfimovirrdopiera, hello what do you think about?15:17
tmazurThey are identical15:18
anfimovirYes, this little strange ;)15:18
rdopieraI think I'm missing something, I have no idea what you are talking about.15:18
anfimovir790504: 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/+/92747815:19
anfimovirIn first you +2, in second -1 =)15:19
tmazurIt was a discussion about implementing server-side pagination15:20
tmazurTo make a proper solution15:20
tmazurHowever Swift code in angular based in Horizon, and pretty unique15:21
anfimovirYes, but the fix suggested there solves some of the problems and can be used.15:21
rdopieraanfimovir: well, I agree with Akihiro's points, which I didn't think about when I first gave my +215:23
rdopieraanfimovir: more like masks than solves15:23
anfimovirYes, from this side you are right ;)15:24
rdopieraas Dijsktra used to say, if correctness is not required, then any response will do15:25
anfimovir can't argue with that =)15:27
tmazurWe will look into implementation of server side pagination when we come to rewriting swift code in python as part of our general effort15:29
tmazurLet's move to the next topic15:30
tmazur#topic Open reviews15:31
tmazurhttp://etherpad.opendev.org/p/horizon-release-priorities#L5815:31
anfimovir951722: 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/+/95172315:31
anfimovirPlease merge both, this clean backport.15:31
tmazurHuge list of open reviews, thanks anfimovir15:31
anfimovirtmazur, please find time for 920959: Support choice of volume type in instance create | https://review.opendev.org/c/openstack/horizon/+/92095915:34
anfimovirAnd we need continue update documentation.15:35
tmazurWill review it later today15:35
anfimovirThank you.15:36
tmazurCool.15:37
tmazurLet's move to the next topic15:38
tmazur#topic Open discussion15:38
tmazurSo we have this discussion in ML:15:38
tmazurhttps://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/W73J7ZHA6ZZJVYLOUV3FZQFTAKD6MMMQ/15:38
anfimovirYes, opened from our side.15:39
anfimovirOps....15:40
anfimovirSorry, I forgot...15:40
anfimovirRadomir, can you look comments in https://review.opendev.org/c/openstack/requirements/+/950400 - mr. Jens said problem from another side...15:41
anfimovirtmazur, what do you think about question with instances/servers? We wait PGT or...?)15:42
tmazurIt'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 term15:44
rdopieraanfimovir: yeah, I have no idea what to do there15:44
jjasekMy 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
rdopieraanfimovir: 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 works15:45
anfimovir@jjasek @tmazur, well, no problem, wait PTG.15:46
tmazurI'll add to ML about this15:47
anfimovir@tmazur, thank you, if you find time please look https://review.opendev.org/c/openstack/requirements/+/95040015:47
tmazurLooking right now, no idea so far15:48
tmazurWhy it says it depends on the Horizon patch?15:49
rdopieraI 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
rdopieratmazur: because otherwise the tests fail15:50
tmazurI see, thanks15:50
omcgonagmy two cents on the naming of Instances topic in Horizon - I agree, wait until PTG15:52
omcgonagI need to drop, cya15:57
tmazurDo we want anything else to discuss today?15:57
tmazurIf not, let's wrap it up15:58
anfimovirNo, have a good week15:58
tmazurThanks everyone for joining and see you next week!15:58
tmazur#endmeeting15:58
opendevmeetMeeting ended Wed Jun  4 15:58:37 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:58
opendevmeetMinutes:        https://meetings.opendev.org/meetings/horizon/2025/horizon.2025-06-04-15.00.html15:58
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/horizon/2025/horizon.2025-06-04-15.00.txt15:58
opendevmeetLog:            https://meetings.opendev.org/meetings/horizon/2025/horizon.2025-06-04-15.00.log.html15:58
opendevreviewMerged openstack/horizon master: Support choice of volume type in instance create  https://review.opendev.org/c/openstack/horizon/+/92095920:49

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!