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 the --paused flag to verdi process list #4213

Merged

Conversation

sphuber
Copy link
Contributor

@sphuber sphuber commented Jul 1, 2020

Fixes #4212

This flag will filter for processes that are currently paused which is
useful to find calculation jobs that may have hit the exponential
backoff mechanism, among other things.

@sphuber sphuber requested a review from chrisjsewell July 1, 2020 21:58
This flag will filter for processes that are currently paused which is
useful to find calculation jobs that may have hit the exponential
backoff mechanism, among other things.
@sphuber sphuber force-pushed the feature/4212/verdi-process-list-paused branch from 66d3213 to 95b7435 Compare July 1, 2020 21:58
@codecov
Copy link

codecov bot commented Jul 1, 2020

Codecov Report

Merging #4213 into develop will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #4213      +/-   ##
===========================================
+ Coverage    79.16%   79.17%   +0.01%     
===========================================
  Files          468      468              
  Lines        34463    34467       +4     
===========================================
+ Hits         27280    27285       +5     
+ Misses        7183     7182       -1     
Flag Coverage Δ
#django 71.09% <100.00%> (+0.01%) ⬆️
#sqlalchemy 71.92% <100.00%> (+0.01%) ⬆️
Impacted Files Coverage Δ
aiida/cmdline/utils/query/mapping.py 80.29% <ø> (ø)
aiida/cmdline/commands/cmd_process.py 74.78% <100.00%> (+0.12%) ⬆️
aiida/cmdline/utils/query/calculation.py 91.43% <100.00%> (+0.39%) ⬆️
aiida/cmdline/utils/query/formatting.py 92.00% <0.00%> (+4.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 855ae82...95b7435. Read the comment docs.

@@ -324,6 +324,8 @@ def decorator(command):
help='Only include entries with this process state.'
)

PAUSED = OverridableOption('--paused', 'paused', is_flag=True, help='Only include entries that are paused.')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Running out of letters lol; I see p and P are already taken

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I figured this might not be used that often so it's okay to only have the long form, which is not that long anyway

@sphuber sphuber merged commit 7af3b48 into aiidateam:develop Jul 2, 2020
@sphuber sphuber deleted the feature/4212/verdi-process-list-paused branch July 2, 2020 05:50
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.

Add the flag --paused to verdi process list
2 participants