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

Jump to bottom

Update version of debian for Docker images #573

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update version of debian for Docker images #573

wants to merge 3 commits into from

Conversation

sheevy
Copy link
Contributor

@sheevy sheevy commented Jun 22, 2023

Bullseye -> Bookworm

@alculquicondor
Copy link
Collaborator

It looks like MPICH is failing, but others are passing.

@sheevy
Copy link
Contributor Author

sheevy commented Jun 22, 2023

For me it was Intel which failed locally.
I've started this PR to ask if people have any hints what could be happening?

terrytangyuan
terrytangyuan approved these changes Jun 22, 2023
View reviewed changes
Copy link
Member

@terrytangyuan terrytangyuan left a comment

Choose a reason for hiding this comment

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

Thanks!

/lgtm

@google-oss-prow
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: terrytangyuan

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@alculquicondor
Copy link
Collaborator

From:

https://github.com/kubeflow/mpi-operator/actions/runs/5346361756/jobs/9693279621?pr=573 (1.25)

== BEGIN pi-launcher-qlgqn pod logs ==
Couldn't resolve pi-launcher... Retrying
Couldn't resolve pi-launcher... Retrying
Couldn't resolve pi-launcher... Retrying
Couldn't resolve pi-launcher... Retrying
Couldn't resolve pi-launcher... Retrying
Couldn't resolve pi-launcher... Retrying
Couldn't resolve pi-launcher... Retrying
Couldn't resolve pi-launcher... Retrying
Couldn't resolve pi-launcher... Retrying
Resolved pi-launcher
Resolved pi-worker-0.pi-worker.e2e-gs9st.svc
Resolved pi-worker-1.pi-worker.e2e-gs9st.svc
Warning: Permanently added '[pi-worker-0.pi-worker.e2e-gs9st.svc]:2222' (ED25519) to the list of known hosts.
Warning: Permanently added '[pi-worker-1.pi-worker.e2e-gs9st.svc]:2222' (ED25519) to the list of known hosts.
Workers: 2
Rank 1 on host pi-worker-1
Rank 0 on host pi-worker-0
pi is approximately 3.1410376000000002

It kind of looks like the job succeeded?

No idea what could be happening. Maybe the timeouts are just too tight?

@google-oss-prow
Copy link

New changes are detected. LGTM label has been removed.

@google-oss-prow google-oss-prow bot removed the lgtm label Jun 26, 2023
@alculquicondor
Copy link
Collaborator

Still failing... can you investigate locally?

@tenzen-y tenzen-y mentioned this pull request Aug 18, 2023
Open
@tenzen-y
Copy link
Member

@sheevy Do you have any progress? We need to move this forward to update the OpenMPI version.
ref: #588

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@terrytangyuan terrytangyuan terrytangyuan approved these changes

@carmark carmark Awaiting requested review from carmark

@tenzen-y tenzen-y Awaiting requested review from tenzen-y

Assignees

@terrytangyuan terrytangyuan

Labels
approved size/S
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@sheevy @alculquicondor @tenzen-y @terrytangyuan