Skip to content

Bugfix/update container action#1620

Closed
stiggy87 wants to merge 4 commits intoJCSDA:developfrom
stiggy87:bugfix/update-container-action
Closed

Bugfix/update container action#1620
stiggy87 wants to merge 4 commits intoJCSDA:developfrom
stiggy87:bugfix/update-container-action

Conversation

@stiggy87
Copy link
Copy Markdown
Contributor

Summary

Removes the intel container build to use the oneapi container build.

Testing

(in-progress) testing via pull-request

Issue(s) addressed

Link the issues addressed or resolved by this PR (use Fixes #??? for fully resolved issues)

Fixes #1619

Checklist

  • This PR addresses one issue/problem/enhancement, or has a very good reason for not doing so.
  • These changes have been tested on the affected systems and applications.
  • All dependency PRs/issues have been resolved and this PR can be merged.

@stiggy87 stiggy87 added the INFRA JEDI Infrastructure label Apr 28, 2025
@stiggy87 stiggy87 self-assigned this Apr 28, 2025
@stiggy87 stiggy87 marked this pull request as draft April 28, 2025 20:12
@climbfuji
Copy link
Copy Markdown
Collaborator

@stiggy87 You'l need to check if the container spec is the one you want to test, because it choses the spec/compiler depending on the day of the week.

on:
# Uncomment this to test for PRs (but do not submit)
#pull_request:
# pull_request:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
# pull_request:
#pull_request:

@climbfuji
Copy link
Copy Markdown
Collaborator

@stiggy87 I pushed your branch with a few modifications to the JCSDA repo, see #1625. This allowed me to run the correct container build in GitHub actions using workflow_dispatch. The oneAPI container build failed in cairo, see https://github.com/JCSDA/spack-stack/actions/runs/14754068408/job/41417962385.

I'll take a look today, if I can't figure it out quickly, I'll let you know.

@climbfuji
Copy link
Copy Markdown
Collaborator

Superseded by #1625

@climbfuji climbfuji closed this Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

INFRA JEDI Infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Intel container build no longer supported - need to update to oneAPI

2 participants