| *** kong_ has joined #openstack-mistral | 00:26 | |
| *** catintheroof has joined #openstack-mistral | 01:58 | |
| *** catintheroof has quit IRC | 02:06 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Modify some codes in '.mistral/tests/unit/engine/base.py' https://review.openstack.org/406691 | 04:02 |
|---|---|---|
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: std.email action requires a smtp_password https://review.openstack.org/406694 | 04:11 |
| *** janki has joined #openstack-mistral | 05:34 | |
| *** sharatss has quit IRC | 05:47 | |
| *** sharatss has joined #openstack-mistral | 05:47 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: std.email action requires a smtp_password https://review.openstack.org/406694 | 05:56 |
| rakhmerov | sharatss: what patch? | 06:08 |
| sharatss | rakhmerov, hi.. it has some problem. i am working on it. will update you onc eit is done | 06:11 |
| sharatss | rakhmerov, regarding that UTC and localtime | 06:12 |
| rakhmerov | ok | 06:12 |
| rakhmerov | I see | 06:12 |
| *** igormarnat__ has joined #openstack-mistral | 06:46 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: change the cron-trigger execution time from localtime to UTC https://review.openstack.org/406740 | 06:46 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: change the cron-trigger execution time from localtime to UTC https://review.openstack.org/406740 | 06:48 |
| openstackgerrit | Merged openstack/mistral-dashboard: Updated from global requirements https://review.openstack.org/406457 | 06:48 |
| *** kong_ has quit IRC | 06:50 | |
| *** igormarnat has quit IRC | 06:53 | |
| *** igormarnat__ is now known as igormarnat | 06:54 | |
| openstackgerrit | Sharat Sharma proposed openstack/mistral: Added releasenote for retry policy update https://review.openstack.org/404142 | 06:58 |
| *** d0ugal has joined #openstack-mistral | 07:51 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: change the cron-trigger execution time from localtime to UTC https://review.openstack.org/406740 | 07:53 |
| *** openstackgerrit has quit IRC | 08:03 | |
| rakhmerov | ddeja_, sharatss, d0ugal, hparekh, kong: any ideas why this is happening? http://logs.openstack.org/93/405893/2/check/gate-python-mistralclient-devstack-dsvm/0c872b9/console.html | 08:27 |
| d0ugal | no idea | 08:29 |
| rakhmerov | I guess this is related to the new keystone client | 08:29 |
| rakhmerov | ok | 08:29 |
| d0ugal | ah, maybe | 08:29 |
| d0ugal | I didn't notice there was a new keystone client | 08:30 |
| rakhmerov | ddeja_: do you think your questions are answered here? https://review.openstack.org/#/c/401188/ | 08:31 |
| rakhmerov | d0ugal: yeah, we recently fixed something related to that already.. looking | 08:31 |
| *** jistr is now known as jistr|mtgs | 08:36 | |
| *** jamielennox is now known as jamielennox|away | 08:40 | |
| sharatss | rakhmerov: i have to see.. right now i have no idea :( | 08:48 |
| rakhmerov | np | 08:48 |
| *** ddeja_ is now known as ddeja | 08:51 | |
| *** sharatss has quit IRC | 09:04 | |
| *** sharatss has joined #openstack-mistral | 09:04 | |
| ddeja | rakhmerov: just dropped to the office... let me take a look on those links | 09:06 |
| rakhmerov | ok | 09:06 |
| *** openstackgerrit has joined #openstack-mistral | 09:13 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: change the cron-trigger execution time from localtime to UTC https://review.openstack.org/406740 | 09:13 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Modify the function "_get_spec_version(spec_dict)" https://review.openstack.org/404510 | 09:20 |
| *** kong_ has joined #openstack-mistral | 09:31 | |
| *** dtantsur|afk is now known as dtantsur | 09:34 | |
| d0ugal | so, I am looking at the feedback on https://review.openstack.org/#/c/405253 | 09:47 |
| d0ugal | Adding database tests feels wrong here | 09:48 |
| d0ugal | it basically means adding workbook service tests here | 09:48 |
| d0ugal | and importing the db_api etc. | 09:49 |
| ddeja | d0ugal: I was rather thinking to add such tests to existing workbook service tests | 09:49 |
| d0ugal | I could add another test in the test_workbook_service tests - but that seems a bit strange too | 09:49 |
| d0ugal | hmm | 09:49 |
| ddeja | d0ugal: why? | 09:49 |
| d0ugal | ddeja: I think it will add lots of duplication | 09:51 |
| d0ugal | ddeja: I could rename a workflow in the service tests to include version in the name? | 09:52 |
| ddeja | hm, the longer I think about it... | 09:53 |
| d0ugal | I guess I can try and see | 09:53 |
| ddeja | after all, you are testing the only place when validation is done | 09:53 |
| ddeja | so if it works there... | 09:53 |
| ddeja | OK, +2 +W | 09:54 |
| d0ugal | ddeja: cool, thanks! | 09:55 |
| openstackgerrit | zhangdetong proposed openstack/mistral: Add descriptions for on_task_state_change parameters. https://review.openstack.org/406863 | 10:05 |
| openstackgerrit | Renat Akhmerov proposed openstack/python-mistralclient: WIP: add domain name to openstack client https://review.openstack.org/406866 | 10:10 |
| openstackgerrit | fengchaoyang proposed openstack/mistral: Add email address validate for SendEmailAction https://review.openstack.org/406873 | 10:15 |
| openstackgerrit | zhangdetong proposed openstack/mistral: Add descriptions for on_task_state_change parameters. https://review.openstack.org/406863 | 10:23 |
| ddeja | rakhmerov: you know what happend there, or still someone should look? http://logs.openstack.org/93/405893/2/check/gate-python-mistralclient-devstack-dsvm/0c872b9/console.html | 10:35 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: change the cron-trigger execution time from localtime to UTC https://review.openstack.org/406740 | 10:47 |
| ddeja | oooh, those are mistralclient tests... | 10:51 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Add some transactions in some files https://review.openstack.org/406691 | 10:56 |
| sharatss | ddeja: did u find out the cause of it? | 10:57 |
| ddeja | sharatss: nope | 10:58 |
| ddeja | but I'll try | 10:58 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Add some transactions in some files https://review.openstack.org/406691 | 10:58 |
| sharatss | ddeja: ok ok.. then i wont continue on it | 10:59 |
| ddeja | well, IMO it was due to some networking problems, seems it doesn'a appear after recheck | 11:00 |
| ddeja | I'm droping for lunch, I'll be back on it in an hour | 11:01 |
| *** sharatss has quit IRC | 11:23 | |
| *** sharatss has joined #openstack-mistral | 11:24 | |
| *** sharatss has quit IRC | 11:28 | |
| *** thrash is now known as thrash|g0ne | 11:39 | |
| *** thrash|g0ne is now known as thrash | 11:39 | |
| *** szaher has joined #openstack-mistral | 11:40 | |
| *** sharatss has joined #openstack-mistral | 11:42 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Add some transactions in some files https://review.openstack.org/406691 | 11:56 |
| rakhmerov | ddeja: what issue were you talking about when you said it seemed like a networking issue? | 12:19 |
| rakhmerov | as far as those failing client tests, I thought it was because we don't pass 'domain' to openstack client | 12:20 |
| rakhmerov | but I tried https://review.openstack.org/#/c/406866/ and it didn't help | 12:21 |
| rakhmerov | the error is: Expecting to find domain in project - the server could not comply with the request since it is either malformed or otherwise incorrect. The client is assumed to be in error | 12:21 |
| *** jpich has joined #openstack-mistral | 12:24 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: change the cron-trigger execution time from localtime to UTC https://review.openstack.org/406740 | 12:27 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral: Remove obsolete method from engine client https://review.openstack.org/406918 | 12:36 |
| ddeja | rakhmerov: It was rather my experience - when there was some random failures on the gate, it usually was due to network condition | 12:37 |
| ddeja | but this one happens regulary? | 12:37 |
| rakhmerov | yes | 12:37 |
| rakhmerov | every time | 12:37 |
| ddeja | well, one strange thing | 12:37 |
| rakhmerov | something changed recently either in devstack or in tempest, or in keystone | 12:38 |
| rakhmerov | dunno | 12:38 |
| ddeja | http://logs.openstack.org/93/405893/2/check/gate-python-mistralclient-devstack-dsvm/0c872b9/console.html#_2016-12-05_07_11_22_828072 | 12:38 |
| ddeja | look on this line in trace | 12:38 |
| rakhmerov | yes | 12:38 |
| rakhmerov | what's wrong with it? | 12:38 |
| ddeja | what is called after it is 'local/lib/python2.7/site-packages/tempest/lib/cli/base.py' | 12:39 |
| ddeja | but when I went through the mistralclientcode | 12:39 |
| ddeja | it should called another method inside mistralclient | 12:39 |
| ddeja | not in tempest | 12:39 |
| ddeja | but I don't know why is it happening... | 12:39 |
| ddeja | oh, wait | 12:40 |
| rakhmerov | it's ok | 12:40 |
| rakhmerov | it's self.admin_clients.openstack("project show", ...) | 12:41 |
| rakhmerov | which is in tempest | 12:41 |
| rakhmerov | I'll reboot | 12:42 |
| ddeja | yes, my mistake, I was looking on a wrong line... | 12:42 |
| rakhmerov | np | 12:47 |
| ddeja | just to make everything clear - we are sure that it is not a problem in client code itself | 12:48 |
| *** bobh has joined #openstack-mistral | 12:48 | |
| *** bobh has quit IRC | 12:48 | |
| ddeja | since if it was, the change that started it would fail on checks, right? | 12:48 |
| *** bobh has joined #openstack-mistral | 12:48 | |
| rakhmerov | sure | 12:49 |
| rakhmerov | the problem is that something changed in devstack or keystone I guess | 12:50 |
| rakhmerov | recently | 12:50 |
| ddeja | rakhmerov: ok, one strange thing | 12:53 |
| rakhmerov | yep | 12:54 |
| ddeja | http://logs.openstack.org/93/405893/2/check/gate-python-mistralclient-devstack-dsvm/0c872b9/console.html#_2016-12-05_07_11_22_825288 | 12:54 |
| ddeja | here the parameter is "demo" | 12:54 |
| ddeja | http://logs.openstack.org/93/405893/2/check/gate-python-mistralclient-devstack-dsvm/0c872b9/console.html#_2016-12-05_07_11_22_837373 | 12:54 |
| ddeja | but here it is alt_demo | 12:54 |
| ddeja | I can't see any place where 'alt' is added | 12:55 |
| ddeja | and I don't know why it is added | 12:55 |
| rakhmerov | hm.. yes | 12:55 |
| ddeja | but probably we got return code = 1, becouse the project alt_demo doesn't exist | 12:55 |
| rakhmerov | well, I know that we have two projects, it's ok | 12:55 |
| rakhmerov | but these two values are in the same stack trace | 12:55 |
| rakhmerov | it should exist | 12:56 |
| rakhmerov | in some tests we use two projets | 12:56 |
| rakhmerov | projects | 12:56 |
| rakhmerov | and two clients for them | 12:56 |
| rakhmerov | "Expecting to find domain in project" | 12:56 |
| rakhmerov | what does this mean? | 12:56 |
| rakhmerov | doesn't it mean "we should provide a domain name" | 12:57 |
| ddeja | where you see this? | 12:57 |
| rakhmerov | a couple of lines below your last link | 12:58 |
| rakhmerov | in the end of the stack trace | 12:58 |
| ddeja | yup, found it | 12:58 |
| rakhmerov | but you're right, I don't understand why the project changes down the stack | 13:00 |
| ddeja | rakhmerov: it's an old bug, but it's something https://bugs.launchpad.net/python-openstackclient/+bug/1385338 | 13:01 |
| openstack | Launchpad bug 1385338 in python-openstackclient "Improve domain related defaults when using v3 identity API" [High,Fix released] - Assigned to Steve Martinelli (stevemar) | 13:01 |
| rakhmerov | yeah, I saw it today too | 13:02 |
| rakhmerov | hm.. | 13:02 |
| rakhmerov | ddeja: ooh, it's all really weird... Look at this: https://github.com/openstack/python-mistralclient/blob/master/mistralclient/tests/functional/cli/v2/base_v2.py#L143 | 13:04 |
| rakhmerov | then https://github.com/openstack/python-mistralclient/blob/master/mistralclient/tests/functional/cli/base.py#L98 | 13:05 |
| openstackgerrit | zhangdetong proposed openstack/mistral: Add descriptions for on_task_state_change parameters. https://review.openstack.org/406863 | 13:05 |
| rakhmerov | and then https://github.com/openstack/python-mistralclient/blob/master/mistralclient/tests/functional/cli/base.py#L28 | 13:05 |
| rakhmerov | initially it's called "project" | 13:05 |
| rakhmerov | but in the function "credentials" it's called "group" | 13:06 |
| rakhmerov | do you understand why? | 13:06 |
| rakhmerov | and then on the line 38 | 13:06 |
| rakhmerov | if group == 'admin': | 13:06 |
| ddeja | I don't understand why we do this, but at least I know why demo changes to alt_demo | 13:06 |
| rakhmerov | this is false which means that we initialize 'alt' project | 13:07 |
| rakhmerov | yes | 13:07 |
| rakhmerov | I know see this too | 13:07 |
| rakhmerov | but it doesn't make sense to me at all | 13:07 |
| rakhmerov | well, it's kind of "it's not an admin project we always use the same alternative project" | 13:09 |
| rakhmerov | but why give it a different name in the first place | 13:09 |
| rakhmerov | I don't know | 13:09 |
| ddeja | rakhmerov: OK, one thing | 13:09 |
| rakhmerov | ok, anyway, maybe the issue is that this project 'alt_demo' doesn't exist? | 13:09 |
| ddeja | one good thing | 13:09 |
| rakhmerov | in devstack | 13:09 |
| rakhmerov | ok | 13:09 |
| ddeja | I'm able to reproduce it on my devstack | 13:10 |
| rakhmerov | ok | 13:10 |
| *** kong_ has quit IRC | 13:10 | |
| ddeja | and alt_demo exists on my env, but still I got excalcy same error | 13:10 |
| rakhmerov | hm... | 13:10 |
| *** sharatss has quit IRC | 13:11 | |
| *** sharatss has joined #openstack-mistral | 13:12 | |
| rakhmerov | sharatss: I abandoned my patch removing on_task_state_change() | 13:14 |
| rakhmerov | please unblock the other one | 13:14 |
| sharatss | rakhmerov: i have removed my review from the other :) | 13:14 |
| rakhmerov | ok | 13:14 |
| rakhmerov | I need to reboot again | 13:15 |
| ddeja | rakhmerov: ok, one thing: this works "openstack --os-username admin --os-tenant-name admin --os-password secret --os-auth-url http://192.168.56.2:5000/v2.0 project show alt_demo" but changing the "2.0" to "3" brokes it | 13:16 |
| *** rbrady has joined #openstack-mistral | 13:24 | |
| rakhmerov | ddeja: | 13:29 |
| rakhmerov | ok | 13:29 |
| rakhmerov | so we need to find out how to run it for V3 correctly? | 13:29 |
| rakhmerov | would adding "--domain Default" help? | 13:29 |
| openstackgerrit | Merged openstack/mistral: Updated from global requirements https://review.openstack.org/406236 | 13:30 |
| ddeja | rakhmerov: nope | 13:30 |
| ddeja | but yes, we need to learn how to run v3 properly | 13:30 |
| ddeja | I found this http://docs.openstack.org/developer/python-openstackclient/authentication.html#authenticating-using-identity-server-api-v3 | 13:31 |
| *** janki has quit IRC | 13:31 | |
| rakhmerov | ok | 13:31 |
| rakhmerov | d0ugal, ddeja: guys do you have any serious updates for the meeting? | 13:31 |
| rakhmerov | I'd like to skip it actually | 13:32 |
| d0ugal | rakhmerov: I don't have much for it. | 13:32 |
| d0ugal | rbrady: did you have anything re:cusutom actions? | 13:32 |
| rakhmerov | we can cancel it completely or you can have it w/o me | 13:32 |
| d0ugal | I'll see what ddeja thinks :) | 13:33 |
| ddeja | I wanted to talk a little bit about the time problems, that I've fixed last week | 13:33 |
| rakhmerov | ddeja: yeah, we need to see how to configure for v3 | 13:33 |
| ddeja | but we can do it tommorow on the channel | 13:33 |
| rbrady | I don't have anything I need to discuss about custom actions today | 13:33 |
| rakhmerov | ok, then let's cancel it and we'll talk with ddeja in our channel tomorrow | 13:34 |
| d0ugal | Sure | 13:34 |
| rakhmerov | ddeja: Dawid, what surprizing for is that it changed to v3 unexpectedly | 13:34 |
| rakhmerov | for me.. | 13:34 |
| rakhmerov | I didn't see any updates on it | 13:35 |
| rakhmerov | anything | 13:35 |
| ddeja | rakhmerov: got it working | 13:35 |
| rakhmerov | really? | 13:35 |
| *** clenimar has joined #openstack-mistral | 13:35 | |
| ddeja | openstack --os-username admin --os-project-name admin --os-default-domain default --os-password secret --os-auth-url http://192.168.56.2:5000/v3 --os-identity-api-version 3 project show alt_demo | 13:35 |
| ddeja | this works | 13:35 |
| rakhmerov | ooh, it's --os-default-domain default | 13:35 |
| rakhmerov | I see | 13:35 |
| ddeja | not only | 13:35 |
| rakhmerov | ok | 13:36 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Add some transactions in some files https://review.openstack.org/406691 | 13:36 |
| ddeja | also, we ned to add --os-identity-api-version 3 | 13:36 |
| rakhmerov | ddeja: I have to run now, will you have time to send a patch today? | 13:36 |
| ddeja | and we need to change the --os-tenant-name to --os-project-name | 13:36 |
| rakhmerov | or I can do it tomorrow | 13:36 |
| ddeja | I'll try | 13:36 |
| rakhmerov | ok | 13:37 |
| *** toure is now known as toure|biab | 13:37 | |
| *** toure|biab is now known as toure | 13:37 | |
| *** catintheroof has joined #openstack-mistral | 13:38 | |
| sharatss | rakhmerov: i dont know if its too late but can we change the commit msg of https://review.openstack.org/#/c/406863/3 | 13:47 |
| *** sharatss has quit IRC | 13:55 | |
| *** catinthe_ has joined #openstack-mistral | 14:02 | |
| *** catintheroof has quit IRC | 14:04 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: change the cron-trigger execution time from localtime to UTC https://review.openstack.org/406740 | 14:09 |
| *** bobh has quit IRC | 14:35 | |
| openstackgerrit | Dawid Deja proposed openstack/python-mistralclient: Fix for failing dsvm gate https://review.openstack.org/406994 | 14:35 |
| *** bobh has joined #openstack-mistral | 14:42 | |
| *** bobh has quit IRC | 14:52 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral: [WIP] Handle unserializable objects returned by custom actions https://review.openstack.org/407005 | 14:54 |
| *** bobh has joined #openstack-mistral | 15:00 | |
| *** bobh has quit IRC | 15:36 | |
| *** Ravikiran_K has joined #openstack-mistral | 15:46 | |
| openstackgerrit | Merged openstack/mistral: Add descriptions for on_task_state_change parameters. https://review.openstack.org/406863 | 15:58 |
| *** sharatss has joined #openstack-mistral | 16:01 | |
| sharatss | rakhmerov: meeting? | 16:05 |
| *** catintheroof has joined #openstack-mistral | 16:13 | |
| *** jpich has quit IRC | 16:14 | |
| *** catinthe_ has quit IRC | 16:16 | |
| openstackgerrit | Dawid Deja proposed openstack/python-mistralclient: Fix for failing dsvm gate https://review.openstack.org/406994 | 16:22 |
| *** bobh has joined #openstack-mistral | 16:22 | |
| openstackgerrit | Merged openstack/mistral: Allow "version" to be within workflow names in workbooks https://review.openstack.org/405253 | 16:27 |
| *** dtantsur is now known as dtantsur|brb | 16:29 | |
| *** catinthe_ has joined #openstack-mistral | 17:02 | |
| *** catintheroof has quit IRC | 17:04 | |
| *** d0ugal has quit IRC | 17:07 | |
| *** dtantsur|brb is now known as dtantsur | 17:12 | |
| *** bobh has quit IRC | 17:23 | |
| *** Ravikiran_K has quit IRC | 17:43 | |
| *** harlowja has joined #openstack-mistral | 17:50 | |
| *** dtantsur is now known as dtantsur|afk | 18:02 | |
| *** bobh has joined #openstack-mistral | 18:18 | |
| *** sharatss has quit IRC | 18:27 | |
| *** d0ugal has joined #openstack-mistral | 18:54 | |
| *** ^Gal^_ has joined #openstack-mistral | 19:25 | |
| *** ^Gal^ has quit IRC | 19:25 | |
| *** ^Gal^_ is now known as ^Gal^ | 19:25 | |
| *** weshay_ has joined #openstack-mistral | 19:27 | |
| *** clenimar has quit IRC | 19:28 | |
| *** weshay has quit IRC | 19:28 | |
| *** clenimar has joined #openstack-mistral | 19:34 | |
| *** bobh has quit IRC | 19:50 | |
| *** kong_ has joined #openstack-mistral | 20:20 | |
| *** d0ugal has quit IRC | 20:32 | |
| *** catinthe_ has quit IRC | 20:53 | |
| *** Ephur has joined #openstack-mistral | 20:59 | |
| *** jamielennox|away is now known as jamielennox | 21:13 | |
| *** catintheroof has joined #openstack-mistral | 21:39 | |
| *** bobh has joined #openstack-mistral | 21:52 | |
| *** bobh has quit IRC | 22:16 | |
| *** bobh has joined #openstack-mistral | 22:34 | |
| *** enykeev has quit IRC | 23:51 | |
| *** enykeev has joined #openstack-mistral | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!