*** harlowja has quit IRC | 00:06 | |
*** thrash has quit IRC | 00:09 | |
*** thrash has joined #openstack-mistral | 00:20 | |
*** thrash has joined #openstack-mistral | 00:20 | |
*** harlowja has joined #openstack-mistral | 00:34 | |
*** jamielennox is now known as jamielennox|away | 00:57 | |
*** jamielennox|away is now known as jamielennox | 01:01 | |
*** harlowja has quit IRC | 01:25 | |
*** jamielennox is now known as jamielennox|away | 01:41 | |
*** jamielennox|away is now known as jamielennox | 01:47 | |
*** thrash is now known as thrash|g0ne | 03:05 | |
*** catintheroof has quit IRC | 04:06 | |
*** catintheroof has joined #openstack-mistral | 04:06 | |
*** catintheroof has quit IRC | 04:06 | |
*** bobh has joined #openstack-mistral | 04:25 | |
openstackgerrit | Spencer Yu proposed openstack/mistral: Replace six.iteritems() with .items() https://review.openstack.org/411628 | 04:57 |
---|---|---|
*** bobh has quit IRC | 05:00 | |
*** sharatss has quit IRC | 05:02 | |
*** sharatss has joined #openstack-mistral | 05:03 | |
*** bobh has joined #openstack-mistral | 05:07 | |
*** bobh has quit IRC | 05:09 | |
*** jaosorior has joined #openstack-mistral | 06:02 | |
*** ist has joined #openstack-mistral | 06:16 | |
openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Test - Do not merge https://review.openstack.org/411645 | 06:18 |
rakhmerov | ddeja, d0ugal: hi, what did I miss yesterday? :) | 06:27 |
rakhmerov | figured out what's wrong with our gates? | 06:27 |
rakhmerov | kong_: hi Lingxian, did you make a release? | 06:30 |
rakhmerov | ooh, I see the patch | 06:31 |
rakhmerov | ok | 06:31 |
rakhmerov | https://review.openstack.org/#/c/411489/ | 06:31 |
openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Test - Do not merge https://review.openstack.org/411645 | 06:32 |
openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Test - Do not merge https://review.openstack.org/411645 | 06:44 |
*** ist has quit IRC | 06:44 | |
*** ist has joined #openstack-mistral | 06:49 | |
openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Test - Do not merge https://review.openstack.org/411645 | 07:06 |
rakhmerov | ddeja, d0ugal, kong_, hparekh: guys, please review my 'with-items' patches, they are pretty important (it was broken for a long time) | 07:17 |
*** jamielennox is now known as jamielennox|away | 08:07 | |
*** jamielennox|away is now known as jamielennox | 08:14 | |
openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Test do not merge https://review.openstack.org/411691 | 08:16 |
*** shardy has joined #openstack-mistral | 08:16 | |
openstackgerrit | Spencer Yu proposed openstack/mistral: Replace six.iteritems() with .items() https://review.openstack.org/411628 | 08:26 |
*** jaosorior has quit IRC | 08:32 | |
*** jaosorior has joined #openstack-mistral | 08:33 | |
ddeja | rakhmerov: Hi | 08:36 |
rakhmerov | hi | 08:36 |
ddeja | heat wasn't installed properly | 08:36 |
ddeja | I've submitted patch to openstack-infra that should fix it | 08:37 |
rakhmerov | ooh, ok | 08:37 |
rakhmerov | thank you | 08:37 |
ddeja | https://review.openstack.org/#/c/411283/ | 08:39 |
rakhmerov | ok | 08:40 |
sharatss | ddeja: similarly any idea on mistral-dashboard docs gate | 08:43 |
*** jaosorior has quit IRC | 08:55 | |
ddeja | rakhmerov: It got merged | 09:04 |
ddeja | let me check if it works | 09:04 |
ddeja | sharatss: no, I didn't look on the mistral-dashboard gate | 09:04 |
rakhmerov | ok | 09:05 |
sharatss | ddeja: i was looking into it.. didnt yet come up with a solution :( | 09:07 |
ddeja | sharatss: I may have some time later today to look on it | 09:17 |
sharatss | ddeja: okay, thanks :) | 09:18 |
*** jamielennox is now known as jamielennox|away | 09:48 | |
*** jamielennox|away is now known as jamielennox | 09:55 | |
sharatss | rakhmerov: hi | 10:10 |
*** openstackgerrit has quit IRC | 10:18 | |
rakhmerov | hi | 10:21 |
sharatss | rakhmerov: https://blueprints.launchpad.net/mistral/+spec/mistral-dashboard-add-doc-links-in-create-views | 10:54 |
sharatss | according to that BP, should the link be added to the create-workflow dialog box? | 10:54 |
rakhmerov | yes | 10:55 |
rakhmerov | create workflow, update workflow | 10:55 |
rakhmerov | create workbook, update workbook | 10:55 |
sharatss | rakhmerov: yes. "Refer DSL Document for sample workbook template" would be ok? | 10:56 |
sharatss | DSL Document would be hyperlink | 10:56 |
rakhmerov | I would do something like "Refer to Mistral workflow language documentation for syntax details" | 10:57 |
rakhmerov | d0ugal: do we still have issues with TripleO gate? | 11:00 |
rakhmerov | I rechecked https://review.openstack.org/#/c/408515/ again and it failed again | 11:00 |
d0ugal | rakhmerov: Yeah, so I think there have been three big issues that all seemed to happen this week :/ | 11:00 |
d0ugal | I can try and check on the status shortly | 11:01 |
d0ugal | rakhmerov: if it is only blocking my patches, then that is fine :) but if it is stopping anyone else progressing do let me know. | 11:01 |
sharatss | rakhmerov: okay :) | 11:02 |
rakhmerov | d0ugal: it fails often for other patches too | 11:03 |
rakhmerov | I'm just confused why it fails always on your patch | 11:04 |
rakhmerov | it looks really innocent | 11:04 |
rakhmerov | maybe TripleO gate does some checks on what's printed out to std.out? | 11:05 |
d0ugal | Nah, I don't think so | 11:05 |
rakhmerov | because now we print also 'event_engine' | 11:05 |
rakhmerov | yeah, just brainstorming.. | 11:05 |
d0ugal | Yeah, I'll take a look at that failure and see if it is always the same | 11:06 |
rakhmerov | щл | 11:07 |
rakhmerov | ok, thanks | 11:07 |
rakhmerov | d0ugal: btw, did you check if you use 'with-items' somewhere in TripleO? | 11:08 |
d0ugal | rakhmerov: Yeah, we do in a few places. | 11:11 |
d0ugal | rakhmerov: is that likely to cause us problems? | 11:13 |
d0ugal | I'm not sure I fully understood the current limitation. | 11:13 |
rakhmerov | so, 5 seconds.. | 11:13 |
rakhmerov | you know that I'm currently working on 'concurrency' | 11:13 |
d0ugal | Yeah | 11:14 |
rakhmerov | once my patches get merged in it will work | 11:14 |
rakhmerov | another thing that I found today and am thinking how to solve is the combination of 'with-items' and 'retry' policy | 11:14 |
rakhmerov | there's a crazy problem with that | 11:14 |
rakhmerov | it won't work correctly now if you try it | 11:15 |
rakhmerov | in some cases | 11:15 |
ddeja | rakhmerov: about the failing dsvm gate | 11:15 |
ddeja | it is still failing | 11:15 |
ddeja | my pathc to infra got merged | 11:16 |
ddeja | but it looks like gate is not using it? | 11:16 |
d0ugal | rakhmerov: we use with-items in 12 places, but never with retry | 11:16 |
d0ugal | ddeja: is there a delay before the infra patch will show up in the CI run? | 11:16 |
ddeja | d0ugal: I think there is no | 11:17 |
ddeja | but I've rechecked again to be sure before I go to the infra channel | 11:18 |
rakhmerov | ddeja: yes, there's a delay | 11:18 |
rakhmerov | that's ok | 11:18 |
rakhmerov | I also added coverage job for mistral and it's not there yet | 11:18 |
rakhmerov | d0ugal: ok, w/o retry it's fine | 11:18 |
rakhmerov | anyway, I'll fix that soon | 11:19 |
rakhmerov | both things | 11:19 |
d0ugal | \o/ | 11:19 |
rakhmerov | I think Andreas told me about this delay | 11:19 |
ddeja | rakhmerov: oh, OK. Then I'll wait till tommorow | 11:21 |
ddeja | or till monday, since it is Friday today | 11:21 |
rakhmerov | it's several hours AFAIK | 11:21 |
*** jaosorior has joined #openstack-mistral | 11:38 | |
*** openstackgerrit has joined #openstack-mistral | 11:44 | |
openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Added links to DSL docs in create/update views https://review.openstack.org/411770 | 11:44 |
openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Added links to DSL docs in create/update views https://review.openstack.org/411770 | 11:46 |
*** thrash|g0ne is now known as thrash | 11:46 | |
ddeja | rakhmerov: OK, dsvm gate got fixed https://review.openstack.org/#/c/411288/1 | 11:57 |
ddeja | and in the same time, postgres/mysql gates start failing... | 11:57 |
rakhmerov | cool | 11:58 |
rakhmerov | yeah :)) | 11:58 |
rakhmerov | btw, this is our coverage report: http://logs.openstack.org/88/411288/1/check/mistral-coverage-ubuntu-xenial/fec3a5b/cover/ | 11:58 |
rakhmerov | 83 % | 11:58 |
rakhmerov | not as bad as I thought it was | 11:59 |
rakhmerov | but we need to make it 90% | 11:59 |
rakhmerov | at least | 11:59 |
ddeja | rakhmerov: OK | 11:59 |
rakhmerov | ddeja: ooh, it's that module vine.five | 11:59 |
rakhmerov | again | 11:59 |
rakhmerov | what was the problem with it last time? We waited for oslo.m new version, right? | 12:00 |
rakhmerov | ddeja: btw, the dinosaur is awesome! :) | 12:04 |
rakhmerov | did you create it manually or used some tool? :) | 12:04 |
*** catintheroof has joined #openstack-mistral | 12:34 | |
*** dprince has joined #openstack-mistral | 12:46 | |
ddeja | rakhmerov: yes, we've just waited for the oslo messaging update | 12:49 |
ddeja | I guess there must be a new version of kombu, this time 4.0.2 | 12:49 |
ddeja | in kombu requirements there is something like kombu>3.0.0 !=4.0.0!=4.0.1 | 12:49 |
ddeja | in oslo requirements* | 12:50 |
ddeja | about the dinosaur: found it on the internet, but you can also use cowsay: cowsay -f stegosaurus Hello, Mistral! | 12:51 |
ddeja | oh, and you can use 'cowsay -l' to list all available cowsays on your OS | 12:53 |
ddeja | rakhmerov ^ | 12:53 |
*** weshay_bbiab is now known as weshay | 13:01 | |
*** rbrady-afk is now known as rbrady | 13:16 | |
*** bobh has joined #openstack-mistral | 13:18 | |
ddeja | rakhmerov: My guesses were right, there is new kombu version 4.0.2 | 13:18 |
*** bobh has quit IRC | 13:27 | |
*** chlong has quit IRC | 14:00 | |
openstackgerrit | Spencer Yu proposed openstack/mistral: Replace six.iteritems() with .items() https://review.openstack.org/411845 | 14:47 |
rakhmerov | ddeja: thanks! | 14:54 |
rakhmerov | got it | 14:54 |
ddeja | rakhmerov: no problem. After they fix the problem with kombu in oslo, we should have all the gates green :) | 14:57 |
rakhmerov | that's very cool :) | 15:03 |
rakhmerov | awesome job, Dawid | 15:03 |
ddeja | rakhmerov: thanks :) | 15:04 |
*** bobh has joined #openstack-mistral | 15:07 | |
*** bobh has quit IRC | 15:13 | |
*** jaosorior has quit IRC | 15:15 | |
*** bobh has joined #openstack-mistral | 15:18 | |
*** bobh has quit IRC | 15:19 | |
*** bobh has joined #openstack-mistral | 15:20 | |
*** chlong has joined #openstack-mistral | 15:21 | |
*** bobh has quit IRC | 15:24 | |
*** szaher has quit IRC | 15:49 | |
*** ist has quit IRC | 15:51 | |
openstackgerrit | Spencer Yu proposed openstack/mistral: Replace six.iteritems() with .items() https://review.openstack.org/411845 | 16:26 |
*** chlong has quit IRC | 16:29 | |
*** chlong has joined #openstack-mistral | 16:53 | |
*** bobh has joined #openstack-mistral | 17:00 | |
*** bobh has quit IRC | 17:04 | |
*** bobh has joined #openstack-mistral | 17:29 | |
*** bobh has quit IRC | 17:37 | |
*** bobh has joined #openstack-mistral | 17:51 | |
*** dprince has quit IRC | 18:02 | |
*** dprince has joined #openstack-mistral | 18:41 | |
*** bobh has quit IRC | 19:04 | |
*** sharatss has quit IRC | 19:11 | |
*** sharatss has joined #openstack-mistral | 19:12 | |
*** rbrady is now known as rbrady-afk | 19:13 | |
*** bobh has joined #openstack-mistral | 19:17 | |
*** bobh has quit IRC | 19:52 | |
*** clenimar has quit IRC | 19:55 | |
*** bobh has joined #openstack-mistral | 19:57 | |
*** jtomasek has quit IRC | 20:33 | |
*** chlong has quit IRC | 20:48 | |
*** bobh has quit IRC | 20:53 | |
*** dprince has quit IRC | 21:08 | |
*** catintheroof has quit IRC | 21:10 | |
*** bobh has joined #openstack-mistral | 21:10 | |
*** catintheroof has joined #openstack-mistral | 21:11 | |
*** catintheroof has quit IRC | 21:15 | |
*** bobh has quit IRC | 21:45 | |
*** weshay is now known as weshay_lata | 22:32 | |
*** toure has quit IRC | 22:34 | |
*** toure has joined #openstack-mistral | 22:36 | |
*** bobh has joined #openstack-mistral | 22:49 | |
*** bobh has quit IRC | 22:53 | |
*** bobh has joined #openstack-mistral | 22:53 | |
*** bobh has quit IRC | 22:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!