| *** bobh has quit IRC | 01:02 | |
| *** bobh has joined #openstack-mistral | 01:09 | |
| *** bobh has quit IRC | 01:19 | |
| *** bobh has joined #openstack-mistral | 01:22 | |
| *** yangyapeng has joined #openstack-mistral | 01:26 | |
| *** bobh has quit IRC | 01:27 | |
| *** bobh has joined #openstack-mistral | 01:35 | |
| *** bobh has quit IRC | 01:39 | |
| *** openstackgerrit has quit IRC | 01:49 | |
| *** FL1SK has quit IRC | 01:57 | |
| *** bobh has joined #openstack-mistral | 01:58 | |
| *** bobh has quit IRC | 02:02 | |
| *** bobh has joined #openstack-mistral | 02:15 | |
| *** bobh has quit IRC | 02:20 | |
| *** bobh has joined #openstack-mistral | 02:25 | |
| *** bobh has quit IRC | 02:30 | |
| *** bobh has joined #openstack-mistral | 02:51 | |
| *** bobh has quit IRC | 03:39 | |
| *** hardikjasani has joined #openstack-mistral | 04:05 | |
| *** yangyapeng has quit IRC | 05:14 | |
| *** nguyenhai has joined #openstack-mistral | 05:21 | |
| *** apetrich has joined #openstack-mistral | 05:23 | |
| *** quiquell|off is now known as quiquell|rover | 05:41 | |
| *** yangyapeng has joined #openstack-mistral | 06:03 | |
| *** yangyapeng has quit IRC | 06:09 | |
| *** yangyapeng has joined #openstack-mistral | 06:09 | |
| *** josecastroleon has joined #openstack-mistral | 06:20 | |
| *** quiquell|rover is now known as quique|rover|bbl | 06:25 | |
| *** nguyenhai_ has joined #openstack-mistral | 06:26 | |
| *** nguyenhai has quit IRC | 06:29 | |
| *** threestrands has joined #openstack-mistral | 06:33 | |
| *** threestrands has quit IRC | 06:33 | |
| *** threestrands has joined #openstack-mistral | 06:33 | |
| *** threestrands has quit IRC | 06:37 | |
| *** quique|rover|bbl is now known as quiquell|rover | 07:00 | |
| *** josecastroleon has quit IRC | 07:01 | |
| *** josecastroleon has joined #openstack-mistral | 07:11 | |
| *** kimamisa has joined #openstack-mistral | 07:12 | |
| rakhmerov | pgaxatte: hi | 07:22 |
|---|---|---|
| rakhmerov | on the policy question: yes, it's not possible now | 07:22 |
| pgaxatte | hello rakhmerov | 07:23 |
| rakhmerov | the granularity of this policy mechanism now is an endpoint | 07:23 |
| pgaxatte | I could not find the policy to set to prevent publishing workflows | 07:23 |
| rakhmerov | we can only allow/deny certain endpoints(or their methods) to various roles | 07:23 |
| rakhmerov | that's it | 07:23 |
| rakhmerov | you won't find it, yes | 07:24 |
| rakhmerov | it doesn't exist | 07:24 |
| pgaxatte | there is a little more granularity at some place though | 07:24 |
| pgaxatte | with workflows:list:all_projects | 07:24 |
| *** kimamisa has quit IRC | 07:25 | |
| pgaxatte | I should submit a small starting patch for the workflows today | 07:25 |
| pgaxatte | so if the approach is good I can do the same for the other resources that can be published | 07:26 |
| rakhmerov | pgaxatte: yes, right, at some places | 07:28 |
| rakhmerov | we can implement this, yes, but it doesn't exist yet | 07:28 |
| rakhmerov | pgaxatte: sounds good | 07:29 |
| pgaxatte | it shouldn't be too hard :) | 07:29 |
| *** kimamisa has joined #openstack-mistral | 07:32 | |
| *** openstackgerrit has joined #openstack-mistral | 07:33 | |
| openstackgerrit | Pierre Gaxatte proposed openstack/mistral master: Add a policy to control the right to publish resources https://review.openstack.org/579477 | 07:33 |
| *** kimamisa has quit IRC | 07:45 | |
| *** kimamisa has joined #openstack-mistral | 07:47 | |
| *** kimamisa has quit IRC | 07:54 | |
| *** kimamisa has joined #openstack-mistral | 07:55 | |
| openstackgerrit | Pierre Gaxatte proposed openstack/mistral master: Add a policy to control the right to publish resources https://review.openstack.org/579477 | 07:56 |
| *** d0ugal has joined #openstack-mistral | 08:01 | |
| *** d0ugal has quit IRC | 08:01 | |
| *** d0ugal has joined #openstack-mistral | 08:01 | |
| *** shardy has joined #openstack-mistral | 08:05 | |
| d0ugal | rakhmerov: Hey | 08:05 |
| d0ugal | rakhmerov: I'm around today actually, but I'll send the email for the rest of the week | 08:06 |
| rakhmerov | d0ugal: sure :) | 08:06 |
| d0ugal | sorry had a slight plan change and forgot to update you | 08:06 |
| pgaxatte | rakhmerov: can you take a look at https://review.openstack.org/#/c/579477/? If the approach is good, I'll extend it to wb and actions | 08:20 |
| pgaxatte | rakhmerov: to not break anything, I've put the publicize policy at RULE_ADMIN_OR_OWNER by default | 08:24 |
| rakhmerov | d0ugal: np | 08:25 |
| rakhmerov | pgaxatte: yes, I'll have a look a little later | 08:25 |
| rakhmerov | ok | 08:25 |
| pgaxatte | thanks | 08:25 |
| *** kimamisa has quit IRC | 08:32 | |
| *** josecastroleon has quit IRC | 08:35 | |
| d0ugal | wow, lots of tests :) | 08:40 |
| pgaxatte | d0ugal: that's only the tip of the iceberg :) | 08:47 |
| pgaxatte | in the end if all goes well I should add tests for each policy | 08:48 |
| d0ugal | Great. | 08:49 |
| pgaxatte | btw, d0ugal, rakhmerov, there's no list_all_projects for workbook, is it intended? | 08:51 |
| pgaxatte | I read somewhere that maybe workbook are not meant to stay around so maybe it is not needed? | 08:51 |
| d0ugal | Workbooks feel kinda weird to me :) | 08:52 |
| d0ugal | I think they only ever got implemented in a very limited way | 08:52 |
| *** kimamisa has joined #openstack-mistral | 08:52 | |
| d0ugal | i.e. if you delete a workbook it doesn't delete the workflows that are in it. That seems to confuse everyone | 08:53 |
| openstackgerrit | Merged openstack/mistral master: Add better output to the user deleting executions https://review.openstack.org/578300 | 08:56 |
| openstackgerrit | Merged openstack/mistral master: Add missing Tacker actions to Mistral https://review.openstack.org/578245 | 08:56 |
| rakhmerov | yeah, right | 08:56 |
| rakhmerov | although we could implement that | 08:56 |
| rakhmerov | but we just chose to let workbooks exist (since they were originally the main entity in the first API) but decided to make them not important | 08:57 |
| rakhmerov | they just allow to upload a number of other entities and after it they don't matter at all | 08:58 |
| pgaxatte | alright so I won't put effort in that direction :) | 09:05 |
| *** gkadam has joined #openstack-mistral | 09:06 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: [WIP] Experimental work adding a Zaqar event publisher https://review.openstack.org/547666 | 09:14 |
| d0ugal | pgaxatte: btw looks like you have a PEP8 error | 09:20 |
| *** pgaxatte has left #openstack-mistral | 09:21 | |
| *** pgaxatte has joined #openstack-mistral | 09:22 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 09:23 | |
| pgaxatte | d0ugal: oh yeah some docstring error, I need to lookup what these are supposed to look like :) | 09:23 |
| *** pgaxatte has left #openstack-mistral | 09:23 | |
| *** pgaxatte has joined #openstack-mistral | 09:23 | |
| d0ugal | pgaxatte: https://github.com/openstack/mistral/blob/master/mistral/engine/base.py#L63-L78 | 09:26 |
| d0ugal | There is a good example, the blank line after the title/first line is important. | 09:26 |
| pgaxatte | d0ugal: ohhh that's what it means! I'll fix it with the next commit, thanks | 09:28 |
| *** AlexeyAbashkin has quit IRC | 09:32 | |
| d0ugal | np | 09:35 |
| d0ugal | I do find it to be one of the more annoying checks :) | 09:35 |
| *** kimamisa has quit IRC | 09:45 | |
| *** d0ugal has quit IRC | 09:55 | |
| *** d0ugal has joined #openstack-mistral | 09:55 | |
| *** d0ugal has quit IRC | 09:55 | |
| *** d0ugal has joined #openstack-mistral | 09:55 | |
| openstackgerrit | Merged openstack/mistral-lib master: Fixed the documentation of 'run' params https://review.openstack.org/578436 | 10:14 |
| *** kimamisa has joined #openstack-mistral | 10:20 | |
| openstackgerrit | Merged openstack/mistral master: Update install guide about running mistral https://review.openstack.org/562121 | 10:24 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: [WIP] Experimental work adding a Zaqar event publisher https://review.openstack.org/547666 | 10:25 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: [WIP] Experimental work adding a Zaqar event publisher https://review.openstack.org/547666 | 10:48 |
| openstackgerrit | Merged openstack/python-mistralclient master: Add release note link in README https://review.openstack.org/578654 | 11:00 |
| *** quiquell|rover is now known as quique|rover|lch | 11:19 | |
| *** quique|rover|lch is now known as quiquell|rover | 11:44 | |
| *** thrash|g0ne is now known as thrash | 11:46 | |
| *** kimamisa has quit IRC | 12:04 | |
| *** jrist has joined #openstack-mistral | 12:34 | |
| openstackgerrit | Merged openstack/mistral master: switch mysql functional tests to PyMySQL https://review.openstack.org/575197 | 12:46 |
| *** gkadam has quit IRC | 12:47 | |
| openstackgerrit | Merged openstack/mistral master: Add missing query paramater `scope` to the workbook api https://review.openstack.org/555030 | 12:54 |
| *** gkadam has joined #openstack-mistral | 12:58 | |
| *** kimamisa has joined #openstack-mistral | 12:59 | |
| kimamisa | Hi ! Some ad for this change: https://review.openstack.org/#/c/560859/3 It's already +1'ed, but need a +2 ! Thx | 13:02 |
| *** pgaxatte has left #openstack-mistral | 13:09 | |
| *** d0ugal has quit IRC | 13:13 | |
| *** toure|gone is now known as toure | 13:14 | |
| *** bobh has joined #openstack-mistral | 13:21 | |
| *** hardikjasani has quit IRC | 13:24 | |
| *** d0ugal has joined #openstack-mistral | 13:35 | |
| *** yangyapeng has quit IRC | 13:43 | |
| *** gkadam_ has joined #openstack-mistral | 13:57 | |
| *** quiquell|rover is now known as quiquell|off | 13:58 | |
| *** gkadam has quit IRC | 13:58 | |
| *** apetrich has quit IRC | 13:59 | |
| *** apetrich has joined #openstack-mistral | 14:00 | |
| *** pgaxatte has joined #openstack-mistral | 14:08 | |
| *** yangyapeng has joined #openstack-mistral | 14:08 | |
| pgaxatte | :'( mistralclient works on multiple regions with the workflowv2 but the openstackclient does not | 14:11 |
| pgaxatte | I just tested against master | 14:12 |
| pgaxatte | and openstackclient ignores my region and takes the first one in the catalog | 14:14 |
| openstackgerrit | Dougal Matthews proposed openstack/python-mistralclient master: Add the root execution ID to the CLI output https://review.openstack.org/579570 | 14:22 |
| d0ugal | pgaxatte: huh, that is strange. I wonder why. Can you open a bug? | 14:24 |
| pgaxatte | yes we are collecting some info on this | 14:25 |
| pgaxatte | but this is reproducible | 14:25 |
| *** yangyapeng has quit IRC | 14:40 | |
| *** yangyapeng has joined #openstack-mistral | 14:41 | |
| *** yangyapeng has quit IRC | 14:45 | |
| *** gkadam_ has quit IRC | 14:52 | |
| *** gkadam_ has joined #openstack-mistral | 14:52 | |
| d0ugal | #startmeeting mistral | 15:00 |
| openstack | Meeting started Mon Jul 2 15:00:18 2018 UTC and is due to finish in 60 minutes. The chair is d0ugal. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| *** openstack changes topic to " (Meeting topic: mistral)" | 15:00 | |
| openstack | The meeting name has been set to 'mistral' | 15:00 |
| d0ugal | Happy Monday! | 15:00 |
| d0ugal | It is office hour time - who is around? | 15:00 |
| d0ugal | https://etherpad.openstack.org/p/mistral-office-hours | 15:00 |
| d0ugal | rakhmerov, apetrich, bobh, mcdoker181818: PING | 15:00 |
| d0ugal | As usual, add your nick to line 16 in the etherpad if you want a ping in here at the start of meetings. | 15:01 |
| apetrich | o/ | 15:05 |
| pgaxatte | yes i'm around too | 15:06 |
| d0ugal | I don't have an agenda this week :) There was one new bug that I triaged | 15:07 |
| d0ugal | I am on vacation for the rest of this week - so look for rakhmerov if anything comes up | 15:07 |
| openstackgerrit | Dougal Matthews proposed openstack/python-mistralclient master: Add the root execution ID to the CLI output https://review.openstack.org/579570 | 15:12 |
| openstackgerrit | Dougal Matthews proposed openstack/python-mistralclient master: Add missing oslo.serialization requirement https://review.openstack.org/579592 | 15:12 |
| d0ugal | apetrich: ^ not sure how nobody missed that missing requirement before :) | 15:12 |
| apetrich | oh | 15:13 |
| apetrich | d0ugal, good catch | 15:13 |
| openstackgerrit | Merged openstack/mistral master: Fix typos https://review.openstack.org/576482 | 15:24 |
| *** yangyapeng has joined #openstack-mistral | 15:29 | |
| openstackgerrit | Quentin GROLLEAU proposed openstack/python-mistralclient master: Fixing region filter for getting workflow endpoint https://review.openstack.org/579612 | 15:48 |
| pgaxatte | d0ugal: a colleague of mine found the origin of the multi region | 15:49 |
| d0ugal | Oh, good | 15:50 |
| pgaxatte | i helped submit a change, here it is: https://review.openstack.org/#/c/579612/ | 15:50 |
| pgaxatte | I tested it and it seems to work but i haven't checked if there are any tests to fix. I guess Zuul will tell us :) | 15:54 |
| d0ugal | Indeed, it will :) | 15:54 |
| d0ugal | The change looks good - I hoped it would be something simple. | 15:54 |
| *** weshay is now known as weshay|ruck | 15:56 | |
| d0ugal | #endmeeting | 15:58 |
| *** openstack changes topic to " (Meeting topic: test)" | 15:58 | |
| openstack | Meeting ended Mon Jul 2 15:58:40 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:58 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-07-02-15.00.html | 15:58 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-07-02-15.00.txt | 15:58 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/mistral/2018/mistral.2018-07-02-15.00.log.html | 15:58 |
| openstackgerrit | Dougal Matthews proposed openstack/python-mistralclient master: Add the root execution ID to the CLI output https://review.openstack.org/579570 | 16:03 |
| *** jrist has quit IRC | 17:00 | |
| *** jrist has joined #openstack-mistral | 17:06 | |
| *** kimamisa has quit IRC | 18:09 | |
| *** kimamisa has joined #openstack-mistral | 18:29 | |
| *** kimamisa has quit IRC | 18:38 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 18:57 | |
| *** AlexeyAbashkin has quit IRC | 19:17 | |
| *** shardy has quit IRC | 19:32 | |
| *** d0ugal has quit IRC | 19:34 | |
| *** kimamisa has joined #openstack-mistral | 19:57 | |
| *** bobh has quit IRC | 20:28 | |
| *** bobh has joined #openstack-mistral | 20:55 | |
| *** bobh has quit IRC | 20:59 | |
| *** bobh has joined #openstack-mistral | 21:03 | |
| *** bobh has quit IRC | 21:07 | |
| *** bobh has joined #openstack-mistral | 21:13 | |
| *** bobh has quit IRC | 21:17 | |
| *** bobh has joined #openstack-mistral | 21:40 | |
| *** thrash is now known as thrash|g0ne | 21:44 | |
| *** bobh has quit IRC | 21:44 | |
| *** bobh has joined #openstack-mistral | 21:45 | |
| *** bobh has quit IRC | 21:49 | |
| *** bobh has joined #openstack-mistral | 21:56 | |
| *** bobh has quit IRC | 22:00 | |
| *** bobh has joined #openstack-mistral | 22:35 | |
| openstackgerrit | Merged openstack/python-mistralclient master: Fixing region filter for getting workflow endpoint https://review.openstack.org/579612 | 22:36 |
| *** bobh has quit IRC | 22:39 | |
| *** bobh has joined #openstack-mistral | 22:57 | |
| *** bobh has quit IRC | 23:01 | |
| *** bobh has joined #openstack-mistral | 23:07 | |
| *** bobh_ has joined #openstack-mistral | 23:10 | |
| *** bobh_ has quit IRC | 23:11 | |
| *** bobh_ has joined #openstack-mistral | 23:11 | |
| *** bobh has quit IRC | 23:14 | |
| *** bobh_ has quit IRC | 23:15 | |
| *** threestrands has joined #openstack-mistral | 23:43 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!