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.

Sign up for GitHub

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: build newer openpmix / ompi releases #103

Merged
merged 3 commits into from
Dec 11, 2024
Merged

ci: build newer openpmix / ompi releases #103

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
5cb828c
testsuite: allow pmix.nsdir to exist
garlick Dec 10, 2024
d7f3723
ci: update openpmix v4 version
garlick Dec 10, 2024
3ecff1c
ci: update ompi v5 version
garlick Dec 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 9 additions & 9 deletions .github/workflows/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,15 +24,15 @@ jobs:
include:
- name: "jammy - chain_lint"
image: "jammy"
ompi_branch: "v5.0.0rc12"
openpmix_branch: "v4.2.3"
ompi_branch: "v5.0.6"
openpmix_branch: "v4.2.9"
coverage: false
env:
chain_lint: t
- name: "el8 - distcheck"
image: "el8"
ompi_branch: "v5.0.0rc12"
openpmix_branch: "v4.2.3"
ompi_branch: "v5.0.6"
openpmix_branch: "v4.2.9"
coverage: false
env:
DISTCHECK: t
Expand All @@ -44,15 +44,15 @@ jobs:
env: {}
- name: "jammy - coverage"
image: "jammy"
ompi_branch: "v5.0.0rc12"
openpmix_branch: "v4.2.3"
ompi_branch: "v5.0.6"
openpmix_branch: "v4.2.9"
coverage: true
env:
COVERAGE: t
- name: "fedora34 - ompi v5.0.x"
- name: "fedora34 - ompi v5.0.6"
image: "fedora34"
ompi_branch: "v5.0.0rc12"
openpmix_branch: "v4.2.3"
ompi_branch: "v5.0.6"
openpmix_branch: "v4.2.9"
coverage: false
env: {}
- name: "jammy - ompi v4.1.x"
Expand Down
2 changes: 1 addition & 1 deletion src/test/docker/jammy/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ FROM fluxrm/flux-core:jammy
ARG USER=fluxuser
ARG UID=1000
ARG OMPI_BRANCH=v5.0.x
ARG OPENPMIX_BRANCH=v4.2.4
ARG OPENPMIX_BRANCH=v4.2.9

RUN \
if test "$USER" != "fluxuser"; then \
Expand Down
7 changes: 4 additions & 3 deletions t/t0002-basic.t
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -108,10 +108,11 @@ test_expect_success 'pmix.job.napps is set to 1' '
${GETKEY} --proc=* pmix.job.napps >pmix.job.napps.out &&
test_cmp pmix.job.napps.exp pmix.job.napps.out
'
# pmix.nsdir appears to be automatically set in openpmix 4.2.9, but not
# in the earlier releases we were testing (up to 4.2.4).
test_expect_success 'pmix.nsdir is NOT set' '
test_must_fail flux run \
${GETKEY} --proc=* pmix.nsdir 2>pmix.nsdir.err &&
grep NOT-FOUND pmix.nsdir.err
test_might_fail flux run \
${GETKEY} --proc=* pmix.nsdir
'
test_expect_success '2n4p pmix.hname is set' '
run_timeout 30 flux run -N2 -n4 \
Expand Down
Loading