opendevreview | Takashi Kajinami proposed openstack/horizon master: Skip tests for plugin-registry.csv update https://review.opendev.org/c/openstack/horizon/+/951109 | 01:34 |
---|---|---|
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases https://review.opendev.org/c/openstack/horizon/+/951000 | 07:30 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove line breaks to translation system https://review.opendev.org/c/openstack/horizon/+/949402 | 07:39 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove line breaks to translation system https://review.opendev.org/c/openstack/horizon/+/949402 | 07:42 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove line breaks to translation system https://review.opendev.org/c/openstack/horizon/+/949402 | 07:43 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove line breaks to translation system https://review.opendev.org/c/openstack/horizon/+/949402 | 07:46 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove line breaks to translation system https://review.opendev.org/c/openstack/horizon/+/949402 | 07:47 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove line breaks to translation system https://review.opendev.org/c/openstack/horizon/+/949402 | 07:48 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove line breaks to translation system https://review.opendev.org/c/openstack/horizon/+/949402 | 07:51 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove line breaks to translation system https://review.opendev.org/c/openstack/horizon/+/949402 | 07:57 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases https://review.opendev.org/c/openstack/horizon/+/951000 | 08:43 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases https://review.opendev.org/c/openstack/horizon/+/951000 | 08:43 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases https://review.opendev.org/c/openstack/horizon/+/951000 | 08:51 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases https://review.opendev.org/c/openstack/horizon/+/951000 | 10:24 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Remove spaces from notifications phrases https://review.opendev.org/c/openstack/horizon/+/951000 | 10:28 |
opendevreview | Radomir 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/+/951753 | 10:37 |
opendevreview | Ivan 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/+/951753 | 12:03 |
*** ralonsoh_ is now known as ralonsoh | 12:36 | |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1 https://review.opendev.org/c/openstack/horizon/+/950966 | 13:03 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1 https://review.opendev.org/c/openstack/horizon/+/950966 | 13:06 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1 https://review.opendev.org/c/openstack/horizon/+/950966 | 13:06 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1 https://review.opendev.org/c/openstack/horizon/+/950966 | 13:18 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1 https://review.opendev.org/c/openstack/horizon/+/950966 | 13:18 |
opendevreview | Ivan Anfimov proposed openstack/horizon master: Instances replaced to Servers | Part 1 https://review.opendev.org/c/openstack/horizon/+/950966 | 13:20 |
opendevreview | Takashi Kajinami proposed openstack/horizon master: Allow adding a subnet without gateway ip to a router https://review.opendev.org/c/openstack/horizon/+/794084 | 14:13 |
anfimovir | Hello | 15:00 |
rdopiera | o/ | 15:00 |
tmazur | #startmeeting horizon | 15:00 |
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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
opendevmeet | The meeting name has been set to 'horizon' | 15:00 |
jjasek | Hello everyone :-) | 15:00 |
tmazur | Hello everyone and welcome to the weekly meeting! | 15:01 |
tmazur | Let's start | 15:05 |
tmazur | #topic Notices | 15:05 |
tmazur | Today's agenda: https://etherpad.opendev.org/p/horizon-release-priorities#L40 | 15:05 |
tmazur | This week is R-17 | 15:05 |
tmazur | Flamingo Schedule: https://releases.openstack.org/flamingo/schedule.html | 15:06 |
tmazur | No other announcements, so moving forward | 15:07 |
tmazur | #topic Release priorities | 15:07 |
tmazur | https://etherpad.opendev.org/p/horizon-release-priorities#L44 | 15:07 |
tmazur | There's two SDK patches on review: | 15:08 |
tmazur | 946269: SDK for Neutron Security Groups | https://review.opendev.org/c/openstack/horizon/+/946269 | 15:08 |
tmazur | 949764: SDK for Neutron QoS | https://review.opendev.org/c/openstack/horizon/+/949764 | 15:08 |
tmazur | And there's two patches on credentials: | 15:09 |
tmazur | 951409: pytest-based selenium tests for 2FA | https://review.opendev.org/c/openstack/horizon/+/951409 | 15:09 |
anfimovir | Second SDK not ready, wait finish works from author. | 15:09 |
tmazur | Oh I see this one is already being merged, thanks rdopiera | 15:09 |
tmazur | anfimovir I thought the author removed WIP tag? Need to take a closer look | 15:10 |
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 |
tmazur | This one needs dependencies added | 15:11 |
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 |
tmazur | So team, please review 951580: Add qrcode | https://review.opendev.org/c/openstack/requirements/+/951580 | 15:11 |
anfimovir | tmazur, ok, now run test again for received clean results | 15:11 |
tmazur | jjasek, thanks! | 15:12 |
opendevreview | Merged openstack/horizon master: pytest-based selenium tests for 2FA https://review.opendev.org/c/openstack/horizon/+/951409 | 15:13 |
tmazur | Perfect timing :D | 15:14 |
jjasek | :D | 15:14 |
tmazur | Next topic | 15:14 |
tmazur | #topic Bug deputy report | 15:14 |
anfimovir | Yes, one question from provios week... Buth MR for one problem, in one -1 in another +2 | 15:16 |
tmazur | I had no time to look into them last week | 15:16 |
anfimovir | rdopiera, hello what do you think about? | 15:17 |
tmazur | They are identical | 15:18 |
anfimovir | Yes, this little strange ;) | 15:18 |
rdopiera | I think I'm missing something, I have no idea what you are talking about. | 15:18 |
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 |
anfimovir | In first you +2, in second -1 =) | 15:19 |
tmazur | It was a discussion about implementing server-side pagination | 15:20 |
tmazur | To make a proper solution | 15:20 |
tmazur | However Swift code in angular based in Horizon, and pretty unique | 15:21 |
anfimovir | Yes, but the fix suggested there solves some of the problems and can be used. | 15:21 |
rdopiera | anfimovir: well, I agree with Akihiro's points, which I didn't think about when I first gave my +2 | 15:23 |
rdopiera | anfimovir: more like masks than solves | 15:23 |
anfimovir | Yes, from this side you are right ;) | 15:24 |
rdopiera | as Dijsktra used to say, if correctness is not required, then any response will do | 15:25 |
anfimovir | can't argue with that =) | 15:27 |
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:29 |
tmazur | Let's move to the next topic | 15:30 |
tmazur | #topic Open reviews | 15:31 |
tmazur | http://etherpad.opendev.org/p/horizon-release-priorities#L58 | 15:31 |
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 |
anfimovir | Please merge both, this clean backport. | 15:31 |
tmazur | Huge list of open reviews, thanks anfimovir | 15:31 |
anfimovir | tmazur, please find time for 920959: Support choice of volume type in instance create | https://review.opendev.org/c/openstack/horizon/+/920959 | 15:34 |
anfimovir | And we need continue update documentation. | 15:35 |
tmazur | Will review it later today | 15:35 |
anfimovir | Thank you. | 15:36 |
tmazur | Cool. | 15:37 |
tmazur | Let's move to the next topic | 15:38 |
tmazur | #topic Open discussion | 15:38 |
tmazur | So we have this discussion in ML: | 15:38 |
tmazur | https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/W73J7ZHA6ZZJVYLOUV3FZQFTAKD6MMMQ/ | 15:38 |
anfimovir | Yes, opened from our side. | 15:39 |
anfimovir | Ops.... | 15:40 |
anfimovir | Sorry, I forgot... | 15:40 |
anfimovir | Radomir, can you look comments in https://review.opendev.org/c/openstack/requirements/+/950400 - mr. Jens said problem from another side... | 15:41 |
anfimovir | tmazur, what do you think about question with instances/servers? We wait PGT or...?) | 15:42 |
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 |
rdopiera | anfimovir: yeah, I have no idea what to do there | 15:44 |
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 |
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:45 |
anfimovir | @jjasek @tmazur, well, no problem, wait PTG. | 15:46 |
tmazur | I'll add to ML about this | 15:47 |
anfimovir | @tmazur, thank you, if you find time please look https://review.opendev.org/c/openstack/requirements/+/950400 | 15:47 |
tmazur | Looking right now, no idea so far | 15:48 |
tmazur | Why it says it depends on the Horizon patch? | 15:49 |
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 |
rdopiera | tmazur: because otherwise the tests fail | 15:50 |
tmazur | I see, thanks | 15:50 |
omcgonag | my two cents on the naming of Instances topic in Horizon - I agree, wait until PTG | 15:52 |
omcgonag | I need to drop, cya | 15:57 |
tmazur | Do we want anything else to discuss today? | 15:57 |
tmazur | If not, let's wrap it up | 15:58 |
anfimovir | No, have a good week | 15:58 |
tmazur | Thanks everyone for joining and see you next week! | 15:58 |
tmazur | #endmeeting | 15:58 |
opendevmeet | Meeting ended Wed Jun 4 15:58:37 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:58 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/horizon/2025/horizon.2025-06-04-15.00.html | 15:58 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/horizon/2025/horizon.2025-06-04-15.00.txt | 15:58 |
opendevmeet | Log: https://meetings.opendev.org/meetings/horizon/2025/horizon.2025-06-04-15.00.log.html | 15:58 |
opendevreview | Merged openstack/horizon master: Support choice of volume type in instance create https://review.opendev.org/c/openstack/horizon/+/920959 | 20:49 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!