| opendevreview | ali proposed openstack/horizon master: Add dark theme support for Horizon dashboard https://review.opendev.org/c/openstack/horizon/+/997929 | 06:31 |
|---|---|---|
| opendevreview | Radomir Dopieralski proposed openstack/horizon master: Don't pass domain_id to policy's credentials by default https://review.opendev.org/c/openstack/horizon/+/997715 | 08:06 |
| opendevreview | Radomir Dopieralski proposed openstack/horizon master: Don't pass domain_id to policy's credentials by default https://review.opendev.org/c/openstack/horizon/+/997715 | 08:07 |
| opendevreview | Radomir Dopieralski proposed openstack/horizon master: Don't pass domain_id to policy's credentials by default https://review.opendev.org/c/openstack/horizon/+/997715 | 08:12 |
| *** M00SE7 is now known as M00SE | 08:30 | |
| opendevreview | Radomir Dopieralski proposed openstack/horizon master: Don't pass domain_id to policy's credentials by default https://review.opendev.org/c/openstack/horizon/+/997715 | 08:50 |
| jjasek | Hello Everyone who contributes to Horizon. If you are facing a failing integration test (test_create_flavor, because of “Unable to locate element: Create Flavor”) in your patch, the reason is update to oslo policy 6.0.0: https://bugs.launchpad.net/oslo.policy/+bug/2161292 | 09:08 |
| jjasek | No need to try to recheck your patch again and again. We are working on a fix. Thanks! | 09:08 |
| opendevreview | Radomir Dopieralski proposed openstack/horizon master: Don't pass domain_id to policy's credentials by default https://review.opendev.org/c/openstack/horizon/+/997715 | 09:35 |
| opendevreview | Merged openstack/horizon master: Don't pass domain_id to policy's credentials by default https://review.opendev.org/c/openstack/horizon/+/997715 | 12:17 |
| opendevreview | Merged openstack/horizon master: Imported Translations from Zanata https://review.opendev.org/c/openstack/horizon/+/997673 | 13:15 |
| tkajinam | jjasek, you might need to consult gmaan about that issue caused by policy scope enforcement. I've left a comment in 997715 but that "fix" effectively disables policy check and we need a better solution for it. | 13:49 |
| tkajinam | (I'm pinging you because Radomir and Tatiana are not in irc now and you may be the only core online at this moment. | 13:50 |
| tkajinam | we probably need to understand better about the scope/credential handling for domain-level access. if it assumes that domain-scoped token can access project-scoped resources then that's no longer possible with scope isolation and that likely brings a large problem. | 13:51 |
| tkajinam | I also left a similar comment in the bug Radomir reported | 13:51 |
| *** noonedeadpunk is now known as noonedeadpunk_ | 13:51 | |
| *** noonedeadpunk_ is now known as noonedeadpunk | 13:51 | |
| jjasek | Hi tkajinam, yes, the fix that Radomir merged few hours ago is only to unblock a gate. We are planning to discuss how to fix it properly tomorrow. | 13:54 |
| tkajinam | ok, thansk | 13:54 |
| tkajinam | I just wanted to make you all aware that that temporal fix voids policy check, and should be fixed again before a release | 13:55 |
| jjasek | Yes, we know about it, we wanted to unblock a gate so everything else (other non related patches) are not completely blocked. | 13:59 |
| jjasek | And it is definitely not final fix. | 13:59 |
| jjasek | Thank you for your activity and being around! tkajinam | 14:01 |
| tkajinam | :-) | 14:22 |
| tkajinam | my pleasure | 14:22 |
| gmaan | jjasek: tkajinam this is what we need but it seems horizon need more work for SRBAC https://review.opendev.org/c/openstack/horizon/+/927342 | 14:41 |
| opendevreview | Jan Jasek proposed openstack/horizon master: WIP: Add periodic job catching risky requirements changes early https://review.opendev.org/c/openstack/horizon/+/998068 | 21:16 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!