*** rook-ugh is now known as rook | 00:03 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Trim yaql/jinja evaluation log https://review.openstack.org/447741 | 00:33 |
---|---|---|
*** catintheroof has joined #openstack-mistral | 00:35 | |
*** gongysh has quit IRC | 00:42 | |
*** catintheroof has quit IRC | 01:08 | |
*** jamielennox is now known as jamielennox|away | 01:16 | |
*** jamielennox|away is now known as jamielennox | 01:28 | |
*** gongysh has joined #openstack-mistral | 01:50 | |
*** zhurong has joined #openstack-mistral | 01:57 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Trim yaql/jinja operation log https://review.openstack.org/447741 | 02:39 |
*** zhurong has quit IRC | 03:05 | |
*** zhurong has joined #openstack-mistral | 03:21 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Change some 3rd party package log levels https://review.openstack.org/447777 | 03:22 |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Change some 3rd party package log levels https://review.openstack.org/447777 | 03:23 |
*** jamielennox is now known as jamielennox|away | 04:12 | |
*** zhurong has quit IRC | 04:12 | |
*** d0ugal has quit IRC | 04:16 | |
*** d0ugal has joined #openstack-mistral | 04:18 | |
openstackgerrit | hnyang proposed openstack/mistral master: remove log translations https://review.openstack.org/447795 | 04:45 |
*** zhurong has joined #openstack-mistral | 05:00 | |
*** sharatss has joined #openstack-mistral | 05:24 | |
*** zhurong has quit IRC | 05:40 | |
*** jaosorior has joined #openstack-mistral | 06:31 | |
*** jtomasek has quit IRC | 07:23 | |
*** shardy has joined #openstack-mistral | 08:07 | |
d0ugal | o/ | 08:18 |
*** shardy has quit IRC | 08:24 | |
*** jpich has joined #openstack-mistral | 08:34 | |
*** zhurong has joined #openstack-mistral | 08:35 | |
*** amoralej|off is now known as amoralej | 08:41 | |
kong | hi, guys, in yaql, how do you assert if a key is defined in context or not? | 09:01 |
*** openstackgerrit has quit IRC | 09:03 | |
d0ugal | kong: I'm not toally sure, but maybe $.containsKey('key') | 09:04 |
d0ugal | $ is a dict, right? | 09:04 |
d0ugal | kong: or if you want to default it, you can do this... | 09:05 |
d0ugal | $.get('key', 'DEFAULT') | 09:05 |
d0ugal | We use that quite often. https://github.com/openstack/tripleo-common/blob/master/workbooks/baremetal.yaml#L88 | 09:05 |
kong | d0ugal: that's what i want | 09:07 |
kong | thanks :) | 09:07 |
d0ugal | np | 09:07 |
d0ugal | YAQL is pretty confusing :) | 09:07 |
kong | d0ugal: i have to read its unit test to learn more use case | 09:10 |
d0ugal | Yeah, they are useful | 09:19 |
*** jtomasek has joined #openstack-mistral | 09:42 | |
*** openstackgerrit has joined #openstack-mistral | 09:57 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Fix cinder base import https://review.openstack.org/447984 | 09:57 |
*** mgershen1 has joined #openstack-mistral | 10:02 | |
*** mgershen has quit IRC | 10:05 | |
*** zhurong has quit IRC | 10:05 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Add missing swift actions https://review.openstack.org/447988 | 10:09 |
*** jaosorior is now known as jaosorior_brb | 10:14 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Fix cinder/heat base import https://review.openstack.org/447984 | 10:16 |
*** zhurong has joined #openstack-mistral | 10:24 | |
*** jamielennox|away is now known as jamielennox | 10:30 | |
*** thrash|g0ne is now known as thrash | 10:35 | |
kong | http://logs.openstack.org/80/447380/4/check/python-mistralclient-coverage-ubuntu-xenial-nv/46b6862/console.html | 10:37 |
kong | problem in python-mistralclient gate | 10:37 |
kong | 2017-03-20 15:23:08.123387 | + tox -ecover | 10:37 |
kong | 2017-03-20 15:23:08.298411 | ERROR: unknown environment 'cover' | 10:37 |
d0ugal | kong: yeah, I noticed - it was never setup fully | 10:43 |
d0ugal | it is non-voting at least | 10:43 |
d0ugal | This patch gets it a bit closer, but it wasn't finished either: https://review.openstack.org/#/c/417902/ | 10:44 |
*** jkilpatr has quit IRC | 10:44 | |
*** shardy has joined #openstack-mistral | 10:45 | |
kong | i'm curious where 'venv=cover' comes from | 10:45 |
kong | in the link i mentioned | 10:45 |
kong | i'm not familiar with gate, but i prefer we fix the issue before we merge more patches | 10:46 |
d0ugal | kong: I believe that will come from infra code, that isn't the issue | 10:50 |
d0ugal | cover isn't defined in the tox.ini | 10:50 |
d0ugal | https://github.com/openstack/python-mistralclient/blob/master/tox.ini | 10:50 |
d0ugal | for example in mistral it is: https://github.com/openstack/mistral/blob/master/tox.ini#L34-L39 | 10:50 |
d0ugal | kong: I agree it would be better to fix it, but I think it isn't as important. | 10:51 |
d0ugal | isn't as important as the other gates | 10:51 |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Change some 3rd party package default log levels https://review.openstack.org/447777 | 10:55 |
kong | d0ugal: ooh, sorry, i didn't notice it's coverage job :P | 10:56 |
kong | thought it's tempest at the first glance | 10:57 |
d0ugal | aha, np | 10:57 |
*** mgershen1 has quit IRC | 11:01 | |
*** mgershen has joined #openstack-mistral | 11:01 | |
*** jkilpatr has joined #openstack-mistral | 11:02 | |
*** jamielennox is now known as jamielennox|away | 11:05 | |
*** mgershen has quit IRC | 11:05 | |
thrash | sharatss: https://review.openstack.org/#/c/417902/ you need to add .testr.conf with proper values before it will pass the coverage job. | 11:10 |
thrash | sharatss: http://logs.openstack.org/02/417902/7/check/python-mistralclient-coverage-ubuntu-xenial-nv/6773f37/console.html#_2017-03-21_06_26_47_178151 | 11:10 |
sharatss | thrash, thank u | 11:11 |
sharatss | i will try to add it | 11:11 |
thrash | sharatss: it also looks like you need to add coverage package to test-requirements.txt | 11:11 |
*** jamielennox|away is now known as jamielennox | 11:12 | |
thrash | sharatss: that's probably the bigger problem. :) | 11:13 |
sharatss | thrash, yes. i will check the possible solution | 11:14 |
sharatss | thrash, will update soon :) | 11:14 |
openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient master: Add script for unit test coverage job https://review.openstack.org/417902 | 11:24 |
openstackgerrit | Merged openstack/python-mistralclient master: Accept keystone session in client https://review.openstack.org/447046 | 11:26 |
openstackgerrit | hnyang proposed openstack/mistral master: remove log translations Log messages are no longer being translated. This removes all use of the _LE, _LI, and _LW translation markers to simplify logging and to avoid confusion with new contributions. https://review.openstack.org/448013 | 11:29 |
openstackgerrit | Merged openstack/python-mistralclient master: Use session from OSC plugin https://review.openstack.org/447380 | 11:30 |
thrash | sharatss: merge conflict. :) | 11:32 |
sharatss | thrash, :( i have to do it again? :'( | 11:32 |
*** gongysh has quit IRC | 11:33 | |
thrash | sharatss: looks like it needs a rebase and merge resolve. | 11:34 |
sharatss | thrash, yea.. will check it | 11:35 |
*** rbrady has joined #openstack-mistral | 11:36 | |
*** mgershen has joined #openstack-mistral | 12:03 | |
*** fultonj_ has joined #openstack-mistral | 12:15 | |
*** jaosorior_brb is now known as jaosorior | 12:38 | |
openstackgerrit | yfzhao proposed openstack/python-mistralclient master: Remove log translations https://review.openstack.org/448068 | 12:39 |
*** zhurong has quit IRC | 12:39 | |
openstackgerrit | hnyang proposed openstack/mistral master: remove log translations Log messages are no longer being translated. This removes all use of the _LE, _LI, and _LW translation markers to simplify logging and to avoid confusion with new contributions. https://review.openstack.org/448013 | 12:41 |
*** dprince has joined #openstack-mistral | 12:48 | |
*** fultonj_ is now known as fultonj | 12:50 | |
*** shardy has quit IRC | 12:57 | |
*** gongysh has joined #openstack-mistral | 13:02 | |
*** shardy has joined #openstack-mistral | 13:03 | |
*** pratapa has joined #openstack-mistral | 13:07 | |
*** gongysh has quit IRC | 13:07 | |
*** gongysh has joined #openstack-mistral | 13:13 | |
*** gongysh has quit IRC | 13:15 | |
*** amoralej is now known as amoralej|lunch | 13:20 | |
*** jtomasek has quit IRC | 13:29 | |
*** pratapa has quit IRC | 13:32 | |
*** jtomasek has joined #openstack-mistral | 13:43 | |
*** amoralej|lunch is now known as amoralej | 13:53 | |
*** catintheroof has joined #openstack-mistral | 13:57 | |
*** sharatss has quit IRC | 14:00 | |
*** bobh has joined #openstack-mistral | 14:05 | |
*** chlong has joined #openstack-mistral | 14:17 | |
*** jkilpatr has quit IRC | 14:21 | |
*** chlong has quit IRC | 14:24 | |
*** jkilpatr has joined #openstack-mistral | 14:34 | |
*** vishwanathj has joined #openstack-mistral | 14:36 | |
*** shardy is now known as coding_shardy | 15:23 | |
*** jrist has quit IRC | 15:33 | |
*** thrash is now known as thrash|bbl | 15:39 | |
*** jrist has joined #openstack-mistral | 15:40 | |
*** jaosorior has quit IRC | 16:29 | |
*** jpich has quit IRC | 16:43 | |
*** rbrady has quit IRC | 16:47 | |
*** jkilpatr_ has joined #openstack-mistral | 16:51 | |
*** jkilpatr has quit IRC | 16:54 | |
*** jaosorior has joined #openstack-mistral | 16:55 | |
*** coding_shardy is now known as shardy | 16:57 | |
*** thrash|bbl is now known as thrash | 16:58 | |
*** rbrady has joined #openstack-mistral | 17:02 | |
*** Qiming has quit IRC | 17:15 | |
d0ugal | Weird. I have a workflow that is "RUNNING" but all the tasks and actions have state SUCCESS | 17:18 |
rbrady | d0gual: any RPC timeouts in the log? | 17:19 |
*** Qiming has joined #openstack-mistral | 17:19 | |
d0ugal | rbrady: nope | 17:20 |
d0ugal | jaosorior: Are you still around? | 17:24 |
d0ugal | I wonder if we managed to break Mistral :) | 17:24 |
d0ugal | rbrady: http://paste.openstack.org/show/603646/ | 17:28 |
jaosorior | d0ugal: I'm kind of around | 17:28 |
jaosorior | d0ugal: what happened? | 17:29 |
d0ugal | jaosorior: I'm not sure yet | 17:29 |
d0ugal | jaosorior: http://paste.openstack.org/show/603647/ | 17:29 |
d0ugal | That is the only error that I've found | 17:29 |
*** catintheroof has quit IRC | 17:30 | |
*** catintheroof has joined #openstack-mistral | 17:30 | |
openstackgerrit | Dougal Matthews proposed openstack/mistral master: Gate Test https://review.openstack.org/448206 | 17:31 |
rbrady | d0ugal: do all mistral cli commands fail? does environment-list work? | 17:33 |
*** bobh has quit IRC | 17:34 | |
d0ugal | rbrady: no, the env-list works fine | 17:34 |
d0ugal | I think the issue is related to the mistral client | 17:34 |
d0ugal | so only the mistral actions | 17:34 |
d0ugal | but really, not sure | 17:34 |
d0ugal | I'll try another actions | 17:34 |
rbrady | d0gual: ack, that was my next question for run-action | 17:35 |
d0ugal | rbrady: okay, seems like it is all actions. | 17:35 |
d0ugal | including std.noop | 17:35 |
jaosorior | d0ugal: are the mistral actions using sessions? | 17:36 |
d0ugal | but I think it is because the Mistral engine had fallen over | 17:36 |
d0ugal | jaosorior: no | 17:36 |
jaosorior | d0ugal: well that's strange; it should be using the legacy code if sessions aren't specified | 17:36 |
jaosorior | d0ugal: is that with the latest mistral? I was testing out an undercloud/overcloud deployment and had it working with the session bits | 17:38 |
d0ugal | jaosorior: yeah, I just installed today | 17:38 |
d0ugal | and I've seen this problem 3 times in a row on fresh environments :) | 17:38 |
d0ugal | I thought the first one just went weird, so I reprovisioned it and another one at the same time | 17:38 |
d0ugal | they are all bust :( | 17:38 |
jaosorior | d0ugal: is there more traceback we could see? | 17:41 |
d0ugal | jaosorior: no, not that I've managed to get yet | 17:42 |
d0ugal | this is weird :) | 17:42 |
*** jtomasek has quit IRC | 17:43 | |
jaosorior | d0ugal: got an environment I could log into? | 17:44 |
d0ugal | jaosorior: http://paste.openstack.org/show/603650/ | 17:44 |
d0ugal | I added "raise" here: | 17:45 |
d0ugal | https://github.com/openstack/mistral/blob/master/mistral/actions/openstack/base.py#L139 | 17:45 |
d0ugal | to get the real traceback | 17:45 |
d0ugal | and now I see it was logged a few lines above, not sure how I missed that. | 17:45 |
jaosorior | d0ugal: try to get the URL it tried to use | 17:46 |
jaosorior | d0ugal: that's the non-session branch | 17:46 |
d0ugal | jaosorior: yeah, so likely something else | 17:47 |
d0ugal | Maybe I'm just doing something odd. I'll keep digging for a bit | 17:48 |
jaosorior | d0ugal: well, that URL will give some answers | 17:48 |
d0ugal | jaosorior: which URL? | 17:48 |
jaosorior | d0ugal: the paste that you showed me http://paste.openstack.org/show/603650/ failed cause of a 404 | 17:48 |
jaosorior | d0ugal: that means that it tried to access an invalid URL | 17:48 |
d0ugal | ah, of course | 17:49 |
jaosorior | d0ugal: so checking what URL that was will tell us a bit more what happened | 17:49 |
jaosorior | d0ugal: cause then we can trace, from where did that URL come from | 17:49 |
jaosorior | d0ugal: you could print the auth_url that's given to the keystone client here https://github.com/openstack/python-mistralclient/blob/master/mistralclient/auth/keystone.py#L90 | 17:52 |
d0ugal | jaosorior: doing almost exactly that :) | 17:53 |
d0ugal | jaosorior: http://192.168.24.1:5000 | 17:53 |
jaosorior | d0ugal: oh... that's funky. The stack trace you showed me says the failure was in line 105. However, in the current code, line 105 is this https://github.com/openstack/python-mistralclient/blob/master/mistralclient/auth/keystone.py#L105 | 17:54 |
jaosorior | d0ugal: dude, you're not using this patch https://github.com/openstack/python-mistralclient/commit/ca313b0de4c1398d7e330892b7ce8d64a7ab911e | 17:55 |
d0ugal | oh | 17:56 |
d0ugal | jaosorior: nope, I seem to be on an old mistral | 17:56 |
jaosorior | d0ugal: it fails because you are using versionless endpoints, and your mistral is old | 17:56 |
d0ugal | maybe I am doing quickstart wrong, I thought it was supposed to install master of everything | 17:56 |
jaosorior | d0ugal: nope, it installs what's promoted | 17:56 |
jaosorior | d0ugal: and there hasn't been promotion for 8 days | 17:56 |
jaosorior | d0ugal: so yeah, that's why :( | 17:57 |
jaosorior | anyway, now we know :D | 17:57 |
d0ugal | jaosorior: indeed, thanks for helping | 17:57 |
jaosorior | d0ugal: no biggie man | 17:57 |
jaosorior | pretty glad we figured it out | 17:57 |
jaosorior | alright, I'm off now | 17:57 |
d0ugal | cya | 17:57 |
jaosorior | have a good one! | 17:57 |
d0ugal | sorry for the hold up! | 17:57 |
*** jaosorior has quit IRC | 17:57 | |
*** dprince has quit IRC | 18:10 | |
*** rbrady has quit IRC | 18:13 | |
*** shardy has quit IRC | 18:18 | |
*** weshay_sick is now known as weshay_brb | 18:25 | |
*** rbrady has joined #openstack-mistral | 18:28 | |
*** bobh has joined #openstack-mistral | 18:31 | |
*** weshay_brb is now known as weshay | 18:42 | |
*** dprince has joined #openstack-mistral | 18:43 | |
*** chlong has joined #openstack-mistral | 19:13 | |
*** amoralej is now known as amoralej|off | 19:44 | |
mwhahaha | i don't suppose anyone has a decent set of examples for interacting with the swift actions from a workbook? | 19:50 |
*** chlong has quit IRC | 20:21 | |
*** clenimar has joined #openstack-mistral | 20:26 | |
*** jkilpatr_ has quit IRC | 20:28 | |
*** jkilpatr has joined #openstack-mistral | 20:44 | |
*** chlong has joined #openstack-mistral | 20:48 | |
*** bobh has quit IRC | 21:49 | |
*** dprince has quit IRC | 21:54 | |
*** catintheroof has quit IRC | 21:57 | |
*** catintheroof has joined #openstack-mistral | 21:58 | |
*** catintheroof has quit IRC | 21:58 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Trim yaql/jinja operation log https://review.openstack.org/447741 | 22:08 |
*** Qiming has quit IRC | 22:09 | |
*** Qiming has joined #openstack-mistral | 22:12 | |
*** chlong has quit IRC | 22:27 | |
*** harlowja has quit IRC | 23:06 | |
*** jamielennox is now known as jamielennox|away | 23:35 | |
*** jamielennox|away is now known as jamielennox | 23:39 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!