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

ci: move compiler compatibility tests to separate workflow #1390

Merged
merged 11 commits into from
Oct 19, 2023

Conversation

wpbonelli
Copy link
Contributor

@wpbonelli wpbonelli commented Oct 7, 2023

  • rethink ci: test with gfortran 13 #1298, ci: test more Intel compiler versions #1303 to reduce CI load
  • redundant to test a wide range of versions on push/PR, test only latest compatible version of each toolchain
  • move compatibility tests to new workflow triggered weekly, on master/release branches, and manual dispatch — create compiler compatibility report artifacts, autoupdate DEVELOPER.md and open PR if compatibility changes, except if on release/master branch
  • check in compatibility reports comp.csv, test.csv in .github/compat/ so they can be diffed, update DEVELOPER.md

There is an issue installing ifort/ifx on ubuntu 20.04 with the setup-fortran action, corresponding entries are marked unknown until resolution (though ifort 2021.7 should work)

ubuntu-20.04 / intel issue resolved and workarounds removed

@wpbonelli wpbonelli force-pushed the compiler-compat branch 3 times, most recently from 1c508b9 to 0300863 Compare October 7, 2023 18:39
@wpbonelli wpbonelli marked this pull request as ready for review October 7, 2023 19:27
@wpbonelli wpbonelli force-pushed the compiler-compat branch 8 times, most recently from 32a78cc to 7a6026b Compare October 7, 2023 22:21
@wpbonelli wpbonelli changed the title ci: move compiler compatibility tests to manual workflow ci: move compiler compatibility tests to separate workflow Oct 7, 2023
@wpbonelli wpbonelli marked this pull request as draft October 8, 2023 02:23
@wpbonelli wpbonelli force-pushed the compiler-compat branch 8 times, most recently from f146abd to fb801c8 Compare October 10, 2023 11:47
@wpbonelli wpbonelli marked this pull request as ready for review October 10, 2023 12:15
@wpbonelli wpbonelli force-pushed the compiler-compat branch 3 times, most recently from 2c7a56f to a5e4528 Compare October 13, 2023 12:48
* trigger weekly, on release/master branches, and on manual dispatch
* create compatibility report for platform/toolchain/version combos
* if changed and not on release/master, open PR autoupdating README
@wpbonelli wpbonelli force-pushed the compiler-compat branch 2 times, most recently from 3dac1d8 to 211c732 Compare October 19, 2023 01:47
@wpbonelli wpbonelli merged commit f895fe5 into MODFLOW-USGS:develop Oct 19, 2023
22 of 30 checks passed
@wpbonelli wpbonelli deleted the compiler-compat branch October 19, 2023 08:58
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.

1 participant