*** bobh has joined #openstack-mistral | 00:48 | |
* kong continue to work on transition to oslo.context | 01:23 | |
*** rbrady has quit IRC | 01:30 | |
*** rbrady has joined #openstack-mistral | 01:32 | |
*** chlong has joined #openstack-mistral | 01:43 | |
*** zhurong has joined #openstack-mistral | 01:52 | |
*** chlong has quit IRC | 02:04 | |
*** apetrich has quit IRC | 02:06 | |
*** apetrich has joined #openstack-mistral | 02:11 | |
*** gongysh has joined #openstack-mistral | 02:24 | |
*** bobh has quit IRC | 02:28 | |
*** chlong has joined #openstack-mistral | 02:36 | |
*** openstackgerrit has joined #openstack-mistral | 03:23 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Refactor mistral context using oslo_context https://review.openstack.org/455407 | 03:23 |
---|---|---|
*** gongysh has quit IRC | 03:39 | |
*** gongysh has joined #openstack-mistral | 03:42 | |
*** zhurong has quit IRC | 03:49 | |
*** gongysh has quit IRC | 03:53 | |
*** chlong has quit IRC | 04:02 | |
*** gkadam has joined #openstack-mistral | 04:09 | |
*** chlong has joined #openstack-mistral | 04:15 | |
*** sharatss has joined #openstack-mistral | 05:07 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Refactor mistral context using oslo_context https://review.openstack.org/455407 | 05:10 |
openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Set the default value of --limit parameter https://review.openstack.org/476110 | 05:15 |
*** gongysh has joined #openstack-mistral | 05:47 | |
*** Dinesh_Bhor has quit IRC | 05:57 | |
*** Dinesh_Bhor has joined #openstack-mistral | 06:06 | |
openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Use keystoneauth plugins and session instead of keystoneclient https://review.openstack.org/455174 | 06:39 |
openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Set the default value of --limit parameter https://review.openstack.org/476110 | 06:47 |
*** sharatss has quit IRC | 07:06 | |
*** gkadam is now known as gkadam-afk | 07:08 | |
*** jpich has joined #openstack-mistral | 07:22 | |
openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Add pagination options for workflows and actions https://review.openstack.org/458515 | 07:26 |
*** dtantsur|afk is now known as dtantsur | 07:35 | |
*** apetrich has quit IRC | 07:39 | |
*** amoralej|off is now known as amoralej | 07:40 | |
openstackgerrit | Michal Gershenzon proposed openstack/mistral-specs master: Create and run workflows within a namespace https://review.openstack.org/470996 | 08:24 |
*** zhurong has joined #openstack-mistral | 08:27 | |
*** dmk0202 has joined #openstack-mistral | 08:40 | |
*** gkadam-afk is now known as gkadam | 08:40 | |
openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Add pagination options for workflows and actions https://review.openstack.org/458515 | 08:46 |
*** zhurong has quit IRC | 09:16 | |
*** dtantsur is now known as dtantsur|bbl | 09:25 | |
*** sharatss has joined #openstack-mistral | 09:29 | |
*** apetrich has joined #openstack-mistral | 09:29 | |
sharatss | therve, hi | 09:29 |
therve | sharatss, Hello! | 09:36 |
sharatss | therve, https://review.openstack.org/#/c/455174/ is messed up again | 09:38 |
therve | sharatss, What did you do? :) | 09:38 |
sharatss | therve, test_httpclient has brought in merge conflict | 09:39 |
therve | That requests-mock fix was not really necessary... | 09:40 |
*** dmk0202 has quit IRC | 09:40 | |
therve | Heh, you made that change | 09:41 |
sharatss | therve, yup.. i realised it very late | 09:42 |
openstackgerrit | Thomas Herve proposed openstack/python-mistralclient master: Use keystoneauth plugins and session instead of keystoneclient https://review.openstack.org/455174 | 09:42 |
therve | Hum tests still failing, one sec | 09:43 |
therve | sharatss, https://review.openstack.org/#/c/455174/49..50/mistralclient/auth/keystone.py | 09:53 |
therve | That looks highly incorrect :) | 09:53 |
therve | Sigh | 09:55 |
openstackgerrit | Thomas Herve proposed openstack/python-mistralclient master: Use keystoneauth plugins and session instead of keystoneclient https://review.openstack.org/455174 | 10:04 |
*** rook has quit IRC | 10:18 | |
openstackgerrit | Michal Gershenzon proposed openstack/mistral-specs master: Create and run workflows within a namespace https://review.openstack.org/470996 | 10:25 |
*** rook has joined #openstack-mistral | 10:27 | |
*** rook is now known as Guest250 | 10:28 | |
*** zhurong has joined #openstack-mistral | 10:32 | |
openstackgerrit | Sharat Sharma proposed openstack/mistral master: Centralize session creation and authorization from OS clients https://review.openstack.org/447399 | 10:33 |
*** dtantsur|bbl is now known as dtantsur | 10:34 | |
*** gongysh has quit IRC | 10:47 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Refactor mistral context using oslo_context https://review.openstack.org/455407 | 11:01 |
*** jkilpatr has joined #openstack-mistral | 11:05 | |
openstackgerrit | Andras Kovi proposed openstack/mistral master: Centralize session creation and authorization from OS clients https://review.openstack.org/447399 | 11:29 |
*** amoralej is now known as amoralej|lunch | 11:47 | |
*** zhurong has quit IRC | 12:01 | |
*** thrash|g0ne is now known as thrash | 12:04 | |
*** rbrady has quit IRC | 12:08 | |
*** dprince has joined #openstack-mistral | 12:12 | |
*** zhurong has joined #openstack-mistral | 12:16 | |
*** zhurong has quit IRC | 12:18 | |
xavierhardy | hi everyone | 12:23 |
xavierhardy | I have strange behavior with stable/ocata | 12:23 |
xavierhardy | I have activated authentication through a custom plugin | 12:23 |
xavierhardy | works great | 12:23 |
xavierhardy | but now I have problems with permissions | 12:23 |
xavierhardy | Our workflows are pushed directly in the DB with scope=public and project_id=<default_project> | 12:24 |
xavierhardy | they cannot be edited, which is fine | 12:24 |
xavierhardy | we have done the same thing for our actions | 12:25 |
xavierhardy | the thing is, the actions can be edited | 12:25 |
xavierhardy | ... | 12:25 |
*** chlong has quit IRC | 12:26 | |
*** gongysh has joined #openstack-mistral | 12:27 | |
xavierhardy | rakhmerov? any idea? | 12:34 |
openstackgerrit | Andrea Visnyei proposed openstack/mistral master: WIP: Mistral install guide https://review.openstack.org/476499 | 12:38 |
xavierhardy | there's a discrepancy between what is done for workflow updates and action updates | 12:48 |
xavierhardy | @b.session_aware() | 12:49 |
xavierhardy | def update_workflow_definition(identifier, values, session=None): | 12:49 |
xavierhardy | wf_def = get_workflow_definition(identifier) | 12:49 |
xavierhardy | ctx = auth_ctx.ctx() | 12:49 |
xavierhardy | if not ctx.is_admin and wf_def.project_id != security.get_project_id(): | 12:49 |
xavierhardy | raise exc.NotAllowedException( | 12:49 |
xavierhardy | "Can not update workflow of other tenants. " | 12:49 |
xavierhardy | "[workflow_identifier=%s]" % identifier | 12:49 |
xavierhardy | ) | 12:49 |
xavierhardy | @b.session_aware() | 12:50 |
xavierhardy | def update_action_definition(identifier, values, session=None): | 12:50 |
xavierhardy | a_def = get_action_definition(identifier) | 12:50 |
xavierhardy | a_def.update(values.copy()) | 12:50 |
xavierhardy | return a_def | 12:50 |
xavierhardy | writing a PR about it | 12:51 |
xavierhardy | sorry a BR | 12:51 |
xavierhardy | and a patch | 12:51 |
*** bobh has joined #openstack-mistral | 12:53 | |
*** gkadam has quit IRC | 12:55 | |
xavierhardy | Bug report created | 13:00 |
*** rbrady has joined #openstack-mistral | 13:03 | |
*** rbrady has joined #openstack-mistral | 13:03 | |
openstackgerrit | Andrea Visnyei proposed openstack/mistral master: WIP: Mistral install guide https://review.openstack.org/476499 | 13:03 |
xavierhardy | same thing for deletion BTW | 13:05 |
*** sharatss has quit IRC | 13:06 | |
*** Guest250 is now known as rook | 13:06 | |
*** bobh has quit IRC | 13:12 | |
*** catintheroof has joined #openstack-mistral | 13:20 | |
openstackgerrit | Xavier Hardy proposed openstack/mistral master: Fix decrepancy between action and workflow update https://review.openstack.org/476514 | 13:23 |
*** abhishekk has quit IRC | 13:35 | |
*** amoralej|lunch is now known as amoralej | 13:36 | |
*** chlong has joined #openstack-mistral | 13:39 | |
*** bobh has joined #openstack-mistral | 13:45 | |
xavierhardy | mistral.exceptions.ApplicationContextNotFoundException: Application context not found | 13:47 |
xavierhardy | in the tests | 13:48 |
*** rbrady is now known as rbrady-brb | 13:48 | |
xavierhardy | I'll look at sync_db | 13:48 |
xavierhardy | OK, I found the difference for workflows: you wipe everything out :s and then create new ones. For actions, you create or update them, so you can't update them now with my patch (since it needs the application context...) | 13:52 |
xavierhardy | Is that OK to modify that behavior to wipe out actions too ? | 13:52 |
xavierhardy | see comment in the review | 14:01 |
*** rbrady-brb is now known as rbrady | 14:07 | |
*** livelace has joined #openstack-mistral | 14:11 | |
openstackgerrit | Ryan Brady proposed openstack/mistral master: TEST DO NOT MERGE https://review.openstack.org/476535 | 14:23 |
*** dprince has quit IRC | 14:43 | |
*** dtantsur is now known as dtantsur|afk | 15:40 | |
openstackgerrit | Andras Kovi proposed openstack/mistral master: Centralize session creation and authorization from OS clients https://review.openstack.org/447399 | 15:46 |
*** dprince has joined #openstack-mistral | 15:54 | |
*** gongysh has quit IRC | 16:05 | |
*** jpich has quit IRC | 16:08 | |
*** thrash is now known as thrash|biab | 16:22 | |
*** dprince has quit IRC | 16:31 | |
*** fultonj has quit IRC | 16:45 | |
*** fultonj has joined #openstack-mistral | 16:53 | |
*** thrash|biab is now known as thrash | 17:29 | |
*** livelace has quit IRC | 17:37 | |
*** dprince has joined #openstack-mistral | 17:59 | |
*** amoralej is now known as amoralej|off | 19:04 | |
*** dprince has quit IRC | 19:25 | |
*** dprince has joined #openstack-mistral | 19:26 | |
*** dprince has quit IRC | 20:07 | |
*** jrist has quit IRC | 20:09 | |
*** jrist has joined #openstack-mistral | 20:16 | |
*** jkilpatr has quit IRC | 20:20 | |
*** thrash is now known as thrash|bbl | 20:26 | |
*** jkilpatr has joined #openstack-mistral | 20:40 | |
*** dmellado has quit IRC | 21:00 | |
*** dmellado has joined #openstack-mistral | 21:03 | |
*** catintheroof has quit IRC | 21:18 | |
*** bobh has quit IRC | 21:25 | |
*** chlong has quit IRC | 21:57 | |
*** weshay has quit IRC | 22:29 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Refactor mistral context using oslo_context https://review.openstack.org/455407 | 22:32 |
*** weshay has joined #openstack-mistral | 22:33 | |
*** dmellado has quit IRC | 22:33 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Refactor mistral context using oslo_context https://review.openstack.org/455407 | 22:35 |
*** dmellado has joined #openstack-mistral | 22:37 | |
*** amoralej|off has quit IRC | 22:37 | |
*** weshay has quit IRC | 22:39 | |
*** weshay has joined #openstack-mistral | 22:42 | |
*** weshay has quit IRC | 22:51 | |
*** weshay has joined #openstack-mistral | 22:52 | |
*** amoralej has joined #openstack-mistral | 22:54 | |
*** weshay has quit IRC | 23:00 | |
*** weshay has joined #openstack-mistral | 23:03 | |
*** dmellado has quit IRC | 23:03 | |
*** dmellado has joined #openstack-mistral | 23:06 | |
*** chlong has joined #openstack-mistral | 23:10 | |
*** dmellado has quit IRC | 23:15 | |
*** dmellado has joined #openstack-mistral | 23:18 | |
*** gongysh has joined #openstack-mistral | 23:31 | |
*** dmellado has quit IRC | 23:31 | |
*** amoralej has quit IRC | 23:33 | |
*** gongysh has quit IRC | 23:33 | |
*** gongysh has joined #openstack-mistral | 23:37 | |
*** gongysh has quit IRC | 23:38 | |
*** amoralej has joined #openstack-mistral | 23:50 | |
*** dmellado has joined #openstack-mistral | 23:51 | |
*** amoralej has quit IRC | 23:55 | |
*** dmellado has quit IRC | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!