*** bobh has quit IRC | 00:06 | |
*** vishwanathj has joined #openstack-mistral | 00:39 | |
*** vishwanathj has quit IRC | 00:44 | |
*** zhurong has joined #openstack-mistral | 01:26 | |
*** jamielennox is now known as jamielennox|away | 01:39 | |
*** vishwanathj has joined #openstack-mistral | 01:40 | |
*** bobh has joined #openstack-mistral | 01:44 | |
*** vishwanathj has quit IRC | 01:45 | |
*** jamielennox|away is now known as jamielennox | 01:48 | |
*** gongysh has joined #openstack-mistral | 01:49 | |
*** vishwanathj has joined #openstack-mistral | 02:06 | |
*** bobh has quit IRC | 02:07 | |
*** vishwanathj has quit IRC | 02:07 | |
*** vishwanathj has joined #openstack-mistral | 02:09 | |
*** vishwanathj has quit IRC | 02:10 | |
*** vishwanathj has joined #openstack-mistral | 02:11 | |
*** bobh has joined #openstack-mistral | 02:15 | |
*** bobh has quit IRC | 03:15 | |
*** gongysh has quit IRC | 03:20 | |
*** zhurong has quit IRC | 04:07 | |
rakhmerov | d0ugal: well, we mostly write them (release notes) | 04:43 |
---|---|---|
*** Kevin_Zheng has quit IRC | 05:13 | |
*** zhurong has joined #openstack-mistral | 05:16 | |
openstackgerrit | Merged openstack/python-mistralclient master: Take DST into account when converting to UTC https://review.openstack.org/449117 | 05:18 |
*** gongysh has joined #openstack-mistral | 05:29 | |
*** sharatss has joined #openstack-mistral | 06:50 | |
*** gongysh has quit IRC | 06:51 | |
*** gongysh has joined #openstack-mistral | 07:02 | |
d0ugal | rakhmerov: sort-of, there are not that many release notes for how many changes there are... | 07:24 |
rakhmerov | :) | 07:26 |
rakhmerov | what I really meant is that it's not an excuse to not write release notes | 07:27 |
rakhmerov | we should be doing it | 07:27 |
d0ugal | sure, we should be stricter about doing it in reviews | 07:51 |
d0ugal | rakhmerov: are there any special rules for the mistral-sepcs repo? | 07:52 |
d0ugal | in terms of reviewing | 07:52 |
d0ugal | I see you +2'ed my spec - do I just need another core? mgershen1 or kong maybe? | 07:53 |
rakhmerov | d0ugal: well, we've been following pretty much the same 2 +2 rule | 07:54 |
rakhmerov | for specs | 07:54 |
d0ugal | okay, cool | 07:54 |
rakhmerov | to me it would even make more sense if we had 3 +2 rule because on specs we usually expect wider feedback | 07:54 |
d0ugal | Yup | 07:55 |
*** jaosorior has joined #openstack-mistral | 07:55 | |
rakhmerov | we can mention it in the policy that you wrote btw | 07:55 |
d0ugal | Good idea. | 07:55 |
rakhmerov | as additional patch | 07:55 |
rakhmerov | yeah | 07:55 |
rakhmerov | if people agree with that we'll accept that change | 07:55 |
rakhmerov | d0ugal: btw, I already started implementing https://review.openstack.org/#/c/446960/ | 07:56 |
rakhmerov | could you pls have a look at it? | 07:56 |
d0ugal | Sure | 07:56 |
rakhmerov | it should be familiar after global wf context to you | 07:57 |
rakhmerov | it's just more generic and elegant IMO and people seem to like it | 07:57 |
rakhmerov | d0ugal: no urgency, just once you have time.. | 07:57 |
d0ugal | sure | 07:58 |
d0ugal | I need to review the spec first :) | 07:58 |
rakhmerov | yeah, this is a spec ) | 07:58 |
rakhmerov | I asked you to review the spec | 07:58 |
d0ugal | yup, it has been on my list. I'll do the spec now. | 07:59 |
d0ugal | so much stuff to review :) | 07:59 |
rakhmerov | true :) | 07:59 |
rakhmerov | this is rather good than bad | 08:00 |
d0ugal | Yup | 08:00 |
d0ugal | Looking at review stats, Pike might be the first time I review more Mistral than TripleO patches! | 08:01 |
rakhmerov | haha :)) | 08:01 |
rakhmerov | that is making me happy :) | 08:01 |
*** shardy_afk is now known as shardy | 08:09 | |
*** _gryf_ has joined #openstack-mistral | 08:21 | |
*** _gryf_ has quit IRC | 08:22 | |
*** gongysh has quit IRC | 08:22 | |
*** Kevin_Zheng has joined #openstack-mistral | 08:22 | |
*** amoralej|off is now known as amoralej | 08:23 | |
d0ugal | rakhmerov: I am seeing tons and tons of RPC timeouts lately | 08:24 |
d0ugal | has something changed?: | 08:24 |
d0ugal | Example: 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_346 | 08:25 |
rakhmerov | nothing I'd be aware of | 08:25 |
d0ugal | (that is just a tripleo patch I am helping debug) | 08:25 |
rakhmerov | d0ugal: oslo.messaging might have changed lately | 08:25 |
rakhmerov | as a hypothesis | 08:26 |
openstackgerrit | 鲍亚萍 proposed openstack/mistral master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/449483 | 08:27 |
rakhmerov | d0ugal: yeah, looks terrible | 08:27 |
d0ugal | rakhmerov: it looks scary if nothing else :) | 08:27 |
rakhmerov | are 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 TripleO | 08:29 |
rakhmerov | anyway, it requires serious investigation | 08:29 |
d0ugal | rakhmerov: no, I'm not sure | 08:29 |
rakhmerov | ok | 08:29 |
rakhmerov | the other question: is it 100% reproducible now or just sometimes? | 08:30 |
d0ugal | rakhmerov: good question - I'll try and find out more. | 08:30 |
rakhmerov | ok | 08:30 |
rakhmerov | d0ugal: let me know if you need help | 08:30 |
d0ugal | Thanks. | 08:31 |
*** gongysh has joined #openstack-mistral | 08:38 | |
*** gongysh has quit IRC | 08:43 | |
d0ugal | rakhmerov: where will the global context be stored? | 08:57 |
rakhmerov | wf_ex.context | 09:03 |
rakhmerov | d0ugal: ^ | 09:03 |
rakhmerov | so it's essentially a dict that already exists and attached to workflow execution | 09:03 |
d0ugal | rakhmerov: will sub-workflows executions have their own global context? | 09:04 |
rakhmerov | it is really one object per workflow | 09:04 |
d0ugal | ok, cool | 09:04 |
d0ugal | I think the answer is "yes" :) | 09:04 |
rakhmerov | yes | 09:06 |
rakhmerov | sub-workflows were never meant to share anything with parent workflows | 09:07 |
d0ugal | yup, good :) | 09:07 |
d0ugal | that would be confusing | 09:07 |
rakhmerov | d0ugal: thanks for reviewing, I replied. I'll wait till what mgershen1 and others say and make changes if necessary | 09:14 |
d0ugal | rakhmerov: great, thanks | 09:14 |
*** gongysh has joined #openstack-mistral | 09:20 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add missing unit tests for 'publish-on-error' https://review.openstack.org/449525 | 09:32 |
*** jrist has quit IRC | 09:34 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add missing schema validation and unit tests for 'publish-on-error' https://review.openstack.org/449525 | 09:34 |
*** brunograz has quit IRC | 09:49 | |
*** brunograz has joined #openstack-mistral | 09:49 | |
*** gongysh has quit IRC | 10:37 | |
*** zhurong has quit IRC | 10:40 | |
*** thrash|g0ne is now known as thrash | 11:07 | |
*** gongysh has joined #openstack-mistral | 11:28 | |
*** sharatss has quit IRC | 11:48 | |
*** sharatss has joined #openstack-mistral | 11:48 | |
*** rbrady-afk is now known as rbrady | 11:55 | |
*** gongysh has quit IRC | 12:17 | |
*** d0ugal has quit IRC | 12:24 | |
*** dprince has joined #openstack-mistral | 12:25 | |
*** dprince has quit IRC | 12:26 | |
*** dprince has joined #openstack-mistral | 12:26 | |
*** shardy is now known as shardy_lunch | 12:31 | |
*** thrash is now known as thrash|brb | 12:51 | |
*** thrash|brb is now known as thrash | 13:13 | |
*** shardy_lunch is now known as shardy | 13:17 | |
*** sharat has joined #openstack-mistral | 13:17 | |
*** sharatss has quit IRC | 13:18 | |
*** jrist has joined #openstack-mistral | 13:22 | |
*** sharat has quit IRC | 13:22 | |
*** bobh has joined #openstack-mistral | 13:28 | |
*** bobh has quit IRC | 13:28 | |
*** bobh has joined #openstack-mistral | 13:28 | |
*** jkilpatr has quit IRC | 13:44 | |
*** d0ugal has joined #openstack-mistral | 13:45 | |
*** amoralej is now known as amoralej|lunch | 14:21 | |
*** chlong has joined #openstack-mistral | 14:25 | |
*** jkilpatr has joined #openstack-mistral | 14:58 | |
*** jaosorior has quit IRC | 15:30 | |
*** jkilpatr has quit IRC | 15:35 | |
*** d0ugal has quit IRC | 15:52 | |
*** amoralej|lunch is now known as amoralej | 15:52 | |
*** vishwanathj has quit IRC | 15:58 | |
*** vishwanathj has joined #openstack-mistral | 15:58 | |
*** d0ugal has joined #openstack-mistral | 16:14 | |
*** thrash is now known as thrash|f00dz | 16:18 | |
*** d0ugal has quit IRC | 16:22 | |
*** bobh has quit IRC | 16:29 | |
*** thrash|f00dz is now known as thrash | 16:44 | |
*** catintheroof has joined #openstack-mistral | 17:14 | |
*** catintheroof has quit IRC | 17:15 | |
*** catintheroof has joined #openstack-mistral | 17:15 | |
*** d0ugal has joined #openstack-mistral | 17:18 | |
openstackgerrit | Merged openstack/mistral master: Add release note for 'created_at' support in execution() https://review.openstack.org/449331 | 17:34 |
openstackgerrit | Merged openstack/python-mistralclient master: Add release note for region name bugfix https://review.openstack.org/449333 | 17:35 |
openstackgerrit | Merged openstack/python-mistralclient master: Update the gitingore https://review.openstack.org/448530 | 17:36 |
*** evgenyl has quit IRC | 18:08 | |
*** evgenyl has joined #openstack-mistral | 18:09 | |
*** jkilpatr has joined #openstack-mistral | 18:16 | |
*** d0ugal has quit IRC | 18:17 | |
*** chlong has quit IRC | 18:24 | |
*** jkilpatr has quit IRC | 19:24 | |
*** dprince has quit IRC | 19:44 | |
*** toure is now known as toure|afk | 19:49 | |
*** dprince has joined #openstack-mistral | 20:03 | |
*** rbrady is now known as rbrady-afk | 20:09 | |
*** amoralej is now known as amoralej|off | 20:25 | |
*** dprince has quit IRC | 21:05 | |
*** joanna has quit IRC | 21:21 | |
*** thrash is now known as thrash|g0ne | 21:34 | |
*** joanna has joined #openstack-mistral | 23:12 | |
*** toure|afk is now known as toure|gone | 23:33 | |
*** bobh has joined #openstack-mistral | 23:34 | |
*** bobh has quit IRC | 23:34 | |
*** bobh has joined #openstack-mistral | 23:35 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!