Skip to content

Conversation

KornevNikita
Copy link
Contributor

@KornevNikita KornevNikita commented Feb 10, 2025

It takes ~10mins for gzip to compress the cov-int dir. Use pigz (parallel implementation of gzip) to speed it up.
formats supported by coverity: gzip, zip, lzma, xz or bz2
#16945

It takes ~10mins for gzip to compress the cov-int dir. Use pigz
(parallel implementation of gzip) to speed it up.
@KornevNikita KornevNikita requested a review from a team as a code owner February 10, 2025 13:11
@KornevNikita
Copy link
Contributor Author

@intel/dpcpp-devops-reviewers actually there is no pigz in our images. For some reason "apt install pigz" doesn't work, see - https://github.com/intel/llvm/actions/runs/13241613101/job/36958055373:

sudo apt-get install pigz
  pigz --version
  shell: sh -e {0}
Reading package lists...
Building dependency tree...
Reading state information...
You might want to run 'apt --fix-broken install' to correct these.
The following packages have unmet dependencies:
 intel-igc-cm : Depends: intel-igc-core (>= 1.0.999[5](https://github.com/intel/llvm/actions/runs/13241613101/job/36958055373#step:3:6)) but it is not installable
E: Unmet dependencies. Try 'apt --fix-broken install' with no packages (or specify a solution).
Error: Process completed with exit code 100.

Should I use apt --fix-broken install?
Instead we can build pigz manually from sources or try to add to https://github.com/intel/llvm/blob/sycl/devops/scripts/install_build_tools.sh
Which way is more preferable?

@sarnex
Copy link
Contributor

sarnex commented Feb 10, 2025

It's because the IGC CI creates weird deb packges. Locally, you can use apt --fix-broken install but if you want to actually change the images you should change here maybe

@KornevNikita
Copy link
Contributor Author

@sarnex @aelovikov-intel can we merge this now?

runs-on: [Linux, build]
container:
image: ghcr.io/intel/llvm/ubuntu2404_intel_drivers:alldeps
image: ghcr.io/intel/llvm/ubuntu2404_build:latest
Copy link
Contributor

Choose a reason for hiding this comment

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

Why this? We want to gradually switch to "alldeps" everywhere.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ubuntu2404_intel_drivers is the only one that have alldeps, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved back to alldeps - addd00f

@KornevNikita
Copy link
Contributor Author

@intel/llvm-gatekeepers could you please merge?

@sommerlukas sommerlukas merged commit 1d5c967 into intel:sycl Feb 12, 2025
17 checks passed
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.

4 participants