Tuesday, 2017-11-21

*** bobh has quit IRC00:06
*** jrist-afk is now known as jrist00:12
*** AlexeyAbashkin has joined #openstack-mistral00:13
*** AlexeyAbashkin has quit IRC00:17
*** AlexeyAbashkin has joined #openstack-mistral01:12
*** AlexeyAbashkin has quit IRC01:16
*** daidv has quit IRC01:18
*** daidv has joined #openstack-mistral01:19
*** yangyapeng has joined #openstack-mistral01:31
*** thrash is now known as thrash|g0ne01:31
*** yangyapeng has quit IRC01:35
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code (1)  https://review.openstack.org/50932501:41
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code - action execution (2)  https://review.openstack.org/51116901:42
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code - action (3)  https://review.openstack.org/51138101:42
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code - cron trigger (4)  https://review.openstack.org/51168801:42
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code - environment (5)  https://review.openstack.org/51169601:42
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code - execution (6)  https://review.openstack.org/51169801:43
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code - member (7)  https://review.openstack.org/51169901:43
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code - service and task (8)  https://review.openstack.org/51170001:43
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code - workbook (9)  https://review.openstack.org/51170101:43
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code - workflow (10)  https://review.openstack.org/51171901:43
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code - event trigger (11)  https://review.openstack.org/51173801:43
openstackgerritDai Dang Van proposed openstack/mistral master: Implement policy in code - docs and reno (end)  https://review.openstack.org/51176901:43
*** yangyapeng has joined #openstack-mistral01:49
*** yangyapeng has quit IRC01:54
*** yangyapeng has joined #openstack-mistral02:15
*** yangyapeng has quit IRC02:19
*** yangyapeng has joined #openstack-mistral02:34
*** bobh has joined #openstack-mistral02:37
*** yangyapeng has quit IRC02:38
*** bobh has quit IRC02:40
*** bobh has joined #openstack-mistral02:52
*** yangyapeng has joined #openstack-mistral02:59
*** yangyapeng has quit IRC03:04
*** bobh has quit IRC03:10
*** yangyapeng has joined #openstack-mistral03:18
*** harlowja has quit IRC03:22
*** yangyapeng has quit IRC03:22
*** bobh has joined #openstack-mistral03:23
daidvHi 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-code03:33
*** harlowja has joined #openstack-mistral04:01
*** yangyapeng has joined #openstack-mistral04:03
*** yangyapeng has quit IRC04:08
*** yangyapeng has joined #openstack-mistral04:24
*** haint has quit IRC04:25
*** yangyapeng has quit IRC04:28
openstackgerritBob Haddleton proposed openstack/mistral master: Add type definitions to support on-complete, on-error, on-success  https://review.openstack.org/49979004:37
*** bobh has quit IRC04:38
*** gkadam has joined #openstack-mistral04:46
*** yangyapeng has joined #openstack-mistral04:48
*** yangyapeng has quit IRC04:52
*** nguyentrihai has joined #openstack-mistral05:34
*** yangyapeng has joined #openstack-mistral05:37
rakhmerovdaidv: hi, no worries, we'll review these patches05:40
rakhmerovwe just had other priorities05:40
*** haint has joined #openstack-mistral05:40
rakhmerovbut now we'll time to look at them05:40
*** nguyentrihai has quit IRC05:43
*** harlowja has quit IRC05:52
daidvrakhmerov, thanks06:07
*** jtomasek has joined #openstack-mistral06:49
*** dtantsur|afk is now known as dtantsur07:01
*** oidgar has joined #openstack-mistral07:12
*** gkadam has quit IRC07:20
*** mcdoker181818 has joined #openstack-mistral07:47
*** shardy has joined #openstack-mistral07:49
mcdoker181818Hi all, can i modify a execution in runtime, when it has RUNNING state?07:52
mcdoker181818For example, to add or remove tasks, to change a dependency between tasks07:52
*** d0ugal has joined #openstack-mistral08:08
*** gkadam has joined #openstack-mistral09:01
*** jpich has joined #openstack-mistral09:07
*** oidgar has quit IRC09:13
*** dtantsur_ has joined #openstack-mistral09:21
*** dtantsur has quit IRC09:22
*** dtantsur_ is now known as dtantsur09:22
*** mcdoker181818 has quit IRC09:30
*** daidv has quit IRC10:05
*** apetrich has quit IRC11:16
*** thrash|g0ne is now known as thrash11:19
*** AlexeyAbashkin has joined #openstack-mistral11:24
*** jkilpatr_ has quit IRC11:39
d0ugalrakhmerov: Hey11:46
d0ugalrakhmerov: https://review.openstack.org/#/c/521611/11:46
*** oidgar has joined #openstack-mistral11:48
openstackgerritDougal Matthews proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session  https://review.openstack.org/52161111:56
*** jkilpatr has joined #openstack-mistral12:04
openstackgerritBrad P. Crochet proposed openstack/mistral master: Switch clients to use a session  https://review.openstack.org/52161112:36
thrashd0ugal: rakhmerov we really need to get the devstack jobs voting again.12:41
*** yangyapeng has quit IRC12:44
*** yangyapeng has joined #openstack-mistral12:45
*** yangyapeng has quit IRC12:49
d0ugalthrash: I don't think they have ever been stable enough to be voting.12:52
thrashd0ugal: yeah... that's a problem.12:53
d0ugalaye12:54
d0ugalI looked into it once but didn't want to learn Tempest12:54
d0ugallol12:54
d0ugalbut since that happened anyway, maybe I should try again.12:54
*** rbrady has quit IRC13:08
d0ugalthrash: looks like cinder still failed13:09
d0ugalhttp://logs.openstack.org/11/521611/4/check/mistral-devstack-dsvm/c39f0b7/job-output.txt.gz#_2017-11-21_13_05_49_66544013:09
thrash:/13:10
thrashd0ugal: I may not be doing it right either. :)13:10
thrashOk. I'm think I'm going to pull the cinder part, since it doesn't add anything.13:11
*** bobh has joined #openstack-mistral13:12
openstackgerritBrad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session  https://review.openstack.org/52161113:13
*** dprince has joined #openstack-mistral13:15
*** yangyapeng has joined #openstack-mistral13:16
thrashd0ugal: Going to add a couple of tests... Looks like the tempest tests did not include swift or zaqar.13:16
*** rbrady has joined #openstack-mistral13:22
*** rbrady has quit IRC13:22
*** rbrady has joined #openstack-mistral13:22
*** shardy is now known as shardy_lunch13:22
d0ugalthrash: seems so - I wondered what projects are available in the env13:26
thrashhmmm... that could be a problem.13:26
thrashI probably have to add them to devstack.13:27
thrashglad you thought of that.13:27
d0ugal:)13:27
d0ugalI kinda expect swift might be there, but not sure about zaqar13:27
thrashDoubt zaqar....13:29
d0ugalaye, that is what I mean really :)13:30
*** apetrich has joined #openstack-mistral13:35
*** shardy_lunch is now known as shardy13:45
openstackgerritBrad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session  https://review.openstack.org/52161113:47
thrashd0ugal: let's see if I'm insane or not. lol13:47
d0ugalthrash: nice, it looks good anyway13:48
apetrichthrash, nice one13:53
apetrichthrash, I thought that it would need changes to mistral/utils/openstack/keystone.py also13:57
*** jkilpatr_ has joined #openstack-mistral14:00
*** jkilpatr has quit IRC14:03
*** toure_biab is now known as toure14:19
*** bobh has quit IRC14:45
apetrichthrash, https://review.openstack.org/#/c/521611 just failed but a different failure. not a 401 unauthorized14:48
thrashapetrich: that's progress14:48
thrashI think. :D14:49
apetrichthrash, ZaqarAction.queue_post failed: 'NoneType' object has no attribute 'find': ActionException: ZaqarAction.queue_post failed: 'NoneType' object has no attribute 'find'14:49
apetrichit is14:49
apetrichhttp://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_17414:49
apetrichI meant14:49
apetrichthrash, ^^14:49
thrashsomething missing from the session it looks like..14:50
apetrichauth_url in maybe? zaqarclient/auth/keystone.py, line 14014:55
*** weshay is now known as weshay_pto14:55
*** oidgar has quit IRC15:03
openstackgerritBrad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session  https://review.openstack.org/52161115:08
thrashlet's try that.15:08
apetrichfingers crossed15:11
*** bobh has joined #openstack-mistral15:20
openstackgerritBrad P. Crochet proposed openstack/mistral master: DNM: TEST Zuul v3 jobs  https://review.openstack.org/50942815:27
thrashapetrich: devstack still failed, but I'm going to let the tripleo job finish before I push a fix for devstack15:48
apetrichok15:48
apetrichthrash, how did it failed?15:48
thrashapetrich: http://logs.openstack.org/11/521611/7/check/mistral-devstack-dsvm/e31784a/logs/devstacklog.txt.gz#_2017-11-21_15_27_39_24115:49
apetrichI see15:50
apetrichcheers15:50
*** jistr is now known as jistr|mtg15:59
openstackgerritBrad P. Crochet proposed openstack/mistral master: DNM: TEST Zuul v3 jobs  https://review.openstack.org/50942816:02
*** yangyapeng has quit IRC16:02
*** yangyapeng has joined #openstack-mistral16:03
*** yangyapeng has quit IRC16:08
openstackgerritBrad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session  https://review.openstack.org/52161116:09
apetrichthrash, woot16:11
thrashapetrich: closer16:11
thrashstill need to fix something...16:12
thrashneed to pass username too I think.16:12
*** oidgar has joined #openstack-mistral16:12
apetrichthrash, I'm relocating in a bit. bbl16:13
thrashapetrich: ack16:13
openstackgerritBrad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session  https://review.openstack.org/52161116:14
*** jistr|mtg is now known as jistr16:16
*** toure is now known as toure_biab16:16
*** apetrich has quit IRC16:18
*** AlexeyAbashkin has quit IRC16:28
*** toure_biab is now known as toure16:36
*** dtantsur is now known as dtantsur|afk16:55
*** yangyapeng has joined #openstack-mistral17:09
*** yangyapeng has quit IRC17:14
*** gkadam has quit IRC17:26
*** d0ugal has quit IRC17:30
openstackgerritBrad P. Crochet proposed openstack/mistral master: Switch zaqarclient and swiftclient to use a session  https://review.openstack.org/52161117:33
openstackgerritBrad P. Crochet proposed openstack/mistral master: Add new tempest tests for swift and zaqar client actions  https://review.openstack.org/52196417:33
*** d0ugal has joined #openstack-mistral17:51
*** jistr is now known as jistr|off|trng18:04
*** jpich has quit IRC18:23
*** shardy has quit IRC18:26
*** AlexeyAbashkin has joined #openstack-mistral18:40
*** AlexeyAbashkin has quit IRC18:44
*** bobh has quit IRC18:59
*** oidgar has quit IRC19:02
*** bobh has joined #openstack-mistral19:06
*** apetrich has joined #openstack-mistral19:08
EmilienMd0ugal, thrash, apetrich: could you please take a look at https://review.openstack.org/521633 ?19:09
thrashEmilienM: that's missing all of mistral's jobs.19:10
EmilienMthrash: yep19:10
EmilienMthrash: because you didn't backport the zuul layout in your stable branches19:10
thrashso how is that useful?19:10
EmilienMthrash: I won't do that because I don't know what you want to test19:10
EmilienMthrash: my backports don't override your legacy jobs running on stable19:10
EmilienMthrash: although it makes sure tripleo jobs run on stable19:10
EmilienM+ the legacy jobs defined in project-config19:11
EmilienMin other words, my patches just add tripleo jobs. They don't remove anything.19:11
thrashEmilienM: but it does..19:11
EmilienMlet me check19:11
*** AlexeyAbashkin has joined #openstack-mistral19:11
thrashEmilienM: Look at what's on that review. There are no devstack jobs or anything.19:11
EmilienMindeed19:11
EmilienMthrash: yeah because you didn't backport the zuul layout from master19:12
EmilienMlet me look again19:12
thrashEmilienM: Would you like me to do that? :D19:12
EmilienMI can do it for you19:12
EmilienMhow much $ ?19:12
thrashYou have to pay me.19:12
EmilienMlol19:12
EmilienMthrash: https://review.openstack.org/#/c/521984/19:13
EmilienMyou don't have IRC notifs for stable/pile19:13
EmilienMpike19:13
EmilienMyou need to patch gerritbot in project-config, I can do it19:13
thrashstable/pile/of/....19:14
EmilienMthrash: https://review.openstack.org/52198619:14
EmilienMyou're welcome19:14
thrashEmilienM: also, there are some additional patches to complete it.19:14
EmilienMthrash: I know19:15
thrashEmilienM: thanks!19:15
EmilienMi'm working on them now19:15
EmilienMthrash: it fails badly, I'll just git stash all the patches ok?19:16
*** AlexeyAbashkin has quit IRC19:16
EmilienMcause it's really hard to backport otherwise19:16
thrashYou mean you're going to collapse them?19:16
EmilienMthrash: I'm going to manually backport .zuul.yaml19:16
thrashEmilienM: ack19:17
EmilienMthrash: https://review.openstack.org/521633 done19:19
thrashEmilienM: need the playbooks dir19:19
EmilienMthrash: damn.19:19
EmilienMthrash: so much beers19:20
EmilienMthrash: I like IPA19:20
thrashEmilienM: hehe19:20
thrashEmilienM: So do I! What a coincidence!19:20
EmilienMthrash: perfect, so next time when YOU buy once, you'll actually buy TWO19:21
thrashhaha19:21
EmilienMthrash: it's maths19:21
EmilienM1 = 219:21
apetrichthrash, https://review.openstack.org/#/c/521611/ is green for scenario 3! Great work!19:23
apetrichand much cleaner19:23
thrashwoot!19:23
thrashapetrich: ok. need to get some +2 action on that.19:23
apetrichthrash, not root yet (working on that :)19:23
thrashrakhmerov: d0ugal https://review.openstack.org/#/c/521611/19:23
thrashany other cores on?19:23
thrashTripleO REALLY needs that patch ^^^^19:24
EmilienMyeah I go tit19:25
EmilienMI got it*19:25
thrashlol19:25
EmilienMbut once merge, we need promotion :(19:25
EmilienMthrash: don't say lol to that typo please19:25
EmilienMi'm trying to be serious here19:25
thrashEmilienM: we're in a chicken/egg then.19:25
EmilienMthrash: no I think it's fine19:26
thrashEmilienM: We need that patch in order to promote.19:26
thrashBut that should work, yeah?19:26
EmilienMyeah19:26
EmilienMland it19:26
thrashEmilienM: I'm not sure we have any Mistral cores on ATM19:26
apetrichEmilienM, rlandy is on that once we got it merged she'll look after the promotion19:27
EmilienMyou can self approve and hope nobody will see it19:28
EmilienMI usually do that on weekends19:28
EmilienM(ok I'll stop jokes here before folks think I'm serious)19:28
thrashEmilienM: I'm not core.19:29
thrashEmilienM: And *I* know you aren't serious. :D19:30
EmilienMlol19:30
apetrich:)19:33
d0ugalI hear there is some serioous chat here.19:43
d0ugalkong: Are you around? Could you take a look at https://review.openstack.org/#/c/521611/ ?19:44
apetrichoh yeah good idea19:44
d0ugalthrash: I don't want to panic you, but Renat goes on PTO for two weeks soon.19:46
d0ugal:)19:47
*** jtomasek has quit IRC19:51
*** bobh has quit IRC19:59
*** AlexeyAbashkin has joined #openstack-mistral20:11
kongd0ugal: sure, will take a look this morning20:13
*** AlexeyAbashkin has quit IRC20:15
thrashkong: thanks. TripleO production chain promotions are held up until that lands. So no pressure. :)20:16
thrashkong: 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/120:16
kongthrash: cool, coincidently i have used swift client and zaqar client recently, so will have a quick look :-)20:18
thrashkong: awesome20:18
openstackgerritBrad P. Crochet proposed openstack/mistral master: Add new tempest tests for swift and zaqar client actions  https://review.openstack.org/52196420:22
*** apetrich has quit IRC20:30
*** apetrich has joined #openstack-mistral20:38
*** bobh has joined #openstack-mistral20:41
*** bobh has quit IRC20:46
*** bobh has joined #openstack-mistral20:59
*** bobh has quit IRC21:05
thrashkong: 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
kongthrash: ok, cool, as long as current code works21:08
kongthrash: because i don't have an available environment, did you test in real deployment?21:09
thrashkong: The zaqar code gets direct testing from the tripleo scenario003 job21:09
kongok, never mind, let's merge for now considering tripleo requirement, and fix any possible bug in following patch21:10
kongthrash: done21:11
thrashkong: Thanks!21:11
kongnp21:11
thrashkong: I'm working on 1) fixing the devstack jobs 2) expanding the tempest tests that cover these client actions21:12
kongthrash: cool, thanks!21:12
*** vishwanathj has joined #openstack-mistral21:23
openstackgerritBrad P. Crochet proposed openstack/mistral master: Add new tempest tests for swift and zaqar client actions  https://review.openstack.org/52196421:23
openstackgerritBrad P. Crochet proposed openstack/mistral master: Use a session for keystone auth  https://review.openstack.org/52202321:35
thrashkong: ^^^^ First pass21:35
*** dprince has quit IRC21:37
*** jkilpatr_ has quit IRC21:47
*** threestrands has joined #openstack-mistral21:48
*** bobh has joined #openstack-mistral21:52
*** jkilpatr has joined #openstack-mistral22:05
*** yangyapeng has joined #openstack-mistral22:10
*** yangyapeng has quit IRC22:14
*** thrash is now known as thrash|bbl22:22
*** bobh has quit IRC22:55
*** bobh has joined #openstack-mistral23:00
*** bobh has quit IRC23:10
*** AlexeyAbashkin has joined #openstack-mistral23:12
*** AlexeyAbashkin has quit IRC23:16
openstackgerritMerged openstack/mistral master: Switch zaqarclient and swiftclient to use a session  https://review.openstack.org/52161123:24
*** d0ugal has quit IRC23:58
*** d0ugal has joined #openstack-mistral23:59

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