Wednesday, 2017-12-13

*** bobh has quit IRC00:03
*** dprince has quit IRC00:10
*** bobh has joined #openstack-mistral00:17
*** bobh has quit IRC00:21
*** bobh has joined #openstack-mistral00:26
*** yangyapeng has quit IRC00:28
*** yangyapeng has joined #openstack-mistral00:29
*** bobh has quit IRC00:31
*** yangyapeng has quit IRC00:33
*** zhurong has joined #openstack-mistral00:37
*** bobh has joined #openstack-mistral00:40
*** bobh has quit IRC00:44
*** bobh has joined #openstack-mistral00:51
*** bobh has quit IRC00:56
*** yangyapeng has joined #openstack-mistral01:15
*** bobh has joined #openstack-mistral01:17
*** bobh has quit IRC01:21
*** bobh has joined #openstack-mistral01:23
*** bobh has quit IRC01:28
*** bobh_ has joined #openstack-mistral01:28
*** bobh_ has quit IRC01:33
*** bobh has joined #openstack-mistral01:41
*** bobh has quit IRC01:46
*** bobh has joined #openstack-mistral01:50
*** bobh has quit IRC01:54
*** zhurong has quit IRC02:23
*** bobh has joined #openstack-mistral02:28
*** bobh has quit IRC02:32
*** zhurong has joined #openstack-mistral02:35
*** bobh has joined #openstack-mistral02:45
*** bobh has quit IRC02:51
*** bobh has joined #openstack-mistral03:10
*** bobh has quit IRC03:15
*** bobh has joined #openstack-mistral03:17
*** bobh has quit IRC03:21
*** bobh has joined #openstack-mistral03:29
*** bobh has quit IRC03:34
*** bobh has joined #openstack-mistral03:40
*** bobh has quit IRC03:44
*** bobh has joined #openstack-mistral03:56
*** bobh has quit IRC04:02
*** bobh has joined #openstack-mistral04:06
*** threestrands has joined #openstack-mistral04:14
*** threestrands has quit IRC04:14
*** threestrands has joined #openstack-mistral04:14
*** jappleii__ has quit IRC04:15
*** threestrands has quit IRC04:15
*** threestrands has joined #openstack-mistral04:16
*** threestrands has quit IRC04:16
*** threestrands has joined #openstack-mistral04:16
*** threestrands has quit IRC04:36
*** threestrands has joined #openstack-mistral04:43
*** threestrands has quit IRC04:43
*** threestrands has joined #openstack-mistral04:43
*** threestrands has quit IRC04:44
*** threestrands has joined #openstack-mistral04:44
*** threestrands has quit IRC04:44
*** threestrands has joined #openstack-mistral04:44
*** threestrands has quit IRC04:45
*** threestrands has joined #openstack-mistral04:46
*** threestrands has quit IRC04:46
*** threestrands has joined #openstack-mistral04:46
*** threestrands has quit IRC04:47
*** threestrands has joined #openstack-mistral04:47
*** threestrands has quit IRC04:47
*** threestrands has joined #openstack-mistral04:47
*** bobh has quit IRC05:03
*** bobh has joined #openstack-mistral05:10
*** bobh has quit IRC05:16
*** bobh has joined #openstack-mistral05:18
*** bobh has quit IRC05:23
*** bobh has joined #openstack-mistral05:28
*** bobh has quit IRC05:33
*** bobh has joined #openstack-mistral05:39
*** AlexeyAbashkin has joined #openstack-mistral05:43
*** bobh has quit IRC05:44
*** threestrands has quit IRC06:03
*** d0ugal has quit IRC06:04
*** bobh has joined #openstack-mistral06:08
*** d0ugal has joined #openstack-mistral06:09
*** bobh has quit IRC06:13
*** bobh has joined #openstack-mistral06:17
*** bobh has quit IRC06:22
*** bobh has joined #openstack-mistral06:39
*** bobh has quit IRC06:43
*** AlexeyAbashkin has quit IRC06:57
*** bobh has joined #openstack-mistral07:27
*** bobh has quit IRC07:32
*** daidv has quit IRC07:41
*** daidv has joined #openstack-mistral07:41
*** shardy has joined #openstack-mistral07:46
therved0ugal, https://review.openstack.org/#/c/527348/ looks promising :)07:50
*** bobh has joined #openstack-mistral07:57
rakhmerovtherve: hi, approved )07:57
therveAwesome, thanks07:58
rakhmerovnp07:58
*** bobh has quit IRC08:02
rakhmerovd0ugal: can you please review these patches? https://review.openstack.org/#/c/526720/, https://review.openstack.org/#/c/521572/, https://review.openstack.org/#/c/520016/ and https://review.openstack.org/#/c/518641/08:07
rakhmerovthe last one is pretty big (tests mostly) but others are fairly small08:08
*** openstackgerrit has joined #openstack-mistral08:12
openstackgerritMerged openstack/mistral-dashboard master: Expand parameter list for workflow execution  https://review.openstack.org/49622008:12
*** bobh has joined #openstack-mistral08:16
d0ugaltherve: interestingly I rechecked my patches and they all passed without yours too08:19
d0ugal¯\_(ツ)_/¯08:19
d0ugaloh, actually, 1 failed08:19
d0ugal(but a different error)08:20
*** bobh has quit IRC08:21
*** apetrich has quit IRC08:25
*** apetrich has joined #openstack-mistral08:26
*** AlexeyAbashkin has joined #openstack-mistral08:27
d0ugalrakhmerov: do you know who I can get to review this? https://review.openstack.org/#/c/506185/08:32
d0ugalMaybe Andras?08:32
*** AlexeyAbashkin has quit IRC08:32
rakhmerovyes, I'll ask him08:36
rakhmerovbut he's OOO now, as far as I know08:36
rakhmerovNikolay can do too, probably08:36
*** bobh has joined #openstack-mistral08:36
*** AlexeyAbashkin has joined #openstack-mistral08:37
*** bobh has quit IRC08:41
*** AlexeyAbashkin has quit IRC08:43
*** jtomasek has joined #openstack-mistral08:43
*** daidv has quit IRC08:46
*** daidv has joined #openstack-mistral08:46
*** bobh has joined #openstack-mistral08:51
*** bobh has quit IRC08:56
*** AlexeyAbashkin has joined #openstack-mistral08:57
openstackgerritMerged openstack/python-mistralclient master: Fix limit handling to not send value of -1  https://review.openstack.org/52157209:07
*** jpich has joined #openstack-mistral09:11
d0ugalrakhmerov: we need to stop merging tempest patches09:21
rakhmerovd0ugal: understood, yes09:21
d0ugalrakhmerov: they should go here: https://github.com/openstack/mistral-tempest-plugin09:21
rakhmerovyep09:21
rakhmerovagree09:21
d0ugalI am going to cherry-pick the patches that are missing, and then try and merge the patch that removes them from Mistral09:21
rakhmerovok09:22
*** bobh has joined #openstack-mistral09:22
openstackgerritDougal Matthews proposed openstack/mistral-tempest-plugin master: Allow filtering executions by their root_execution_id  https://review.openstack.org/52764009:25
openstackgerritDougal Matthews proposed openstack/mistral-tempest-plugin master: Disable unstable tempest multi_vim_authentication test  https://review.openstack.org/52764109:26
openstackgerritDougal Matthews proposed openstack/mistral-tempest-plugin master: Disable unstable tempest test_create_action_execution_sync test  https://review.openstack.org/52764209:26
openstackgerritDougal Matthews proposed openstack/mistral-tempest-plugin master: Re-work the direct action call tempest test  https://review.openstack.org/52764309:26
*** bobh has quit IRC09:27
*** bobh has joined #openstack-mistral09:28
d0ugalrakhmerov: ^ is it okay with you if I just +W these?09:29
d0ugalThey are all my patches :)09:29
d0ugalI'll cherry-pick therve's once it merges too.09:29
rakhmerovd0ugal: sure, that's totally fine09:30
rakhmerovgo ahead09:30
d0ugalWe only have PEP8 jobs on mistral-tempest-plugin, we need to fix that asap09:31
rakhmerovaah09:32
*** bobh has quit IRC09:32
*** bobh has joined #openstack-mistral09:34
*** bobh has quit IRC09:39
rakhmerovd0ugal: who is the best person who can do that?09:42
d0ugalrakhmerov: CI?09:42
rakhmerovmaybe thrash|g0ne could help?09:42
rakhmerovyes09:42
d0ugalgood question :)09:42
d0ugalI was going to ask thrash|g0ne, he seems to know it best.09:42
rakhmerovseems like he's heavily involved into this09:43
rakhmerovyes09:43
d0ugalhaha09:43
rakhmerov:)09:43
rakhmerovdeal!09:43
d0ugalI'm not sure he wants to be heavily involved, but he is :-D09:43
d0ugalthanks thrash|g0ne!09:43
rakhmerovyeah :)09:43
openstackgerritMerged openstack/mistral-tempest-plugin master: Allow filtering executions by their root_execution_id  https://review.openstack.org/52764009:45
openstackgerritMerged openstack/mistral-tempest-plugin master: Disable unstable tempest multi_vim_authentication test  https://review.openstack.org/52764109:45
openstackgerritMerged openstack/mistral-tempest-plugin master: Disable unstable tempest test_create_action_execution_sync test  https://review.openstack.org/52764209:45
openstackgerritMerged openstack/mistral-tempest-plugin master: Re-work the direct action call tempest test  https://review.openstack.org/52764309:51
*** yangyapeng has quit IRC10:06
*** yangyapeng has joined #openstack-mistral10:07
therved0ugal, You jinxed it10:09
*** yangyapeng has quit IRC10:09
*** bobh has joined #openstack-mistral10:11
d0ugaluh oh10:14
d0ugaldamn, the non-apache is usually the most reliable10:15
*** bobh has quit IRC10:16
rakhmerov:)10:21
d0ugalhttp://logs.openstack.org/48/527348/2/gate/mistral-devstack-dsvm-non-apache/c342f83/logs/screen-mistral-executor.txt.gz#_Dec_13_08_35_02_58975310:23
d0ugalI wonder if adding a retry to that test would be good, I suspect ssh could fail randomly for no reason.10:23
*** katkapilatova has joined #openstack-mistral10:23
*** bobh has joined #openstack-mistral10:26
*** bobh has quit IRC10:30
rakhmerovyeah, we tried to investigate earlier why it happens but with no result10:35
*** bobh has joined #openstack-mistral10:37
*** zhurong has quit IRC10:37
*** bobh has quit IRC10:41
*** bobh has joined #openstack-mistral10:57
d0ugalrakhmerov: do you have a way to ping Nikolay about https://review.openstack.org/#/c/506185/?11:00
rakhmerovyes11:01
rakhmerovsecond..11:01
d0ugalThanks.11:01
rakhmerovdone11:01
rakhmerovAndras is on vacation11:01
rakhmerovfor about 2 weeks11:01
*** bobh has quit IRC11:02
d0ugalGood to know, thanks.11:02
*** bobh has joined #openstack-mistral11:05
openstackgerritMerged openstack/mistral master: Log a warning log message if the task isn't found  https://review.openstack.org/52635911:08
*** bobh has quit IRC11:10
openstackgerritRenat Akhmerov proposed openstack/mistral master: Fix inconsistencies when setting policy values  https://review.openstack.org/51864111:13
*** bobh has joined #openstack-mistral11:19
*** bobh has quit IRC11:23
*** jkilpatr has quit IRC11:23
openstackgerritMerged openstack/mistral master: Modify log infomation to achieve the same format  https://review.openstack.org/52001611:24
openstackgerritRenat Akhmerov proposed openstack/mistral master: Cache action definitions  https://review.openstack.org/52021811:25
*** bobh has joined #openstack-mistral11:27
rakhmerovd0ugal: I reviewed Mike's patch https://review.openstack.org/#/c/520218/ and replied to your concern11:29
rakhmerovplease have another look11:30
rakhmerovI rebased it, it's now on CI11:30
d0ugalrakhmerov: okay, will do11:30
d0ugaloh, I didn't notice it was load_11:31
rakhmerovyep11:31
d0ugalwe should have a better name to make the difference more obvious :)11:31
rakhmerovI actually borrowed naming from Hibernate :)11:31
*** bobh has quit IRC11:31
d0ugaloh, I have never used that11:31
rakhmerovit's in Java11:31
d0ugalYeah, I've heard of it11:32
rakhmerovyep, very powerful ORM11:32
d0ugalwith confusing method names? :P11:32
rakhmerov:))11:32
rakhmerovsometimes if we are straightforward with naming then names become too long11:33
rakhmerovso we have to make trade-offs between clearness and conciseness11:33
d0ugalSure11:34
*** bobh has joined #openstack-mistral11:54
*** jkilpatr has joined #openstack-mistral11:54
*** bobh has quit IRC12:00
*** bobh has joined #openstack-mistral12:08
*** bobh has quit IRC12:12
*** bobh has joined #openstack-mistral12:27
*** dprince has joined #openstack-mistral13:09
*** thrash|g0ne is now known as thrash13:09
*** bobh has quit IRC13:14
*** dprince has quit IRC13:15
*** dprince has joined #openstack-mistral13:16
openstackgerritBrad P. Crochet proposed openstack/mistral master: Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942813:24
d0ugalthrash: hey, did you see we signed you up for a new task? :)13:25
thrashd0ugal: no... What is it?13:25
d0ugalthrash: we need CI jobs for mistral-tempest-plugin13:25
thrashd0ugal: ha... Joke's on you.. I was planning on doing that!13:26
thrash:P13:26
d0ugalyay13:26
d0ugalat the moment it only does PEP813:26
d0ugaland if we make any changes we wont know if they break anything until they merge and a mistral change runs CI13:26
d0ugalmaybe a depends-on could be used for now...13:26
thrashd0ugal: yeah... I got it.13:26
thrash:D13:26
thrashactually already started the split.13:26
d0ugalthrash: great.13:27
d0ugalI cherry-picked the missing changes over to the new repo, other than one that is close to merging I'll start to -2 tempest changes in mistral13:27
*** rbrady has joined #openstack-mistral13:34
*** rbrady has joined #openstack-mistral13:34
*** dprince has quit IRC13:37
*** dprince has joined #openstack-mistral13:46
*** dprince has quit IRC14:02
*** dprince has joined #openstack-mistral14:36
*** openstack has quit IRC14:39
*** openstack has joined #openstack-mistral14:41
*** ChanServ sets mode: +o openstack14:41
*** openstack has quit IRC14:43
*** openstack has joined #openstack-mistral14:46
*** ChanServ sets mode: +o openstack14:46
*** katkapilatova has quit IRC14:52
openstackgerritToure Dunnon proposed openstack/mistral master: Running new workflow based on an existing execution.  https://review.openstack.org/50665214:55
*** bobh has joined #openstack-mistral15:00
openstackgerritMerged openstack/mistral master: Clear error info  https://review.openstack.org/52672015:03
openstackgerritBrad P. Crochet proposed openstack/mistral master: Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942815:19
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Add zuul jobs for mistral-tempest-plugin  https://review.openstack.org/52771315:23
openstackgerritBrad P. Crochet proposed openstack/mistral master: Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942815:23
thrashd0ugal: I *think* that should do it.15:23
thrashWe shall see.15:23
d0ugalthrash: nice!15:24
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Add zuul jobs for mistral-tempest-plugin  https://review.openstack.org/52771315:24
openstackgerritBrad P. Crochet proposed openstack/mistral master: Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942815:24
thrashOk. That :P15:24
d0ugallol15:24
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Add zuul jobs for mistral-tempest-plugin  https://review.openstack.org/52771315:26
*** dprince has quit IRC15:29
openstackgerritAdriano Petrich proposed openstack/mistral master: Hide env from logging  https://review.openstack.org/49766315:30
*** dprince has joined #openstack-mistral15:32
*** jtomasek has quit IRC15:47
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Add zuul jobs for mistral-tempest-plugin  https://review.openstack.org/52771315:48
*** bobh has quit IRC15:48
openstackgerritBrad P. Crochet proposed openstack/mistral master: Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942815:52
openstackgerritBrad P. Crochet proposed openstack/mistral master: Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942815:58
*** dprince has quit IRC16:01
*** dprince has joined #openstack-mistral16:01
*** jtomasek has joined #openstack-mistral16:03
openstackgerritMerged openstack/mistral master: Pass the new ActionContext to mistral-lib  https://review.openstack.org/50618516:21
*** dprince has quit IRC16:30
*** rbrady has quit IRC16:31
*** AlexeyAbashkin has quit IRC16:47
d0ugalYAY! IT MERGED!17:00
*** jkilpatr has quit IRC17:07
*** yangyapeng has joined #openstack-mistral17:09
*** yangyapeng has quit IRC17:13
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Add zuul jobs for mistral-tempest-plugin  https://review.openstack.org/52771317:15
*** jpich has quit IRC17:15
d0ugalthrash: I lol'ed at your TODO :)17:16
thrashd0ugal: :D17:16
d0ugalthrash: does something look for the pre.yaml?17:19
d0ugalI mean, is it just ran based on the naming convention?17:19
thrashah crap17:19
thrashno. apparently that change didn't make it. :P17:19
thrashone sec.17:19
d0ugallol17:19
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Add zuul jobs for mistral-tempest-plugin  https://review.openstack.org/52771317:20
d0ugalaha17:20
d0ugalthrash: What is the deal with required-projects, should mistral-lib be inlcuded there?17:21
thrashd0ugal: perhaps?17:21
d0ugal:)17:21
thrashyeah... probably.. That way, any changes that are depends-on would get pulled in.17:22
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Add zuul jobs for mistral-tempest-plugin  https://review.openstack.org/52771317:22
*** thrash is now known as thrash|biab17:23
*** jkilpatr has joined #openstack-mistral17:23
*** yangyapeng has joined #openstack-mistral17:24
*** yangyapeng has quit IRC17:28
*** bobh has joined #openstack-mistral17:30
*** bobh has quit IRC17:31
*** bobh has joined #openstack-mistral17:31
*** jkilpatr has quit IRC17:38
*** jkilpatr has joined #openstack-mistral17:52
openstackgerritMerged openstack/mistral master: Fix inconsistencies when setting policy values  https://review.openstack.org/51864117:54
openstackgerritMerged openstack/mistral master: Use the new action context in MistralHTTPAction  https://review.openstack.org/52034817:54
*** yangyapeng has joined #openstack-mistral18:04
*** yangyapeng has quit IRC18:09
*** jtomasek has quit IRC18:15
*** AlexeyAbashkin has joined #openstack-mistral18:17
*** AlexeyAbashkin has quit IRC18:21
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Add zuul jobs for mistral-tempest-plugin  https://review.openstack.org/52771318:36
*** dprince has joined #openstack-mistral18:41
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Add zuul jobs for mistral-tempest-plugin  https://review.openstack.org/52771319:31
*** thrash|biab is now known as thrash19:56
thrashd0ugal: you around?19:56
*** jkilpatr has quit IRC20:03
*** jkilpatr has joined #openstack-mistral20:39
*** dprince has quit IRC21:10
*** dprince has joined #openstack-mistral21:11
*** threestrands has joined #openstack-mistral21:18
*** threestrands has quit IRC21:18
*** threestrands has joined #openstack-mistral21:18
*** bobh has quit IRC21:47
*** bobh has joined #openstack-mistral21:52
*** AlexeyAbashkin has joined #openstack-mistral21:54
*** bobh has quit IRC21:57
*** apetrich has quit IRC21:58
*** AlexeyAbashkin has quit IRC21:58
*** apetrich has joined #openstack-mistral21:59
*** bobh has joined #openstack-mistral22:04
*** bobh has quit IRC22:08
*** bobh has joined #openstack-mistral22:17
*** dprince has quit IRC22:20
d0ugalthrash: not really :)22:31
openstackgerritDougal Matthews proposed openstack/mistral master: Remove the __init__ method from the test action  https://review.openstack.org/51306122:33
openstackgerritDougal Matthews proposed openstack/mistral master: Remove all class methods in tempest tests  https://review.openstack.org/52734822:34
*** harlowja has quit IRC22:54
*** harlowja has joined #openstack-mistral23:42
*** bobh has quit IRC23:44

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