Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix Intel CI workflow and migrate off Intel container images #1876

Open
vithikashah001 wants to merge 15 commits into
NOAA-GFDL:mainfrom
vithikashah001:main
Open

Fix Intel CI workflow and migrate off Intel container images #1876
vithikashah001 wants to merge 15 commits into
NOAA-GFDL:mainfrom
vithikashah001:main

Conversation

@vithikashah001

Copy link
Copy Markdown
Contributor

Description
Fix Intel CI workflow by replacing broken Intel container image usage and updating environment variables for compatibility.

Fixes #1872

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note
any relevant details for your test configuration (e.g. compiler, OS). Include
enough information so someone can reproduce your tests.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • New check tests, if applicable, are included
  • make distcheck passes

@rem1776 rem1776 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It looks like this is failing from not having the intel compilers in the PATH during the hdf5 build, even though its able to run configure sucessfully. The install action you added should be making those changes for us automatically, so not sure why its not there.

I would try adding a couple of debugging commands like echo $PATH and which mpiicx as a step after the compilers are installed so we can see where things are on the filesystem and get a better idea of whats going on.

uses: NOAA-EMC/ci-install-intel-toolkit@develop
with:
install-classic: true
install-oneapi: true

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would set this one to false instead, we'll only need the classic compiler for this workflow.

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.

fix broken intel classic workflow and migrate from intel images

2 participants