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

ShellJob: Customize the _build_process_label method #17

Merged
merged 1 commit into from
Nov 6, 2022

Conversation

sphuber
Copy link
Owner

@sphuber sphuber commented Nov 4, 2022

Fixes #7

This method returns the string that is set as the process_label attribute on the nodes representing executions of the process class. By default this is the name of the process class, i.e., ShellJob in this case. This is not very informative as all ShellJob runs, regardless of the shell command it ran, will get the same label.

Here we override the _build_process_label to include the full label of the code, which in turn will include the label (typically the name of the shell command) and the name of the computer on which is was run. An example is ShellJob<echo@localhost>.

@sphuber sphuber added pr/blocked PR is blocked by another PR that should be merged first and removed pr/blocked PR is blocked by another PR that should be merged first labels Nov 4, 2022
@sphuber sphuber force-pushed the feature/007/custom-process-label branch from 8bfc496 to 6ffe0db Compare November 6, 2022 13:13
This method returns the string that is set as the `process_label`
attribute on the nodes representing executions of the process class. By
default this is the name of the process class, i.e., `ShellJob` in this
case. This is not very informative as all `ShellJob` runs, regardless of
the shell command it ran, will get the same label.

Here we override the `_build_process_label` to include the full label of
the code, which in turn will include the label (typically the name of
the shell command) and the name of the computer on which is was run.
An example is `ShellJob<echo@localhost>`.
@sphuber sphuber force-pushed the feature/007/custom-process-label branch from 6ffe0db to a754891 Compare November 6, 2022 13:22
@sphuber sphuber merged commit 506fe91 into master Nov 6, 2022
@sphuber sphuber deleted the feature/007/custom-process-label branch November 6, 2022 13:24
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.

See if the process_label can be more specific
1 participant