Skip to content

Conversation

kdvalin
Copy link
Member

@kdvalin kdvalin commented Aug 25, 2025

Description

This removes all pbench arguments as it is no longer supported by Zathras.

Any configuration files will need to remove the use of these arguments or at least avoid going down code paths that do use them.

Before/After Comparison

Before

--pbench* options are available but will inevitably lead to broken functionality

After

The options are removed to prevent problems and broken functionality.

Clerical Stuff

Issue TBD

Relates to JIRA: RPOPC-<TICKET_NUMBER>

This removes all pbench arguments as it is no longer supported by
Zathras.  Any configuration files will need to remove the use of these
 arguments or at least avoid going down code paths that do use them.

BREAKING CHANGE: Pbench argument removal

Co-developed-by: Cursor
@kdvalin kdvalin requested a review from a team August 25, 2025 18:16
Copy link
Contributor

@frival frival left a comment

Choose a reason for hiding this comment

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

LGTM. We'll have to add back run_label at some point as part of the PCP work when we get to long-term storage support, but for now it's a do-nothing arg.

@dvalinrh dvalinrh added the group_review_lgtm Indicates approval after a group review meeting label Sep 2, 2025

to_puser=`whoami`
to_run_user=`whoami`
Copy link
Contributor

Choose a reason for hiding this comment

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

Needs to be present.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
group_review_lgtm Indicates approval after a group review meeting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants