-
Notifications
You must be signed in to change notification settings - Fork 31
Commpath #117
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
Open
rountree
wants to merge
34
commits into
llnl:devel
Choose a base branch
from
rountree:commpath
base: devel
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Commpath #117
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
c98f70f
Cachepath: parse_loc utilities.
rountree-alt 77fd7d3
Cachepath: remove/rename [orig_]location.
rountree-alt c3dba38
Cachepath: Configure-time support.
rountree-alt 7e1715d
Cachepath: Internal messaging for path resolution
rountree-alt 2020634
Cachepath: Adds cobo_allreduce()
rountree-alt dfdf166
Cachepath: Adds parameters to config_mgr
rountree-alt 38bacd3
Cachepath: adds flux parameter support
rountree-alt 6d34e18
Cachepath: Adds logging support.
rountree-alt 69f03de
Cachepath: Removes out-of-root cleanup checks.
rountree-alt 36540be
Cachepath: Set of small, miscellaneous patches.
rountree-alt 0eb779a
Fixes per Matt's comments.
rountree-alt 61ba31c
Single source of truth for client cachepath.
rountree-alt 78ee5ff
Comments the cachepath variables.
rountree-alt f9aa472
Removes internal vars from spindle_launch.h
rountree-alt ef75381
Client cachepath message now uses single response.
rountree-alt 3c83f6c
Removes assert(0) in network error paths.
rountree-alt 1134a1c
Renames ldcs_audit_server_md_consensus().
rountree-alt 57eb3fe
Adds explicit enum values to CmdlineShortOptions.
rountree-alt 4a43875
Return instead of exit on network errors.
rountree-alt 97ae725
Apply rename to configuration and parameters.
rountree-alt 9756688
Renaming location variables/fields to commpath.
rountree-alt 2b0acc0
Renames enums.
rountree-alt fa47cf1
Use strdup() for commpath instead of stack var.
rountree-alt e025e83
Adds LDCS_COMMPATH
rountree-alt a2c25ac
Restores checkLinkForLeak() to test_driver.c
rountree-alt 77b7375
Replacing "location" with "commpath" as needed.
rountree-alt 332e67f
Continues location rename.
rountree-alt 193cba2
Fixes -Wsign-compare warning in new code.
rountree-alt 5f3a1df
LDCS_CHOSEN_PARSED_CACHEPATH set in bootstrap.
rountree-alt 857f9e7
Updates test_driver.c to ignore FIFO files.
rountree-alt 5e508a4
Bump docker/setup-buildx-action in /.github/workflows
dependabot[bot] 7ce216c
Bump actions/checkout from 6.0.0 to 6.0.1 in /.github/workflows
dependabot[bot] 8de2636
Sets TMPDIR=/tmp in each Dockerfile
rountree-alt 5771e6c
It's a trivial change...
rountree-alt File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The configure script will give a warning for unrecognized arguments, so removing the
--with-localstoragecauses builds that still specify--with-localstorageto configure without error, but the option is unused. For example, the script in the CI environment that builds Spindle uses--with-localstorage=/tmp, and you can see the warning in the CI output:https://github.com/llnl/Spindle/actions/runs/21076617750/job/60620001124?pr=117#step:4:826
I wonder if it would make sense to either:
--with-localstorageis specified, where the error message says that--with-cachepathsand--with-commpathshould be used instead; or,--with-localstorageand use it to set the cachepaths and commpath value.We should also change the Spindle configure line used in CI to use the new arguments instead of
--with-localstorage, incontainers/spindle-serial-ubuntu/scripts/build_spindle.sh,containers/spindle-flux-ubuntu/scripts/build_spindle.sh, andcontainers/spindle-slurm-ubuntu/testing/scripts/build_spindle.sh.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per Matt, map this to both cache and compaths, although those take precedence.