Friday, 2017-03-24

*** bobh has quit IRC00:06
*** vishwanathj has joined #openstack-mistral00:39
*** vishwanathj has quit IRC00:44
*** zhurong has joined #openstack-mistral01:26
*** jamielennox is now known as jamielennox|away01:39
*** vishwanathj has joined #openstack-mistral01:40
*** bobh has joined #openstack-mistral01:44
*** vishwanathj has quit IRC01:45
*** jamielennox|away is now known as jamielennox01:48
*** gongysh has joined #openstack-mistral01:49
*** vishwanathj has joined #openstack-mistral02:06
*** bobh has quit IRC02:07
*** vishwanathj has quit IRC02:07
*** vishwanathj has joined #openstack-mistral02:09
*** vishwanathj has quit IRC02:10
*** vishwanathj has joined #openstack-mistral02:11
*** bobh has joined #openstack-mistral02:15
*** bobh has quit IRC03:15
*** gongysh has quit IRC03:20
*** zhurong has quit IRC04:07
rakhmerovd0ugal: well, we mostly write them (release notes)04:43
*** Kevin_Zheng has quit IRC05:13
*** zhurong has joined #openstack-mistral05:16
openstackgerritMerged openstack/python-mistralclient master: Take DST into account when converting to UTC  https://review.openstack.org/44911705:18
*** gongysh has joined #openstack-mistral05:29
*** sharatss has joined #openstack-mistral06:50
*** gongysh has quit IRC06:51
*** gongysh has joined #openstack-mistral07:02
d0ugalrakhmerov: sort-of, there are not that many release notes for how many changes there are...07:24
rakhmerov:)07:26
rakhmerovwhat I really meant is that it's not an excuse to not write release notes07:27
rakhmerovwe should be doing it07:27
d0ugalsure, we should be stricter about doing it in reviews07:51
d0ugalrakhmerov: are there any special rules for the mistral-sepcs repo?07:52
d0ugalin terms of reviewing07:52
d0ugalI see you +2'ed my spec - do I just need another core? mgershen1 or kong maybe?07:53
rakhmerovd0ugal: well, we've been following pretty much the same 2 +2 rule07:54
rakhmerovfor specs07:54
d0ugalokay, cool07:54
rakhmerovto me it would even make more sense if we had 3 +2 rule because on specs we usually expect wider feedback07:54
d0ugalYup07:55
*** jaosorior has joined #openstack-mistral07:55
rakhmerovwe can mention it in the policy that you wrote btw07:55
d0ugalGood idea.07:55
rakhmerovas additional patch07:55
rakhmerovyeah07:55
rakhmerovif people agree with that we'll accept that change07:55
rakhmerovd0ugal: btw, I already started implementing https://review.openstack.org/#/c/446960/07:56
rakhmerovcould you pls have a look at it?07:56
d0ugalSure07:56
rakhmerovit should be familiar after global wf context to you07:57
rakhmerovit's just more generic and elegant IMO and people seem to like it07:57
rakhmerovd0ugal: no urgency, just once you have time..07:57
d0ugalsure07:58
d0ugalI need to review the spec first :)07:58
rakhmerovyeah, this is a spec )07:58
rakhmerovI asked you to review the spec07:58
d0ugalyup, it has been on my list. I'll do the spec now.07:59
d0ugalso much stuff to review :)07:59
rakhmerovtrue :)07:59
rakhmerovthis is rather good than bad08:00
d0ugalYup08:00
d0ugalLooking at review stats, Pike might be the first time I review more Mistral than TripleO patches!08:01
rakhmerovhaha :))08:01
rakhmerovthat is making me happy :)08:01
*** shardy_afk is now known as shardy08:09
*** _gryf_ has joined #openstack-mistral08:21
*** _gryf_ has quit IRC08:22
*** gongysh has quit IRC08:22
*** Kevin_Zheng has joined #openstack-mistral08:22
*** amoralej|off is now known as amoralej08:23
d0ugalrakhmerov: I am seeing tons and tons of RPC timeouts lately08:24
d0ugalhas something changed?:08:24
d0ugalExample: http://logs.openstack.org/87/446887/1/gate/gate-tripleo-ci-centos-7-nonha-multinode-oooq/cf58322/logs/undercloud/var/log/mistral/executor.log.txt.gz#_2017-03-24_07_21_56_34608:25
rakhmerovnothing I'd be aware of08:25
d0ugal(that is just a tripleo patch I am helping debug)08:25
rakhmerovd0ugal: oslo.messaging might have changed lately08:25
rakhmerovas a hypothesis08:26
openstackgerrit鲍亚萍 proposed openstack/mistral master: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/44948308:27
rakhmerovd0ugal: yeah, looks terrible08:27
d0ugalrakhmerov: it looks scary if nothing else :)08:27
rakhmerovare you sure it's a problem of Mistral? I mean maybe something really hangs in TripleO and Mistral just times out on waiting a response from TripleO08:29
rakhmerovanyway, it requires serious investigation08:29
d0ugalrakhmerov: no, I'm not sure08:29
rakhmerovok08:29
rakhmerovthe other question: is it 100% reproducible now or just sometimes?08:30
d0ugalrakhmerov: good question - I'll try and find out more.08:30
rakhmerovok08:30
rakhmerovd0ugal: let me know if you need help08:30
d0ugalThanks.08:31
*** gongysh has joined #openstack-mistral08:38
*** gongysh has quit IRC08:43
d0ugalrakhmerov: where will the global context be stored?08:57
rakhmerovwf_ex.context09:03
rakhmerovd0ugal: ^09:03
rakhmerovso it's essentially a dict that already exists and attached to workflow execution09:03
d0ugalrakhmerov: will sub-workflows executions have their own global context?09:04
rakhmerovit is really one object per workflow09:04
d0ugalok, cool09:04
d0ugalI think the answer is "yes" :)09:04
rakhmerovyes09:06
rakhmerovsub-workflows were never meant to share anything with parent workflows09:07
d0ugalyup, good :)09:07
d0ugalthat would be confusing09:07
rakhmerovd0ugal: thanks for reviewing, I replied. I'll wait till what mgershen1 and others say and make changes if necessary09:14
d0ugalrakhmerov: great, thanks09:14
*** gongysh has joined #openstack-mistral09:20
openstackgerritRenat Akhmerov proposed openstack/mistral master: Add missing unit tests for 'publish-on-error'  https://review.openstack.org/44952509:32
*** jrist has quit IRC09:34
openstackgerritRenat Akhmerov proposed openstack/mistral master: Add missing schema validation and unit tests for 'publish-on-error'  https://review.openstack.org/44952509:34
*** brunograz has quit IRC09:49
*** brunograz has joined #openstack-mistral09:49
*** gongysh has quit IRC10:37
*** zhurong has quit IRC10:40
*** thrash|g0ne is now known as thrash11:07
*** gongysh has joined #openstack-mistral11:28
*** sharatss has quit IRC11:48
*** sharatss has joined #openstack-mistral11:48
*** rbrady-afk is now known as rbrady11:55
*** gongysh has quit IRC12:17
*** d0ugal has quit IRC12:24
*** dprince has joined #openstack-mistral12:25
*** dprince has quit IRC12:26
*** dprince has joined #openstack-mistral12:26
*** shardy is now known as shardy_lunch12:31
*** thrash is now known as thrash|brb12:51
*** thrash|brb is now known as thrash13:13
*** shardy_lunch is now known as shardy13:17
*** sharat has joined #openstack-mistral13:17
*** sharatss has quit IRC13:18
*** jrist has joined #openstack-mistral13:22
*** sharat has quit IRC13:22
*** bobh has joined #openstack-mistral13:28
*** bobh has quit IRC13:28
*** bobh has joined #openstack-mistral13:28
*** jkilpatr has quit IRC13:44
*** d0ugal has joined #openstack-mistral13:45
*** amoralej is now known as amoralej|lunch14:21
*** chlong has joined #openstack-mistral14:25
*** jkilpatr has joined #openstack-mistral14:58
*** jaosorior has quit IRC15:30
*** jkilpatr has quit IRC15:35
*** d0ugal has quit IRC15:52
*** amoralej|lunch is now known as amoralej15:52
*** vishwanathj has quit IRC15:58
*** vishwanathj has joined #openstack-mistral15:58
*** d0ugal has joined #openstack-mistral16:14
*** thrash is now known as thrash|f00dz16:18
*** d0ugal has quit IRC16:22
*** bobh has quit IRC16:29
*** thrash|f00dz is now known as thrash16:44
*** catintheroof has joined #openstack-mistral17:14
*** catintheroof has quit IRC17:15
*** catintheroof has joined #openstack-mistral17:15
*** d0ugal has joined #openstack-mistral17:18
openstackgerritMerged openstack/mistral master: Add release note for 'created_at' support in execution()  https://review.openstack.org/44933117:34
openstackgerritMerged openstack/python-mistralclient master: Add release note for region name bugfix  https://review.openstack.org/44933317:35
openstackgerritMerged openstack/python-mistralclient master: Update the gitingore  https://review.openstack.org/44853017:36
*** evgenyl has quit IRC18:08
*** evgenyl has joined #openstack-mistral18:09
*** jkilpatr has joined #openstack-mistral18:16
*** d0ugal has quit IRC18:17
*** chlong has quit IRC18:24
*** jkilpatr has quit IRC19:24
*** dprince has quit IRC19:44
*** toure is now known as toure|afk19:49
*** dprince has joined #openstack-mistral20:03
*** rbrady is now known as rbrady-afk20:09
*** amoralej is now known as amoralej|off20:25
*** dprince has quit IRC21:05
*** joanna has quit IRC21:21
*** thrash is now known as thrash|g0ne21:34
*** joanna has joined #openstack-mistral23:12
*** toure|afk is now known as toure|gone23:33
*** bobh has joined #openstack-mistral23:34
*** bobh has quit IRC23:34
*** bobh has joined #openstack-mistral23:35

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