| opendevreview | OpenStack Proposal Bot proposed openstack/horizon master: Imported Translations from Zanata https://review.opendev.org/c/openstack/horizon/+/858599 | 02:15 |
|---|---|---|
| *** tobias-urdin6 is now known as tobias-urdin | 08:49 | |
| opendevreview | OpenStack Release Bot proposed openstack/horizon stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/horizon/+/858691 | 12:08 |
| opendevreview | OpenStack Release Bot proposed openstack/horizon stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/horizon/+/858692 | 12:08 |
| opendevreview | OpenStack Release Bot proposed openstack/horizon master: Update master for stable/zed https://review.opendev.org/c/openstack/horizon/+/858693 | 12:08 |
| opendevreview | OpenStack Release Bot proposed openstack/horizon master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/horizon/+/858694 | 12:08 |
| opendevreview | Merged openstack/horizon master: Imported Translations from Zanata https://review.opendev.org/c/openstack/horizon/+/858599 | 14:57 |
| opendevreview | Merged openstack/horizon stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/horizon/+/858691 | 14:57 |
| vishalmanchanda | #startmeeting horizon | 15:00 |
| opendevmeet | Meeting started Wed Sep 21 15:00:33 2022 UTC and is due to finish in 60 minutes. The chair is vishalmanchanda. 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 |
| tmazur | o/ | 15:00 |
| opendevreview | Merged openstack/horizon stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/horizon/+/858692 | 15:01 |
| opendevreview | Merged openstack/horizon master: Update master for stable/zed https://review.opendev.org/c/openstack/horizon/+/858693 | 15:01 |
| ganso | o/ | 15:01 |
| rdopiera | o/ | 15:01 |
| vishalmanchanda | Hello everyone | 15:01 |
| vishalmanchanda | ok let's start the meeting | 15:02 |
| vishalmanchanda | Agenda of meeting can be found here https://etherpad.opendev.org/p/horizon-release-priorities#L39 | 15:02 |
| vishalmanchanda | #topic Notices | 15:03 |
| vishalmanchanda | few announcements from my side | 15:03 |
| vishalmanchanda | As you may noticed, a stable/zed branch is created for horizon. | 15:03 |
| vishalmanchanda | https://review.opendev.org/c/openstack/horizon/+/858691 | 15:03 |
| vishalmanchanda | Now master branch is Antelope(2023.1) | 15:03 |
| vishalmanchanda | Now if any bug you wants to fix in zed cycle you have to backport it to stable/zed branch. | 15:04 |
| vishalmanchanda | moving to next announcement | 15:05 |
| vishalmanchanda | TC & PTL election result is out. | 15:05 |
| vishalmanchanda | For more info, please refer https://lists.openstack.org/pipermail/openstack-discuss/2022-September/030541.html | 15:05 |
| vishalmanchanda | that's all announcements from my side. if anyone have any announcement to make, please go ahead. | 15:06 |
| vishalmanchanda | moving to next topic | 15:07 |
| vishalmanchanda | #topic On-Demand Agenda | 15:08 |
| vishalmanchanda | there is a request for patch review https://review.opendev.org/c/openstack/horizon/+/856308 | 15:08 |
| ganso | that's ^ me =) | 15:08 |
| vishalmanchanda | ganso: that's you | 15:08 |
| ganso | so, I wanted to discuss it as it might be a controversial bug | 15:09 |
| vishalmanchanda | ganso: honestly I am not sure about the bug | 15:09 |
| ganso | and/or a controversial fix | 15:09 |
| ganso | vishalmanchanda: yup I suspect that might be the case =) | 15:09 |
| vishalmanchanda | Just before the meeting I try to resproduce it but it's work fine for me. | 15:09 |
| ganso | vishalmanchanda: did you try as a cloud adminsitrator or domain adminsitrator? | 15:10 |
| vishalmanchanda | I am unable to reproduce this bug https://bugs.launchpad.net/ubuntu/+source/horizon/+bug/1827120 | 15:10 |
| vishalmanchanda | cloud adminsitrator | 15:10 |
| vishalmanchanda | I have also done some changes in my keystone.conf file | 15:11 |
| vishalmanchanda | [resource] | 15:11 |
| vishalmanchanda | admin_project_domain_name = admin_domain | 15:11 |
| vishalmanchanda | admin_project_name = admin | 15:11 |
| ganso | vishalmanchanda: so, I don't remember all the details, but I remember I wasn't able to reproduce it early on as well, but then upon adjusting the roles I was able to | 15:12 |
| ganso | and I reproduced it on Yoga, so I'm not sure about master | 15:12 |
| ganso | but, the main use case here is not as a cloud admin | 15:12 |
| ganso | because the cloud admin has an easy workaround | 15:12 |
| ganso | that I wasn't able to find for domain admins | 15:12 |
| ganso | and looking at the code, the bug is very clear | 15:12 |
| ganso | because every time there is a domain_token, that overrides the user_token, and it sends that token to keystone when doing authentication | 15:13 |
| ganso | keystone, using that token, will not include a project_id context | 15:13 |
| ganso | so it is about which token you send to keystone, as the token contain the context, it doesn't seem to me as a keystone fault | 15:14 |
| ganso | the trick to reproduce the issue, is that the user HAS to have a domain admin role | 15:14 |
| ganso | if it is just a user that is member of a project, it always works. When the user becomes the domain admin, it enables the bug | 15:15 |
| ganso | so, I'd love to know a workaround for the domain admin user, if there is any. The only workaround I've found is to inject the missing project_id in the database, in the app credential entry, until the fix lands | 15:16 |
| ganso | I suspect it might be possible there is a workaround by playing with all knobs | 15:16 |
| ganso | but I haven't found any | 15:16 |
| ganso | vishalmanchanda: so if you are unable to reproduce it as a non-cloud user with a domain admin role, I'd love to hear more details | 15:17 |
| vishalmanchanda | ganso: ok thanks for your input. I will definately try to reproduce it. | 15:19 |
| ganso | but the key point I intended to discuss here in the meeting was, assuming that the bug could be reproduced, is whether the bug is actually intended behavior, this is the "controversial" aspect I was referring to | 15:19 |
| ganso | because I am not sure about how things are expected to work. Maybe it was intended behavior | 15:19 |
| vishalmanchanda | ganso: I still have few doubts, will get back to you once I try again. | 15:19 |
| ganso | vishalmanchanda: thank you | 15:19 |
| vishalmanchanda | ganso: I am also not sure. | 15:19 |
| vishalmanchanda | amotoki: maybe amotoki rdopiera knows more. | 15:20 |
| vishalmanchanda | ganso: thanks for working on this. I will try to understand this bug and get back to you. | 15:24 |
| ganso | vishalmanchanda: thanks! | 15:24 |
| vishalmanchanda | ganso: if it's working with CLI then it should definately works with horizon. | 15:25 |
| ganso | vishalmanchanda: yes, with the CLI it is fine, even if the user has the domain admin role | 15:25 |
| vishalmanchanda | ganso: So you have enabled Multi domain in your env. to reproduce it? | 15:26 |
| vishalmanchanda | Do we have any other topics to discuss? | 15:30 |
| vishalmanchanda | If nothing more to discuss, Let's end this meeting. | 15:32 |
| vishalmanchanda | Thanks evryone for joining! | 15:32 |
| vishalmanchanda | #endmeeting | 15:33 |
| opendevmeet | Meeting ended Wed Sep 21 15:33:04 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:33 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/horizon/2022/horizon.2022-09-21-15.00.html | 15:33 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/horizon/2022/horizon.2022-09-21-15.00.txt | 15:33 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/horizon/2022/horizon.2022-09-21-15.00.log.html | 15:33 |
| ganso | vishalmanchanda: I am not sure I enabled it, it was already enabled for me | 15:33 |
| ganso | vishalmanchanda: I remember debugging a flag IS_MULTIDOMAIN_ENABLED and it returned true | 15:34 |
| vishalmanchanda | ganso: ok thanks. | 15:34 |
| vishalmanchanda | zigo: hi, horizon Zed(23.0.0) is released, now you can package it. | 15:35 |
| opendevreview | Merged openstack/horizon master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/horizon/+/858694 | 16:09 |
| opendevreview | Tatiana Ovchinnikova proposed openstack/horizon master: Fix flavor specification at instance overview page https://review.opendev.org/c/openstack/horizon/+/848905 | 23:29 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!