Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Alow more task summary info as event handler args. #2458

Merged
merged 2 commits into from
Nov 3, 2017

Conversation

hjoliver
Copy link
Member

Following on from #2441 - allow remaining task summary data as event handler args: submit_time, start_time, finish_time, user@host. This lets us use event handler scripts for routine reporting, at least until we hash out the details of #2442

@cylc/core do you see any reason not to do this, before I go ahead with documentation and extending the tests?

@hjoliver hjoliver added this to the soon milestone Oct 23, 2017
@hjoliver hjoliver self-assigned this Oct 23, 2017
@matthewrmshin
Copy link
Contributor

Looks like a good move. (One day, we'll get TaskProxy.summary set up correctly so we don't have to build another dict!)

@hjoliver
Copy link
Member Author

Existing test extended to cover the new args.

@matthewrmshin matthewrmshin modified the milestones: soon, next release Nov 2, 2017
@hjoliver hjoliver force-pushed the more-event-handler-args branch from 6d15aef to 36500b8 Compare November 3, 2017 00:51
@hjoliver
Copy link
Member Author

hjoliver commented Nov 3, 2017

Rebased to bring in the pep8 -> pycodestyle change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants