Skip to content

Comments

Removed parallel builds on sanitizer due to memory overhead#1214

Merged
AhmedBM merged 1 commit intodevfrom
ahbenmes/fix_ci_sanitizers
Nov 24, 2025
Merged

Removed parallel builds on sanitizer due to memory overhead#1214
AhmedBM merged 1 commit intodevfrom
ahbenmes/fix_ci_sanitizers

Conversation

@AhmedBM
Copy link
Contributor

@AhmedBM AhmedBM commented Nov 21, 2025

Description

  • Removed parallel builds from sanitizer build due to memory overhead. It appears the workflows would randomly fail due to github runners dying in the build process during linking.

Checklist

  • I have read the contribution guidelines.
  • I added unit-tests to validate my changes. All unit tests are passing.
  • I have merged the latest dev branch prior to this PR submission.
  • I submitted this PR against the dev branch.

@github-actions
Copy link

github-actions bot commented Nov 21, 2025

Test Results

    1 files   -  63     61 suites   - 3   1m 28s ⏱️ - 57m 8s
1 001 tests +993  1 001 ✅ +993  0 💤 ± 0  0 ❌ ±0 
1 001 runs  +745  1 001 ✅ +767  0 💤  - 22  0 ❌ ±0 

Results for commit 1d7a111. ± Comparison against base commit 142d0a0.

This pull request removes 8 and adds 1001 tests. Note that renamed tests count towards both.
/home/user1/UniversalNRP.Tests.ps1 ‑ Validate Universal NRP.Get.Ensure all resource instances have status
/home/user1/UniversalNRP.Tests.ps1 ‑ Validate Universal NRP.Get.Ensure resons are properly populated
/home/user1/UniversalNRP.Tests.ps1 ‑ Validate Universal NRP.Get.Ensure the total resource instances count
/home/user1/UniversalNRP.Tests.ps1 ‑ Validate Universal NRP.Set.Perform remediation
/mnt/vss/_work/azure-osconfig/azure-osconfig/src/tests/universal-nrp-e2e/UniversalNRP.Tests.ps1 ‑ Validate Universal NRP.Get.Ensure all resource instances have status
/mnt/vss/_work/azure-osconfig/azure-osconfig/src/tests/universal-nrp-e2e/UniversalNRP.Tests.ps1 ‑ Validate Universal NRP.Get.Ensure resons are properly populated
/mnt/vss/_work/azure-osconfig/azure-osconfig/src/tests/universal-nrp-e2e/UniversalNRP.Tests.ps1 ‑ Validate Universal NRP.Get.Ensure the total resource instances count
/mnt/vss/_work/azure-osconfig/azure-osconfig/src/tests/universal-nrp-e2e/UniversalNRP.Tests.ps1 ‑ Validate Universal NRP.Set.Perform remediation
AuditdRulesCheckTest ‑ AuditctlCommandFails
AuditdRulesCheckTest ‑ ExcludeOptionsSkipsMatchingRules
AuditdRulesCheckTest ‑ ImmutableRuleCompliant
AuditdRulesCheckTest ‑ ImmutableRuleMissingIsNonCompliant
AuditdRulesCheckTest ‑ InvalidExcludeOptionsRegexReturnsError
AuditdRulesCheckTest ‑ InvalidOverridePathReturnsNonCompliant
AuditdRulesCheckTest ‑ InvalidRequiredOptionsRegexReturnsError
AuditdRulesCheckTest ‑ MultiSyscallOneMissingInFilesIsNonCompliant
AuditdRulesCheckTest ‑ OverridePathMissingFileRuleIsNonCompliant
AuditdRulesCheckTest ‑ OverridePathWithMatchingFileRuleIsCompliant
…

♻️ This comment has been updated with latest results.

@AhmedBM AhmedBM marked this pull request as ready for review November 22, 2025 00:10
@AhmedBM AhmedBM requested review from a team as code owners November 22, 2025 00:10
Copy link
Contributor

@MariusNi MariusNi left a comment

Choose a reason for hiding this comment

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

:shipit:

@AhmedBM AhmedBM merged commit d6ee0aa into dev Nov 24, 2025
122 checks passed
@AhmedBM AhmedBM deleted the ahbenmes/fix_ci_sanitizers branch November 24, 2025 18:45
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