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

(files/install/installCCS.sh) update CCS install script for use on summit #58

Merged
merged 3 commits into from
May 21, 2024

Conversation

glennmorris
Copy link
Contributor

@glennmorris glennmorris commented May 20, 2024

The installCCS.sh script is not used by puppet, only when doing manual CCS installs (for testing etc).
Now it takes a branch as argument.
Also, it makes more sense for /lsst/ccsadm/dev-package-lists (not created or managed by puppet) to move to /home/ccs.

@glennmorris glennmorris added the enhancement New feature or request label May 20, 2024
@glennmorris glennmorris requested a review from jhoblitt May 20, 2024 23:33
Copy link
Member

@jhoblitt jhoblitt left a comment

Choose a reason for hiding this comment

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

I usually prefer a release PR to be standalone but this looks reasonable.

@glennmorris
Copy link
Contributor Author

I usually prefer a release PR to be standalone but this looks reasonable.

Thanks, I'll try to remember to separate the version bump next time.

@glennmorris glennmorris merged commit 3beb65e into master May 21, 2024
16 checks passed
@glennmorris glennmorris deleted the IT-5368 branch May 21, 2024 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants