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

Fix Houdini CI Caching #1690

Merged
merged 2 commits into from
Oct 19, 2023
Merged

Conversation

Idclip
Copy link
Contributor

@Idclip Idclip commented Oct 18, 2023

# We bumped from the 2021 CI image to 2023 to fix some OpenSSL issues
# with the Houdini download script. In so doing we broke some of the caching
# between this job and the jobs in houdini.yml which _don't_ use the 2023
# image yet. The issue is that the cache action will use zstd if it's
# available to zip the cache and this causes it to be inserted with a unique
# hash which images without zstd (i.e. the 2021/2022 images don't have
# access to). For now, uninstall zstd here instead of installing it
# everywhere and ask the LF to add zstd to the older base images.

Signed-off-by: Nick Avramoussis <4256455+Idclip@users.noreply.github.com>
@Idclip
Copy link
Contributor Author

Idclip commented Oct 18, 2023

This should now pass once in master

@Idclip Idclip enabled auto-merge October 18, 2023 10:56
@Idclip Idclip disabled auto-merge October 18, 2023 10:56
Signed-off-by: Nick Avramoussis <4256455+Idclip@users.noreply.github.com>
@Idclip Idclip enabled auto-merge October 19, 2023 04:05
@Idclip Idclip merged commit 102e56d into AcademySoftwareFoundation:master Oct 19, 2023
39 of 40 checks passed
@Idclip Idclip deleted the hou_ci_fix branch October 19, 2023 08:25
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