*** openstack has joined #openstack-mistral | 06:59 | |
openstackgerrit | Merged openstack/mistral master: Add hide_args=True to @profiler.trace() where it may cause problems https://review.openstack.org/464150 | 07:07 |
---|---|---|
openstackgerrit | Merged openstack/mistral master: Add 'evaluate_env' workflow parameter https://review.openstack.org/464158 | 07:10 |
openstackgerrit | Merged openstack/mistral master: Release notes for "evaluate_env" https://review.openstack.org/464160 | 07:10 |
rakhmerov | kong, d0ugal: stupid question.. When we're backporting a fix to a stable branch, do we need to backport a release note for it also? | 07:22 |
d0ugal | rakhmerov: yes, I think so | 07:32 |
rakhmerov | ok ) | 07:32 |
d0ugal | rakhmerov: which is another reason why we should include release notes and documentation etc. in the same commit ;) | 07:32 |
* d0ugal ducks | 07:33 | |
rakhmerov | aah | 07:33 |
rakhmerov | :) | 07:33 |
*** jpich has joined #openstack-mistral | 07:57 | |
*** jaosorior has joined #openstack-mistral | 08:09 | |
*** Qiming_ is now known as Qiming | 08:45 | |
*** shardy has joined #openstack-mistral | 08:49 | |
*** jistr_ is now known as jistr | 09:30 | |
openstackgerrit | luong tuan proposed openstack/mistral master: Add user creds trusts to db https://review.openstack.org/462056 | 09:36 |
*** ddeja_ is now known as ddeja | 09:48 | |
*** sharat has quit IRC | 10:00 | |
*** abishop has joined #openstack-mistral | 10:07 | |
*** sharat has joined #openstack-mistral | 10:09 | |
breton | first_execution_time must be at least 1 minute in the future. | 10:11 |
breton | nice! | 10:11 |
*** szaher_ has quit IRC | 10:14 | |
*** szaher has joined #openstack-mistral | 10:15 | |
breton | i would really like to see revert https://review.openstack.org/#/c/463455/ reverted | 10:19 |
therve | Well the original patch should be fixed and resubmitted | 10:21 |
breton | i am trying to use a cron trigger and get this: http://paste.openstack.org/show/609534/ | 10:22 |
breton | any idea why this happens? | 10:23 |
breton | workflow works fine outside of cron trigger | 10:24 |
thrash | breton: that looks like a genuine bug. Can you file one in launchpad please? | 10:24 |
*** shardy is now known as shardy_afk | 10:24 | |
breton | >_> | 10:24 |
breton | ok | 10:24 |
thrash | breton: I assume you are running from master? | 10:25 |
breton | thrash: no, Ocata from RDO | 10:26 |
thrash | breton: Interesting... | 10:26 |
therve | breton, Can you check the token, see if it expired? | 10:30 |
breton | therve: it's definitely not expired | 10:30 |
therve | So I don't know why it's requesting another token then | 10:31 |
breton | everything was restarted and recreated in the last 15 minutes, and token expiration is 60 minutes | 10:32 |
breton | the token for the action is issued at 2017-05-15T10:20:00.000000Z and expires at 2017-05-15T11:20:00.000000Z | 10:38 |
d0ugal | rakhmerov: Why do we need to reinstall mistral for new custom actions? | 10:39 |
d0ugal | I never reinstall... | 10:39 |
d0ugal | Just restart. | 10:39 |
rakhmerov | d0ugal: hi, reading the context.. | 10:40 |
d0ugal | rakhmerov: [openstack-dev] [OpenStack][Mistral]A simple tool to discover and manage custom actions | 10:40 |
d0ugal | rakhmerov: I've not read the above chat :) | 10:40 |
rakhmerov | d0ugal: sorry, was distracted | 10:47 |
d0ugal | np | 10:47 |
rakhmerov | well, mistral doesn't have to be reinstalled if actions are implemented in a separate project | 10:48 |
rakhmerov | but, in my understanding, if you add new entry points into setup.cfg then the project needs to be reinstalled so that stevedore can pick them up | 10:49 |
*** gongysh has quit IRC | 10:49 | |
rakhmerov | because, during installation, it registers those additional entry points | 10:49 |
rakhmerov | breton: yeah, it seems to be something bad.. we'll have to look at it | 10:49 |
breton | rakhmerov: https://bugs.launchpad.net/mistral/+bug/1690787 :) | 10:50 |
openstack | Launchpad bug 1690787 in Mistral "cron trigger uses trust-scoped token to create another token" [Undecided,New] | 10:50 |
rakhmerov | hm... that's the area where have a mess | 10:51 |
rakhmerov | unfortunately | 10:51 |
rakhmerov | ...we have.. | 10:51 |
rakhmerov | breton: so that's what we really do with cron triggers. It used to work. When we're creating a cron trigger we create a trust for it immediately | 10:54 |
rakhmerov | and then try to authenticate all operations by creating tokens based on this trust | 10:54 |
rakhmerov | I guess it's changed at some point and stopped working | 10:54 |
*** gongysh has joined #openstack-mistral | 10:56 | |
*** jkilpatr has joined #openstack-mistral | 10:57 | |
rakhmerov | breton: so we need to rethink on this solution, especially given other issues we have with auth (e.g. https://bugs.launchpad.net/mistral/+bug/1595084 ) | 10:58 |
openstack | Launchpad bug 1595084 in Mistral "Workflow execution lifespan is limited by auth token expire time" [High,Confirmed] | 10:58 |
breton | rakhmerov: i guess all this operations still work. But something tries to refetch the token. | 10:58 |
breton | *these | 10:58 |
rakhmerov | yeah | 10:59 |
openstackgerrit | Merged openstack/mistral-dashboard master: Updated from global requirements https://review.openstack.org/464401 | 11:02 |
*** rbrady_ has joined #openstack-mistral | 11:13 | |
*** rbrady_ has quit IRC | 11:13 | |
*** rbrady_ has joined #openstack-mistral | 11:14 | |
*** sharat has quit IRC | 11:17 | |
*** sharat has joined #openstack-mistral | 11:17 | |
*** gongysh has quit IRC | 11:21 | |
*** shardy_afk is now known as shardy | 11:47 | |
*** sharat has quit IRC | 11:48 | |
*** sharat has joined #openstack-mistral | 11:48 | |
*** dprince has joined #openstack-mistral | 11:52 | |
*** dprince has quit IRC | 11:56 | |
*** shardy_ has joined #openstack-mistral | 11:59 | |
d0ugal | rakhmerov: sorry, I also got distrated. Right, that makes sense - the wording in the docs makes it sound like third party actions require a reinstall I think | 12:04 |
breton | so 1690787 is pretty interesting | 12:10 |
breton | i guess it happened because novaclient moved to keystoneauth sessions | 12:10 |
breton | and keystoneauth always tries to reauthenticate | 12:11 |
breton | i am trying to figure out how we can force keystoneauth not to reauth, but the proper answer i guess is to use sessions | 12:19 |
*** sharat has quit IRC | 12:30 | |
*** dprince has joined #openstack-mistral | 12:30 | |
*** flaper87 has quit IRC | 12:43 | |
breton | in the logs i see message http://paste.openstack.org/show/609558/ . There is already a token. Where does that token gets fetched? | 12:49 |
*** jaosorior has quit IRC | 13:15 | |
*** ddeja is now known as zona_ddeja | 13:17 | |
*** zona_ddeja is now known as ddeja | 13:17 | |
*** toure|gone is now known as toure | 13:30 | |
*** fultonj has joined #openstack-mistral | 13:35 | |
*** mbrennan has joined #openstack-mistral | 13:36 | |
*** gongysh has joined #openstack-mistral | 13:44 | |
*** gongysh has quit IRC | 13:44 | |
*** gongysh has joined #openstack-mistral | 13:58 | |
*** gongysh has quit IRC | 14:09 | |
openstackgerrit | Merged openstack/mistral-dashboard master: Added link to Create Action form Edit https://review.openstack.org/453656 | 14:11 |
*** aspiers_ has joined #openstack-mistral | 14:47 | |
*** fultonj has quit IRC | 14:53 | |
d0ugal | rakhmerov: Hey | 14:53 |
*** mbrennan has quit IRC | 14:53 | |
*** fultonj has joined #openstack-mistral | 14:54 | |
*** sharatss_ has joined #openstack-mistral | 14:57 | |
rakhmerov | hi | 14:57 |
sharatss_ | rakhmerov: hi | 14:58 |
rakhmerov | d0ugal, toure, thrash, rbrady, sharatss_: we're about to start the meeting in webex | 14:58 |
d0ugal | rakhmerov: I am just figuring out how to join | 14:58 |
d0ugal | rakhmerov: I need to install something :( | 14:58 |
rakhmerov | ooh | 14:58 |
rakhmerov | yes | 14:59 |
rakhmerov | we'll wait, no problem | 14:59 |
thrash | rakhmerov: where is the webex info? | 15:00 |
*** bobh has joined #openstack-mistral | 15:01 | |
apetrich_ | rakhmerov, ^^ me too | 15:01 |
rakhmerov | https://nokiameetings.webex.com/nokiameetings/j.php?MTID=ma568fbd65db98eeb7b927ec96a14a018 | 15:01 |
apetrich_ | rakhmerov, cheers | 15:03 |
*** dprince has quit IRC | 15:14 | |
*** sharatss_ has quit IRC | 15:21 | |
*** mbrennan has joined #openstack-mistral | 15:40 | |
*** jaosorior has joined #openstack-mistral | 15:51 | |
apetrich_ | rakhmerov, I think it is a great tool for operations | 15:58 |
rakhmerov | yep, it should be | 15:58 |
apetrich_ | rakhmerov, in a sense that you can let people that use mistral without knowing that they are using mistral know what is the status for their workflows | 15:58 |
rakhmerov | I'm looking forward to open sourcing it | 15:58 |
apetrich_ | aye! | 15:59 |
rakhmerov | d0ugal: TripleO gate is failing on https://review.openstack.org/#/c/460351, do you think it's related to the patch? | 16:02 |
d0ugal | rakhmerov: looking | 16:02 |
rakhmerov | ok | 16:02 |
rakhmerov | please leave a comment in the patch | 16:02 |
rakhmerov | I'm out for now | 16:03 |
d0ugal | will do | 16:03 |
openstackgerrit | Bob Haddleton proposed openstack/mistral master: Update AdHoc Actions to support env() references in YAQL/Jinja expressions https://review.openstack.org/464677 | 16:03 |
*** jpich has quit IRC | 16:11 | |
*** dprince has joined #openstack-mistral | 16:14 | |
*** harlowja has joined #openstack-mistral | 16:44 | |
*** bobh has quit IRC | 16:58 | |
*** livelace has quit IRC | 17:41 | |
breton | so... | 17:46 |
breton | does a rpc call convert MistralContext to json? | 17:47 |
breton | i am trying to figure out how token is passed from whatever triggers workflow exection to workflow itself | 17:48 |
breton | i think i have found the place where trust-scoped token is obtained, and wanted to wrap things there into session | 17:48 |
breton | but got this: http://paste.openstack.org/show/609598/ | 17:50 |
breton | i just added a new key to the context in context.py and to the place where the context is set | 17:50 |
*** bobh has joined #openstack-mistral | 17:58 | |
*** shardy is now known as shardy_afk | 17:59 | |
*** shardy_afk is now known as shardy | 18:00 | |
*** shardy is now known as shardy_afk | 18:02 | |
*** bobh has quit IRC | 18:04 | |
*** mbrennan has quit IRC | 18:31 | |
*** bobh has joined #openstack-mistral | 18:32 | |
*** mbrennan has joined #openstack-mistral | 18:32 | |
bobh | !list | 18:44 |
openstack | bobh: Admin, Bugtracker, Channel, ChannelLogger, Config, MeetBot, Misc, Owner, Services, and User | 18:44 |
openstackgerrit | Toure Dunnon proposed openstack/mistral master: [WIP] Workflow Error Analysis (do not merge!) https://review.openstack.org/455447 | 18:45 |
*** shardy_afk has quit IRC | 19:55 | |
*** mbrennan has quit IRC | 20:27 | |
*** mbrennan has joined #openstack-mistral | 20:29 | |
*** Kevin_Zheng has quit IRC | 21:21 | |
*** gongysh has joined #openstack-mistral | 21:26 | |
*** gongysh has quit IRC | 21:27 | |
*** jkilpatr has quit IRC | 21:32 | |
*** jkilpatr has joined #openstack-mistral | 22:00 | |
*** bobh has quit IRC | 22:00 | |
*** bobh has joined #openstack-mistral | 22:01 | |
*** dprince has quit IRC | 22:04 | |
*** bobh has quit IRC | 22:06 | |
*** mbrennan has quit IRC | 22:24 | |
*** gongysh has joined #openstack-mistral | 22:39 | |
*** gongysh has quit IRC | 22:40 | |
*** thrash is now known as thrash|g0ne | 22:43 | |
openstackgerrit | Lingxian Kong proposed openstack/mistral master: Add "action_region" param for OpenStack actions https://review.openstack.org/460351 | 23:31 |
*** harlowja has quit IRC | 23:46 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!