Skip to content

Commit 4dde6ec

Browse files
authored
Merge pull request #3258 from vkarak/enhancement/change-storage-opts-defaults
[enhancement] Change default values for `--performance-report` and `--list-stored-sessions` options
2 parents 262b1f3 + b12f61d commit 4dde6ec

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

docs/manpage.rst

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -349,7 +349,8 @@ There are commands that act upon the selected tests and others that have a helpe
349349

350350
List sessions stored in the results database.
351351

352-
If ``TIME_PERIOD`` is not specified or if ``all`` is passed, all stored sessions will be listed.
352+
If ``TIME_PERIOD`` is ``all``, all stored sessions will be listed.
353+
If not specified, only the sessions of last week will be listed.
353354
For the exact syntax of ``TIME_PERIOD`` check the :ref:`time-period-syntax`.
354355

355356
.. versionadded:: 4.7
@@ -1096,7 +1097,7 @@ Miscellaneous options
10961097

10971098
For each test all of their performance variables are reported and optionally compared to past results based on the ``CMPSPEC`` specified.
10981099

1099-
If not specified, the default ``CMPSPEC`` is ``19700101T0000+0000:now/last:+job_nodelist/+result``, meaning that the current performance will be compared to the last run of the same test grouped additionally by the ``job_nodelist`` and showing also the obtained result (``pass`` or ``fail``).
1100+
If not specified, the default ``CMPSPEC`` is ``now:now/last:/+job_nodelist+result``, meaning that the current performance will not be compared to any past run and, additionally, the ``job_nodelist`` and the test result (``pass`` or ``fail``) will be listed.
11001101

11011102
For the exact syntax of ``CMPSPEC``, refer to :ref:`performance-comparisons`.
11021103

reframe/frontend/cli.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -431,8 +431,8 @@ def main():
431431
'providing more details')
432432
)
433433
action_options.add_argument(
434-
'--list-stored-sessions', nargs='?', action='store', const='all',
435-
metavar='PERIOD', help='List stored sessions'
434+
'--list-stored-sessions', nargs='?', action='store',
435+
const='now-1w:now', metavar='PERIOD', help='List stored sessions'
436436
)
437437
action_options.add_argument(
438438
'--list-stored-testcases', action='store',
@@ -607,7 +607,7 @@ def main():
607607
configvar='general/perf_report_spec',
608608
envvar='RFM_PERF_REPORT_SPEC',
609609
help=('Print a report for performance tests '
610-
'(default: "19700101T0000+0000:now/last:+job_nodelist/+result")')
610+
'(default: "now:now/last:+job_nodelist/+result")')
611611
)
612612

613613
# Miscellaneous options

reframe/schemas/config.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -582,7 +582,7 @@
582582
"general/module_mappings": [],
583583
"general/non_default_craype": false,
584584
"general/perf_info_level": "info",
585-
"general/perf_report_spec": "19700101T0000+0000:now/last:+job_nodelist/+result",
585+
"general/perf_report_spec": "now:now/last:/+job_nodelist+result",
586586
"general/purge_environment": false,
587587
"general/remote_detect": false,
588588
"general/remote_workdir": ".",

0 commit comments

Comments
 (0)