| *** bobh has joined #openstack-mistral | 00:15 | |
| *** bobh has quit IRC | 00:18 | |
| *** bobh has joined #openstack-mistral | 00:19 | |
| *** bobh has quit IRC | 00:22 | |
| *** catintheroof has quit IRC | 00:47 | |
| *** catintheroof has joined #openstack-mistral | 00:48 | |
| *** catintheroof has quit IRC | 00:52 | |
| *** rrecio_ has quit IRC | 00:53 | |
| *** bobh has joined #openstack-mistral | 01:07 | |
| *** bobh has quit IRC | 01:58 | |
| *** bobh has joined #openstack-mistral | 02:02 | |
| *** bobh has quit IRC | 02:32 | |
| *** bobh has joined #openstack-mistral | 02:37 | |
| *** jamielennox is now known as jamielennox|away | 03:07 | |
| *** bobh has quit IRC | 03:12 | |
| *** bobh has joined #openstack-mistral | 03:18 | |
| *** bobh has quit IRC | 03:19 | |
| *** bobh has joined #openstack-mistral | 03:20 | |
| *** jamielennox|away is now known as jamielennox | 03:21 | |
| *** bobh has quit IRC | 03:23 | |
| *** bobh has joined #openstack-mistral | 03:24 | |
| *** thrash is now known as thrash|g0ne | 03:26 | |
| *** bobh has quit IRC | 04:00 | |
| *** jamielennox is now known as jamielennox|away | 04:15 | |
| *** jamielennox|away is now known as jamielennox | 04:17 | |
| *** sharatss has joined #openstack-mistral | 04:22 | |
| *** janki has joined #openstack-mistral | 04:34 | |
| *** jamielennox is now known as jamielennox|away | 04:40 | |
| *** jamielennox|away is now known as jamielennox | 05:03 | |
| *** khamtamtun has joined #openstack-mistral | 05:04 | |
| *** khamtamtun has quit IRC | 05:21 | |
| rakhmerov | d0ugal, xavierhardy: hi | 05:25 |
|---|---|---|
| rakhmerov | yes, we need those patches in stable branch | 05:25 |
| rakhmerov | definitely | 05:25 |
| rakhmerov | but one of them was in conflict | 05:25 |
| rakhmerov | xavierhardy: how did you resolve conflicts in https://review.openstack.org/#/c/397199/ ? | 05:27 |
| rakhmerov | manually? | 05:29 |
| *** hparekh has joined #openstack-mistral | 05:40 | |
| *** jaosorior has joined #openstack-mistral | 06:07 | |
| *** khamtamtun has joined #openstack-mistral | 06:17 | |
| *** khamtamtun has quit IRC | 06:35 | |
| *** sharatss has quit IRC | 06:35 | |
| *** sharatss has joined #openstack-mistral | 06:36 | |
| *** jaosorior has quit IRC | 06:40 | |
| *** jaosorior has joined #openstack-mistral | 06:40 | |
| *** khamtamtun has joined #openstack-mistral | 06:48 | |
| openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient: Updated tox.ini to pick up requirements from upper constraints https://review.openstack.org/397549 | 06:56 |
| *** sharatss has quit IRC | 07:35 | |
| *** sharatss has joined #openstack-mistral | 07:37 | |
| hparekh | sharatss: Hi, Can you please add release note for https://review.openstack.org/#/c/395587/7 ? | 08:09 |
| xavierhardy | rakhmerov: manually? Yes | 08:19 |
| rakhmerov | ok | 08:19 |
| rakhmerov | np | 08:19 |
| xavierhardy | I followed this | 08:19 |
| xavierhardy | http://docs.openstack.org/project-team-guide/stable-branches.html | 08:19 |
| rakhmerov | a approved the backport | 08:19 |
| rakhmerov | yes, ok | 08:19 |
| rakhmerov | I approved | 08:19 |
| xavierhardy | Good, thanks a lot | 08:20 |
| rakhmerov | thanks to you | 08:20 |
| *** janki has quit IRC | 08:22 | |
| rakhmerov | d0ugal: hi | 08:23 |
| d0ugal | rakhmerov: hi | 08:24 |
| rakhmerov | d0ugal: how was the meeting yesterday? | 08:25 |
| d0ugal | rakhmerov: good, it was quick - I have less to say than you I guess | 08:25 |
| d0ugal | so it was a short one, but that was fine :) | 08:25 |
| rakhmerov | ok | 08:25 |
| rakhmerov | did you discuss time slots? | 08:25 |
| d0ugal | rakhmerov: Yeah, briefly. | 08:26 |
| rakhmerov | outcome? | 08:26 |
| d0ugal | rakhmerov: based on the feedback in the etherpad, most people find the current meeting time okay tbh | 08:26 |
| d0ugal | so it is hard to know what to do | 08:26 |
| d0ugal | but we could try a doodlepoll | 08:26 |
| rakhmerov | ooh, I see | 08:26 |
| d0ugal | https://etherpad.openstack.org/p/mistral-meeting-time | 08:26 |
| rakhmerov | I know that, for example, for kong it's not convenient | 08:27 |
| d0ugal | I roughly grouped them by similar times | 08:27 |
| d0ugal | Yeah, it is worst for kong | 08:27 |
| d0ugal | but if we move it to a time for kong it doesn't suit anyone else | 08:27 |
| d0ugal | even you have very little overlap with kong | 08:27 |
| rakhmerov | yeah | 08:27 |
| rakhmerov | ok | 08:28 |
| d0ugal | it might be better for some people if we make it a little earlier | 08:28 |
| d0ugal | like you | 08:28 |
| rakhmerov | I'm ok with it | 08:28 |
| d0ugal | Alright, I guess we will just continue with the current time | 08:28 |
| rakhmerov | I'm just thinking that maybe we should leave this time slot but every 2nd week have it at a different time | 08:29 |
| d0ugal | I had hoped we might find more people that want to attend but can't - but maybe we have already lost those people | 08:29 |
| rakhmerov | more convenient for StackStorm folks and kong | 08:29 |
| d0ugal | Sure, that would be fine | 08:29 |
| rakhmerov | we definitely didn't loose kong and Winson from StackStorm | 08:29 |
| d0ugal | True :) | 08:30 |
| rakhmerov | Winson works almost full time on Mistral | 08:30 |
| rakhmerov | he is just shy :) | 08:30 |
| rakhmerov | and meeting time is not convenient for him | 08:30 |
| rakhmerov | he sends patches rarely but they are big | 08:30 |
| rakhmerov | ok, I'll have a meeting with him this Friday | 08:31 |
| d0ugal | Right, but if they don't make requests it is hard to know what to do | 08:31 |
| rakhmerov | and I'll discuss it with him | 08:31 |
| d0ugal | rakhmerov: Yeah, too big maybe ;) | 08:31 |
| rakhmerov | yes, I know | 08:31 |
| rakhmerov | we need to fix communication with them | 08:31 |
| rakhmerov | :)) | 08:31 |
| d0ugal | rakhmerov: if Kong or Winston can run the other meeting that would work | 08:31 |
| rakhmerov | too big, right | 08:31 |
| rakhmerov | ok | 08:31 |
| rakhmerov | d0ugal: thanks for driving this | 08:31 |
| d0ugal | Winson* | 08:32 |
| d0ugal | rakhmerov: but I guess they wont want the meeting if it is just for two maybe with a bit of proption we will get more | 08:34 |
| d0ugal | rakhmerov: but I guess they wont want the meeting if it is just for two of them. Maybe with a bit of proption we will get more people. | 08:35 |
| *** shardy has joined #openstack-mistral | 08:35 | |
| rakhmerov | yes, right | 08:38 |
| *** jpich has joined #openstack-mistral | 08:40 | |
| sharatss | hparekh: sure. i was waiting for the merge | 08:45 |
| *** jaosorior is now known as jaosorior_lunch | 08:52 | |
| d0ugal | Have all the dangling transaction patches landed? | 08:56 |
| d0ugal | oh, two waiting for stable/newton | 08:56 |
| openstackgerrit | Sharat Sharma proposed openstack/mistral: Added releasenote for Senlin Action Pack https://review.openstack.org/397592 | 09:02 |
| openstackgerrit | yushangbin proposed openstack/mistral: Remove unused pylintrc https://review.openstack.org/397598 | 09:11 |
| *** szaher has joined #openstack-mistral | 09:18 | |
| *** openstackgerrit has quit IRC | 09:18 | |
| *** openstackgerrit has joined #openstack-mistral | 09:18 | |
| *** jaosorior_lunch is now known as jaosorior | 09:42 | |
| openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient: Updated tox.ini to pick up requirements from upper constraints https://review.openstack.org/397549 | 09:42 |
| openstackgerrit | Gal Margalit proposed openstack/mistral-dashboard: mistral-dashboard: entities name change for system consistency https://review.openstack.org/397637 | 10:01 |
| openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient: Remove unused pylint https://review.openstack.org/397642 | 10:08 |
| openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Cleanup unused files https://review.openstack.org/397648 | 10:15 |
| openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient: Updated tox.ini to pick up requirements from upper constraints https://review.openstack.org/397549 | 10:20 |
| openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Adding files to .gitignore https://review.openstack.org/397656 | 10:27 |
| *** khamtamtun has quit IRC | 10:39 | |
| openstackgerrit | Sharat Sharma proposed openstack/mistral-lib: Added the Options for Internationalization output in conf.py https://review.openstack.org/397675 | 10:58 |
| *** khamtamtun has joined #openstack-mistral | 11:01 | |
| openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient: Removed the extra space from tox.ini https://review.openstack.org/397683 | 11:03 |
| *** hparekh has quit IRC | 11:36 | |
| *** thrash|g0ne is now known as thrash | 11:43 | |
| *** sharatss has quit IRC | 11:55 | |
| *** sharatss has joined #openstack-mistral | 11:57 | |
| sharatss | rakhmerov d0ugal ddeja rbrady: anyone free? | 12:01 |
| d0ugal | sharatss: What's up? | 12:01 |
| d0ugal | Those reviews? :) | 12:01 |
| sharatss | http://logs.openstack.org/83/397683/1/check/gate-python-mistralclient-python35-nv/a152aa5/console.html | 12:02 |
| * d0ugal looks | 12:02 | |
| sharatss | I am getting Assertion Error for all mistralclient commits | 12:02 |
| sharatss | am i going wrong somewhere? | 12:02 |
| d0ugal | sharatss: I think you can ignore the python35 check for now. | 12:03 |
| d0ugal | sharatss: (it is non-voting, I assume because it isn't stable) | 12:03 |
| sharatss | d0ugal: ok. this might sound silly now | 12:03 |
| d0ugal | sharatss: but I don't understand what that error means. | 12:03 |
| sharatss | d0ugal: how can i get jenkins to pass that? | 12:04 |
| d0ugal | sharatss: Jenkins will still +1 if it fails | 12:04 |
| d0ugal | sharatss: but if you actually want to fix the check... I'm not sure. | 12:04 |
| sharatss | d0ugal: thanks for the quick response as always | 12:05 |
| sharatss | d0ugal: the devstack-dvsm also often fails | 12:05 |
| d0ugal | sharatss: Yeah, that is a bigger problem :) | 12:05 |
| d0ugal | because jenkins will -1 you | 12:05 |
| d0ugal | so lets see what is wrong there.. | 12:06 |
| sharatss | d0ugal: unit tests fail there | 12:06 |
| sharatss | d0ugal: the patch i am referring to now is just given as a test patch since it removes only an extra space | 12:07 |
| d0ugal | sharatss: I wonder if this is related to the transaction changes | 12:07 |
| d0ugal | sharatss: because it is related to lazy loading | 12:07 |
| d0ugal | i.e. http://logs.openstack.org/83/397683/1/check/gate-python-mistralclient-devstack-dsvm/6a79fa7/console.html#_2016-11-15_11_48_37_161580 | 12:07 |
| sharatss | d0ugal: yes. is there a way we can fix it? | 12:08 |
| * ddeja is reading the conversation | 12:08 | |
| d0ugal | sharatss: good question. I am looking for the traceback | 12:09 |
| d0ugal | http://logs.openstack.org/83/397683/1/check/gate-python-mistralclient-devstack-dsvm/6a79fa7/logs/screen-mistral-api.txt.gz#_2016-11-15_11_37_24_717 | 12:09 |
| d0ugal | That looks bad | 12:10 |
| sharatss | d0ugal: even i am looking if there is a permanent fix for this | 12:10 |
| d0ugal | If you grep that link for DetachedInstanceError it happens a few times. | 12:10 |
| d0ugal | sharatss: Yeah, me too now :) | 12:10 |
| sharatss | d0ugal: should find a way to pass this lazy load | 12:11 |
| sharatss | d0ugal: i will spend today on this issue | 12:12 |
| d0ugal | sharatss: okay, I'll keep looking for a bit and let you know if I find anything | 12:15 |
| sharatss | d0ugal: sure. thanks for taking time to help me | 12:15 |
| d0ugal | sharatss: np, I think we might have broken something for everyone | 12:16 |
| d0ugal | so I am helping myself too :-D | 12:16 |
| sharatss | d0ugal: :) | 12:16 |
| ddeja | d0ugal, sharatss I think I found the issue | 12:17 |
| sharatss | ddeja: great. where is the problem? | 12:17 |
| ddeja | starting from the begging of traceback that d0ugal showed | 12:18 |
| ddeja | http://logs.openstack.org/83/397683/1/check/gate-python-mistralclient-devstack-dsvm/6a79fa7/logs/screen-mistral-api.txt.gz#_2016-11-15_11_37_24_717 | 12:18 |
| ddeja | firstly we call get_all and then _get_action_executions from file /opt/stack/new/mistral/mistral/api/controllers/v2/action_execution.py | 12:18 |
| ddeja | then it calls 'get_all' from /opt/stack/new/mistral/mistral/utils/rest_utils.py | 12:19 |
| ddeja | which, in line 167 calls the db directly | 12:19 |
| ddeja | aaand this call is not in the transaction | 12:19 |
| ddeja | co then, when we try to load additional data in line 176, there is no sessios so the lazy loading fails | 12:20 |
| d0ugal | I think I just found this too | 12:20 |
| d0ugal | I have replecated the issue in my dev env | 12:20 |
| * ddeja is wondering why it happens only on the mistrialclient | 12:20 | |
| d0ugal | ddeja: https://review.openstack.org/#/c/395696/ | 12:21 |
| d0ugal | devstack is non-voting | 12:21 |
| d0ugal | we need to fix that | 12:21 |
| ddeja | oh | 12:21 |
| d0ugal | Yeah, it failed with the same error. | 12:22 |
| d0ugal | http://logs.openstack.org/96/395696/22/check/gate-mistral-devstack-dsvm/d43f8f3/console.html.gz#_2016-11-11_13_22_03_864245 | 12:22 |
| ddeja | we need this patch in, I guess? https://review.openstack.org/#/c/396128/3/mistral/api/controllers/v2/action_execution.py | 12:22 |
| ddeja | d0ugal: ^ | 12:22 |
| ddeja | it passed the dsvm gate | 12:22 |
| d0ugal | ddeja: oh, cool. yeah | 12:23 |
| d0ugal | ddeja: that is roughly what I was just testing | 12:23 |
| d0ugal | I missed that patch | 12:23 |
| d0ugal | actually, I already reviewed it, I just forgot about it lol | 12:23 |
| ddeja | but hm, this function that is failin | 12:24 |
| ddeja | still doesn't have any transactions | 12:24 |
| ddeja | oh, it has this | 12:24 |
| ddeja | https://review.openstack.org/#/c/396128/3/mistral/utils/rest_utils.py | 12:24 |
| ddeja | OK, reviewing briefly | 12:25 |
| d0ugal | thanks | 12:25 |
| ddeja | sharatss: thanks for birnging it up! | 12:25 |
| sharatss | ddeja: np :) anyway i will also be benifited if this gets fixed | 12:26 |
| d0ugal | ddeja, sharatss: https://review.openstack.org/#/c/397717/ | 12:28 |
| *** catintheroof has joined #openstack-mistral | 12:28 | |
| *** dprince has joined #openstack-mistral | 12:33 | |
| *** bobh has joined #openstack-mistral | 13:09 | |
| *** bobh has quit IRC | 13:09 | |
| *** bobh has joined #openstack-mistral | 13:09 | |
| *** khamtamtun has quit IRC | 13:16 | |
| xavierhardy | ddeja: I have replied to your comments | 13:17 |
| xavierhardy | I agree with them. If there's no opposition, I will modify them the way you want it | 13:17 |
| * ddeja is looking | 13:18 | |
| ddeja | xavierhardy: you mean for this patch? https://review.openstack.org/#/c/396128/3 | 13:18 |
| ddeja | If so, I don't see any comments | 13:18 |
| *** gnuoy has quit IRC | 13:19 | |
| xavierhardy | they only appear as drafts... | 13:19 |
| ddeja | oh | 13:19 |
| ddeja | but basically, only the one in file task.py is critical | 13:20 |
| xavierhardy | how do I publish drafts | 13:20 |
| xavierhardy | ? | 13:20 |
| openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient: Added the reno for stable/mitaka stable/newton and stable/liberty https://review.openstack.org/397736 | 13:20 |
| ddeja | you click 'reply' on the https://review.openstack.org/#/c/396128/3 and post | 13:20 |
| xavierhardy | Yeah, found it just now | 13:20 |
| *** bobh has quit IRC | 13:26 | |
| xavierhardy | doing | 13:26 |
| openstackgerrit | Xavier Hardy proposed openstack/mistral: Fix possible DB race conditions in REST controller https://review.openstack.org/396128 | 13:27 |
| xavierhardy | done | 13:28 |
| openstackgerrit | Merged openstack/mistral: Added senlin action pack https://review.openstack.org/395587 | 13:29 |
| openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Added reno for stable/mitaka, stable/liberty and stable/newton https://review.openstack.org/397741 | 13:31 |
| *** mgershen has joined #openstack-mistral | 13:39 | |
| openstackgerrit | Sharat Sharma proposed openstack/mistral-lib: Added reno for stable/mitaka, stable/liberty and stable/newton https://review.openstack.org/397745 | 13:40 |
| openstackgerrit | Sharat Sharma proposed openstack/python-mistralclient: Remove unused pylint https://review.openstack.org/397642 | 13:42 |
| d0ugal | rakhmerov: https://review.openstack.org/#/c/397717/ | 13:45 |
| d0ugal | sharatss: Hey, just looking at the i18n blueprint. | 13:46 |
| d0ugal | sharatss: I think it looks good | 13:46 |
| d0ugal | I don't have any other feedback at the moment :) | 13:46 |
| sharatss | d0ugal: thank you :) | 13:46 |
| d0ugal | sharatss: For something a bit related, do you have any idea how we can help users translate strings in Mistral workflows? | 13:47 |
| sharatss | d0ugal: yes. i will start working once it gets approved | 13:47 |
| sharatss | d0ugal: i am not sure about it | 13:48 |
| sharatss | d0ugal: may be i should try to find that out | 13:48 |
| d0ugal | sharatss: I think it is a different problem, because workflows are user content rather than something in Mistral | 13:49 |
| d0ugal | but I have been thinking about it and I don't have any good ideas. | 13:49 |
| sharatss | d0ugal: that sounds interesting | 13:50 |
| sharatss | d0ugal: i will try to find something if possible on this and let you know if I find something | 13:50 |
| d0ugal | sharatss: Great, thanks! | 13:52 |
| sharatss | d0ugal: My pleasure :) i am happy to be a part of Mistral | 13:53 |
| ddeja | xavierhardy: thanks for the patch. I'll give +2 if jenkins pass | 13:54 |
| xavierhardy | Cool, thanks | 13:55 |
| *** khamtamtun has joined #openstack-mistral | 13:58 | |
| openstackgerrit | Merged openstack/mistral: Few changes in the doc https://review.openstack.org/397067 | 14:00 |
| *** mgershen has quit IRC | 14:16 | |
| *** khamtamtun has quit IRC | 14:16 | |
| openstackgerrit | Sharat Sharma proposed openstack/mistral: Add timestamp at the bottom of every page https://review.openstack.org/397761 | 14:18 |
| *** mgershen has joined #openstack-mistral | 14:25 | |
| *** jaosorior has quit IRC | 14:36 | |
| openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Remove mox and mox3 in test-requirement.txt https://review.openstack.org/397770 | 14:42 |
| *** khamtamtun has joined #openstack-mistral | 14:45 | |
| *** khamtamtun has quit IRC | 14:50 | |
| d0ugal | ddeja: when Mistral states something is "RUNNING" does that sometimes mean it is scheduled and waiting to run? | 14:53 |
| *** jpich has left #openstack-mistral | 14:54 | |
| *** jpich has joined #openstack-mistral | 14:54 | |
| openstackgerrit | Sharat Sharma proposed openstack/mistral-dashboard: Remove mox in test-requirement.txt https://review.openstack.org/397770 | 14:57 |
| ddeja | d0ugal: I'm not sure | 14:58 |
| ddeja | let me check | 14:58 |
| ddeja | d0ugal: yes, scheduled actions have the state set to 'RUNNING' | 15:00 |
| d0ugal | ddeja: that's a shame, we should have a SCHEDULED state IMO | 15:00 |
| ddeja | https://github.com/openstack/mistral/blob/master/mistral/engine/actions.py#L124-L136 | 15:00 |
| d0ugal | ddeja: thanks | 15:00 |
| ddeja | I agree that this is misleading | 15:01 |
| ddeja | but right now we don't reallly have a scheduler | 15:02 |
| ddeja | sorry | 15:02 |
| ddeja | let me re-phrase | 15:02 |
| ddeja | we don't use scheduler for running actions | 15:02 |
| sharatss | ddeja: d0ugal hi | 15:03 |
| d0ugal | ddeja: right | 15:03 |
| d0ugal | ddeja: but I think it is the same with workflows - they are created as "RUNNING" | 15:03 |
| ddeja | so the requests to run action would be send almost immediately | 15:03 |
| sharatss | py35 jobs may be failing because https://review.openstack.org/#/c/391025/1/mistralclient/openstack/common/cliutils.py got it fixed | 15:03 |
| d0ugal | ddeja: oh, no, workflows are IDLE to start with. hmm | 15:03 |
| d0ugal | ddeja: I was testing something and I ended up with hundreds of running actions and workflows - I don't believe they are all running :) | 15:04 |
| ddeja | d0ugal: they may not be running | 15:04 |
| *** bobh has joined #openstack-mistral | 15:05 | |
| d0ugal | ddeja: right | 15:05 |
| ddeja | but if they are on the RPC call, we can not say what are really the status | 15:05 |
| d0ugal | ddeja: and these were all triggered by workflows btw, no direct action calls. | 15:05 |
| ddeja | sharatss: thanks, I'll take a look on this | 15:05 |
| d0ugal | ddeja: anyway, it isn't important for now, I am just thinking :) | 15:05 |
| sharatss | ddeja: and the complete openstack directory has been removed now?? | 15:06 |
| ddeja | d0ugal: for direct action call is OK, because we create the action in the DB and then send a request to do it to executor | 15:06 |
| ddeja | sharatss: which directory? | 15:06 |
| sharatss | ddeja: mistralclient/openstack | 15:07 |
| ddeja | sharatss: I'm not aware of such change... | 15:07 |
| ddeja | oh, you mean that patch https://review.openstack.org/#/c/391025/1 fixed the gate and then whole directory was deleted? | 15:08 |
| sharatss | ddeja: now i dont find the openstack/common folder in the repo | 15:08 |
| ddeja | including the fix? | 15:08 |
| mgershen | d0ugal: BTW we do have WAITING for tasks that have join on them and waiting for other task executions to finish. | 15:08 |
| sharatss | ddeja: yes | 15:08 |
| ddeja | sharatss: that patch removed it https://review.openstack.org/#/c/395082/ | 15:09 |
| ddeja | sharatss: hm, it seems that you are right | 15:10 |
| d0ugal | mgershen: oh yeah, thanks. I guess I need to look into this more. | 15:11 |
| sharatss | ddeja: what can be done now? | 15:11 |
| ddeja | sharatss: we should check where the inspect.getargspec() was used and change it to signature() | 15:12 |
| sharatss | ddeja: i tried grepping it. i will try again and see if i can find some possible solution | 15:13 |
| ddeja | sharatss: If I see correctly, only this test fails rght now mistralclient.tests.unit.v2.test_cli_bash_completion.TestCLIBashCompletionV2.test_bash_completion | 15:14 |
| sharatss | ddeja: yes. i will try to look into this | 15:19 |
| sharatss | ddeja: a minor change should fix this i guess | 15:19 |
| ddeja | d0ugal: the alternative is to change the state of task from 'WAITING' to 'RUNNING' once we got the response from RPC, but it would double our db calls | 15:21 |
| ddeja | and to be honest, I'm not sure if we still need the method schedule from /mistral/enigne/actions.py | 15:23 |
| ddeja | It logically does nearly the same what run does | 15:24 |
| ddeja | oh, wait, there is a difference in what is returned | 15:24 |
| ddeja | but it's the only one | 15:25 |
| openstackgerrit | Sharat Sharma proposed openstack/mistral: Add timestamp at the bottom of every page https://review.openstack.org/397761 | 15:27 |
| *** openstackgerrit has quit IRC | 15:48 | |
| *** openstackgerrit has joined #openstack-mistral | 15:48 | |
| openstackgerrit | Merged openstack/mistral: Fix possible DB race conditions in REST controller https://review.openstack.org/396128 | 15:50 |
| *** sharatss has quit IRC | 15:50 | |
| *** sharatss has joined #openstack-mistral | 15:51 | |
| *** rrecio_ has joined #openstack-mistral | 15:53 | |
| *** mgershen has quit IRC | 17:05 | |
| *** thrash is now known as thrash|f00dz | 17:14 | |
| *** thrash|f00dz is now known as thrash | 17:50 | |
| *** jpich has quit IRC | 17:51 | |
| *** dprince has quit IRC | 18:21 | |
| *** dprince has joined #openstack-mistral | 18:21 | |
| *** harlowja has quit IRC | 18:38 | |
| *** shardy has quit IRC | 18:39 | |
| *** harlowja has joined #openstack-mistral | 19:06 | |
| *** dprince has quit IRC | 19:23 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/mistral: Updated from global requirements https://review.openstack.org/397899 | 19:31 |
| *** sharatss has quit IRC | 21:21 | |
| *** sharatss has joined #openstack-mistral | 21:22 | |
| *** harlowja has quit IRC | 21:55 | |
| *** harlowja has joined #openstack-mistral | 22:13 | |
| *** catinthe_ has joined #openstack-mistral | 22:18 | |
| *** catintheroof has quit IRC | 22:21 | |
| *** khamtamtun has joined #openstack-mistral | 22:29 | |
| *** catintheroof has joined #openstack-mistral | 22:35 | |
| *** catinthe_ has quit IRC | 22:36 | |
| *** khamtamtun has quit IRC | 22:59 | |
| *** bobh has quit IRC | 23:05 | |
| *** rrecio_ has quit IRC | 23:05 | |
| *** catintheroof has quit IRC | 23:52 | |
| *** catintheroof has joined #openstack-mistral | 23:54 | |
| *** sharatss has quit IRC | 23:54 | |
| *** sharatss has joined #openstack-mistral | 23:55 | |
| *** catintheroof has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!