Skip to content

Bumping default iozone version to 508#63

Open
malucius-rh wants to merge 1 commit into
masterfrom
bump_to_508
Open

Bumping default iozone version to 508#63
malucius-rh wants to merge 1 commit into
masterfrom
bump_to_508

Conversation

@malucius-rh
Copy link
Copy Markdown
Contributor

@malucius-rh malucius-rh commented May 8, 2026

Description

This bumps the default version of IOzone installed by the wrapper from 3.490 (dated and not marked "stable" upstream) to 3.508 (current stable release). There are no changes to wrapper logic or output or usage of iozone itself.

Before/After Comparison

Before: If no iozone exists on the test system the wrapper will install version 3.490.
After: If no iozone exists on the test system the wrapper will install version 3.508

Clerical Stuff

This closes #60
Relates to JIRA: RPOPC-920

Logs iozone_490_to_508.txt

@malucius-rh malucius-rh requested a review from a team May 8, 2026 15:41
@malucius-rh malucius-rh self-assigned this May 8, 2026
@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

Update default IOzone version to 3.508

✨ Enhancement

Grey Divider

Walkthroughs

Description
• Updates default IOzone version from 3.490 to 3.508
• Installs current stable release instead of dated version
• No changes to wrapper logic or functionality
Diagram
flowchart LR
  A["IOzone Wrapper"] -- "Install default version" --> B["iozone3_508<br/>Current Stable"]
  C["iozone3_490<br/>Dated Version"] -.->|replaced| B
Loading

Grey Divider

File Changes

1. iozone/iozone_run.sh ⚙️ Configuration changes +1/-1

Bump default IOzone version to 3.508

• Updates iozone_kit variable from iozone3_490 to iozone3_508
• Changes default IOzone version installed when not present on system
• No modifications to wrapper logic or output behavior

iozone/iozone_run.sh


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

qodo-code-review Bot commented May 8, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

Copy link
Copy Markdown

@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, but in the spirit of putting useful stuff in the PR I'd really like to see before and after output from the test. Extra eyes on test output changes are useful to spot things the developer missed, and we've been bitten too many times in the past to not ask for those test artifacts rather than just "I tested it".

@malucius-rh
Copy link
Copy Markdown
Contributor Author

LGTM, but in the spirit of putting useful stuff in the PR I'd really like to see before and after output from the test. Extra eyes on test output changes are useful to spot things the developer missed, and we've been bitten too many times in the past to not ask for those test artifacts rather than just "I tested it".

They're coming, I couldn't find the ones I did before so I'm getting fresh and didn't bother marking "draft."

@malucius-rh malucius-rh requested a review from frival May 12, 2026 15:18
Copy link
Copy Markdown

@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.

Thanks for the logs, it all LGTM now.

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.

IOzone kit should be updated to 508

2 participants