AcademySoftwareFoundation / OpenColorIO

A color management framework for visual effects and animation.
https://opencolorio.org
BSD 3-Clause "New" or "Revised" License
1.76k stars 437 forks source link

Update CI workflows #1770

Closed remia closed 1 year ago

remia commented 1 year ago

As discussed during TSC meetings, here are some update and new CI workflows.

This PR contains the following changes:

Note some jobs changed their names in the CI workflow, updating the required list is again going to be needed, hopefully this will happen less in the future as I removed some variability from these name like compiler versions which are not always fixed even in ci-20xx docker images (Clang). I also had a recurring issue where the GCC based linux platform latest build was killed by GitHub Action during compilation of OpenEXR with no diagnostic messages, apparently this can happen when some hardware resources limits are reached. This was on my fork but may be different here, this could require further investigations, usually manual relaunch of the failed jobs fix the issue.

remia commented 1 year ago

Chatting with @JeanChristopheMorinPerso, I disabled parallel build on OpenEXR ExternalProject, I'll try to trigger the build a couple of times but on the first try it seemed to have resolved the issue!

doug-walker commented 1 year ago

For anyone else reviewing the new CI settings, I made a spreadsheet showing the current and new matrix: https://docs.google.com/spreadsheets/d/1Iw5uRw6voB54wFpkSSOGrcAXkciZfx9nlBSVHz6N4P8/edit?usp=sharing

remia commented 1 year ago

Thanks for the review @doug-walker, most of the points are about the old analysis workflow (new dependencies_latest) which I didn't touch and is currently failing in nightly for the past few months. From my understanding @cedrik-fuoco-adsk was about to work on these so I think it's best to push the comments for that upcoming work? I'm not really confortable updating things in there knowing the builds are going to fail early and I won't be able to test.