| *** bobh has quit IRC | 00:02 | |
| *** mfisch has joined #openstack-mistral | 00:16 | |
| mfisch | hi, can anyone tell me the right way to generate config for stable/newton? I see the tox_install.sh but not how to use it properly | 00:17 |
|---|---|---|
| *** catintheroof has quit IRC | 00:25 | |
| openstackgerrit | Lingxian Kong proposed openstack/mistral: Support openstack context for workflow and task https://review.openstack.org/407240 | 00:26 |
| kong_ | mfisch: try to use 'tox -e genconfig' | 00:27 |
| *** mfisch has quit IRC | 00:28 | |
| *** thrash is now known as thrash|g0ne | 01:15 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: change the cron-trigger execution time from localtime to UTC https://review.openstack.org/406740 | 02:06 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Modify the function "_get_spec_version(spec_dict)" https://review.openstack.org/404510 | 02:12 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Add some transactions in some files https://review.openstack.org/406691 | 02:43 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Change version '1.0' to '2.0' https://review.openstack.org/405292 | 02:44 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: std.email action requires a smtp_password https://review.openstack.org/406694 | 02:44 |
| *** Ephur has quit IRC | 02:44 | |
| rakhmerov | kong_: hi | 02:50 |
| kong_ | rakhmerov: hi | 02:50 |
| rakhmerov | are you back? | 02:50 |
| rakhmerov | :) | 02:50 |
| kong_ | rakhmerov: yeah, back to normal | 02:51 |
| kong_ | rakhmerov: how's it going? | 02:51 |
| rakhmerov | ok, we missed you :) | 02:51 |
| kong_ | rakhmerov: miss you too :-) | 02:51 |
| rakhmerov | kong_: overall good, we have a lot of work ) | 02:51 |
| kong_ | that need to be finished before O release? | 02:51 |
| rakhmerov | kong_: is there anything specific that you'd like to work on? | 02:51 |
| kong_ | considering we have a very short cycle | 02:51 |
| kong_ | :-) | 02:51 |
| rakhmerov | yes, the cycle is short this time | 02:52 |
| kong_ | I just uploaded a patch for supporting region name in workflow | 02:52 |
| kong_ | working on that | 02:52 |
| kong_ | seems i need to fix the jenkins failure first | 02:52 |
| rakhmerov | ooh, I see | 02:53 |
| kong_ | other than that, i have no specific requirement for mistral | 02:53 |
| rakhmerov | kong_: in fact, there's a BP for doing this region related work | 02:53 |
| rakhmerov | sec | 02:53 |
| kong_ | ooh, i didn't know i am doing duplicate work | 02:54 |
| rakhmerov | https://blueprints.launchpad.net/mistral/+spec/mistral-multi-region-support | 02:54 |
| rakhmerov | no, you are not | 02:54 |
| rakhmerov | I believe nobody is working on it now | 02:54 |
| kong_ | rakhmerov: ok, cool | 02:54 |
| kong_ | i will link that bp | 02:54 |
| rakhmerov | it's assigned to Istvan Imre (my colleage) but I believe he's not actively working on it | 02:54 |
| rakhmerov | although he did some related work | 02:54 |
| rakhmerov | his IRC nick is ist | 02:55 |
| rakhmerov | we can talk to him once he wakes up (he's in Budapest) | 02:55 |
| kong_ | rakhmerov: ok, sure. I think my patch will cover this feature | 02:55 |
| rakhmerov | yep | 02:55 |
| kong_ | that's a requirement i am facing | 02:55 |
| rakhmerov | which is great because it's been wanted for a long time | 02:56 |
| rakhmerov | yes, it's needed by many people | 02:56 |
| kong_ | yeah | 02:56 |
| rakhmerov | good | 02:56 |
| kong_ | rakhmerov: do we have changed the meeting time? | 02:56 |
| rakhmerov | so, you're ready to continue being a release liaison? | 02:56 |
| kong_ | rakhmerov: of course, if you think i can continue do that :-) | 02:56 |
| kong_ | you are the boss, man | 02:57 |
| rakhmerov | yes, you can ) | 02:57 |
| rakhmerov | yes, I need you | 02:57 |
| kong_ | honoured | 02:57 |
| rakhmerov | we did Ocata-1 already and 2.1.2 for the clienet | 02:57 |
| rakhmerov | client | 02:57 |
| rakhmerov | d0ugal needed something in the client so we decided to release it | 02:58 |
| kong_ | i will check the release schedule, and prepare to do the next one | 02:58 |
| rakhmerov | yep | 02:58 |
| rakhmerov | just FYI, I made a huge patch recently that changed how we launch our components (engine, even_engine, executor) | 02:59 |
| rakhmerov | so we might face some issues with that | 02:59 |
| kong_ | rakhmerov: saw that, didn't have a chance to read the code | 02:59 |
| rakhmerov | if we do, I'm ready to react quickly | 02:59 |
| rakhmerov | yeah, that's no problem | 02:59 |
| rakhmerov | I was just fixing the bug related to usage of oslo_service and realized that the whole machinery demands serious refactoring | 03:00 |
| rakhmerov | so did I | 03:00 |
| rakhmerov | kong_: do I need to reassign this multi-region BP to you? | 03:00 |
| kong_ | rakhmerov: yeah, maybe there was some misunderstanding of using oslo_service | 03:00 |
| kong_ | rakhmerov: sure, feel free to do that | 03:01 |
| rakhmerov | done | 03:01 |
| rakhmerov | kong_: we also have a problem with the client now | 03:02 |
| kong_ | what's that | 03:02 |
| rakhmerov | trying to fix it but it's not done yet | 03:03 |
| rakhmerov | https://review.openstack.org/#/c/406994/2 | 03:03 |
| rakhmerov | look at the failing tests | 03:03 |
| rakhmerov | seems like we have to use keystone v3 now | 03:03 |
| rakhmerov | and it's missing domain name | 03:03 |
| rakhmerov | if you know how to fix it please feel free | 03:03 |
| rakhmerov | client development is now blocked | 03:04 |
| kong_ | rakhmerov: ok, will dig it a little bit | 03:04 |
| kong_ | rakhmerov: so, is Dawid still working on that? | 03:04 |
| rakhmerov | yeah, ddeja investigated it yesterday a little bit but the sooner we fix it the better | 03:05 |
| kong_ | ok | 03:06 |
| kong_ | my top priority on mistral now | 03:06 |
| rakhmerov | :) | 03:06 |
| rakhmerov | other than, feel free to take any bugs or talk to me if you want some exciting task | 03:07 |
| kong_ | rakhmerov: sure, will do if i have more bandwidth | 03:08 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: cancel comments in 'setup.cfg' https://review.openstack.org/407290 | 03:12 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: change the cron-trigger execution time from localtime to UTC https://review.openstack.org/406740 | 03:24 |
| openstackgerrit | Lingxian Kong proposed openstack/python-mistralclient: Fix for failing dsvm gate https://review.openstack.org/406994 | 03:50 |
| rakhmerov | kong_: your last patchset passed | 04:24 |
| rakhmerov | :) | 04:24 |
| rakhmerov | do you think we can merge it in this form? | 04:24 |
| rakhmerov | kong_: so is it because tempest switched to using V3 by default? | 04:25 |
| *** sharatss has joined #openstack-mistral | 05:26 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral: Handling MistralException in default executor https://review.openstack.org/407330 | 05:36 |
| sharatss | rakhmerov: hi | 05:42 |
| *** janki has joined #openstack-mistral | 05:46 | |
| openstackgerrit | zhangdetong proposed openstack/mistral: Fix a syntax error in yaml example https://review.openstack.org/407334 | 06:11 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: Fix a syntax error in yaml example https://review.openstack.org/407334 | 06:23 |
| rakhmerov | sharatss: hi | 06:31 |
| sharatss | rakhmerov: yesterday the meeting was cancelled or did i miss it? | 06:33 |
| rakhmerov | it was cancelled, I sent out an email | 06:34 |
| sharatss | rakhmerov: ok. maybe i am not in that list :( | 06:34 |
| rakhmerov | openstack-dev | 06:34 |
| rakhmerov | with tag [mistral] in subject | 06:34 |
| rakhmerov | please make a subscription | 06:34 |
| sharatss | rakhmerov: ok ok.. i will subscribe now | 06:35 |
| openstackgerrit | zhangdetong proposed openstack/mistral: Fix a syntax error in yaml example https://review.openstack.org/407334 | 06:36 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral: Handling MistralException in default executor https://review.openstack.org/407330 | 06:37 |
| *** Kiall has quit IRC | 06:56 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral: Handling MistralException in default executor https://review.openstack.org/407330 | 07:22 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral: Handling MistralException in default executor https://review.openstack.org/407330 | 07:43 |
| ddeja | oh, thanks kong_ for fixing my patch | 07:48 |
| * ddeja fells stupid now for previous attepmts... | 07:49 | |
| ddeja | I have a meeting starting in 10 minutes that would last for 2 hours | 07:50 |
| ddeja | rakhmerov: would you be availabl after that time? | 07:50 |
| rakhmerov | ddeja: yes | 07:50 |
| ddeja | rakhmerov: great! | 07:50 |
| *** d0ugal has joined #openstack-mistral | 07:57 | |
| *** d0ugal has joined #openstack-mistral | 07:57 | |
| *** FL1SK has quit IRC | 08:04 | |
| *** sharatss has quit IRC | 08:06 | |
| *** sharatss has joined #openstack-mistral | 08:06 | |
| rakhmerov | ddeja: so, do you think this is a complete fix for failing devstack gate? | 08:18 |
| rakhmerov | if yes, I'll merge it | 08:18 |
| ddeja | rakhmerov: yes, I think so | 08:30 |
| rakhmerov | ok | 08:30 |
| ddeja | due to the fact that tempest is not able to use keystone v3... | 08:31 |
| openstackgerrit | fengchaoyang proposed openstack/mistral: Fix two errors in YAML example and a error in action doc https://review.openstack.org/404229 | 08:36 |
| *** Jizhaoxuan has joined #openstack-mistral | 08:43 | |
| *** no-thing_ has joined #openstack-mistral | 08:58 | |
| openstackgerrit | Merged openstack/python-mistralclient: Fix for failing dsvm gate https://review.openstack.org/406994 | 09:00 |
| *** shardy has joined #openstack-mistral | 09:11 | |
| openstackgerrit | Renat Akhmerov proposed openstack/python-mistralclient: Updated from global requirements https://review.openstack.org/405893 | 09:24 |
| rakhmerov | ddeja: yes, but why did it work before? | 09:27 |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: change the cron-trigger execution time from localtime to UTC https://review.openstack.org/406740 | 09:28 |
| rakhmerov | does anyone know why patches in python-mistralclient don't get into 'gate' queue ? Like https://review.openstack.org/#/c/405893/ | 09:39 |
| openstackgerrit | Renat Akhmerov proposed openstack/python-mistralclient: Filter workflow executions by creating task execution id https://review.openstack.org/403844 | 09:41 |
| *** shardy is now known as shardy_mtg | 09:45 | |
| *** jpich has joined #openstack-mistral | 09:50 | |
| openstackgerrit | Jizhaoxuan proposed openstack/mistral: modify something in 'dsl_v2.rst' https://review.openstack.org/407430 | 09:52 |
| *** FL1SK has joined #openstack-mistral | 10:05 | |
| kong_ | ddeja: you are welcome, i was just trying my luck ;-） | 10:07 |
| ddeja | rakhmerov: it looks like there was some change in devstack so it switched the default url to v3 | 10:11 |
| ddeja | i mean, the default value of OS_AUTH_URL was changed | 10:12 |
| ddeja | kong_: yup. And you did it in simple and elegant way. I was having some thinking problem when writing my patch... It was like code in C rather than in python... | 10:12 |
| rakhmerov | :) | 10:21 |
| rakhmerov | ok | 10:21 |
| rakhmerov | thanks to you both ) | 10:21 |
| openstackgerrit | Merged openstack/python-mistralclient: Updated from global requirements https://review.openstack.org/405893 | 10:39 |
| rakhmerov | d0ugal: btw, look at https://review.openstack.org/#/c/407330/ once you have a chance | 10:39 |
| rakhmerov | this fixes the problem you recently reported | 10:39 |
| rakhmerov | with invalid unicode | 10:39 |
| d0ugal | rakhmerov: I spotted it, and started looking :) | 10:39 |
| d0ugal | Then I got distracted by docs ;) | 10:39 |
| d0ugal | I'll get back to it soon. | 10:39 |
| rakhmerov | np | 10:40 |
| rakhmerov | it might have an overlap with your https://review.openstack.org/#/c/407005/1/mistral/tests/unit/engine/test_error_result.py | 10:40 |
| rakhmerov | https://review.openstack.org/#/c/407005/ | 10:40 |
| rakhmerov | but maybe it's ok to have it too | 10:40 |
| rakhmerov | I believe it can become mostly additional testing for that fix | 10:40 |
| rakhmerov | not only when unicode is broken but some other cases | 10:41 |
| openstackgerrit | Merged openstack/mistral: Fix two errors in YAML example and a error in action doc https://review.openstack.org/404229 | 10:43 |
| openstackgerrit | Renat Akhmerov proposed openstack/python-mistralclient: Make python mistralclient readme better https://review.openstack.org/404249 | 10:43 |
| *** Ephur has joined #openstack-mistral | 10:47 | |
| openstackgerrit | Gal Margalit proposed openstack/mistral-dashboard: mistral-dashboard: added action executions screens https://review.openstack.org/401188 | 10:49 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral: Update the wording in the actions terminology docs https://review.openstack.org/407454 | 10:51 |
| *** Ephur has quit IRC | 10:52 | |
| *** dtantsur|afk is now known as dtantsur | 11:01 | |
| ddeja | hey, rakhmerov, do you have a moment to talk about https://review.openstack.org/#/c/406143/ | 11:10 |
| rakhmerov | ddeja: yes, let's do | 11:12 |
| ddeja | ok, so let me describe what the problem is | 11:12 |
| rakhmerov | ok | 11:13 |
| ddeja | when we create any table using oslo db, it automatically adds column 'created_at' | 11:13 |
| ddeja | it has a default value which is set using some internal oslo.utils functions, but basically we can think of it as a datetime.datetime object | 11:14 |
| ddeja | what is important, the value is set from the python code, not in the db | 11:14 |
| ddeja | ok, so let's take a table action_definition_v2, as an example | 11:15 |
| rakhmerov | ok | 11:15 |
| ddeja | when we add new raw to it | 11:15 |
| ddeja | the created_at value is set from the python interpreter as a datetime object | 11:16 |
| ddeja | and then it is passed to the DB | 11:16 |
| ddeja | this tame consist of time up to microseconds | 11:16 |
| ddeja | s/tame/time/ | 11:16 |
| rakhmerov | yep | 11:16 |
| ddeja | and, if it sqlite/postgres/mysql in version 5.6+ there is no problem | 11:17 |
| ddeja | (ok, there is still some problems even with mysql 5.6+, but it can be handle) | 11:17 |
| rakhmerov | yeah, but if it's mysql < 5.6 then it rounds the value | 11:17 |
| ddeja | to be honest, it does same in 5.6+, since you need to provide special flag in SQLalchemy so it stores the microseconds rather than round it | 11:18 |
| rakhmerov | ok | 11:18 |
| ddeja | so the problem is: since the time is set from python code, not from the DB | 11:18 |
| rakhmerov | so, what's the solution you are proposing? | 11:19 |
| ddeja | if mysql is used, then user gets different time than is stored in the db | 11:19 |
| ddeja | for now, I see two options | 11:19 |
| ddeja | 1) change the default for 'created_up', 'updated_at' so it only uses seconds | 11:19 |
| ddeja | (this is current patch) | 11:19 |
| ddeja | 2) change the default, but only if mysql is used | 11:20 |
| rakhmerov | pl | 11:20 |
| rakhmerov | ok | 11:20 |
| ddeja | to be honest, I don't know which solution is better. Or there may be 3rd one, that is better than those 2 | 11:20 |
| rakhmerov | let me think.. | 11:20 |
| rakhmerov | so utc.now() give milliseconds, right? | 11:21 |
| rakhmerov | or seconds? | 11:21 |
| ddeja | to give you some more info - I've dug into nova code | 11:21 |
| ddeja | rakhmerov: microseconds even | 11:21 |
| ddeja | so nova drops the microseconds for instances | 11:22 |
| rakhmerov | yeah, I see | 11:22 |
| rakhmerov | that's what I thought too | 11:22 |
| rakhmerov | maybe we simply should do the same | 11:22 |
| ddeja | OK | 11:22 |
| rakhmerov | second.. | 11:22 |
| rakhmerov | if utils.utcnow() give microseconds then why do you use it for default value? | 11:23 |
| ddeja | it's mistral utils, not oslo.utils :) | 11:23 |
| ddeja | maybe I should rename it, it can be confusing | 11:23 |
| rakhmerov | ooh, and it gives milliseconds? | 11:23 |
| rakhmerov | or seconds? | 11:23 |
| rakhmerov | :) | 11:23 |
| ddeja | it's my implementation, it gives only seconds | 11:24 |
| rakhmerov | ooh, I see the file | 11:24 |
| rakhmerov | yes | 11:24 |
| ddeja | what is most suprising for me, is why it is only happening on kombu gate.. | 11:25 |
| ddeja | since the tests that fails is not using even a single RPC call | 11:25 |
| rakhmerov | maybe it's configured with a different DB? | 11:25 |
| rakhmerov | mysql of different version, no? | 11:25 |
| ddeja | hm, maybe... maybe one uses postgres and one mysql.. | 11:26 |
| rakhmerov | ok | 11:26 |
| rakhmerov | but the problem still exists anyway | 11:26 |
| rakhmerov | we know it | 11:26 |
| ddeja | yes, it is there | 11:26 |
| ddeja | this change made some unit tests to fail - those which where relaing on order of return from DB | 11:27 |
| rakhmerov | so why did this patch fail now? | 11:27 |
| rakhmerov | tests | 11:27 |
| ddeja | I'm fixing it | 11:27 |
| ddeja | rakhmerov: ^ | 11:27 |
| rakhmerov | oh, I see | 11:27 |
| rakhmerov | ok | 11:27 |
| ddeja | I've run it multiple times on my env, but If there are two elements for which order matters, it may pass 10-15 times before we hit the error | 11:28 |
| ddeja | I'll recheck my patchset a few more times before we can merge it | 11:28 |
| rakhmerov | so on one hand it may be useful to retain microseconds | 11:28 |
| rakhmerov | in cases when it's possible | 11:29 |
| rakhmerov | (not mysql < 5.6) | 11:29 |
| ddeja | well, I'm not sure really | 11:29 |
| rakhmerov | on the other hand I really don't like any kind of special handling | 11:29 |
| ddeja | despite tests, why it should matter for anyone? | 11:29 |
| rakhmerov | just a precision | 11:29 |
| ddeja | OK | 11:29 |
| rakhmerov | if, for example, I run Mistral under load I may be interested in seeing which action ran first and which second | 11:30 |
| rakhmerov | although time precision doesn't really solve this reliably too | 11:30 |
| *** shardy_mtg has quit IRC | 11:30 | |
| rakhmerov | there always can be time collisions | 11:30 |
| ddeja | to be honest, it doesnt give any information | 11:31 |
| rakhmerov | if we simply sort by time | 11:31 |
| rakhmerov | yeah, maybe... | 11:31 |
| ddeja | even if we keep the microseconds in the DB | 11:31 |
| rakhmerov | hm | 11:31 |
| ddeja | this times was created when the object was firstly initialize | 11:31 |
| ddeja | it doesnt say which runs first | 11:31 |
| rakhmerov | true | 11:31 |
| rakhmerov | yes, good point | 11:32 |
| ddeja | well, it even doesn't say which was really first in the DB | 11:32 |
| rakhmerov | hm... yes | 11:32 |
| rakhmerov | yeah, I agree. Ordering should not rely on time | 11:32 |
| ddeja | in a very rare circumstances, we created A, then create B, then store B then store A | 11:32 |
| rakhmerov | it's rather a logical thing related to a particular workflow | 11:32 |
| ddeja | yes | 11:33 |
| rakhmerov | ok, then I'd suggest we round it to seconds | 11:33 |
| ddeja | OK | 11:33 |
| ddeja | so I'm getting back to fixing unit tests | 11:33 |
| rakhmerov | so does mysql < 5.6 support milliseconds at least? | 11:33 |
| rakhmerov | if yes we could round to milliseconds | 11:34 |
| ddeja | no | 11:34 |
| rakhmerov | :) | 11:34 |
| rakhmerov | ok | 11:34 |
| rakhmerov | let's do seconds | 11:34 |
| ddeja | well, once 14.04 is dropped | 11:34 |
| ddeja | (ubuntu 14.04) | 11:34 |
| ddeja | for ubuntu 16.04 default mysql is 5.7 | 11:34 |
| rakhmerov | and get as many people involved in reviewing it as possible | 11:34 |
| ddeja | but the problem is that the default precision for columns of type 'datetime' is still only to seconds | 11:35 |
| rakhmerov | I think it's not a big problem to use only seconds | 11:35 |
| ddeja | well, If you have mysql, you already only have seconds | 11:35 |
| ddeja | only when creating objects you got more precision, but next time you ask for the same object, those mircoseconds are gone | 11:36 |
| rakhmerov | I understand, I just mean internally for those two fields | 11:36 |
| ddeja | ok | 11:36 |
| rakhmerov | to make our testing consistent | 11:36 |
| ddeja | OK | 11:36 |
| ddeja | oh, BTW Renat | 11:36 |
| rakhmerov | when I work with Mistral myself I don't have any problems with time, it's fine. We just need to deal with tests | 11:37 |
| rakhmerov | yes, what's up? | 11:37 |
| ddeja | Next week I'll be giving a speech called 'Introduction to mistral' | 11:37 |
| rakhmerov | really? :) | 11:37 |
| rakhmerov | where? | 11:37 |
| ddeja | In Wroclaw, Poland | 11:37 |
| rakhmerov | ok :) | 11:37 |
| rakhmerov | some internal Intel event? | 11:37 |
| rakhmerov | or a public event? | 11:38 |
| ddeja | https://www.meetup.com/Wroclaw-OpenStack-Meetup/events/235620920/ | 11:38 |
| ddeja | (scroll for english version :) | 11:38 |
| rakhmerov | aah, OpenStack meetup | 11:38 |
| rakhmerov | cool | 11:38 |
| rakhmerov | good luck! | 11:38 |
| ddeja | and would it be possible that you can send me slides that you used in barcelona | 11:38 |
| ddeja | I don't want to use it | 11:38 |
| ddeja | but there was some info I'd like to put into my slides | 11:39 |
| ddeja | and it would be easier to have a slidedeck instead of youtube video ;) | 11:39 |
| rakhmerov | sure, np | 11:39 |
| ddeja | especially the part with 'who uses mistral' | 11:39 |
| rakhmerov | I'll send em shortly | 11:39 |
| ddeja | cool, thanks :) | 11:39 |
| rakhmerov | what format would you prefer? | 11:40 |
| rakhmerov | keynote, pdf, ppt? | 11:40 |
| ddeja | 11:40 | |
| rakhmerov | ok | 11:40 |
| ddeja | thanks a lot! | 11:41 |
| rakhmerov | ooh, it's in Google Docs | 11:42 |
| rakhmerov | just a sec.. | 11:42 |
| rakhmerov | I think I can just give you access to it | 11:42 |
| ddeja | that's also OK :) | 11:42 |
| rakhmerov | ok, check your email | 11:43 |
| *** jamielennox is now known as jamielennox|away | 11:44 | |
| rakhmerov | ddeja: did you get the links? | 11:44 |
| ddeja | rakhmerov: yup, I got them, thanks! | 11:45 |
| rakhmerov | np, ok | 11:45 |
| *** thrash|g0ne is now known as thrash | 12:19 | |
| rakhmerov | ddeja, kong_, d0ugal, hparekh: guys, if somebody has time today, could you please look at https://review.openstack.org/#/c/404249/ ? | 12:30 |
| rakhmerov | I don't understand why it keeps failing | 12:30 |
| rakhmerov | ddeja: btw, it's something about time too | 12:30 |
| ddeja | rakhmerov: let me look | 12:38 |
| rakhmerov | ok | 12:38 |
| ddeja | rakhmerov: this looks like excactly the problem I have described | 12:40 |
| rakhmerov | :) | 12:40 |
| ddeja | but I need to look on code to be sure | 12:40 |
| ddeja | rakhmerov: that's the same problem | 12:42 |
| ddeja | look https://github.com/openstack/python-mistralclient/blob/master/mistralclient/tests/functional/cli/v2/cli_tests_v2.py#L776-L777 | 12:42 |
| ddeja | it even cutting of what is after the '.' | 12:43 |
| ddeja | I'm wondering why it came out recently... maybe some older mysql version was cutting the microseconds, not rounding | 12:44 |
| ddeja | rakhmerov: I can fix tests in client, or we can wait for the patch in mistral | 12:45 |
| d0ugal | ddeja: This patch? https://review.openstack.org/#/c/406143/ | 12:54 |
| ddeja | d0ugal: yup, this one | 12:56 |
| ddeja | (fix on the way) | 12:56 |
| d0ugal | k | 12:57 |
| *** Jizhaoxuan has quit IRC | 13:04 | |
| openstackgerrit | Dawid Deja proposed openstack/mistral: Fix for failing kombu gate https://review.openstack.org/406143 | 13:09 |
| openstackgerrit | Dawid Deja proposed openstack/mistral: Fix for failing kombu gate https://review.openstack.org/406143 | 13:12 |
| *** shardy has joined #openstack-mistral | 13:22 | |
| openstackgerrit | Merged openstack/mistral-dashboard: mistral-dashboard: added action executions screens https://review.openstack.org/401188 | 13:28 |
| *** catintheroof has joined #openstack-mistral | 13:43 | |
| *** hrybacki|l4mG3 is now known as hrybacki | 13:53 | |
| *** mfisch has joined #openstack-mistral | 14:17 | |
| *** mfisch has left #openstack-mistral | 14:17 | |
| *** dprince has joined #openstack-mistral | 14:21 | |
| *** soumiyajit has joined #openstack-mistral | 14:27 | |
| *** soumiyajit has quit IRC | 14:29 | |
| *** Ephur has joined #openstack-mistral | 14:49 | |
| *** Ephur has quit IRC | 14:53 | |
| openstackgerrit | Gal Margalit proposed openstack/mistral-dashboard: mistral-dashboard: removed the false check, false is a valid value - and has a matching design https://review.openstack.org/407573 | 15:50 |
| *** bobh has joined #openstack-mistral | 15:57 | |
| *** lkannan has quit IRC | 15:59 | |
| *** lkannan has joined #openstack-mistral | 16:00 | |
| openstackgerrit | Gal Margalit proposed openstack/mistral-dashboard: mistral-dashboard:fixed boolean field design bug https://review.openstack.org/407573 | 16:08 |
| *** dprince has quit IRC | 16:11 | |
| *** jpich has quit IRC | 16:13 | |
| *** shardy has quit IRC | 16:14 | |
| *** catinthe_ has joined #openstack-mistral | 16:14 | |
| *** catintheroof has quit IRC | 16:16 | |
| *** catinthe_ has quit IRC | 16:18 | |
| *** no-thing_ has quit IRC | 16:39 | |
| *** Ephur has joined #openstack-mistral | 17:03 | |
| *** catintheroof has joined #openstack-mistral | 17:03 | |
| *** catintheroof has quit IRC | 17:03 | |
| *** catintheroof has joined #openstack-mistral | 17:04 | |
| *** janki has quit IRC | 17:04 | |
| *** dtantsur is now known as dtantsur|afk | 17:06 | |
| *** bobh has quit IRC | 17:56 | |
| *** weshay_ has quit IRC | 17:58 | |
| *** weshay has joined #openstack-mistral | 17:59 | |
| *** Kiall has joined #openstack-mistral | 18:41 | |
| *** weshay has quit IRC | 18:55 | |
| *** bobh has joined #openstack-mistral | 18:57 | |
| *** weshay has joined #openstack-mistral | 18:58 | |
| *** bobh has quit IRC | 19:01 | |
| *** jamielennox|away is now known as jamielennox | 19:07 | |
| *** catinthe_ has joined #openstack-mistral | 19:14 | |
| *** catintheroof has quit IRC | 19:16 | |
| *** AlexeyAbashkin has joined #openstack-mistral | 19:22 | |
| *** AlexeyAbashkin has quit IRC | 19:23 | |
| *** thrash is now known as thrash|biab | 19:27 | |
| *** thrash|biab is now known as thrash | 19:54 | |
| *** bobh has joined #openstack-mistral | 19:58 | |
| *** catintheroof has joined #openstack-mistral | 20:02 | |
| *** bobh has quit IRC | 20:03 | |
| *** catinthe_ has quit IRC | 20:04 | |
| *** bobh has joined #openstack-mistral | 20:59 | |
| *** bobh has quit IRC | 21:03 | |
| *** catintheroof has quit IRC | 21:06 | |
| *** toure is now known as toure|biab | 21:07 | |
| *** chlong has joined #openstack-mistral | 21:43 | |
| *** Ephur has quit IRC | 22:01 | |
| *** chlong has quit IRC | 22:06 | |
| *** chlong has joined #openstack-mistral | 22:07 | |
| *** chlong has quit IRC | 22:12 | |
| *** harlowja has quit IRC | 22:43 | |
| *** harlowja has joined #openstack-mistral | 22:43 | |
| *** jamielennox is now known as jamielennox|away | 23:05 | |
| *** jamielennox|away is now known as jamielennox | 23:06 | |
| *** catintheroof has joined #openstack-mistral | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!