| *** catintheroof has joined #openstack-mistral | 00:25 | |
| *** catintheroof has quit IRC | 01:02 | |
| *** thrash is now known as thrash|g0ne | 01:44 | |
| bobh | rakhmerov: ping | 01:56 |
|---|---|---|
| bobh | Question - is the env() available in action execution, or just in tasks? It appears that I can reference env() variables in tasks but I get an error when trying to reference them in actions | 02:04 |
| *** harlowja has quit IRC | 02:06 | |
| kong | bobh: hi, any log i can see? | 02:36 |
| kong | bobh: rakhmerov may be still sleeping now :) | 02:36 |
| bobh | kong: thanks - give me a minute and I'll copy the error | 02:45 |
| *** harlowja has joined #openstack-mistral | 02:50 | |
| bobh | kong: I have an action that extends an installed action, and I try to set one of the base-input values of the installed action to an environment value: | 02:55 |
| bobh | host: '{{ env().msgbus_hostname }}' | 02:55 |
| bobh | and I get a Jinja error: | 02:55 |
| bobh | JinjaEvaluationException: Can not evaluate Jinja expression [expression= env().msgbus_hostname , error='__env', data={u'to': u'Workflow_07cf0462-18df-11e7-a36a-fa163e791a1a', | | 02:55 |
| bobh | | | u'message': u'Mode: https, LFP: nfvfp, Port: 5000, MsgBus: nfvfp', u'status_code': 200}] | 02:55 |
| bobh | it seems like it knows to map env() to __env in the context but it cant find __env in the context when the action executes | 02:56 |
| bobh | kong: interesting - it seems to work if I use YAQL instead of Jinja | 02:59 |
| bobh | kong: I'll dig into it some more and file a bug if I can get it isolated | 03:02 |
| kong | bobh: sorry, was distracted by something else. yeah, jinja was supported recently, so maybe it's not fully verified. | 03:06 |
| kong | would appriciate if you could file a bug | 03:07 |
| kong | bobh: please let me know if you need any help | 03:07 |
| kong | bobh: btw, next time if you want to paste something, you could use paste.openstack.org | 03:08 |
| bobh | kong: thanks, will do | 03:09 |
| *** bobh has quit IRC | 03:21 | |
| kong | d0ugal, ddeja, mgershen, akuznetsova, need your review on https://review.openstack.org/#/c/448340/ | 03:50 |
| *** harlowja has quit IRC | 03:56 | |
| *** harlowja has joined #openstack-mistral | 04:07 | |
| rakhmerov | bobh, kong: yes, there might be a problem with Jinja | 04:22 |
| kong | rakhmerov :-) | 04:22 |
| rakhmerov | kong: just got to work ) | 04:22 |
| * kong hands over the channel to renat | 04:23 | |
| rakhmerov | :)) | 04:23 |
| rakhmerov | kong: question | 04:23 |
| rakhmerov | wait a sec... | 04:23 |
| kong | sure | 04:24 |
| rakhmerov | https://review.openstack.org/#/c/414508/ | 04:24 |
| rakhmerov | I'm still not sure if we understand each other on this spec | 04:24 |
| rakhmerov | as far as implementing "region" parameter for OpenStack actions first | 04:24 |
| rakhmerov | as the first step I mean | 04:25 |
| rakhmerov | what do you think? | 04:25 |
| kong | rakhmerov: it's doable, although it's a little weird to me | 04:26 |
| rakhmerov | why? | 04:26 |
| kong | after all, 'region' is not included in the action original param list | 04:27 |
| rakhmerov | to me it seems logical to be able to run openstack actions in different regions | 04:27 |
| kong | that's right | 04:27 |
| rakhmerov | yes, I know, but look at this from the user perspective | 04:27 |
| rakhmerov | user story: as a user I want to run any nova stuff in any region | 04:27 |
| rakhmerov | so under the hood we can make this possible | 04:28 |
| kong | i mean, weird to me because i know the exact params when i want to create a instance in Nova | 04:28 |
| kong | as a developer :) | 04:28 |
| rakhmerov | :) | 04:28 |
| kong | but as i said, it's doable and it's more clear from the workflow perspective | 04:29 |
| rakhmerov | yes, but 'region' would be just a cool bonus from Mistral :) | 04:29 |
| rakhmerov | and btw one more reason to use Mistral ) | 04:29 |
| rakhmerov | kong: ok, if we're on the same page here what do we need to do with this spec in your opinion? | 04:30 |
| kong | rakhmerov: do you know it still needs a spec if we only want to add one more param to actions? | 04:31 |
| rakhmerov | hm.. maybe | 04:31 |
| rakhmerov | then I'd suggest we either abandon it for now or make it a WIP | 04:31 |
| kong | ok, i will update the spec anyway | 04:32 |
| rakhmerov | yes | 04:32 |
| rakhmerov | or we can update it | 04:32 |
| rakhmerov | I just thought that besides this param do you think we still need anything else? | 04:32 |
| rakhmerov | on top of it | 04:32 |
| kong | hmm...not for now from what i can see | 04:33 |
| kong | maybe people will have some other ideas when reviewing | 04:33 |
| *** harlowja has quit IRC | 04:33 | |
| rakhmerov | ok | 04:33 |
| rakhmerov | kong: then please take care of the spec one way or another | 04:34 |
| rakhmerov | I'd like to progress with it anyway | 04:34 |
| kong | rakhmerov: sure | 04:34 |
| kong | me too | 04:34 |
| kong | rakhmerov: btw, do you know if 'region' is supported in Heat for different resources? | 04:34 |
| rakhmerov | I don't know, honestly | 04:34 |
| kong | if nested stack is not used | 04:34 |
| rakhmerov | most likely yes | 04:34 |
| rakhmerov | yep | 04:35 |
| kong | i remember 'region' is only supported in stack level | 04:35 |
| kong | not resource | 04:35 |
| kong | just like 'region' is supported in workflow level, not task | 04:35 |
| kong | but not sure, will ask heat guys | 04:35 |
| rakhmerov | ok | 04:39 |
| kong | rakhmerov: do you think it's time for us to deprecate using name as identifier in the api request? | 05:00 |
| rakhmerov | hm.. don't know | 05:01 |
| kong | currently, i'm implementing rbac, admin use will be allowed to operate other tenants resouces | 05:01 |
| rakhmerov | yes, I see | 05:01 |
| kong | :) | 05:01 |
| rakhmerov | hm.. yes, may be | 05:01 |
| rakhmerov | maybe it's time | 05:01 |
| kong | the rbac feature is important to me, not sure we have requirments from others | 05:02 |
| rakhmerov | ok, np | 05:03 |
| rakhmerov | btw, I'm now reviewing your rbac patches ) | 05:03 |
| kong | rakhmerov: thanks | 05:05 |
| kong | i plan to implement workflow and execution first, becuase those are most commonly used | 05:05 |
| *** vishwana_ has joined #openstack-mistral | 05:05 | |
| kong | and after that, will implement client side. | 05:06 |
| kong | then will do for other resources such as cron-triggers, task, action, etc. | 05:06 |
| *** doriftoshoes__ has joined #openstack-mistral | 05:07 | |
| *** szaher_ has joined #openstack-mistral | 05:10 | |
| openstackgerrit | Lingxian Kong proposed openstack/mistral master: Allow admin user to get workflow of other tenants https://review.openstack.org/453020 | 05:11 |
| *** harlowja has joined #openstack-mistral | 05:11 | |
| *** histrio_ has joined #openstack-mistral | 05:13 | |
| kong | rakhmerov: you can see the whole series here: https://review.openstack.org/#/q/topic:bp/mistral-rbac | 05:13 |
| kong | all related patches | 05:13 |
| rakhmerov | ok | 05:13 |
| *** histrio has quit IRC | 05:14 | |
| *** szaher has quit IRC | 05:14 | |
| *** vishwanathj has quit IRC | 05:14 | |
| *** zhenguo has quit IRC | 05:14 | |
| *** doriftoshoes_ has quit IRC | 05:14 | |
| *** histrio_ is now known as histrio | 05:14 | |
| *** jamielennox is now known as jamielennox|away | 05:15 | |
| *** doriftoshoes__ is now known as doriftoshoes_ | 05:15 | |
| * kong is off | 05:16 | |
| rakhmerov | good night :) | 05:17 |
| kong | rakhmerov: enjoy your day :) | 05:17 |
| rakhmerov | thanks buddy | 05:17 |
| openstackgerrit | Merged openstack/mistral master: Remove unnecessary setUp function in testcase https://review.openstack.org/451760 | 05:21 |
| *** jamielennox|away is now known as jamielennox | 05:37 | |
| *** jaosorior_away is now known as jaosorior | 06:01 | |
| *** harlowja has quit IRC | 06:02 | |
| *** shardy has joined #openstack-mistral | 07:09 | |
| *** sharatss has joined #openstack-mistral | 07:12 | |
| *** zhenguo has joined #openstack-mistral | 07:15 | |
| *** shardy has quit IRC | 07:25 | |
| openstackgerrit | Merged openstack/mistral master: Change Http action result content encoding https://review.openstack.org/449143 | 07:26 |
| *** jamielennox is now known as jamielennox|away | 07:45 | |
| openstackgerrit | Kupai József proposed openstack/mistral master: Limit the number of finished executions. https://review.openstack.org/446680 | 07:50 |
| *** amoralej|off is now known as amoralej | 07:53 | |
| jenner_ | rakhmerov: hi again! I've got more questions :) HA - I can see how mistral api or mistra executors can be run in a cluster, i.e. multiple instances of both, balanced (round robin or whatever), but what about the engine? | 07:53 |
| *** jpich has joined #openstack-mistral | 07:54 | |
| jenner_ | also, how can I monitor for execution failures or status changes? I see a couple of *mq sections in the config file, does that mean I can subscribe to e.g. a rabbitmq channel and mistral will publish the results? | 07:58 |
| *** tuan_ has joined #openstack-mistral | 08:03 | |
| *** sharat has joined #openstack-mistral | 08:17 | |
| *** sharatss has quit IRC | 08:17 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Don't create actions when inspection fails https://review.openstack.org/412433 | 08:37 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Don't create actions when inspection fails https://review.openstack.org/412433 | 08:39 |
| rakhmerov | jenner_: hi, as far as engine: same thing, you can run many instances and they will balance workflow processing | 08:40 |
| rakhmerov | even one workflow can be processed by many engines | 08:40 |
| rakhmerov | one task by engine #1, another task by engine #2 etc. | 08:41 |
| rakhmerov | as far as monitoring: currently we don't have a special asynchronous mechanism (discussed it many times but didn't come to agreement among contributors on how it should work) | 08:42 |
| rakhmerov | two options that are available now: polling (via API) and notifying an external system from the workflow itself | 08:42 |
| rakhmerov | the latter means that you can explicitly have WF steps that will send notifications to your system (http or anything else) | 08:43 |
| *** warface has joined #openstack-mistral | 09:06 | |
| *** szaher_ has quit IRC | 09:19 | |
| *** szaher has joined #openstack-mistral | 09:19 | |
| openstackgerrit | Merged openstack/mistral master: Add check for idempotent id in tempest tests https://review.openstack.org/452420 | 09:34 |
| openstackgerrit | Merged openstack/mistral master: Remove unnecessary tearDown function in testcase https://review.openstack.org/452396 | 09:34 |
| *** shardy has joined #openstack-mistral | 09:39 | |
| openstackgerrit | Lingxian Kong proposed openstack/mistral master: Role based resource access control - delete executions https://review.openstack.org/451255 | 09:46 |
| openstackgerrit | Lingxian Kong proposed openstack/mistral master: Role based resource access control - update executions https://review.openstack.org/451160 | 09:46 |
| openstackgerrit | Lingxian Kong proposed openstack/mistral master: Allow admin user to get workflow of other tenants https://review.openstack.org/453020 | 09:46 |
| *** thrash|g0ne is now known as thrash | 09:48 | |
| *** rook is now known as rook|mobile | 10:15 | |
| *** jkilpatr has quit IRC | 10:38 | |
| *** shardy has quit IRC | 10:38 | |
| *** shardy has joined #openstack-mistral | 10:40 | |
| *** shardy_ has joined #openstack-mistral | 10:43 | |
| *** shardy has quit IRC | 10:44 | |
| *** warface has quit IRC | 10:52 | |
| *** warface has joined #openstack-mistral | 10:54 | |
| *** jamielennox|away is now known as jamielennox | 11:03 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Don't create actions when inspection fails https://review.openstack.org/412433 | 11:06 |
| *** warface has quit IRC | 11:09 | |
| *** warface has joined #openstack-mistral | 11:11 | |
| *** jkilpatr has joined #openstack-mistral | 11:18 | |
| jenner_ | rakhmerov: thanks, balancing tasks between engines sounds really nice :) | 11:29 |
| rakhmerov | yep | 11:29 |
| jenner_ | hm, since I'm using a postgres underneatch I could actually LISTEN on a couple of tables and create a trigger to call NOTIFY on updates/inserts | 11:35 |
| jenner_ | *underneath | 11:35 |
| *** bobh has joined #openstack-mistral | 11:58 | |
| *** catintheroof has joined #openstack-mistral | 12:05 | |
| *** amoralej is now known as amoralej|lunch | 12:08 | |
| *** bobh has quit IRC | 12:08 | |
| jenner_ | rakhmerov: can't find anything about load balancing and corresponding setup in the docs :( | 12:08 |
| *** shardy_ is now known as shardy_lunch | 12:08 | |
| *** chlong has joined #openstack-mistral | 12:10 | |
| *** dprince has joined #openstack-mistral | 12:22 | |
| *** jkilpatr has quit IRC | 12:24 | |
| *** warface has quit IRC | 12:25 | |
| *** warface has joined #openstack-mistral | 12:26 | |
| *** shardy_lunch is now known as shardy | 12:40 | |
| *** jkilpatr has joined #openstack-mistral | 12:40 | |
| *** rook|mobile is now known as rook-tower | 12:45 | |
| *** tuan_ has quit IRC | 12:57 | |
| *** tuan_ has joined #openstack-mistral | 13:02 | |
| *** xavierhardy has left #openstack-mistral | 13:06 | |
| *** shardy is now known as shardy_mtg | 13:10 | |
| *** amoralej|lunch is now known as amoralej | 13:14 | |
| *** toure|biab is now known as toure | 13:24 | |
| *** bobh has joined #openstack-mistral | 13:30 | |
| d0ugal | rakhmerov: hey, are you around still? | 13:34 |
| *** jaosorior is now known as jaosorior_away | 13:36 | |
| d0ugal | kong: or maybe I need to speak with you actually :) | 13:44 |
| *** shardy_mtg is now known as shardy | 13:57 | |
| *** brunograz has quit IRC | 14:04 | |
| warface | hey, is anyone there? | 14:15 |
| *** sharat has quit IRC | 14:35 | |
| *** bobh has quit IRC | 14:40 | |
| *** warface has quit IRC | 14:41 | |
| *** tuan_ has quit IRC | 14:41 | |
| * d0ugal is here | 14:42 | |
| *** tuan has joined #openstack-mistral | 14:44 | |
| *** brunograz has joined #openstack-mistral | 14:58 | |
| *** bobh has joined #openstack-mistral | 15:01 | |
| jenner_ | ахахаха https://www.axios.com/computer-programmers-may-no-longer-be-eligible-for-h-1b-visas-2342531251.html | 15:11 |
| d0ugal | :( | 15:14 |
| jenner_ | damn, wrong channel, sorry :) | 15:16 |
| *** jpich has quit IRC | 16:23 | |
| *** amoralej is now known as amoralej|off | 17:20 | |
| *** d0ugal has quit IRC | 17:48 | |
| *** d0ugal has joined #openstack-mistral | 17:49 | |
| *** dprince has quit IRC | 18:27 | |
| *** shardy has quit IRC | 18:32 | |
| *** tuan has quit IRC | 18:51 | |
| *** tuan has joined #openstack-mistral | 18:53 | |
| *** dprince has joined #openstack-mistral | 18:57 | |
| *** harlowja has joined #openstack-mistral | 19:24 | |
| *** dprince has quit IRC | 19:26 | |
| *** dprince has joined #openstack-mistral | 19:29 | |
| *** tuan has quit IRC | 20:17 | |
| *** tuan_ has joined #openstack-mistral | 20:20 | |
| *** jkilpatr has quit IRC | 20:45 | |
| *** dprince has quit IRC | 21:01 | |
| *** jkilpatr has joined #openstack-mistral | 21:23 | |
| kong | d0ugal: hi, i'm online now | 21:24 |
| bobh | kong: got a minute? | 21:40 |
| kong | bobh: i will have stard up meeting soon, you can leave message to me, will ping you when i come back | 21:41 |
| bobh | kong: Thanks. Just following up on env() support in actions - looks like it's not supported for either YAQL or Jinja | 21:43 |
| bobh | kong: just wondering if it would be a good candidate to be added to the action_context | 21:43 |
| kong | bobh: could you please paste your workflow(jinja version) to paste.openstack.org? | 21:51 |
| bobh | kong: will do tonight when I get home. Thanks | 21:56 |
| *** bobh has quit IRC | 22:00 | |
| *** thrash is now known as thrash|g0ne | 22:27 | |
| *** catintheroof has quit IRC | 22:57 | |
| *** bobh has joined #openstack-mistral | 23:11 | |
| *** bobh has quit IRC | 23:40 | |
| *** bobh has joined #openstack-mistral | 23:48 | |
| *** rook-tower is now known as rook | 23:49 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!