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

Use Arcus for CI #182

Merged
merged 37 commits into from
May 16, 2022
Merged

Use Arcus for CI #182

merged 37 commits into from
May 16, 2022

Conversation

sjpb
Copy link
Collaborator

@sjpb sjpb commented May 4, 2022

Add CI on Arcus. Unlike the existing smslabs CI, this uses direct-type VNICs so can test RDMA (using "inbox" drivers).

To avoid duplication between smslabs and arcus environments, this PR:

  • Replaces the (outdated, unused, unloved) terraform in the cookiecutter skeleton with terraform based on smslabs, but modified to optionally support direct-mode ports VNICs
  • Uses that terraform as a module from both arcus and smslabs environments
  • Uses the everything layout as the arcus and smslabs environment's groups.
  • Uses a single workflow file with matrix to select which cloud.

NB: the same image (see environments/*/terraform/main.tf) is used for both arcus and smslabs.

Arcus also exposed a bug in hpctests pingmatrix output (divide-by-zero when no difference between min and max latency), fixed here in baea143 & 02f9475 so that CI passes.

TODO:

  • check packer build base

@sjpb sjpb force-pushed the ci/arcus-basic branch from fc256e0 to 2e66f3d Compare May 5, 2022 08:46
@sjpb sjpb force-pushed the ci/arcus-basic branch from 2d84953 to 4bd48c5 Compare May 11, 2022 11:17
@sjpb sjpb requested a review from m-bull May 12, 2022 19:22
ansible/ci/test_reimage.yml Show resolved Hide resolved
ansible/ci/test_reimage.yml Outdated Show resolved Hide resolved
environments/arcus/hooks/check_slurm.yml Show resolved Hide resolved
@sjpb sjpb merged commit a287a99 into main May 16, 2022
@sjpb sjpb deleted the ci/arcus-basic branch May 16, 2022 14:29
@sjpb sjpb mentioned this pull request May 24, 2022
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.

2 participants