Wednesday, 2018-02-14

*** itlinux has joined #openstack-mistral00:16
*** AlexeyAbashkin has joined #openstack-mistral00:40
*** AlexeyAbashkin has quit IRC00:44
*** pengdake_ has joined #openstack-mistral00:55
*** pengdake_ has quit IRC01:11
*** pengdake_ has joined #openstack-mistral01:11
*** pengdake has joined #openstack-mistral01:16
*** pengdake_ has quit IRC01:16
*** itlinux has quit IRC01:19
*** itlinux has joined #openstack-mistral01:44
*** pengdake has quit IRC01:59
*** itlinux has quit IRC02:10
*** itlinux has joined #openstack-mistral02:32
*** pengdake has joined #openstack-mistral02:36
*** itlinux has quit IRC02:44
*** d0ugal has quit IRC02:49
*** d0ugal has joined #openstack-mistral03:00
*** gkadam has joined #openstack-mistral03:24
*** harlowja has quit IRC03:30
*** pengdake has quit IRC03:33
*** pengdake has joined #openstack-mistral03:33
*** d0ugal has quit IRC03:55
*** d0ugal has joined #openstack-mistral04:00
*** pengdake has quit IRC04:08
*** harlowja has joined #openstack-mistral04:12
*** gkadam_ has joined #openstack-mistral04:13
*** gkadam_ has quit IRC04:14
*** gkadam_ has joined #openstack-mistral04:14
*** gkadam has quit IRC04:15
*** ykarel_ has joined #openstack-mistral04:22
*** harlowja has quit IRC04:30
*** pengdake has joined #openstack-mistral05:01
rakhmerovd0ugal: hi, for some reason I don't see any support for root_execution_id on the client? Didn't we add it? I thought we did long ago..05:04
*** harlowja has joined #openstack-mistral05:14
*** hardikjasani has joined #openstack-mistral06:00
*** harlowja has quit IRC06:24
*** ykarel_ is now known as ykarel06:31
*** AlexeyAbashkin has joined #openstack-mistral07:16
*** AlexeyAbashkin has quit IRC07:21
*** AlexeyAbashkin has joined #openstack-mistral07:26
*** ykarel_ has joined #openstack-mistral07:47
*** ykarel has quit IRC07:48
apetrichrakhmerov, weird I do remember a patch for that. was it in tripleo-common instead?07:51
rakhmerovapetrich: yeah, I thought I saw that patch as well07:51
apetrichugly regression patch seems fixed07:52
rakhmerovnot sure about triplo-common07:52
* apetrich is happier07:52
*** ykarel__ has joined #openstack-mistral07:52
rakhmerovapetrich: what regression patch?07:52
apetrichyeah so very tripleo-y :)07:52
apetrichrakhmerov, it has been blocking promotion for a while. so we changed the python-mistralclient to return mistral exceptions but some other code were expecting keystone errors07:53
apetrichhttps://review.openstack.org/#/c/543916/07:53
rakhmerovok )07:53
apetrichthe patches for changing to mistral exceptions were from november but some clients are just tested on release. hence the problem07:53
*** ykarel has joined #openstack-mistral07:54
apetrichit was like the second regression with a patch that had landed more than 4 months in a row07:54
apetrich in mistralclient07:54
*** ykarel_ has quit IRC07:55
apetrich I think we will need some better actions to test that more frequently07:55
*** ykarel__ has quit IRC07:57
apetrichrakhmerov, not finding the root_execution_id patch. I guess we didn't have one?08:05
rakhmerovapetrich: that's what I'd like to clarify with d0ugal08:05
rakhmerovI failed to find it too although I seem to remember it08:05
rakhmerovI just wanted to demo this functionality to someone08:06
rakhmerovand realized that it's not supported by the client08:06
*** akazakov has joined #openstack-mistral08:35
*** gkadam__ has joined #openstack-mistral08:50
*** oidgar has joined #openstack-mistral08:51
*** gkadam_ has quit IRC08:53
*** jpich has joined #openstack-mistral08:59
*** ykarel is now known as ykarel|lunch09:19
d0ugalrakhmerov: correct, I never added it to the client.09:50
rakhmerovooh, interesting..09:50
d0ugalwell, it works via --filter doesn't it?09:50
d0ugalI forget09:50
rakhmerovhah!09:50
rakhmerovyes, it actually should! %)09:51
d0ugalso I don't think there is specific support - should there be?09:51
d0ugalIt might be easier if there was...09:51
rakhmerovcompletely forgot about it..09:51
d0ugalor maybe the client could at least show the valid filter options09:51
rakhmerovno-no, I think it is fine (if it works) but we just need to put a section about it in the doc09:51
rakhmerovbecause it's very very useful09:52
rakhmerovin many cases09:52
rakhmerovshowing the valid filter options is a good idea but we'll have to hardcode them on the client09:52
rakhmerovwhich may not be a big deal though..09:52
d0ugalrakhmerov: see the commends on: https://review.openstack.org/#/c/543402/309:52
d0ugalcomments*09:53
d0ugalbrb09:53
rakhmerovd0ugal: oh, so I need to send an email with the FFE request to myself or what? :)09:55
rakhmerovlet me send something anyway..09:56
*** ykarel|lunch is now known as ykarel10:00
*** shardy has joined #openstack-mistral10:02
rakhmerovd0ugal: just sent an email10:04
rakhmerov"openstack-dev][mistral][release][ffe] Requesting FFE for supporting source execution id in the Mistral client"10:04
rakhmerovyou can also say a word or two to support this10:05
d0ugalrakhmerov: great, thanks10:17
d0ugalrakhmerov: sorry, I got called away longer than I thought. Today is a bit crazy10:18
d0ugalI'll reply and agree, thanks for doing it.10:18
*** ykarel_ has joined #openstack-mistral10:26
rakhmerovnp10:26
*** ykarel has quit IRC10:29
*** ykarel_ is now known as ykarel10:37
*** ykarel_ has joined #openstack-mistral10:52
*** ykarel has quit IRC10:54
*** AlexeyAbashkin has quit IRC11:05
*** AlexeyAbashkin has joined #openstack-mistral11:38
d0ugalrakhmerov: Can we land this? https://review.openstack.org/#/c/543447/11:50
d0ugalIt adds quite a bit of time to mistralclient patches, but we would have caught the regression we landed months ago.11:50
openstackgerritOpenStack Proposal Bot proposed openstack/mistral master: Updated from global requirements  https://review.openstack.org/54426712:10
*** gkadam has joined #openstack-mistral12:17
*** gkadam__ has quit IRC12:19
*** katkapilatova has joined #openstack-mistral12:31
*** gkadam has quit IRC12:38
*** thrash|g0ne is now known as thrash12:57
*** weshay_PTO is now known as weshay13:01
*** ykarel__ has joined #openstack-mistral13:03
*** ykarel_ has quit IRC13:05
openstackgerritBrad P. Crochet proposed openstack/python-mistralclient master: Migrate python-mistralclient jobs to Zuul v3  https://review.openstack.org/50918413:18
*** oidgar has quit IRC13:21
*** hardikjasani has quit IRC13:30
openstackgerritOpenStack Release Bot proposed openstack/mistral master: Update reno for stable/queens  https://review.openstack.org/54432013:42
openstackgerritOpenStack Release Bot proposed openstack/mistral-dashboard master: Update reno for stable/queens  https://review.openstack.org/54432313:42
openstackgerritMerged openstack/mistral-dashboard master: Update reno for stable/queens  https://review.openstack.org/54432313:53
*** oidgar has joined #openstack-mistral14:16
*** ykarel__ is now known as ykarel14:26
*** bobh has joined #openstack-mistral15:05
*** gkadam has joined #openstack-mistral15:12
openstackgerritMerged openstack/mistral master: Update reno for stable/queens  https://review.openstack.org/54432015:16
*** AlexeyAbashkin has quit IRC15:17
*** AlexeyAbashkin has joined #openstack-mistral15:42
*** itlinux has joined #openstack-mistral15:50
*** gkadam_ has joined #openstack-mistral15:52
*** katkapilatova has quit IRC15:52
*** katkapilatova has joined #openstack-mistral15:53
*** katkapilatova has quit IRC15:57
*** ykarel is now known as ykarel|away15:57
*** gkadam has quit IRC16:04
*** ykarel|away has quit IRC16:05
d0ugalapetrich: sigh. https://review.openstack.org/#/c/543447/16:14
apetrichd0ugal, test_executions_list_with_pagination again?16:16
d0ugalapetrich: nah, another one16:16
d0ugalapetrich: mistralclient.tests.functional.cli.v2.cli_tests_v2.TaskCLITests.test_task_list_with_filter16:16
apetrichreally? why I am seing thise http://logs.openstack.org/47/543447/5/gate/python-mistralclient-devstack-dsvm/fbb570d/job-output.txt.gz#_2018-02-14_13_04_10_49356816:17
apetrichoh also16:17
apetrichyay16:17
d0ugalhuh16:17
d0ugalInteresting16:17
d0ugalI didn't notice there was more than one failure :(16:18
* apetrich cant type16:21
* apetrich can't type "can't type"16:22
apetrichis that a race condition? because it smells like one16:23
d0ugalapetrich: aye16:23
d0ugalthey all will be16:23
d0ugalapetrich: in the test_task_list_with_filter it fails when listing the tasks, I assume there are tasks from an execution in a different test at that point16:23
apetrichaye16:24
d0ugalif tempest wasn't concurrent we would have less issues16:24
d0ugal... and slower tests16:24
apetrichconsidering that we wait for ages to zuul to spin our tests how much time are we saving by deployment?16:24
d0ugalapetrich: no idea, but I don't think we can change how people run tempest easily - so better to fix it at our end.16:26
d0ugalIMO the tempest tests were just incorrectly done from the start and need to be rethought, but that is quite a big job16:27
d0ugalMaybe I can try with all my spare time...16:27
*** itlinux has quit IRC16:41
*** itlinux has joined #openstack-mistral16:52
*** oidgar has quit IRC17:01
*** AlexeyAbashkin has quit IRC17:07
*** AlexeyAbashkin has joined #openstack-mistral17:15
*** AlexeyAbashkin has quit IRC17:20
*** jpich has quit IRC17:24
*** thrash is now known as thrash|f00dz17:42
openstackgerritOpenStack Proposal Bot proposed openstack/mistral stable/pike: Updated from global requirements  https://review.openstack.org/54459318:30
*** openstackgerrit has quit IRC18:33
*** shardy has quit IRC18:38
*** prometheanfire has joined #openstack-mistral18:58
*** bobh has quit IRC18:58
prometheanfirewe plan on removing mistral-extra from requirements, can we get an ack from the ptl? https://review.openstack.org/54290718:58
apetrichprometheanfire, why? if anything mistral-extra will be more present. the plan is to migrate openstack actions out of mistral into mistral-extra19:16
prometheanfireapetrich: it's not used anywhere, http://codesearch.openstack.org/?q=mistral-extra&i=nope&files=.*requirements.*&repos=19:17
apetrichyeah I know.19:18
apetrichbecause releasing it was a pain. now that we have it released we are moving functionality into it this cycle19:18
apetrichprometheanfire, ^^19:18
prometheanfireok, if it's happening this cycle that works for me19:18
apetrichwe have a couple of cards to do so19:18
apetrichcool19:19
prometheanfiremind leaving a comment in the review?19:19
apetrichI'm typing it19:19
prometheanfire:D19:19
apetrichcheers19:19
*** harlowja has joined #openstack-mistral19:21
*** prometheanfire has left #openstack-mistral19:22
*** bobh has joined #openstack-mistral19:38
d0ugalapetrich: gj20:23
*** thrash|f00dz is now known as thrash20:23
*** bobh has quit IRC22:17
*** threestrands has joined #openstack-mistral22:31
*** itlinux has quit IRC22:53

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!