Tuesday, 2018-01-23

*** mattybre_ has quit IRC00:00
*** bobh has quit IRC00:04
*** bobh has joined #openstack-mistral00:12
*** AlexeyAbashkin has joined #openstack-mistral00:16
*** AlexeyAbashkin has quit IRC00:21
*** yangyapeng has quit IRC00:26
*** thrash is now known as thrash|g0ne00:27
*** jkilpatr has quit IRC00:28
*** daidv has quit IRC00:39
*** daidv has joined #openstack-mistral00:39
*** zhurong has joined #openstack-mistral00:48
*** d0ugal has quit IRC00:55
*** d0ugal has joined #openstack-mistral01:00
*** zhurong has quit IRC01:13
*** yangyapeng has joined #openstack-mistral01:13
*** bobh has quit IRC01:21
*** bobh has joined #openstack-mistral01:42
*** mattybrennan has joined #openstack-mistral02:00
*** yangyapeng has quit IRC02:08
*** yangyapeng has joined #openstack-mistral02:13
*** rbrady has quit IRC02:18
*** harlowja has quit IRC02:27
*** rbrady has joined #openstack-mistral02:33
*** rbrady has quit IRC02:33
*** rbrady has joined #openstack-mistral02:33
*** mattybrennan has quit IRC02:46
*** kushalagrawal has quit IRC02:49
*** kushalagrawal has joined #openstack-mistral02:51
*** openstackgerrit has joined #openstack-mistral03:04
openstackgerritchen jiao proposed openstack/mistral master: change import order  https://review.openstack.org/53530403:04
*** daidv has quit IRC03:30
*** hardikjasani has joined #openstack-mistral03:34
*** daidv has joined #openstack-mistral03:44
*** kushalagrawal has quit IRC04:16
*** kushalagrawal has joined #openstack-mistral04:17
*** bobh has quit IRC04:21
*** daidv has quit IRC04:28
*** daidv has joined #openstack-mistral04:28
rakhmerovapetrich: yeah :)04:36
openstackgerritRenat Akhmerov proposed openstack/mistral master: Fixed integration of the unit tests with PosrgeSQL  https://review.openstack.org/53186104:37
rakhmerovthrash|g0ne: thanks!04:37
rakhmerovthrash|g0ne: the first one looks good. But if you look at the tail of http://logs.openstack.org/13/527713/26/check/mistral-devstack/66460d1/job-output.txt.gz you'll see huge amount of json printings. Any idea where this comes from? If we look at the corresponding log in 'mistral' we don't see that04:41
rakhmerovbut the rest looks fine, tests run etc.04:41
*** harlowja has joined #openstack-mistral05:12
*** harlowja has quit IRC05:13
openstackgerritMerged openstack/mistral-tempest-plugin master: Add zuul jobs for mistral-tempest-plugin  https://review.openstack.org/52771305:26
openstackgerritMerged openstack/mistral-tempest-plugin master: Updated from global requirements  https://review.openstack.org/53631605:26
rakhmerovtoure: is there a chance you can finish https://blueprints.launchpad.net/mistral/+spec/mistral-run-workflow-from-execution this week?05:27
rakhmerovI'm still hoping :)05:27
rakhmerovI think you're very very close )05:27
*** zhurong has joined #openstack-mistral05:29
*** kushalagrawal has quit IRC05:36
*** kushalagrawal has joined #openstack-mistral05:37
openstackgerritRenat Akhmerov proposed openstack/mistral master: task name can not be reserved keyword  https://review.openstack.org/53529705:42
*** jaosorior has quit IRC05:57
*** jaosorior has joined #openstack-mistral05:57
*** kushalagrawal has quit IRC06:21
*** kushalagrawal has joined #openstack-mistral06:22
rakhmerovapetrich: hi Adriano, would you please review https://review.openstack.org/#/c/528540/ ?06:27
*** jtomasek has joined #openstack-mistral06:40
apetrichrakhmerov, done06:45
rakhmerovapetrich: thanks06:51
openstackgerritHardik Jasani proposed openstack/mistral master: Fix std.http action doc  https://review.openstack.org/53668906:51
mcdoker181818rakhmerov: Hi, can you merge this patch https://review.openstack.org/#/c/536256/ to the pike branch?06:56
openstackgerritRenat Akhmerov proposed openstack/mistral stable/pike: Added session.flush() before update_on_match()  https://review.openstack.org/53669206:57
rakhmerovmcdoker181818: https://review.openstack.org/#/c/53669206:57
rakhmerovhi :)06:57
mcdoker181818thank you06:58
*** kushalagrawal has quit IRC07:02
*** kushalagrawal has joined #openstack-mistral07:03
*** jrist has quit IRC07:15
openstackgerritRenat Akhmerov proposed openstack/mistral master: Fixed the length of a task name  https://review.openstack.org/52849407:35
*** mattybrennan has joined #openstack-mistral07:47
*** mattybrennan has quit IRC07:51
*** jrist has joined #openstack-mistral07:53
*** kushalagrawal has quit IRC08:03
*** kushalagrawal has joined #openstack-mistral08:04
*** AlexeyAbashkin has joined #openstack-mistral08:05
*** kushalagrawal has quit IRC08:22
*** kushalagrawal has joined #openstack-mistral08:22
*** daidv has quit IRC08:38
*** daidv has joined #openstack-mistral08:38
rakhmerovmcdoker181818: seems like CI is broken again08:43
rakhmerovall devstack jobs don't pass08:43
*** oidgar has joined #openstack-mistral08:44
rakhmerovthat's where the problem is: http://logs.openstack.org/61/531861/6/gate/mistral-devstack-dsvm/341f171/job-output.txt.gz#_2018-01-23_08_21_52_08092808:48
rakhmerovI wonder why it stopped working..08:48
rakhmerovapetrich: hi, does it look any familiar to you? By any chance..08:48
*** oidgar has quit IRC08:52
apetrichrakhmerov, wait a second I think I saw something ssh related08:52
rakhmerovmaybe it was ssh_proxied08:52
rakhmerovthat one is disabled for now08:52
rakhmerovbecause it wasn't stable08:52
rakhmerovthis looks a lot like it runs in a different env now08:53
rakhmerovbecause essentially the key file is not accessible08:53
rakhmerovapetrich: ^08:54
apetrichthere was a change to ssh that if it started with "/" it would use the full directory instead of prepending it with some variable08:55
*** shardy has joined #openstack-mistral08:56
apetrichhttps://review.openstack.org/#/c/534744/2/mistral/utils/ssh_utils.py08:57
apetrichcould that be related?08:57
rakhmerovoooh!08:57
rakhmerovmay be, yes08:57
*** jpich has joined #openstack-mistral09:03
rakhmerovapetrich: in this line, we use username 'cirros' but it tries to use a key living under /root/.ssh/ folder09:06
rakhmerovstrange09:06
openstackgerritchen jiao proposed openstack/mistral master: modify the import order  https://review.openstack.org/53672009:07
rakhmerovthrash|g0ne, apetrich: would you be able to look at it today?09:08
apetrichit is very strange09:12
rakhmerovhttps://review.openstack.org/#/c/527713/26/mistral_tempest_tests/tests/scenario/engine/actions/v2/test_ssh_actions.py09:14
rakhmerovit may be related to this change which was just merged today09:15
rakhmerovapetrich: ^09:15
rakhmerovthese two things almost coincide in time09:15
apetrichor lack of promotion09:15
rakhmerovpromotion?09:16
rakhmerovwhat do you mean?09:16
apetrichthat needs the ssh change but mistral doesn't have it09:16
apetrichlet me just check09:16
rakhmerovok09:16
apetrichthe integration test that moves the master code into rpms for tripleo-ci09:17
apetrichthat promotion09:17
apetrichbut yeah removing the self.key_dir = "/opt/stack/new/.ssh/" might be the issue09:20
apetrichon that patch09:20
apetrichrakhmerov, ^09:20
rakhmerovyep09:20
rakhmerovseems like that's the issue09:20
rakhmerovit just tries to find it at a wrong place09:21
apetrichshould we revert to keep the gates going for now?09:24
*** oidgar has joined #openstack-mistral09:25
rakhmerovI think so09:26
rakhmerovthere's a lot of patches waiting to be merged09:27
rakhmerovlet me do that, yes09:27
rakhmerovI'll ask Brad to work further on this patch09:27
openstackgerritRenat Akhmerov proposed openstack/mistral-tempest-plugin master: Revert "Add zuul jobs for mistral-tempest-plugin"  https://review.openstack.org/53673109:27
*** jaosorior has quit IRC09:55
*** kushalagrawal has quit IRC10:01
*** kushalagrawal has joined #openstack-mistral10:01
*** zhurong has quit IRC10:11
*** zhurong has joined #openstack-mistral10:31
*** shardy has quit IRC10:49
*** shardy has joined #openstack-mistral10:51
*** jaosorior has joined #openstack-mistral10:53
openstackgerritMerged openstack/mistral stable/pike: Added session.flush() before update_on_match()  https://review.openstack.org/53669211:00
openstackgerritMerged openstack/mistral-tempest-plugin master: Revert "Add zuul jobs for mistral-tempest-plugin"  https://review.openstack.org/53673111:00
*** AlexeyAbashkin has quit IRC11:06
*** AlexeyAbashkin has joined #openstack-mistral11:23
*** kushalagrawal has quit IRC11:42
*** kushalagrawal has joined #openstack-mistral11:42
*** zhurong has quit IRC12:05
*** jkilpatr has joined #openstack-mistral12:12
*** kushalagrawal has quit IRC12:40
*** kushalagrawal has joined #openstack-mistral12:40
*** yangyapeng has quit IRC12:50
*** yangyapeng has joined #openstack-mistral12:51
*** yangyapeng has quit IRC12:55
*** zhurong has joined #openstack-mistral12:57
*** thrash|g0ne is now known as thrash13:05
*** zhurong has quit IRC13:14
*** kushalagrawal has quit IRC13:14
*** kushalagrawal has joined #openstack-mistral13:15
openstackgerritRenat Akhmerov proposed openstack/mistral master: Fix break_on calculation in before_task_start  https://review.openstack.org/52854013:21
*** hardikjasani has quit IRC13:22
rakhmerovthrash: hi Brad, can you please look at the discussion here?13:24
rakhmerovwith apetrich13:24
rakhmerovseems like the latest change you made in test_ssh_action broke all devstack CI jobs13:24
thrashWhoa.13:25
rakhmerovI reverted the patch for now13:25
rakhmerovyeah13:25
thrashALL?13:25
apetrichthrash, long story short gates are falling here http://logs.openstack.org/61/531861/6/gate/mistral-devstack-dsvm/341f171/job-output.txt.gz#_2018-01-23_08_21_52_08092813:25
rakhmerovyep13:25
rakhmerovall devstack jobs yes13:25
rakhmerovthree13:25
thrashOr you mean all of *mistral's* devstack jobs.13:25
rakhmerovmistral, yes :)13:25
thrashrakhmerov: that's actually not surprising.13:25
rakhmerovit tries to find the ssh key at a wrong place13:25
apetrichthrash, that's more dramatic13:25
rakhmerovunder /root/.ssh13:26
apetrichI think it might be the missing  self.key_dir = "/opt/stack/new/.ssh/" from the patch13:26
rakhmerovand, of course, has no permission for it13:26
apetrichbecause it is user cirrus13:26
thrashrakhmerov: apetrich I can explain that very easily13:26
rakhmerovyes, it was removed in the last patch13:26
rakhmerovexplain what?13:26
*** yangyapeng has joined #openstack-mistral13:26
rakhmerovok, go ahead13:26
thrashrakhmerov: this patch would fix that. :D13:27
thrashhttps://review.openstack.org/#/c/509428/13:27
apetrichoh13:27
thrashrakhmerov: it would move the jobs to the new devstack base jobs.13:27
thrashlooks like I need to make a small change to it though.13:27
rakhmerovaaah!13:27
rakhmerovanyways, I rechecked the failing patches now after revert13:28
rakhmerovcan you please resent it with whatever changes needed?13:28
rakhmerovand what should we do with ?13:28
rakhmerovAndreas still votes -1 on it13:28
thrashrakhmerov: that's the small change. :D13:29
*** rbrady has quit IRC13:29
thrashI'll figure out the back-compatible part.13:29
thrashMerde13:29
rakhmerovyes, please13:29
rakhmerovbut what is it all about?13:29
thrashactually, I know exactly what to do with it.13:29
rakhmerovit's supposed to run on a different environment/os?13:30
thrashI'll mark it as skipped in the original patch based on a bug13:30
rakhmerovok13:30
thrashand then in the mistral patch, I'll mark it as closing that bug.13:30
thrashthat should do it.13:30
rakhmerovokay13:30
thrashAnd move the code into the mistral patch.13:30
thrashKeep the old code in tact in the initial patch.13:31
rakhmerovbut if you still could quickly explain what we're doing there... :))13:31
rakhmerovit would be helpful )13:31
thrashrakhmerov: Sorry... which part?13:31
rakhmerovthe whole idea13:31
thrashrakhmerov: This is to move the mistral jobs away from 'legacy' jobs to the new zuul v3 format13:32
apetrichcool. the lack of promotion is the pain now.13:32
rakhmerovand why it's failing now (before the revert)13:32
thrashand the reason it's failing now, is that the devstack zuulv3 base job functions differently than the legacy devstack job.13:32
rakhmerovthrash: and why was it failing? Because it should run on a different env?13:32
rakhmerovok13:33
thrashrakhmerov: yeah. It's not being run as root for one thing.13:33
thrashrakhmerov: which, obviously is a good thing. :)13:33
rakhmerovsure13:33
rakhmerovyes13:33
rakhmerovok, I more or less understand now13:33
rakhmerovapetrich: what about promotion?13:33
thrashrakhmerov: I'll rework it so I don't break the world. :D13:34
thrashI'll make sure I add a patch that tests the existing jobs.13:34
apetrichwith the changes if we gate mistral we just rebase and have the code but other projects cannot use it until it has been promoted to current-passed-ci13:35
*** rbrady has joined #openstack-mistral13:42
*** dprince has joined #openstack-mistral13:46
*** mattybrennan has joined #openstack-mistral13:48
*** mattybrennan has quit IRC13:56
*** mattybrennan has joined #openstack-mistral13:57
*** mattybrennan has quit IRC14:01
*** mattybrennan has joined #openstack-mistral14:24
*** mattybrennan has quit IRC14:28
*** mattybrennan has joined #openstack-mistral14:33
*** mattybre_ has joined #openstack-mistral14:34
*** mattybrennan has quit IRC14:38
*** thrash is now known as thrash|biab15:02
*** mattybre_ has quit IRC15:06
*** mattybrennan has joined #openstack-mistral15:08
*** shardy is now known as shardy_afk15:10
*** bobh has joined #openstack-mistral15:16
*** oidgar has quit IRC15:23
*** thrash|biab is now known as thrash15:34
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Revert "Revert "Add zuul jobs for mistral-tempest-plugin""  https://review.openstack.org/53686615:35
*** shardy_afk is now known as shardy15:46
*** mattybrennan has quit IRC15:54
*** mattybrennan has joined #openstack-mistral16:01
*** openstackgerrit has quit IRC16:03
*** openstackgerrit has joined #openstack-mistral16:19
openstackgerritBrad P. Crochet proposed openstack/mistral master: Remove any old client actions that no longer exist  https://review.openstack.org/53688616:19
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Revert "Revert "Add zuul jobs for mistral-tempest-plugin""  https://review.openstack.org/53686616:21
*** oidgar has joined #openstack-mistral16:21
tourerakhmerov ping16:23
tourerakhmerov in regards to the ping earlier, I am refactoring it now16:23
toureshould be ready to review in a couple hours16:23
*** mattybrennan has quit IRC16:26
*** mattybrennan has joined #openstack-mistral16:28
*** mattybrennan has quit IRC16:28
openstackgerritBrad P. Crochet proposed openstack/mistral master: TEST: DNM Test the new zuul jobs don't break existing  https://review.openstack.org/53689016:28
*** mattybrennan has joined #openstack-mistral16:29
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Simplify the ssh tests and where it gets it's key from  https://review.openstack.org/53689216:32
openstackgerritBrad P. Crochet proposed openstack/mistral master: Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942816:33
*** jkilpatr_ has joined #openstack-mistral16:58
*** jkilpatr has quit IRC17:00
*** oidgar has quit IRC17:01
*** AlexeyAbashkin has quit IRC17:02
*** kushalagrawal has quit IRC17:04
*** kushalagrawal has joined #openstack-mistral17:04
*** mattybre_ has joined #openstack-mistral17:10
*** mattybrennan has quit IRC17:10
*** jkilpatr_ has quit IRC17:12
*** kushalagrawal has quit IRC17:23
*** kushalagrawal has joined #openstack-mistral17:25
*** mattybre_ has quit IRC17:30
*** mattybrennan has joined #openstack-mistral17:31
*** jpich has quit IRC17:39
*** kushalagrawal has quit IRC17:57
*** oidgar has joined #openstack-mistral17:57
*** kushalagrawal has joined #openstack-mistral17:57
*** mattybrennan has quit IRC17:59
*** mattybrennan has joined #openstack-mistral18:04
*** thrash is now known as thrash|biab18:04
openstackgerritMerged openstack/mistral master: task name can not be reserved keyword  https://review.openstack.org/53529718:13
*** oidgar has quit IRC18:18
*** jkilpatr has joined #openstack-mistral18:20
*** shardy has quit IRC18:22
*** AlexeyAbashkin has joined #openstack-mistral18:31
*** AlexeyAbashkin has quit IRC18:35
*** thrash|biab is now known as thrash18:36
openstackgerritToure Dunnon proposed openstack/mistral master: Running new workflow based on an existing execution.  https://review.openstack.org/50665218:43
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Revert "Revert "Add zuul jobs for mistral-tempest-plugin""  https://review.openstack.org/53686618:56
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Simplify the ssh tests and where it gets it's key from  https://review.openstack.org/53689218:57
*** kushalagrawal has quit IRC18:57
*** kushalagrawal has joined #openstack-mistral18:58
*** jkilpatr has quit IRC18:59
*** mattybrennan has quit IRC19:00
*** bobh has quit IRC19:02
*** mattybrennan has joined #openstack-mistral19:03
*** bobh has joined #openstack-mistral19:10
*** harlowja has joined #openstack-mistral19:34
*** kushalagrawal has quit IRC19:43
*** kushalagrawal has joined #openstack-mistral19:45
*** mattybrennan has quit IRC19:53
*** mattybrennan has joined #openstack-mistral19:54
*** bobh has quit IRC19:55
*** kushalagrawal has quit IRC20:09
*** kushalagrawal has joined #openstack-mistral20:15
*** bobh has joined #openstack-mistral20:26
*** mattybrennan has quit IRC20:28
*** bobh has quit IRC20:31
*** mattybrennan has joined #openstack-mistral20:32
*** mattybrennan has quit IRC20:32
*** mattybrennan has joined #openstack-mistral20:33
*** jkilpatr has joined #openstack-mistral20:39
*** bobh has joined #openstack-mistral21:03
*** mattybrennan has quit IRC21:24
*** dprince has quit IRC21:38
openstackgerritBrad P. Crochet proposed openstack/mistral-tempest-plugin master: Revert "Revert "Add zuul jobs for mistral-tempest-plugin""  https://review.openstack.org/53686621:47
openstackgerritBrad P. Crochet proposed openstack/mistral master: Remove any old client actions that no longer exist  https://review.openstack.org/53688622:06
openstackgerritMerged openstack/mistral master: Fix std.http action doc  https://review.openstack.org/53668922:18
*** kushalagrawal has quit IRC23:10
*** kushalagrawal has joined #openstack-mistral23:10
*** toure is now known as toure|gone23:17
*** bobh has quit IRC23:51

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