Skip to content

Conversation

@malucius-rh
Copy link
Contributor

Description

This adds handling of package dependencies to the wrapper based on what's in the wrapper's json file, and adds said json file.

Before/After Comparison

Before: packages are installed by the parent harness
After: packages are installed by the wrapper based on contents of associated json file.
I watched the sequence work properly on rhel and ubuntu

Clerical Stuff

This closes #15

Relates to JIRA: RPOPC-648

Add json file for package_tool
Add handling of package dependencies based on what's in the wrapper's json file
@malucius-rh malucius-rh requested a review from a team October 21, 2025 00:01
@malucius-rh malucius-rh self-assigned this Oct 21, 2025
@malucius-rh malucius-rh added the enhancement New feature or request label Oct 21, 2025
@kdvalin kdvalin added the group_review_lgtm Indicates approval after a group review meeting label Oct 28, 2025
Co-authored-by: Keith Valin <[email protected]>
kdvalin
kdvalin previously approved these changes Oct 28, 2025
Copy link
Member

@kdvalin kdvalin left a comment

Choose a reason for hiding this comment

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

LGTM

done

# Install needed packages based on what's listed in the wrapper's json file
${TOOLS_BIN}/package_tool --no_packages $to_no_pkg_install -wrapper_config ${run_dir}/io_burst-wrapper.json
Copy link

Choose a reason for hiding this comment

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

Is it -wrapper_config or --wrapper_config?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's where the extra dash came from.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request group_review_lgtm Indicates approval after a group review meeting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Need to move to new package dependency management scheme

4 participants