16:00:07 #startmeeting mistral 16:00:09 Meeting started Mon Jul 25 16:00:07 2016 UTC and is due to finish in 60 minutes. The chair is ddeja. Information about MeetBot at http://wiki.debian.org/MeetBot. 16:00:10 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 16:00:12 The meeting name has been set to 'mistral' 16:00:17 hello all 16:00:25 o/ 16:00:26 hello 16:00:26 hey! 16:01:15 Renat can't attend the meeting and asked me to chair. Let's start with first topic 16:01:29 #topic Review action items 16:02:15 there were 2 action items on Renat and one on hparekh, but they are not there 16:02:37 sore there is one left -> rbrady, jpeeler: start with initial proposal on security module for Actions API 16:02:54 so, how's the work going? :) 16:02:58 jpeeler and I have been discussing it 16:03:12 I have a question about the context info 16:03:39 hm? 16:03:48 reading the requirement of the context, it seems the mistral.context solves that and could be ported over to mistral-lib 16:04:27 I guess that would be possible 16:04:30 OR a separate security context could be created and mistral.context could have a property (security_context) 16:05:07 Hey 16:05:08 I've been looking through the code base a bit to see if there is a reason to use one approach over the other 16:05:33 does anyone have any thoughts about this? 16:05:33 Sorry I am late 16:05:43 d0ugal: no problem :) 16:06:01 rbrady: I'm not an expert in this part unfortunately 16:06:11 IMO both approaches are OK 16:06:15 i would think that external actions are going to have all the same authentication requirements as a native mistral action would 16:06:43 jpeeler: as af as I remember yes 16:06:48 +1 16:07:16 so porting sounds to be reasonable if for no reason other than consistency 16:07:36 to me the question is about exposing other details from the mistral context to the custom actions and not just the security attributes listed in the spec 16:07:49 rbrady, jpeeler, are you US based? 16:07:55 yes, east coast 16:08:08 is there any reason not to share the full mistral context to a custom action? 16:08:25 rbrady: let me look on the code... 16:09:48 rbrady: IMO there is no reason, but I may not think about something 16:10:13 Maybe this should be discussed on openstack-dev to get more input? 16:10:25 I can ask Renat about this tommorow on IRC 16:10:30 ddeja: ack. thanks. I think we'll port the context over then and see how the review comments play out 16:11:02 #action ddeja to ask Renat about proting context 16:11:29 ok, thanks guys for the update on this update 16:11:48 even if the context won't be passed now, it can be added later. 16:11:52 #topic Current status (progress, issues, roadblocks, further plans) 16:12:58 My status: Finishing work on alternative RPC layer. One patch still waiting for merge and some small bugs to fix 16:13:03 my status: have some time to start working on this bug https://bugs.launchpad.net/mistral/+bug/1603941 16:13:03 Launchpad bug 1603941 in Mistral "create workflow fails on data too long for column definition" [High,Confirmed] - Assigned to Michal Gershenzon (michal-gershenzon) 16:13:41 I'm working on the actions api utils and execution modules 16:15:33 i'm working on client caching and i appreciate the reviews i received. will incorporate those changes soon 16:16:17 Anyone else wish to share their status? :) 16:17:41 #topic Open discussion 16:17:56 I have been at EuroPython and travelling for work, so not had time for Mistral. 16:18:24 d0ugal: OK, no problem 16:18:27 This week is also a bit crazy for me, hopefully back to normal next week. I really want to figure out how to finish this bug: https://bugs.launchpad.net/mistral/+bug/1597640 16:18:27 Launchpad bug 1597640 in Mistral "with-items does not process ad-hoc action default values" [High,In progress] - Assigned to Dougal Matthews (d0ugal) 16:18:45 If anyone has any ideas where I should go next with it, that would be great. So far I just have a failing test case 16:19:09 d0ugal: you mean some unit test? 16:19:14 or if anyone wants to finish it because I am being slow, I am happy for them to take it on :) - I don't want to block anyone 16:19:30 ddeja: Yup, a failing unit test; https://review.openstack.org/337696 16:19:42 * ddeja is looking 16:20:30 d0ugal: I can look at it (hopefully tommorow) and leave a comment in the review 16:21:33 ddeja: Great, that would be very useful :) 16:21:48 Any other opens? If not, we can end the meeting erlier. 16:22:34 counting to 10... 1 16:22:38 2 16:22:42 3 16:22:45 4 16:22:48 5 16:22:51 6 16:22:54 7 16:22:57 8 16:23:00 9 16:23:06 10. sold! 16:23:19 thank you all very much and see you next week :) 16:23:21 Thanks ddeja! 16:23:30 Thanks bey! 16:23:41 #endmeeting