Skip to content

Need to return the rtc from verify#53

Merged
dvalinrh merged 2 commits into
masterfrom
validate
Feb 6, 2026
Merged

Need to return the rtc from verify#53
dvalinrh merged 2 commits into
masterfrom
validate

Conversation

@dvalinrh

@dvalinrh dvalinrh commented Feb 6, 2026

Copy link
Copy Markdown
Contributor

Description

We are not returning the rtc from the results verification

Before/After Comparison

Before: we verified the data but did not return the rtc.
After: we verify the data and return the rtc.

Clerical Stuff

This closes #52

Relates to JIRA: RPOPC-831

Test Command executed
/home/ec2-user/workloads/specjbb-wrapper/specjbb/specjbb_run --run_user ec2-user --home_parent /home --iterations 1 --tuned_setting tuned_none_sys_file_ --host_config m5.xlarge --sysname m5.xlarge --sys_type aws --use_pcp --java_version 21

csv of successful run
Warehouses,Bops,Numb_JVMs,Start_Date,End_Date
2,109974,1,2026-02-06T10:35:03Z,2026-02-06T10:39:10Z
4,147291,1,2026-02-06T10:35:03Z,2026-02-06T10:39:10Z
6,139581,1,2026-02-06T10:35:03Z,2026-02-06T10:39:10Z
8,131452,1,2026-02-06T10:35:03Z,2026-02-06T10:39:10Z

rtc of successful run: 0

rtc of induced failure: 1

@dvalinrh dvalinrh requested a review from kdvalin February 6, 2026 12:34

@kdvalin kdvalin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@dvalinrh dvalinrh merged commit d154062 into master Feb 6, 2026
2 of 3 checks passed
@dvalinrh dvalinrh deleted the validate branch March 26, 2026 18:16
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.

Need to return the rtc from the verification

2 participants