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

spack: upgrade to 0.22.2 #2042

Open
wants to merge 1 commit into
base: 3.x
Choose a base branch
from

Conversation

adrianreber
Copy link
Member

This includes better integration into OpenHPC (again).

This also removes an old patch which was not applied for a long time. This patch was important to make spack installed software packages available to OpenHPC users.

In its default configuration in OpenHPC only root can install additional software packages using spack, but these packages will be made available to all users again.

Non-root users can also use spack, but it requires additional configuration for each user. Unfortunately spack does not automatically set different defaults when running as non-root like EasyBuild for example seems to do.

This includes better integration into OpenHPC (again).

This also removes an old patch which was not applied for a long time.
This patch was important to make spack installed software packages
available to OpenHPC users.

In its default configuration in OpenHPC only root can install additional
software packages using spack, but these packages will be made available
to all users again.

Non-root users can also use spack, but it requires additional
configuration for each user. Unfortunately spack does not automatically
set different defaults when running as non-root like EasyBuild for
example seems to do.

Signed-off-by: Adrian Reber <areber@redhat.com>
@adrianreber
Copy link
Member Author

@martin-g This tries to solve the problem described in #1995

Copy link

Test Results

20 files   -  4  20 suites   - 4   1m 25s ⏱️ +59s
57 tests  -  6  49 ✅  - 3  4 💤  -  7  4 ❌ +4 
74 runs   - 12  66 ✅  - 2  4 💤  - 14  4 ❌ +4 

For more details on these failures, see this check.

Results for commit 76412d4. ± Comparison against base commit 611b01f.

This pull request removes 10 and adds 4 tests. Note that renamed tests count towards both.
conman ‑ [ConMan] Verify conman binary available
conman ‑ [ConMan] Verify man page availability
conman ‑ [ConMan] Verify rpm version matches binary
ipmitool ‑ [OOB] ipmitool exists
ipmitool ‑ [OOB] ipmitool local bmc ping
ipmitool ‑ [OOB] ipmitool power status
ipmitool ‑ [OOB] ipmitool read CPU1 sensor data
ipmitool ‑ [OOB] ipmitool read sel log
ipmitool ‑ [OOB] istat exists
warewulf-ipmi ‑ [warewulf-ipmi] ipmitool lanplus protocol
spack ‑ [spack] add compiler
spack ‑ [spack] check for RPM
spack ‑ [spack] test build
spack ‑ [spack] test module refresh

@adrianreber
Copy link
Member Author

The test failure is due to a limitation of the test script and our defaults packages.

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.

1 participant