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

Add jobby output to console log #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mgbolivierdahinden
Copy link

For some time I was in need to use jobby with console output. Since
we are working in cloud foundry environment and we capture application
log with a drainlog this was quite critical.

This commit introduces the config flag output_console which takes
a boolean to either activate the output to console. What it actually
does is remove the bash output to logs and puts an echo infront of
the exec().

While at it: Add composer script for tests. Add 2 testcases for
closures and files.

@mgbolivierdahinden
Copy link
Author

I'm sure this is not yet ready since it's not that clean. But maybe it is inspiring and maybe bringing another solution from you guys.

For some time I was in need to use jobby with console output. Since
we are working in cloud foundry environment and we capture application
log with a drainlog this was quite critical.

This commit introduces the config flag `output_console` which takes
a boolean to either activate the output to console. What it actually
does is remove the bash output to logs and puts an echo infront of
the `exec()`.

While at it: Add composer script for tests. Add 2 testcases for
closures and files.
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.

1 participant