Increase mem alloc for license scanning container#4691
Conversation
There was a problem hiding this comment.
Pull request overview
This PR increases the memory allocation for the license scanning container from 6GB to 12GB to address memory-related failures during license scanning operations. The change is motivated by an issue in the scancode-toolkit library that causes out-of-memory errors.
Changes:
- Memory allocation for license scanning containers increased from 6g to 12g
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| eng/pipelines/vmr-license-scan.yml | Updated memory allocation for official license scan pipeline container |
| eng/pipelines/vmr-license-scan-unofficial.yml | Updated memory allocation for unofficial license scan pipeline container |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
/backport to release/10.0.1xx |
|
Started backporting to |
|
/backport to release/10.0.2xx |
|
Started backporting to |
|
/backport to release/10.0.3xx |
|
Started backporting to |
|
@mthalman backporting to git am output$ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch
Applying: Increase mem alloc for license scanning container
Using index info to reconstruct a base tree...
M eng/pipelines/vmr-license-scan-unofficial.yml
M eng/pipelines/vmr-license-scan.yml
Falling back to patching base and 3-way merge...
Auto-merging eng/pipelines/vmr-license-scan-unofficial.yml
CONFLICT (content): Merge conflict in eng/pipelines/vmr-license-scan-unofficial.yml
Auto-merging eng/pipelines/vmr-license-scan.yml
CONFLICT (content): Merge conflict in eng/pipelines/vmr-license-scan.yml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Increase mem alloc for license scanning container
Error: The process '/usr/bin/git' failed with exit code 128 |
|
@mthalman backporting to git am output$ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch
Applying: Increase mem alloc for license scanning container
Using index info to reconstruct a base tree...
M eng/pipelines/vmr-license-scan-unofficial.yml
M eng/pipelines/vmr-license-scan.yml
Falling back to patching base and 3-way merge...
Auto-merging eng/pipelines/vmr-license-scan-unofficial.yml
CONFLICT (content): Merge conflict in eng/pipelines/vmr-license-scan-unofficial.yml
Auto-merging eng/pipelines/vmr-license-scan.yml
CONFLICT (content): Merge conflict in eng/pipelines/vmr-license-scan.yml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Increase mem alloc for license scanning container
Error: The process '/usr/bin/git' failed with exit code 128 |
|
@mthalman backporting to git am output$ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch
Applying: Increase mem alloc for license scanning container
Using index info to reconstruct a base tree...
M eng/pipelines/vmr-license-scan-unofficial.yml
M eng/pipelines/vmr-license-scan.yml
Falling back to patching base and 3-way merge...
Auto-merging eng/pipelines/vmr-license-scan-unofficial.yml
CONFLICT (content): Merge conflict in eng/pipelines/vmr-license-scan-unofficial.yml
Auto-merging eng/pipelines/vmr-license-scan.yml
CONFLICT (content): Merge conflict in eng/pipelines/vmr-license-scan.yml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0001 Increase mem alloc for license scanning container
Error: The process '/usr/bin/git' failed with exit code 128 |
Fixes dotnet/source-build#5470
The root cause is likely aboutcode-org/scancode-toolkit#4709. Increasing the memory allocation in the container fixes the issue.