*** mattybre_ has quit IRC | 00:00 | |
*** bobh has quit IRC | 00:04 | |
*** bobh has joined #openstack-mistral | 00:12 | |
*** AlexeyAbashkin has joined #openstack-mistral | 00:16 | |
*** AlexeyAbashkin has quit IRC | 00:21 | |
*** yangyapeng has quit IRC | 00:26 | |
*** thrash is now known as thrash|g0ne | 00:27 | |
*** jkilpatr has quit IRC | 00:28 | |
*** daidv has quit IRC | 00:39 | |
*** daidv has joined #openstack-mistral | 00:39 | |
*** zhurong has joined #openstack-mistral | 00:48 | |
*** d0ugal has quit IRC | 00:55 | |
*** d0ugal has joined #openstack-mistral | 01:00 | |
*** zhurong has quit IRC | 01:13 | |
*** yangyapeng has joined #openstack-mistral | 01:13 | |
*** bobh has quit IRC | 01:21 | |
*** bobh has joined #openstack-mistral | 01:42 | |
*** mattybrennan has joined #openstack-mistral | 02:00 | |
*** yangyapeng has quit IRC | 02:08 | |
*** yangyapeng has joined #openstack-mistral | 02:13 | |
*** rbrady has quit IRC | 02:18 | |
*** harlowja has quit IRC | 02:27 | |
*** rbrady has joined #openstack-mistral | 02:33 | |
*** rbrady has quit IRC | 02:33 | |
*** rbrady has joined #openstack-mistral | 02:33 | |
*** mattybrennan has quit IRC | 02:46 | |
*** kushalagrawal has quit IRC | 02:49 | |
*** kushalagrawal has joined #openstack-mistral | 02:51 | |
*** openstackgerrit has joined #openstack-mistral | 03:04 | |
openstackgerrit | chen jiao proposed openstack/mistral master: change import order https://review.openstack.org/535304 | 03:04 |
---|---|---|
*** daidv has quit IRC | 03:30 | |
*** hardikjasani has joined #openstack-mistral | 03:34 | |
*** daidv has joined #openstack-mistral | 03:44 | |
*** kushalagrawal has quit IRC | 04:16 | |
*** kushalagrawal has joined #openstack-mistral | 04:17 | |
*** bobh has quit IRC | 04:21 | |
*** daidv has quit IRC | 04:28 | |
*** daidv has joined #openstack-mistral | 04:28 | |
rakhmerov | apetrich: yeah :) | 04:36 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Fixed integration of the unit tests with PosrgeSQL https://review.openstack.org/531861 | 04:37 |
rakhmerov | thrash|g0ne: thanks! | 04:37 |
rakhmerov | thrash|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 that | 04:41 |
rakhmerov | but the rest looks fine, tests run etc. | 04:41 |
*** harlowja has joined #openstack-mistral | 05:12 | |
*** harlowja has quit IRC | 05:13 | |
openstackgerrit | Merged openstack/mistral-tempest-plugin master: Add zuul jobs for mistral-tempest-plugin https://review.openstack.org/527713 | 05:26 |
openstackgerrit | Merged openstack/mistral-tempest-plugin master: Updated from global requirements https://review.openstack.org/536316 | 05:26 |
rakhmerov | toure: is there a chance you can finish https://blueprints.launchpad.net/mistral/+spec/mistral-run-workflow-from-execution this week? | 05:27 |
rakhmerov | I'm still hoping :) | 05:27 |
rakhmerov | I think you're very very close ) | 05:27 |
*** zhurong has joined #openstack-mistral | 05:29 | |
*** kushalagrawal has quit IRC | 05:36 | |
*** kushalagrawal has joined #openstack-mistral | 05:37 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: task name can not be reserved keyword https://review.openstack.org/535297 | 05:42 |
*** jaosorior has quit IRC | 05:57 | |
*** jaosorior has joined #openstack-mistral | 05:57 | |
*** kushalagrawal has quit IRC | 06:21 | |
*** kushalagrawal has joined #openstack-mistral | 06:22 | |
rakhmerov | apetrich: hi Adriano, would you please review https://review.openstack.org/#/c/528540/ ? | 06:27 |
*** jtomasek has joined #openstack-mistral | 06:40 | |
apetrich | rakhmerov, done | 06:45 |
rakhmerov | apetrich: thanks | 06:51 |
openstackgerrit | Hardik Jasani proposed openstack/mistral master: Fix std.http action doc https://review.openstack.org/536689 | 06:51 |
mcdoker181818 | rakhmerov: Hi, can you merge this patch https://review.openstack.org/#/c/536256/ to the pike branch? | 06:56 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral stable/pike: Added session.flush() before update_on_match() https://review.openstack.org/536692 | 06:57 |
rakhmerov | mcdoker181818: https://review.openstack.org/#/c/536692 | 06:57 |
rakhmerov | hi :) | 06:57 |
mcdoker181818 | thank you | 06:58 |
*** kushalagrawal has quit IRC | 07:02 | |
*** kushalagrawal has joined #openstack-mistral | 07:03 | |
*** jrist has quit IRC | 07:15 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Fixed the length of a task name https://review.openstack.org/528494 | 07:35 |
*** mattybrennan has joined #openstack-mistral | 07:47 | |
*** mattybrennan has quit IRC | 07:51 | |
*** jrist has joined #openstack-mistral | 07:53 | |
*** kushalagrawal has quit IRC | 08:03 | |
*** kushalagrawal has joined #openstack-mistral | 08:04 | |
*** AlexeyAbashkin has joined #openstack-mistral | 08:05 | |
*** kushalagrawal has quit IRC | 08:22 | |
*** kushalagrawal has joined #openstack-mistral | 08:22 | |
*** daidv has quit IRC | 08:38 | |
*** daidv has joined #openstack-mistral | 08:38 | |
rakhmerov | mcdoker181818: seems like CI is broken again | 08:43 |
rakhmerov | all devstack jobs don't pass | 08:43 |
*** oidgar has joined #openstack-mistral | 08:44 | |
rakhmerov | that'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_080928 | 08:48 |
rakhmerov | I wonder why it stopped working.. | 08:48 |
rakhmerov | apetrich: hi, does it look any familiar to you? By any chance.. | 08:48 |
*** oidgar has quit IRC | 08:52 | |
apetrich | rakhmerov, wait a second I think I saw something ssh related | 08:52 |
rakhmerov | maybe it was ssh_proxied | 08:52 |
rakhmerov | that one is disabled for now | 08:52 |
rakhmerov | because it wasn't stable | 08:52 |
rakhmerov | this looks a lot like it runs in a different env now | 08:53 |
rakhmerov | because essentially the key file is not accessible | 08:53 |
rakhmerov | apetrich: ^ | 08:54 |
apetrich | there was a change to ssh that if it started with "/" it would use the full directory instead of prepending it with some variable | 08:55 |
*** shardy has joined #openstack-mistral | 08:56 | |
apetrich | https://review.openstack.org/#/c/534744/2/mistral/utils/ssh_utils.py | 08:57 |
apetrich | could that be related? | 08:57 |
rakhmerov | oooh! | 08:57 |
rakhmerov | may be, yes | 08:57 |
*** jpich has joined #openstack-mistral | 09:03 | |
rakhmerov | apetrich: in this line, we use username 'cirros' but it tries to use a key living under /root/.ssh/ folder | 09:06 |
rakhmerov | strange | 09:06 |
openstackgerrit | chen jiao proposed openstack/mistral master: modify the import order https://review.openstack.org/536720 | 09:07 |
rakhmerov | thrash|g0ne, apetrich: would you be able to look at it today? | 09:08 |
apetrich | it is very strange | 09:12 |
rakhmerov | https://review.openstack.org/#/c/527713/26/mistral_tempest_tests/tests/scenario/engine/actions/v2/test_ssh_actions.py | 09:14 |
rakhmerov | it may be related to this change which was just merged today | 09:15 |
rakhmerov | apetrich: ^ | 09:15 |
rakhmerov | these two things almost coincide in time | 09:15 |
apetrich | or lack of promotion | 09:15 |
rakhmerov | promotion? | 09:16 |
rakhmerov | what do you mean? | 09:16 |
apetrich | that needs the ssh change but mistral doesn't have it | 09:16 |
apetrich | let me just check | 09:16 |
rakhmerov | ok | 09:16 |
apetrich | the integration test that moves the master code into rpms for tripleo-ci | 09:17 |
apetrich | that promotion | 09:17 |
apetrich | but yeah removing the self.key_dir = "/opt/stack/new/.ssh/" might be the issue | 09:20 |
apetrich | on that patch | 09:20 |
apetrich | rakhmerov, ^ | 09:20 |
rakhmerov | yep | 09:20 |
rakhmerov | seems like that's the issue | 09:20 |
rakhmerov | it just tries to find it at a wrong place | 09:21 |
apetrich | should we revert to keep the gates going for now? | 09:24 |
*** oidgar has joined #openstack-mistral | 09:25 | |
rakhmerov | I think so | 09:26 |
rakhmerov | there's a lot of patches waiting to be merged | 09:27 |
rakhmerov | let me do that, yes | 09:27 |
rakhmerov | I'll ask Brad to work further on this patch | 09:27 |
openstackgerrit | Renat Akhmerov proposed openstack/mistral-tempest-plugin master: Revert "Add zuul jobs for mistral-tempest-plugin" https://review.openstack.org/536731 | 09:27 |
*** jaosorior has quit IRC | 09:55 | |
*** kushalagrawal has quit IRC | 10:01 | |
*** kushalagrawal has joined #openstack-mistral | 10:01 | |
*** zhurong has quit IRC | 10:11 | |
*** zhurong has joined #openstack-mistral | 10:31 | |
*** shardy has quit IRC | 10:49 | |
*** shardy has joined #openstack-mistral | 10:51 | |
*** jaosorior has joined #openstack-mistral | 10:53 | |
openstackgerrit | Merged openstack/mistral stable/pike: Added session.flush() before update_on_match() https://review.openstack.org/536692 | 11:00 |
openstackgerrit | Merged openstack/mistral-tempest-plugin master: Revert "Add zuul jobs for mistral-tempest-plugin" https://review.openstack.org/536731 | 11:00 |
*** AlexeyAbashkin has quit IRC | 11:06 | |
*** AlexeyAbashkin has joined #openstack-mistral | 11:23 | |
*** kushalagrawal has quit IRC | 11:42 | |
*** kushalagrawal has joined #openstack-mistral | 11:42 | |
*** zhurong has quit IRC | 12:05 | |
*** jkilpatr has joined #openstack-mistral | 12:12 | |
*** kushalagrawal has quit IRC | 12:40 | |
*** kushalagrawal has joined #openstack-mistral | 12:40 | |
*** yangyapeng has quit IRC | 12:50 | |
*** yangyapeng has joined #openstack-mistral | 12:51 | |
*** yangyapeng has quit IRC | 12:55 | |
*** zhurong has joined #openstack-mistral | 12:57 | |
*** thrash|g0ne is now known as thrash | 13:05 | |
*** zhurong has quit IRC | 13:14 | |
*** kushalagrawal has quit IRC | 13:14 | |
*** kushalagrawal has joined #openstack-mistral | 13:15 | |
openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Fix break_on calculation in before_task_start https://review.openstack.org/528540 | 13:21 |
*** hardikjasani has quit IRC | 13:22 | |
rakhmerov | thrash: hi Brad, can you please look at the discussion here? | 13:24 |
rakhmerov | with apetrich | 13:24 |
rakhmerov | seems like the latest change you made in test_ssh_action broke all devstack CI jobs | 13:24 |
thrash | Whoa. | 13:25 |
rakhmerov | I reverted the patch for now | 13:25 |
rakhmerov | yeah | 13:25 |
thrash | ALL? | 13:25 |
apetrich | thrash, 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_080928 | 13:25 |
rakhmerov | yep | 13:25 |
rakhmerov | all devstack jobs yes | 13:25 |
rakhmerov | three | 13:25 |
thrash | Or you mean all of *mistral's* devstack jobs. | 13:25 |
rakhmerov | mistral, yes :) | 13:25 |
thrash | rakhmerov: that's actually not surprising. | 13:25 |
rakhmerov | it tries to find the ssh key at a wrong place | 13:25 |
apetrich | thrash, that's more dramatic | 13:25 |
rakhmerov | under /root/.ssh | 13:26 |
apetrich | I think it might be the missing self.key_dir = "/opt/stack/new/.ssh/" from the patch | 13:26 |
rakhmerov | and, of course, has no permission for it | 13:26 |
apetrich | because it is user cirrus | 13:26 |
thrash | rakhmerov: apetrich I can explain that very easily | 13:26 |
rakhmerov | yes, it was removed in the last patch | 13:26 |
rakhmerov | explain what? | 13:26 |
*** yangyapeng has joined #openstack-mistral | 13:26 | |
rakhmerov | ok, go ahead | 13:26 |
thrash | rakhmerov: this patch would fix that. :D | 13:27 |
thrash | https://review.openstack.org/#/c/509428/ | 13:27 |
apetrich | oh | 13:27 |
thrash | rakhmerov: it would move the jobs to the new devstack base jobs. | 13:27 |
thrash | looks like I need to make a small change to it though. | 13:27 |
rakhmerov | aaah! | 13:27 |
rakhmerov | anyways, I rechecked the failing patches now after revert | 13:28 |
rakhmerov | can you please resent it with whatever changes needed? | 13:28 |
rakhmerov | and what should we do with ? | 13:28 |
rakhmerov | Andreas still votes -1 on it | 13:28 |
thrash | rakhmerov: that's the small change. :D | 13:29 |
*** rbrady has quit IRC | 13:29 | |
thrash | I'll figure out the back-compatible part. | 13:29 |
thrash | Merde | 13:29 |
rakhmerov | yes, please | 13:29 |
rakhmerov | but what is it all about? | 13:29 |
thrash | actually, I know exactly what to do with it. | 13:29 |
rakhmerov | it's supposed to run on a different environment/os? | 13:30 |
thrash | I'll mark it as skipped in the original patch based on a bug | 13:30 |
rakhmerov | ok | 13:30 |
thrash | and then in the mistral patch, I'll mark it as closing that bug. | 13:30 |
thrash | that should do it. | 13:30 |
rakhmerov | okay | 13:30 |
thrash | And move the code into the mistral patch. | 13:30 |
thrash | Keep the old code in tact in the initial patch. | 13:31 |
rakhmerov | but if you still could quickly explain what we're doing there... :)) | 13:31 |
rakhmerov | it would be helpful ) | 13:31 |
thrash | rakhmerov: Sorry... which part? | 13:31 |
rakhmerov | the whole idea | 13:31 |
thrash | rakhmerov: This is to move the mistral jobs away from 'legacy' jobs to the new zuul v3 format | 13:32 |
apetrich | cool. the lack of promotion is the pain now. | 13:32 |
rakhmerov | and why it's failing now (before the revert) | 13:32 |
thrash | and the reason it's failing now, is that the devstack zuulv3 base job functions differently than the legacy devstack job. | 13:32 |
rakhmerov | thrash: and why was it failing? Because it should run on a different env? | 13:32 |
rakhmerov | ok | 13:33 |
thrash | rakhmerov: yeah. It's not being run as root for one thing. | 13:33 |
thrash | rakhmerov: which, obviously is a good thing. :) | 13:33 |
rakhmerov | sure | 13:33 |
rakhmerov | yes | 13:33 |
rakhmerov | ok, I more or less understand now | 13:33 |
rakhmerov | apetrich: what about promotion? | 13:33 |
thrash | rakhmerov: I'll rework it so I don't break the world. :D | 13:34 |
thrash | I'll make sure I add a patch that tests the existing jobs. | 13:34 |
apetrich | with 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-ci | 13:35 |
*** rbrady has joined #openstack-mistral | 13:42 | |
*** dprince has joined #openstack-mistral | 13:46 | |
*** mattybrennan has joined #openstack-mistral | 13:48 | |
*** mattybrennan has quit IRC | 13:56 | |
*** mattybrennan has joined #openstack-mistral | 13:57 | |
*** mattybrennan has quit IRC | 14:01 | |
*** mattybrennan has joined #openstack-mistral | 14:24 | |
*** mattybrennan has quit IRC | 14:28 | |
*** mattybrennan has joined #openstack-mistral | 14:33 | |
*** mattybre_ has joined #openstack-mistral | 14:34 | |
*** mattybrennan has quit IRC | 14:38 | |
*** thrash is now known as thrash|biab | 15:02 | |
*** mattybre_ has quit IRC | 15:06 | |
*** mattybrennan has joined #openstack-mistral | 15:08 | |
*** shardy is now known as shardy_afk | 15:10 | |
*** bobh has joined #openstack-mistral | 15:16 | |
*** oidgar has quit IRC | 15:23 | |
*** thrash|biab is now known as thrash | 15:34 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral-tempest-plugin master: Revert "Revert "Add zuul jobs for mistral-tempest-plugin"" https://review.openstack.org/536866 | 15:35 |
*** shardy_afk is now known as shardy | 15:46 | |
*** mattybrennan has quit IRC | 15:54 | |
*** mattybrennan has joined #openstack-mistral | 16:01 | |
*** openstackgerrit has quit IRC | 16:03 | |
*** openstackgerrit has joined #openstack-mistral | 16:19 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Remove any old client actions that no longer exist https://review.openstack.org/536886 | 16:19 |
openstackgerrit | Brad P. Crochet proposed openstack/mistral-tempest-plugin master: Revert "Revert "Add zuul jobs for mistral-tempest-plugin"" https://review.openstack.org/536866 | 16:21 |
*** oidgar has joined #openstack-mistral | 16:21 | |
toure | rakhmerov ping | 16:23 |
toure | rakhmerov in regards to the ping earlier, I am refactoring it now | 16:23 |
toure | should be ready to review in a couple hours | 16:23 |
*** mattybrennan has quit IRC | 16:26 | |
*** mattybrennan has joined #openstack-mistral | 16:28 | |
*** mattybrennan has quit IRC | 16:28 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: TEST: DNM Test the new zuul jobs don't break existing https://review.openstack.org/536890 | 16:28 |
*** mattybrennan has joined #openstack-mistral | 16:29 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral-tempest-plugin master: Simplify the ssh tests and where it gets it's key from https://review.openstack.org/536892 | 16:32 |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Migrate the jobs to native Zuul v3 format https://review.openstack.org/509428 | 16:33 |
*** jkilpatr_ has joined #openstack-mistral | 16:58 | |
*** jkilpatr has quit IRC | 17:00 | |
*** oidgar has quit IRC | 17:01 | |
*** AlexeyAbashkin has quit IRC | 17:02 | |
*** kushalagrawal has quit IRC | 17:04 | |
*** kushalagrawal has joined #openstack-mistral | 17:04 | |
*** mattybre_ has joined #openstack-mistral | 17:10 | |
*** mattybrennan has quit IRC | 17:10 | |
*** jkilpatr_ has quit IRC | 17:12 | |
*** kushalagrawal has quit IRC | 17:23 | |
*** kushalagrawal has joined #openstack-mistral | 17:25 | |
*** mattybre_ has quit IRC | 17:30 | |
*** mattybrennan has joined #openstack-mistral | 17:31 | |
*** jpich has quit IRC | 17:39 | |
*** kushalagrawal has quit IRC | 17:57 | |
*** oidgar has joined #openstack-mistral | 17:57 | |
*** kushalagrawal has joined #openstack-mistral | 17:57 | |
*** mattybrennan has quit IRC | 17:59 | |
*** mattybrennan has joined #openstack-mistral | 18:04 | |
*** thrash is now known as thrash|biab | 18:04 | |
openstackgerrit | Merged openstack/mistral master: task name can not be reserved keyword https://review.openstack.org/535297 | 18:13 |
*** oidgar has quit IRC | 18:18 | |
*** jkilpatr has joined #openstack-mistral | 18:20 | |
*** shardy has quit IRC | 18:22 | |
*** AlexeyAbashkin has joined #openstack-mistral | 18:31 | |
*** AlexeyAbashkin has quit IRC | 18:35 | |
*** thrash|biab is now known as thrash | 18:36 | |
openstackgerrit | Toure Dunnon proposed openstack/mistral master: Running new workflow based on an existing execution. https://review.openstack.org/506652 | 18:43 |
openstackgerrit | Brad P. Crochet proposed openstack/mistral-tempest-plugin master: Revert "Revert "Add zuul jobs for mistral-tempest-plugin"" https://review.openstack.org/536866 | 18:56 |
openstackgerrit | Brad P. Crochet proposed openstack/mistral-tempest-plugin master: Simplify the ssh tests and where it gets it's key from https://review.openstack.org/536892 | 18:57 |
*** kushalagrawal has quit IRC | 18:57 | |
*** kushalagrawal has joined #openstack-mistral | 18:58 | |
*** jkilpatr has quit IRC | 18:59 | |
*** mattybrennan has quit IRC | 19:00 | |
*** bobh has quit IRC | 19:02 | |
*** mattybrennan has joined #openstack-mistral | 19:03 | |
*** bobh has joined #openstack-mistral | 19:10 | |
*** harlowja has joined #openstack-mistral | 19:34 | |
*** kushalagrawal has quit IRC | 19:43 | |
*** kushalagrawal has joined #openstack-mistral | 19:45 | |
*** mattybrennan has quit IRC | 19:53 | |
*** mattybrennan has joined #openstack-mistral | 19:54 | |
*** bobh has quit IRC | 19:55 | |
*** kushalagrawal has quit IRC | 20:09 | |
*** kushalagrawal has joined #openstack-mistral | 20:15 | |
*** bobh has joined #openstack-mistral | 20:26 | |
*** mattybrennan has quit IRC | 20:28 | |
*** bobh has quit IRC | 20:31 | |
*** mattybrennan has joined #openstack-mistral | 20:32 | |
*** mattybrennan has quit IRC | 20:32 | |
*** mattybrennan has joined #openstack-mistral | 20:33 | |
*** jkilpatr has joined #openstack-mistral | 20:39 | |
*** bobh has joined #openstack-mistral | 21:03 | |
*** mattybrennan has quit IRC | 21:24 | |
*** dprince has quit IRC | 21:38 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral-tempest-plugin master: Revert "Revert "Add zuul jobs for mistral-tempest-plugin"" https://review.openstack.org/536866 | 21:47 |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Remove any old client actions that no longer exist https://review.openstack.org/536886 | 22:06 |
openstackgerrit | Merged openstack/mistral master: Fix std.http action doc https://review.openstack.org/536689 | 22:18 |
*** kushalagrawal has quit IRC | 23:10 | |
*** kushalagrawal has joined #openstack-mistral | 23:10 | |
*** toure is now known as toure|gone | 23:17 | |
*** bobh has quit IRC | 23:51 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!