Tuesday, 2018-02-20

*** jrist has quit IRC00:24
*** AlexeyAbashkin has joined #openstack-mistral00:24
*** AlexeyAbashkin has quit IRC00:28
*** itlinux has joined #openstack-mistral00:29
*** jrist has joined #openstack-mistral00:36
*** mcdoker181818 has quit IRC00:49
*** vishwanathj has joined #openstack-mistral02:20
*** yangyapeng has quit IRC02:47
*** bobh has joined #openstack-mistral02:47
*** yangyapeng has joined #openstack-mistral02:47
*** bobh has quit IRC02:48
*** harlowja has quit IRC02:52
*** yangyape_ has joined #openstack-mistral03:00
*** yangyapeng has quit IRC03:01
*** ykarel|away has joined #openstack-mistral03:10
*** AlexeyAbashkin has joined #openstack-mistral03:24
*** AlexeyAbashkin has quit IRC03:29
*** gkadam has joined #openstack-mistral03:39
*** thrash is now known as thrash|g0ne03:39
*** ykarel|away has quit IRC03:59
*** ykarel has joined #openstack-mistral04:21
*** harlowja has joined #openstack-mistral04:54
*** kushalAgrawal has joined #openstack-mistral05:42
*** gkadam has quit IRC05:43
*** gkadam has joined #openstack-mistral05:44
*** threestrands has quit IRC07:10
*** harlowja has quit IRC07:10
*** gkadam has quit IRC07:34
*** gkadam has joined #openstack-mistral07:34
*** AlexeyAbashkin has joined #openstack-mistral07:53
rakhmerovd0ugal: hi, as far as wiki, yes. We used to use wiki pretty actively for posting some design drafts and other things. But probably we can have a special page in our official docs for that08:02
rakhmerovand we now have spec repo08:02
rakhmerovthrash|g0ne: thanks!08:02
*** akazakov has quit IRC08:09
*** yangyape_ has quit IRC08:10
*** akazakov has joined #openstack-mistral08:12
*** yangyapeng has joined #openstack-mistral08:17
*** yangyapeng has quit IRC08:27
*** openstackgerrit has joined #openstack-mistral08:31
openstackgerritRenat Akhmerov proposed openstack/mistral master: Explain better combinations of task names and engine commands in docs  https://review.openstack.org/54569208:31
*** yangyapeng has joined #openstack-mistral08:38
*** jpich has joined #openstack-mistral08:47
*** yangyape_ has joined #openstack-mistral08:49
*** yangyapeng has quit IRC08:50
*** shardy has joined #openstack-mistral09:09
apetrichrakhmerov, almost done on the massive review. Sorry that I didn't have the focus yesterday to go through it all09:36
rakhmerovit's ok )09:36
rakhmerovthanks a lot! I know it's not easy09:36
d0ugalapetrich: I think it looks pretty good fwiw09:36
d0ugalbut I'd like to test it before it lands09:36
apetrichaye09:36
d0ugalI started writing a Zaqar notifier09:36
apetrichoh neat09:36
apetrichit is lgtm09:37
apetrichbut I will +1 it to let you +2 and +W it if you are going to test it09:37
d0ugalrakhmerov: I am trying to figure out how I can get the security context to create a zaqarclient for a zaqar publisher09:39
d0ugalrakhmerov: would you happen to know?09:39
rakhmerovhah.. good question09:43
rakhmerovwell, first of all, IMO it should be a publisher, not notifier09:43
d0ugalrakhmerov: yeah, I got confused in my first message.09:44
d0ugalrakhmerov: this is what I am playing with so far: http://paste.openstack.org/show/678334/09:44
d0ugalOther than lines 29 and 30, it should be really easy :)09:44
d0ugalbut I can't figure out how to get that information.09:45
d0ugalMaybe I can ask Winson09:45
rakhmerovyep.. looking09:46
rakhmerovok, seems like it's simply missing09:47
d0ugal:)09:47
rakhmerovif you look at https://review.openstack.org/#/c/467799/11/mistral/engine/tasks.py line 7309:48
rakhmerovyou won't see anything related to security context09:48
rakhmerovall info comes from that place09:48
d0ugalrakhmerov: right, so I guess I could just pass it at that point09:48
d0ugalthat should be easy09:48
rakhmerovit only has info about the task itself and the event that doesn't include security context09:48
rakhmerovyes09:48
d0ugalthanks09:48
d0ugalI think adding a publisher is a good test of the design :)09:49
rakhmerovbut maybe the interface of Publisher (and Notifier) should have it explicitly09:49
rakhmerovyes, indeed09:49
rakhmerovthanks for doing that..09:49
rakhmerovI'm planning to do that as well but later09:49
d0ugalIt might be nicer to pass it to the init09:49
rakhmerovpass what and to what init?09:50
d0ugalthe context09:50
rakhmerovto what init?09:50
d0ugalpublisher init09:51
d0ugalsorry :)09:51
d0ugalI am just thinking "out loud"09:51
rakhmerovnope, I think it won't work09:51
d0ugaltrue, it changes09:51
d0ugalnevermind09:51
rakhmerovbecause same publisher can be reused by multiple threads09:51
rakhmerovso I think it should be a part of Notifier and Publisher interface09:51
rakhmerovso, the first missing thing..09:52
rakhmerovbut I'd still suggest we consider this patch the first step towards the full implementation and let it merge and then add what's missing09:52
rakhmerovno? :)09:52
rakhmerov(hoping that I didn't say anything stupid)09:53
d0ugalNo, that seems fine09:53
rakhmerovanyway.. up to you )09:53
rakhmerovthanks09:53
rakhmerovgoing for lunch09:53
d0ugalI just want to verify that it mostly works and doing something like this requires minimal changes.09:53
*** d0ugal_ has joined #openstack-mistral10:32
*** d0ugal has quit IRC10:34
*** d0ugal_ has quit IRC10:50
*** d0ugal has joined #openstack-mistral10:51
*** gkadam has quit IRC11:06
*** gkadam has joined #openstack-mistral11:07
*** gkadam has quit IRC11:12
*** gkadam has joined #openstack-mistral11:12
*** gkadam has quit IRC11:18
*** gkadam has joined #openstack-mistral11:30
*** gkadam has quit IRC12:06
*** oidgar has joined #openstack-mistral12:10
*** gkadam has joined #openstack-mistral12:22
*** oidgar has quit IRC12:30
*** dprince has joined #openstack-mistral12:40
*** oidgar has joined #openstack-mistral12:45
rakhmerovok12:46
*** thrash|g0ne is now known as thrash12:54
*** oidgar has quit IRC14:03
*** oidgar has joined #openstack-mistral14:14
*** ykarel is now known as ykarel|away14:19
*** rbrady has quit IRC14:35
*** ykarel|away is now known as ykarel14:38
*** rbrady has joined #openstack-mistral14:48
*** rbrady has quit IRC14:48
*** rbrady has joined #openstack-mistral14:48
*** ykarel is now known as ykarel|away14:59
*** d0ugal has quit IRC14:59
*** d0ugal has joined #openstack-mistral15:01
*** itlinux has quit IRC15:16
*** thrash is now known as thrash|biab15:25
*** ykarel|away has quit IRC15:35
*** d0ugal has quit IRC15:36
*** d0ugal has joined #openstack-mistral15:40
*** thrash|biab is now known as thrash15:49
*** d0ugal has quit IRC15:52
*** itlinux has joined #openstack-mistral16:06
*** d0ugal has joined #openstack-mistral16:07
*** d0ugal has quit IRC16:17
*** AlexeyAbashkin has quit IRC16:26
*** d0ugal has joined #openstack-mistral16:35
*** oidgar has quit IRC16:54
*** d0ugal has quit IRC16:59
*** itlinux has quit IRC17:02
*** d0ugal has joined #openstack-mistral17:03
*** itlinux has joined #openstack-mistral17:04
*** jpich has quit IRC17:18
*** harlowja has joined #openstack-mistral17:23
*** d0ugal has quit IRC17:38
*** dprince has quit IRC17:47
*** jtomasek has quit IRC17:54
*** AlexeyAbashkin has joined #openstack-mistral17:57
*** dprince has joined #openstack-mistral18:01
*** AlexeyAbashkin has quit IRC18:02
*** openstackgerrit has quit IRC18:03
*** oidgar has joined #openstack-mistral18:06
*** d0ugal has joined #openstack-mistral18:11
*** AlexeyAbashkin has joined #openstack-mistral18:13
*** oidgar has quit IRC18:15
*** AlexeyAbashkin has quit IRC18:17
*** harlowja has quit IRC18:23
*** d0ugal has quit IRC18:38
*** d0ugal has joined #openstack-mistral18:40
*** shardy has quit IRC18:43
*** d0ugal has quit IRC18:51
*** harlowja has joined #openstack-mistral18:55
*** d0ugal_ has joined #openstack-mistral18:57
*** harlowja_ has joined #openstack-mistral18:59
*** harlowja has quit IRC18:59
*** apetrich has quit IRC19:12
*** apetrich has joined #openstack-mistral19:14
*** openstackgerrit has joined #openstack-mistral19:23
openstackgerritToure Dunnon proposed openstack/mistral-extra master: Add Ansible actions to Mistral  https://review.openstack.org/51436019:23
*** d0ugal_ has quit IRC19:45
*** d0ugal_ has joined #openstack-mistral19:57
*** d0ugal_ has quit IRC20:03
*** d0ugal_ has joined #openstack-mistral20:04
*** d0ugal_ has quit IRC20:10
*** d0ugal_ has joined #openstack-mistral20:14
*** AlexeyAbashkin has joined #openstack-mistral20:23
*** AlexeyAbashkin has quit IRC20:27
*** d0ugal_ has quit IRC20:30
*** d0ugal_ has joined #openstack-mistral20:57
*** FL1SK has joined #openstack-mistral21:08
*** dprince has quit IRC21:09
*** AlexeyAbashkin has joined #openstack-mistral21:23
*** AlexeyAbashkin has quit IRC21:27
*** kushalAgrawal has quit IRC21:33
*** d0ugal_ has quit IRC22:03
*** threestrands has joined #openstack-mistral22:23
*** itlinux has quit IRC22:49
*** openstackgerrit has quit IRC23:04
*** AlexeyAbashkin has joined #openstack-mistral23:23
*** AlexeyAbashkin has quit IRC23:27

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!