| johnsom | gmann ack | 00:55 |
|---|---|---|
| johnsom | I will poke at that patch a little later tonight | 01:53 |
| johnsom | Ah, it looks like you are already looking at it. | 02:09 |
| opendevreview | Ghanshyam proposed openstack/designate-tempest-plugin master: Add project_reader in new RBAC tests https://review.opendev.org/c/openstack/designate-tempest-plugin/+/871215 | 02:17 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Move to a batch model for incrementing serial https://review.opendev.org/c/openstack/designate/+/871255 | 05:25 |
| opendevreview | Arkady Shtempler proposed openstack/designate-tempest-plugin master: New boundray scenario tests for a Quota test suite https://review.opendev.org/c/openstack/designate-tempest-plugin/+/860935 | 10:07 |
| opendevreview | Elod Illes proposed openstack/designate-dashboard master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/designate-dashboard/+/874876 | 10:44 |
| opendevreview | Arkady Shtempler proposed openstack/designate-tempest-plugin master: New boundray scenario tests for a Quota test suite https://review.opendev.org/c/openstack/designate-tempest-plugin/+/860935 | 13:20 |
| opendevreview | Michael Johnson proposed openstack/designate-tempest-plugin master: Add tempest tests for shared zones https://review.opendev.org/c/openstack/designate-tempest-plugin/+/730370 | 15:29 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Move to a batch model for incrementing serial https://review.opendev.org/c/openstack/designate/+/871255 | 18:00 |
| gmann | johnsom: it seems project member and reader are able to get the list items instead of [] https://review.opendev.org/c/openstack/designate-tempest-plugin/+/871215 | 18:47 |
| gmann | I did not dig into the code flow but it seems admin cannot get but project member/reader can get list items | 18:48 |
| johnsom | gmann Hi, I'm juggling two RBAC issues, so splitting my time a bit. Yeah, I'm wondering if project member and project reader have the same project ID as os_primary???? That would cause this. | 18:51 |
| johnsom | I just haven't had a minute to test that theory | 18:51 |
| gmann | johnsom: oh, yes they do have same project id as os_primary | 18:51 |
| gmann | johnsom: so in my tempest change I am keeping os_admin as legacy admin with separate project_id and 'os_promary, project_admin, project_member, project_reader' will have same project id | 18:52 |
| johnsom | Ok... So, I guess the question is if that is the "right" expected thing, or not. It's obviously an easy fix in Designate. | 18:52 |
| gmann | os_promary is nothing but the member role in project so it should be same as project_member | 18:52 |
| gmann | any member role in projectA either it is via os_primary should be within same project | 18:53 |
| gmann | for another project we can use os_alt, oa_alt_project_member and so on | 18:53 |
| johnsom | Yeah, ok, I can be ok with that. Let me take another look at our tests and update the patch. | 18:55 |
| gmann | johnsom: I think here we need to create zone with os_alt right ? https://review.opendev.org/c/openstack/designate-tempest-plugin/+/871215/9/designate_tempest_plugin/tests/api/v2/test_zones.py#b227 | 18:55 |
| gmann | johnsom: thanks, I will leave those updates to you. | 18:56 |
| johnsom | Yeah, I think your change is correct, I just need to verify. I will post something in the next 30 minutes | 18:56 |
| gmann | sure. | 18:59 |
| opendevreview | Michael Johnson proposed openstack/designate-tempest-plugin master: Add project_reader in new RBAC tests https://review.opendev.org/c/openstack/designate-tempest-plugin/+/871215 | 19:05 |
| opendevreview | Michael Johnson proposed openstack/designate-tempest-plugin master: Add project_reader in new RBAC tests https://review.opendev.org/c/openstack/designate-tempest-plugin/+/871215 | 19:07 |
| johnsom | gmann We are ready to merge that | 20:20 |
| opendevreview | Michael Johnson proposed openstack/designate master: DNM: Testing openstackdoctheme fix https://review.opendev.org/c/openstack/designate/+/874961 | 20:24 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate stable/zed: [CI] Reduce workers and disable cinder to save on memory https://review.opendev.org/c/openstack/designate/+/874840 | 20:48 |
| gmann | johnsom: perfect, will get tempest one merged which is in gate too https://review.opendev.org/c/openstack/tempest/+/871018 | 21:39 |
| opendevreview | Merged openstack/designate-tempest-plugin master: Add tempest tests for shared zones https://review.opendev.org/c/openstack/designate-tempest-plugin/+/730370 | 22:34 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!