Skip to content

Commit

Permalink
All means "all" (#3622)
Browse files Browse the repository at this point in the history
When I added a keyword to select "upload" or "creation" time statistics, I
omitted "statistics" from the effect of `--all`, but I've come to regret that
minor inconvenience. This PR makes `--all` include `--statistics=creation` by
default.
  • Loading branch information
dbutenhof authored May 21, 2024
1 parent 19957c2 commit aa325ce
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/pbench/cli/server/report.py
Original file line number Diff line number Diff line change
Expand Up @@ -779,8 +779,8 @@ def report(context: object, **kwargs):
if kwargs.get("all") or kwargs.get("cache"):
report_cache(cache_m)
stats = kwargs.get("statistics")
if stats:
if stats == "creation":
if stats or kwargs.get("all"):
if stats == "creation" or stats is None:
report_creation(kwargs)
elif stats == "upload":
report_uploads(kwargs)
Expand Down

0 comments on commit aa325ce

Please sign in to comment.