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: Bring back ccache for DPC++ #749

Merged
merged 1 commit into from
Jul 14, 2023
Merged

CI: Bring back ccache for DPC++ #749

merged 1 commit into from
Jul 14, 2023

Conversation

psalz
Copy link
Contributor

@psalz psalz commented Jul 14, 2023

I just randomly noticed that CI run times are going absolutely through the roof again, with runs taking anywhere from FOUR to SIX hours. While not really addressing the root problem, our fix for this so far was to use ccache, which has worked rather well.

It looks like ccache was lost in the switch to the DPC++ nightly containers (#712), so with this I'm bringing it back.

Note: I haven't tested this, so after the cache has been primed by the CI run for this PR we'll have to launch a second run to see if it actually works.


Side node / reminder: we generate a detailed report of what is taking up the time, looks like the marray operators test is the new champion with a whopping 25 minutes - in case anybody from Intel wants to take a look.

@psalz psalz requested a review from a team as a code owner July 14, 2023 12:59
This was lost in the switch to the DPC++ nightly containers (#712).
Copy link
Member

@keryell keryell left a comment

Choose a reason for hiding this comment

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

@bader bader merged commit 116ea83 into SYCL-2020 Jul 14, 2023
@bader bader deleted the bring-back-dpcpp-ccache branch July 14, 2023 18:47
@bader
Copy link
Contributor

bader commented Jul 17, 2023

@psalz, unfortunately, it seems like it doesn't help.

I still see that cache is not used here: https://github.com/KhronosGroup/SYCL-CTS/actions/runs/5570325262/jobs/10174289342. This is for the #740, where I merged this change.

BTW, I think ccache is installed in the DPC++ image by this script: https://github.com/intel/llvm/blob/sycl/devops/scripts/install_build_tools.sh#L7. We run it to build "base" image, which we use to build the container with "HW drivers". The nightly build is based on the "HW drivers" image.

@psalz
Copy link
Contributor Author

psalz commented Jul 18, 2023

Yes the merge was a bit premature (I did say I'm not sure whether it's working or not). I'm debugging it now.

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.

3 participants