*** bobh has quit IRC | 00:06 | |
*** jrist-afk is now known as jrist | 00:12 | |
*** AlexeyAbashkin has joined #openstack-mistral | 00:13 | |
*** AlexeyAbashkin has quit IRC | 00:17 | |
*** AlexeyAbashkin has joined #openstack-mistral | 01:12 | |
*** AlexeyAbashkin has quit IRC | 01:16 | |
*** daidv has quit IRC | 01:18 | |
*** daidv has joined #openstack-mistral | 01:19 | |
*** yangyapeng has joined #openstack-mistral | 01:31 | |
*** thrash is now known as thrash|g0ne | 01:31 | |
*** yangyapeng has quit IRC | 01:35 | |
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code (1) https://review.openstack.org/509325 | 01:41 |
---|---|---|
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code - action execution (2) https://review.openstack.org/511169 | 01:42 |
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code - action (3) https://review.openstack.org/511381 | 01:42 |
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code - cron trigger (4) https://review.openstack.org/511688 | 01:42 |
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code - environment (5) https://review.openstack.org/511696 | 01:42 |
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code - execution (6) https://review.openstack.org/511698 | 01:43 |
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code - member (7) https://review.openstack.org/511699 | 01:43 |
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code - service and task (8) https://review.openstack.org/511700 | 01:43 |
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code - workbook (9) https://review.openstack.org/511701 | 01:43 |
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code - workflow (10) https://review.openstack.org/511719 | 01:43 |
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code - event trigger (11) https://review.openstack.org/511738 | 01:43 |
openstackgerrit | Dai Dang Van proposed openstack/mistral master: Implement policy in code - docs and reno (end) https://review.openstack.org/511769 | 01:43 |
*** yangyapeng has joined #openstack-mistral | 01:49 | |
*** yangyapeng has quit IRC | 01:54 | |
*** yangyapeng has joined #openstack-mistral | 02:15 | |
*** yangyapeng has quit IRC | 02:19 | |
*** yangyapeng has joined #openstack-mistral | 02:34 | |
*** bobh has joined #openstack-mistral | 02:37 | |
*** yangyapeng has quit IRC | 02:38 | |
*** bobh has quit IRC | 02:40 | |
*** bobh has joined #openstack-mistral | 02:52 | |
*** yangyapeng has joined #openstack-mistral | 02:59 | |
*** yangyapeng has quit IRC | 03:04 | |
*** bobh has quit IRC | 03:10 | |
*** yangyapeng has joined #openstack-mistral | 03:18 | |
*** harlowja has quit IRC | 03:22 | |
*** yangyapeng has quit IRC | 03:22 | |
*** bobh has joined #openstack-mistral | 03:23 | |
daidv | Hi kong , can you review the series of default policy-in-code patches for me? | 03:33 |
daidv | #link https://review.openstack.org/#/q/project:openstack/mistral+topic:policy-and-docs-in-code | 03:33 |
*** harlowja has joined #openstack-mistral | 04:01 | |
*** yangyapeng has joined #openstack-mistral | 04:03 | |
*** yangyapeng has quit IRC | 04:08 | |
*** yangyapeng has joined #openstack-mistral | 04:24 | |
*** haint has quit IRC | 04:25 | |
*** yangyapeng has quit IRC | 04:28 | |
openstackgerrit | Bob Haddleton proposed openstack/mistral master: Add type definitions to support on-complete, on-error, on-success https://review.openstack.org/499790 | 04:37 |
*** bobh has quit IRC | 04:38 | |
*** gkadam has joined #openstack-mistral | 04:46 | |
*** yangyapeng has joined #openstack-mistral | 04:48 | |
*** yangyapeng has quit IRC | 04:52 | |
*** nguyentrihai has joined #openstack-mistral | 05:34 | |
*** yangyapeng has joined #openstack-mistral | 05:37 | |
rakhmerov | daidv: hi, no worries, we'll review these patches | 05:40 |
rakhmerov | we just had other priorities | 05:40 |
*** haint has joined #openstack-mistral | 05:40 | |
rakhmerov | but now we'll time to look at them | 05:40 |
*** nguyentrihai has quit IRC | 05:43 | |
*** harlowja has quit IRC | 05:52 | |
daidv | rakhmerov, thanks | 06:07 |
*** jtomasek has joined #openstack-mistral | 06:49 | |
*** dtantsur|afk is now known as dtantsur | 07:01 | |
*** oidgar has joined #openstack-mistral | 07:12 | |
*** gkadam has quit IRC | 07:20 | |
*** mcdoker181818 has joined #openstack-mistral | 07:47 | |
*** shardy has joined #openstack-mistral | 07:49 | |
mcdoker181818 | Hi all, can i modify a execution in runtime, when it has RUNNING state? | 07:52 |
mcdoker181818 | For example, to add or remove tasks, to change a dependency between tasks | 07:52 |
*** d0ugal has joined #openstack-mistral | 08:08 | |
*** gkadam has joined #openstack-mistral | 09:01 | |
*** jpich has joined #openstack-mistral | 09:07 | |
*** oidgar has quit IRC | 09:13 | |
*** dtantsur_ has joined #openstack-mistral | 09:21 | |
*** dtantsur has quit IRC | 09:22 | |
*** dtantsur_ is now known as dtantsur | 09:22 | |
*** mcdoker181818 has quit IRC | 09:30 | |
*** daidv has quit IRC | 10:05 | |
*** apetrich has quit IRC | 11:16 | |
*** thrash|g0ne is now known as thrash | 11:19 | |
*** AlexeyAbashkin has joined #openstack-mistral | 11:24 | |
*** jkilpatr_ has quit IRC | 11:39 | |
d0ugal | rakhmerov: Hey | 11:46 |
d0ugal | rakhmerov: https://review.openstack.org/#/c/521611/ | 11:46 |
*** oidgar has joined #openstack-mistral | 11:48 | |
openstackgerrit | Dougal Matthews proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session https://review.openstack.org/521611 | 11:56 |
*** jkilpatr has joined #openstack-mistral | 12:04 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Switch clients to use a session https://review.openstack.org/521611 | 12:36 |
thrash | d0ugal: rakhmerov we really need to get the devstack jobs voting again. | 12:41 |
*** yangyapeng has quit IRC | 12:44 | |
*** yangyapeng has joined #openstack-mistral | 12:45 | |
*** yangyapeng has quit IRC | 12:49 | |
d0ugal | thrash: I don't think they have ever been stable enough to be voting. | 12:52 |
thrash | d0ugal: yeah... that's a problem. | 12:53 |
d0ugal | aye | 12:54 |
d0ugal | I looked into it once but didn't want to learn Tempest | 12:54 |
d0ugal | lol | 12:54 |
d0ugal | but since that happened anyway, maybe I should try again. | 12:54 |
*** rbrady has quit IRC | 13:08 | |
d0ugal | thrash: looks like cinder still failed | 13:09 |
d0ugal | http://logs.openstack.org/11/521611/4/check/mistral-devstack-dsvm/c39f0b7/job-output.txt.gz#_2017-11-21_13_05_49_665440 | 13:09 |
thrash | :/ | 13:10 |
thrash | d0ugal: I may not be doing it right either. :) | 13:10 |
thrash | Ok. I'm think I'm going to pull the cinder part, since it doesn't add anything. | 13:11 |
*** bobh has joined #openstack-mistral | 13:12 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session https://review.openstack.org/521611 | 13:13 |
*** dprince has joined #openstack-mistral | 13:15 | |
*** yangyapeng has joined #openstack-mistral | 13:16 | |
thrash | d0ugal: Going to add a couple of tests... Looks like the tempest tests did not include swift or zaqar. | 13:16 |
*** rbrady has joined #openstack-mistral | 13:22 | |
*** rbrady has quit IRC | 13:22 | |
*** rbrady has joined #openstack-mistral | 13:22 | |
*** shardy is now known as shardy_lunch | 13:22 | |
d0ugal | thrash: seems so - I wondered what projects are available in the env | 13:26 |
thrash | hmmm... that could be a problem. | 13:26 |
thrash | I probably have to add them to devstack. | 13:27 |
thrash | glad you thought of that. | 13:27 |
d0ugal | :) | 13:27 |
d0ugal | I kinda expect swift might be there, but not sure about zaqar | 13:27 |
thrash | Doubt zaqar.... | 13:29 |
d0ugal | aye, that is what I mean really :) | 13:30 |
*** apetrich has joined #openstack-mistral | 13:35 | |
*** shardy_lunch is now known as shardy | 13:45 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session https://review.openstack.org/521611 | 13:47 |
thrash | d0ugal: let's see if I'm insane or not. lol | 13:47 |
d0ugal | thrash: nice, it looks good anyway | 13:48 |
apetrich | thrash, nice one | 13:53 |
apetrich | thrash, I thought that it would need changes to mistral/utils/openstack/keystone.py also | 13:57 |
*** jkilpatr_ has joined #openstack-mistral | 14:00 | |
*** jkilpatr has quit IRC | 14:03 | |
*** toure_biab is now known as toure | 14:19 | |
*** bobh has quit IRC | 14:45 | |
apetrich | thrash, https://review.openstack.org/#/c/521611 just failed but a different failure. not a 401 unauthorized | 14:48 |
thrash | apetrich: that's progress | 14:48 |
thrash | I think. :D | 14:49 |
apetrich | thrash, ZaqarAction.queue_post failed: 'NoneType' object has no attribute 'find': ActionException: ZaqarAction.queue_post failed: 'NoneType' object has no attribute 'find' | 14:49 |
apetrich | it is | 14:49 |
apetrich | http://logs.openstack.org/11/521611/6/check/tripleo-ci-centos-7-scenario003-multinode-oooq-container/4e9b52e/logs/undercloud/var/log/mistral/executor.log.txt.gz#_2017-11-21_14_38_14_174 | 14:49 |
apetrich | I meant | 14:49 |
apetrich | thrash, ^^ | 14:49 |
thrash | something missing from the session it looks like.. | 14:50 |
apetrich | auth_url in maybe? zaqarclient/auth/keystone.py, line 140 | 14:55 |
*** weshay is now known as weshay_pto | 14:55 | |
*** oidgar has quit IRC | 15:03 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session https://review.openstack.org/521611 | 15:08 |
thrash | let's try that. | 15:08 |
apetrich | fingers crossed | 15:11 |
*** bobh has joined #openstack-mistral | 15:20 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: DNM: TEST Zuul v3 jobs https://review.openstack.org/509428 | 15:27 |
thrash | apetrich: devstack still failed, but I'm going to let the tripleo job finish before I push a fix for devstack | 15:48 |
apetrich | ok | 15:48 |
apetrich | thrash, how did it failed? | 15:48 |
thrash | apetrich: http://logs.openstack.org/11/521611/7/check/mistral-devstack-dsvm/e31784a/logs/devstacklog.txt.gz#_2017-11-21_15_27_39_241 | 15:49 |
apetrich | I see | 15:50 |
apetrich | cheers | 15:50 |
*** jistr is now known as jistr|mtg | 15:59 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: DNM: TEST Zuul v3 jobs https://review.openstack.org/509428 | 16:02 |
*** yangyapeng has quit IRC | 16:02 | |
*** yangyapeng has joined #openstack-mistral | 16:03 | |
*** yangyapeng has quit IRC | 16:08 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session https://review.openstack.org/521611 | 16:09 |
apetrich | thrash, woot | 16:11 |
thrash | apetrich: closer | 16:11 |
thrash | still need to fix something... | 16:12 |
thrash | need to pass username too I think. | 16:12 |
*** oidgar has joined #openstack-mistral | 16:12 | |
apetrich | thrash, I'm relocating in a bit. bbl | 16:13 |
thrash | apetrich: ack | 16:13 |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session https://review.openstack.org/521611 | 16:14 |
*** jistr|mtg is now known as jistr | 16:16 | |
*** toure is now known as toure_biab | 16:16 | |
*** apetrich has quit IRC | 16:18 | |
*** AlexeyAbashkin has quit IRC | 16:28 | |
*** toure_biab is now known as toure | 16:36 | |
*** dtantsur is now known as dtantsur|afk | 16:55 | |
*** yangyapeng has joined #openstack-mistral | 17:09 | |
*** yangyapeng has quit IRC | 17:14 | |
*** gkadam has quit IRC | 17:26 | |
*** d0ugal has quit IRC | 17:30 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session https://review.openstack.org/521611 | 17:33 |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Add new tempest tests for swift and zaqar client actions https://review.openstack.org/521964 | 17:33 |
*** d0ugal has joined #openstack-mistral | 17:51 | |
*** jistr is now known as jistr|off|trng | 18:04 | |
*** jpich has quit IRC | 18:23 | |
*** shardy has quit IRC | 18:26 | |
*** AlexeyAbashkin has joined #openstack-mistral | 18:40 | |
*** AlexeyAbashkin has quit IRC | 18:44 | |
*** bobh has quit IRC | 18:59 | |
*** oidgar has quit IRC | 19:02 | |
*** bobh has joined #openstack-mistral | 19:06 | |
*** apetrich has joined #openstack-mistral | 19:08 | |
EmilienM | d0ugal, thrash, apetrich: could you please take a look at https://review.openstack.org/521633 ? | 19:09 |
thrash | EmilienM: that's missing all of mistral's jobs. | 19:10 |
EmilienM | thrash: yep | 19:10 |
EmilienM | thrash: because you didn't backport the zuul layout in your stable branches | 19:10 |
thrash | so how is that useful? | 19:10 |
EmilienM | thrash: I won't do that because I don't know what you want to test | 19:10 |
EmilienM | thrash: my backports don't override your legacy jobs running on stable | 19:10 |
EmilienM | thrash: although it makes sure tripleo jobs run on stable | 19:10 |
EmilienM | + the legacy jobs defined in project-config | 19:11 |
EmilienM | in other words, my patches just add tripleo jobs. They don't remove anything. | 19:11 |
thrash | EmilienM: but it does.. | 19:11 |
EmilienM | let me check | 19:11 |
*** AlexeyAbashkin has joined #openstack-mistral | 19:11 | |
thrash | EmilienM: Look at what's on that review. There are no devstack jobs or anything. | 19:11 |
EmilienM | indeed | 19:11 |
EmilienM | thrash: yeah because you didn't backport the zuul layout from master | 19:12 |
EmilienM | let me look again | 19:12 |
thrash | EmilienM: Would you like me to do that? :D | 19:12 |
EmilienM | I can do it for you | 19:12 |
EmilienM | how much $ ? | 19:12 |
thrash | You have to pay me. | 19:12 |
EmilienM | lol | 19:12 |
EmilienM | thrash: https://review.openstack.org/#/c/521984/ | 19:13 |
EmilienM | you don't have IRC notifs for stable/pile | 19:13 |
EmilienM | pike | 19:13 |
EmilienM | you need to patch gerritbot in project-config, I can do it | 19:13 |
thrash | stable/pile/of/.... | 19:14 |
EmilienM | thrash: https://review.openstack.org/521986 | 19:14 |
EmilienM | you're welcome | 19:14 |
thrash | EmilienM: also, there are some additional patches to complete it. | 19:14 |
EmilienM | thrash: I know | 19:15 |
thrash | EmilienM: thanks! | 19:15 |
EmilienM | i'm working on them now | 19:15 |
EmilienM | thrash: it fails badly, I'll just git stash all the patches ok? | 19:16 |
*** AlexeyAbashkin has quit IRC | 19:16 | |
EmilienM | cause it's really hard to backport otherwise | 19:16 |
thrash | You mean you're going to collapse them? | 19:16 |
EmilienM | thrash: I'm going to manually backport .zuul.yaml | 19:16 |
thrash | EmilienM: ack | 19:17 |
EmilienM | thrash: https://review.openstack.org/521633 done | 19:19 |
thrash | EmilienM: need the playbooks dir | 19:19 |
EmilienM | thrash: damn. | 19:19 |
EmilienM | thrash: so much beers | 19:20 |
EmilienM | thrash: I like IPA | 19:20 |
thrash | EmilienM: hehe | 19:20 |
thrash | EmilienM: So do I! What a coincidence! | 19:20 |
EmilienM | thrash: perfect, so next time when YOU buy once, you'll actually buy TWO | 19:21 |
thrash | haha | 19:21 |
EmilienM | thrash: it's maths | 19:21 |
EmilienM | 1 = 2 | 19:21 |
apetrich | thrash, https://review.openstack.org/#/c/521611/ is green for scenario 3! Great work! | 19:23 |
apetrich | and much cleaner | 19:23 |
thrash | woot! | 19:23 |
thrash | apetrich: ok. need to get some +2 action on that. | 19:23 |
apetrich | thrash, not root yet (working on that :) | 19:23 |
thrash | rakhmerov: d0ugal https://review.openstack.org/#/c/521611/ | 19:23 |
thrash | any other cores on? | 19:23 |
thrash | TripleO REALLY needs that patch ^^^^ | 19:24 |
EmilienM | yeah I go tit | 19:25 |
EmilienM | I got it* | 19:25 |
thrash | lol | 19:25 |
EmilienM | but once merge, we need promotion :( | 19:25 |
EmilienM | thrash: don't say lol to that typo please | 19:25 |
EmilienM | i'm trying to be serious here | 19:25 |
thrash | EmilienM: we're in a chicken/egg then. | 19:25 |
EmilienM | thrash: no I think it's fine | 19:26 |
thrash | EmilienM: We need that patch in order to promote. | 19:26 |
thrash | But that should work, yeah? | 19:26 |
EmilienM | yeah | 19:26 |
EmilienM | land it | 19:26 |
thrash | EmilienM: I'm not sure we have any Mistral cores on ATM | 19:26 |
apetrich | EmilienM, rlandy is on that once we got it merged she'll look after the promotion | 19:27 |
EmilienM | you can self approve and hope nobody will see it | 19:28 |
EmilienM | I usually do that on weekends | 19:28 |
EmilienM | (ok I'll stop jokes here before folks think I'm serious) | 19:28 |
thrash | EmilienM: I'm not core. | 19:29 |
thrash | EmilienM: And *I* know you aren't serious. :D | 19:30 |
EmilienM | lol | 19:30 |
apetrich | :) | 19:33 |
d0ugal | I hear there is some serioous chat here. | 19:43 |
d0ugal | kong: Are you around? Could you take a look at https://review.openstack.org/#/c/521611/ ? | 19:44 |
apetrich | oh yeah good idea | 19:44 |
d0ugal | thrash: I don't want to panic you, but Renat goes on PTO for two weeks soon. | 19:46 |
d0ugal | :) | 19:47 |
*** jtomasek has quit IRC | 19:51 | |
*** bobh has quit IRC | 19:59 | |
*** AlexeyAbashkin has joined #openstack-mistral | 20:11 | |
kong | d0ugal: sure, will take a look this morning | 20:13 |
*** AlexeyAbashkin has quit IRC | 20:15 | |
thrash | kong: thanks. TripleO production chain promotions are held up until that lands. So no pressure. :) | 20:16 |
thrash | kong: and if you're wondering about tests, I've got a follow-on patch to fix all of the tempest tests related to openstack actions https://review.openstack.org/#/c/521964/1 | 20:16 |
kong | thrash: cool, coincidently i have used swift client and zaqar client recently, so will have a quick look :-) | 20:18 |
thrash | kong: awesome | 20:18 |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Add new tempest tests for swift and zaqar client actions https://review.openstack.org/521964 | 20:22 |
*** apetrich has quit IRC | 20:30 | |
*** apetrich has joined #openstack-mistral | 20:38 | |
*** bobh has joined #openstack-mistral | 20:41 | |
*** bobh has quit IRC | 20:46 | |
*** bobh has joined #openstack-mistral | 20:59 | |
*** bobh has quit IRC | 21:05 | |
thrash | kong: I appreciate the comments. In the interest of time (and helping TripleO get unwedged) I'd like to address your concerns in a follow-on patch? | 21:07 |
kong | thrash: ok, cool, as long as current code works | 21:08 |
kong | thrash: because i don't have an available environment, did you test in real deployment? | 21:09 |
thrash | kong: The zaqar code gets direct testing from the tripleo scenario003 job | 21:09 |
kong | ok, never mind, let's merge for now considering tripleo requirement, and fix any possible bug in following patch | 21:10 |
kong | thrash: done | 21:11 |
thrash | kong: Thanks! | 21:11 |
kong | np | 21:11 |
thrash | kong: I'm working on 1) fixing the devstack jobs 2) expanding the tempest tests that cover these client actions | 21:12 |
kong | thrash: cool, thanks! | 21:12 |
*** vishwanathj has joined #openstack-mistral | 21:23 | |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Add new tempest tests for swift and zaqar client actions https://review.openstack.org/521964 | 21:23 |
openstackgerrit | Brad P. Crochet proposed openstack/mistral master: Use a session for keystone auth https://review.openstack.org/522023 | 21:35 |
thrash | kong: ^^^^ First pass | 21:35 |
*** dprince has quit IRC | 21:37 | |
*** jkilpatr_ has quit IRC | 21:47 | |
*** threestrands has joined #openstack-mistral | 21:48 | |
*** bobh has joined #openstack-mistral | 21:52 | |
*** jkilpatr has joined #openstack-mistral | 22:05 | |
*** yangyapeng has joined #openstack-mistral | 22:10 | |
*** yangyapeng has quit IRC | 22:14 | |
*** thrash is now known as thrash|bbl | 22:22 | |
*** bobh has quit IRC | 22:55 | |
*** bobh has joined #openstack-mistral | 23:00 | |
*** bobh has quit IRC | 23:10 | |
*** AlexeyAbashkin has joined #openstack-mistral | 23:12 | |
*** AlexeyAbashkin has quit IRC | 23:16 | |
openstackgerrit | Merged openstack/mistral master: Switch zaqarclient and swiftclient to use a session https://review.openstack.org/521611 | 23:24 |
*** d0ugal has quit IRC | 23:58 | |
*** d0ugal has joined #openstack-mistral | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!