Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI, TST: simplified automated build script + CI workflows #986

Merged
merged 11 commits into from
May 3, 2024

Conversation

shanedsnyder
Copy link
Contributor

@shanedsnyder shanedsnyder commented May 2, 2024

  • Add new build script build-darshan.sh intended to simplify building/testing of darshan-runtime and darshan-util in some of our automated testing frameworks. You can provide the following env vars to control some aspects of the build:
    • DARSHAN_INSTALL_PREFIX - path to install Darshan
    • DARSHAN_LOG_PATH - formatted path to store Darshan logs
    • DARSHAN_RUNTIME_CONFIG_ARGS - darshan-runtime config args
    • DARSHAN_UTIL_CONFIG_ARGS - darshan-util config args
    • DARSHAN_RUNTIME_SKIP - skip darshan-runtime build if set
  • Updated GitHub workflows and cibuildwheel process to use this script
  • Removed old automated testing scripts for building Darshan, which were tied to a Jenkins plugin and less flexible.
  • Removed now decomissioned ALCF Theta regression testing environment
  • Updated Python test case to account for new build directory organization from this new script.

The script aims to simplify a lot of our automated builds and
supports the following env vars:
 - DARSHAN_INSTALL_PREFIX - path to install Darshan
 - DARSHAN_LOG_PATH - formatted path to store Darshan logs
 - DARSHAN_RUNTIME_CONFIG_ARGS - darshan-runtime config args
 - DARSHAN_UTIL_CONFIG_ARGS - darshan-util config args
 - DARSHAN_RUNTIME_SKIP - skip darshan-runtime build if set
@github-actions github-actions bot added the CI continuous integration label May 2, 2024
@shanedsnyder shanedsnyder force-pushed the snyder/dev-automated-build-script branch from 39f5cd8 to 2574ca6 Compare May 2, 2024 20:36
@shanedsnyder shanedsnyder force-pushed the snyder/dev-automated-build-script branch from 2707c06 to 9839728 Compare May 2, 2024 20:54
- remove old automated scripts tied to Jenkins plugins
- remove deprecated Theta regression test environment
- updated README
@shanedsnyder shanedsnyder force-pushed the snyder/dev-automated-build-script branch from d49bdf2 to 0af299f Compare May 2, 2024 22:44
@shanedsnyder shanedsnyder requested a review from carns May 3, 2024 17:36
Copy link
Contributor

@carns carns left a comment

Choose a reason for hiding this comment

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

very nice cleanup, looks good to me

@shanedsnyder shanedsnyder merged commit 59f5628 into main May 3, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI continuous integration pydarshan
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants