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

CLI options added #351

Merged
merged 6 commits into from
May 15, 2023
Merged

CLI options added #351

merged 6 commits into from
May 15, 2023

Conversation

cms21
Copy link
Contributor

@cms21 cms21 commented May 10, 2023

This PR aims to address issues #309, #341, #258 and #322. It includes:

  • option to delete all apps from a site

balsam app rm --all --site=my-site

  • option to modify job state based on tags

balsam job modify --tag species=H20 -s RESTART_READY

  • option to force deletion of site, even if path doesn't exist

balsam site rm -f my-site-directory

  • option to verbose list batch jobs

balsam queue ls -v

  • option to list batch jobs by scheduler id

balsam queue ls --id 12345

  • option to list the last N batch jobs

balsam queue ls -n 5

The default behavior of CL commands remain unchanged, except for balsam queue ls. If there are no active batch jobs, balsam queue ls now will return by default the records of the last 3 Batch Jobs.

@codecov-commenter
Copy link

codecov-commenter commented May 10, 2023

Codecov Report

Patch coverage: 12.62% and project coverage change: -0.29 ⚠️

Comparison is base (692d2d9) 61.26% compared to head (6112efe) 60.98%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #351      +/-   ##
==========================================
- Coverage   61.26%   60.98%   -0.29%     
==========================================
  Files         157      157              
  Lines        9541     9598      +57     
  Branches     1233     1254      +21     
==========================================
+ Hits         5845     5853       +8     
- Misses       3437     3486      +49     
  Partials      259      259              
Impacted Files Coverage Δ
balsam/cmdline/app.py 22.38% <7.69%> (-12.62%) ⬇️
balsam/cmdline/scheduler.py 31.11% <11.42%> (-1.36%) ⬇️
balsam/cmdline/site.py 30.00% <16.00%> (-1.75%) ⬇️
balsam/cmdline/job.py 29.53% <50.00%> (+0.05%) ⬆️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

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