| *** rcernin has quit IRC | 02:12 | |
| *** rcernin has joined #heat | 03:17 | |
| *** vishalmanchanda has joined #heat | 04:40 | |
| *** udesale has joined #heat | 05:34 | |
| *** ttsiouts has joined #heat | 07:15 | |
| *** ttsiouts_ has joined #heat | 07:17 | |
| *** ttsiouts_ has quit IRC | 07:20 | |
| *** ttsiouts has quit IRC | 07:21 | |
| *** ttsiouts has joined #heat | 07:21 | |
| *** ttsiouts has quit IRC | 07:26 | |
| *** ttsiouts has joined #heat | 07:27 | |
| openstackgerrit | Samuel BARTEL proposed openstack/heat master: Add tenant filter from context when using List stacks API https://review.opendev.org/736746 | 07:40 |
|---|---|---|
| *** rcernin has quit IRC | 07:59 | |
| *** samuelbartel has joined #heat | 08:12 | |
| samuelbartel | Hello | 08:12 |
| samuelbartel | I am looking at following bug https://bugs.launchpad.net/heat/+bug/1757550 | 08:14 |
| openstack | Launchpad bug 1757550 in OpenStack Heat "List stacks API lists stacks from all tenants for admin" [Undecided,New] | 08:14 |
| samuelbartel | when an admin user use List stacks API, it retrieves the stacks from all users even without the --all-project option | 08:15 |
| samuelbartel | lookingat the code it seems that the all-projects option only cheange the context from user context to admin context | 08:16 |
| samuelbartel | without this option we are using the user context but the tenant_id from context is not used to scope the request | 08:17 |
| samuelbartel | I have submited a review to set by default a filter_params tenant_id to tenant_id from context when not in global_tenant use case and if the user has not explictaly set such filter_params | 08:18 |
| samuelbartel | it seems to solve the issue | 08:19 |
| samuelbartel | bu it breaks zull validation as zull is expacting filter_params oto be empty in several tests | 08:19 |
| *** gfidente has joined #heat | 08:19 | |
| samuelbartel | i woulk like to have your advice. Should it be corrected in an other way or should I submit update to zuul? | 08:20 |
| samuelbartel | looking at nova for example is such case e have a filter param project besides the context set to None for all-project and to tenant_id from context if not | 08:22 |
| *** ttsiouts has quit IRC | 08:38 | |
| *** mugsie has quit IRC | 08:48 | |
| *** mugsie has joined #heat | 08:48 | |
| *** ttsiouts has joined #heat | 08:54 | |
| *** rcernin has joined #heat | 08:54 | |
| *** rcernin has quit IRC | 09:01 | |
| *** tkajinam has quit IRC | 09:21 | |
| *** rcernin has joined #heat | 09:24 | |
| *** samuelbartel has quit IRC | 09:41 | |
| *** rcernin has quit IRC | 09:58 | |
| *** k_mouza has joined #heat | 10:32 | |
| *** rcernin has joined #heat | 10:54 | |
| *** rcernin has quit IRC | 11:08 | |
| *** rcernin has joined #heat | 11:54 | |
| *** tkajinam has joined #heat | 12:04 | |
| *** ttsiouts has quit IRC | 12:04 | |
| *** rcernin has quit IRC | 12:06 | |
| *** ttsiouts has joined #heat | 12:13 | |
| *** k_mouza has quit IRC | 12:24 | |
| *** k_mouza has joined #heat | 12:41 | |
| *** udesale_ has joined #heat | 12:44 | |
| *** udesale has quit IRC | 12:46 | |
| *** samuelbartel has joined #heat | 13:15 | |
| *** ttsiouts has quit IRC | 13:33 | |
| *** ttsiouts has joined #heat | 13:36 | |
| *** mailingsam has joined #heat | 14:12 | |
| mailingsam | Hi All, could you review https://review.opendev.org/#/c/704630/ I'll add tempest test cases once this patchset is merged | 14:13 |
| mailingsam | If any questions we can discuss here | 14:13 |
| *** tkajinam has quit IRC | 14:13 | |
| mailingsam | I updated the patchset with request to review | 14:25 |
| mailingsam | zaneb @review https://review.opendev.org/#/c/704630/ | 14:27 |
| *** ttsiouts has quit IRC | 14:55 | |
| *** dopplergange has quit IRC | 15:03 | |
| *** dopplergange has joined #heat | 15:03 | |
| *** jtomasek has quit IRC | 15:08 | |
| *** dopplergange has quit IRC | 15:32 | |
| *** mailingsam has quit IRC | 15:32 | |
| *** dopplergange has joined #heat | 15:32 | |
| *** mailingsam has joined #heat | 15:32 | |
| openstackgerrit | Samuel BARTEL proposed openstack/heat master: Add tenant filter from context when using List stacks API https://review.opendev.org/736746 | 15:42 |
| *** jmlowe has quit IRC | 15:45 | |
| AJaeger | could I get a review for this heat-agents change to update docstheme, please? https://review.opendev.org/731968 | 15:49 |
| *** ttsiouts has joined #heat | 15:51 | |
| *** ttsiouts has quit IRC | 15:56 | |
| openstackgerrit | Sam Kumar proposed openstack/heat master: S2007837: Fixes domain_id parameter in the user find call https://review.opendev.org/737040 | 15:57 |
| *** samuelbartel has quit IRC | 16:14 | |
| *** mailingsam has quit IRC | 16:20 | |
| *** udesale_ has quit IRC | 16:26 | |
| *** ChanServ has quit IRC | 16:59 | |
| *** k_mouza has quit IRC | 17:05 | |
| *** gmann is now known as gmann_afk | 17:13 | |
| openstackgerrit | Merged openstack/heat-agents master: Switch to newer openstackdocstheme and reno versions https://review.opendev.org/731968 | 17:20 |
| openstackgerrit | Sam Kumar proposed openstack/heat master: S2007837: Fixes domain_id parameter in the user find call https://review.opendev.org/737040 | 17:25 |
| *** ChanServ has joined #heat | 17:36 | |
| *** tepper.freenode.net sets mode: +o ChanServ | 17:36 | |
| *** gfidente is now known as gfidente|afk | 17:38 | |
| *** k_mouza has joined #heat | 17:49 | |
| *** ttsiouts has joined #heat | 17:52 | |
| *** k_mouza has quit IRC | 17:54 | |
| *** dopplergange has quit IRC | 18:03 | |
| *** dopplergange has joined #heat | 18:04 | |
| *** AJaeger has left #heat | 18:07 | |
| *** ttsiouts has quit IRC | 18:25 | |
| *** jmlowe has joined #heat | 18:53 | |
| *** bnemec is now known as beekneemech | 19:08 | |
| *** gmann_afk is now known as gmann | 20:15 | |
| *** vishalmanchanda has quit IRC | 20:40 | |
| *** ttsiouts has joined #heat | 21:01 | |
| *** ttsiouts has quit IRC | 21:35 | |
| *** k_mouza has joined #heat | 23:12 | |
| *** k_mouza has quit IRC | 23:16 | |
| *** ttsiouts has joined #heat | 23:32 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!