Skip to content

Backport #33 to release/3.0: ACL-T kola parity (variant-aware enforcement, 192 MiB ESP, LUKS initramfs modules) - #79

Merged
Jiri Appl (jiria) merged 1 commit into
release/3.0from
user/lanzeliu/backport-33-release-3.0
Sep 23, 2026
Merged

Jiri Appl (jiria) merged 1 commit into
release/3.0from
user/lanzeliu/backport-33-release-3.0

Conversation

@liulanze

@liulanze Lanze Liu (liulanze) commented Sep 22, 2026 •

Copy link
Copy Markdown
Member

Summary

Cherry-pick of #33 (aclmain, 2026-08-13) onto release/3.0.

Production builds of the ACL-T variant use release/3.0 and hit the two gaps #33 closed on aclmain: the arm64 UKI rebuild runs out of ESP space (Low free disk space 0% on /boot, prod nightly 1207248, same as 1193186 on 08-28), and the ACL-T kola runs count the variant-exempt tests as enforced failures (140 QEMU / 125 Azure on 1207247; the same test set passes as exempt on aclmain, e.g. 1206112).

Change Log

Type of Change

  • New kola test
  • Kola test fix/update
  • Platform integration change (Azure, QEMU, etc.)
  • CLI/command change
  • CI/automation change
  • Bug fix
  • Documentation update

Does this affect the image build?

  • Yes
  • No

ESP size of every UKI image and the initramfs module list; aclmain has shipped both in every nightly since 08-13.

Associated Issues

  • Backport of ACL PR #33.
  • Unblocks the ACL-T publish to MCR (acl-pipelines PR 29363).

Test Methodology

  • aclmain nightlies (5304) carry Enable ACL Template kola parity: variant-aware enforcement, ESP sizing, LUKS initramfs modules #33 since 08-13: e.g. 1206112 (ACL-T kola: 28/35 exemptions applied, all enforced tests passed) and 1207313 (09-22, ACL-T build green on both architectures).
  • Production nightly pipelines (amd64 and arm64) runs 1207566 / 1207567 built from this branch (aclScriptsRef=user/lanzeliu/backport-33-release-3.0, 9aab1f2): both succeeded; stock kola all enforced tests passed on both architectures; the arm64 ACL-T build succeeds with the 192 MiB ESP; ACL-T kola applies the imageVariant=acl-t exemptions and passes on both architectures (details in the PR comment).
  • Required CI checks (github-acl-pr-validation: stock build, kola and smoke on both architectures).

Merge Checklist

All applicable boxes should be checked before merging

  • Required CI checks pass
  • Ready to merge

…g, LUKS initramfs modules (#33)

* ado: Readd es-metadata.yml

* feat(acl-t): add parallel ACL Template build path

* Comments addressed.

* Testing verification 0722

* Exempt nfs

* 0723 verifications.

* Comments addressed.

* Correct linux.ntp exemption reason: timesyncd loses via unit Conflicts, not %post

The AzL 3.0 chrony.spec %post only runs daemon-reload/systemd_post/start;
the actual suppression mechanism is Conflicts=systemd-timesyncd.service in
the upstream chronyd.service that AzL packages verbatim.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* Document imageVariants matching and selection semantics in the schema header

Two non-obvious behaviors now spelled out where contributors copy from:
imageVariants values match exactly (unlike the other list keys; 'acl' is
a substring of 'acl-t'), and a rule constrained only by imageVariants
drops the test from kola selection on that variant rather than just
forgiving it at evaluation.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* Comments addressed.

---------

Co-authored-by: Jiri Appl <jiria@microsoft.com>
Co-authored-by: Lanze Liu <lanzeliu@microsoft.com>
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
(cherry picked from commit 97e5ef9)
@liulanze

Lanze Liu (liulanze) commented Sep 23, 2026 •

Copy link
Copy Markdown
Member Author

Production evidence for this branch (9aab1f2): the production nightly pipelines for amd64 and arm64 were queued with aclScriptsRef=user/lanzeliu/backport-33-release-3.0, buildAclTemplate=true, icSourceRef=839a4e8ff (runs 1207566 and 1207567, 2026-09-22).

Run Result Stock kola (Evaluate Test Results) ACL-T
1207566 amd64 succeeded Azure 59 enforced / 28 exempt, QEMU 85 enforced / 2 exempt: all enforced tests passed build, QEMU + Azure smoke, QEMU kola (59 enforced / 28 exempt) and Azure kola (37 enforced / 50 exempt) all green
1207567 arm64 succeeded Azure 59 / 28, QEMU 66 / 21: all enforced tests passed build with the 192 MiB ESP (the UKI rebuild that failed on 1207248), smoke and kola all green: QEMU 52 enforced / 35 exempt, Azure 37 / 50

Same commit as this PR; the two gaps in the description are closed on release/3.0 (arm64 ACL-T build succeeds, ACL-T kola applies the imageVariant=acl-t exemptions and passes). The stock bundles of these two runs were not released: the release analysis they triggered judged them ineligible and queued no publish.

@jiria
Jiri Appl (jiria) merged commit b43eeb7 into release/3.0 Sep 23, 2026
24 of 26 checks passed
@jiria
Jiri Appl (jiria) deleted the user/lanzeliu/backport-33-release-3.0 branch September 23, 2026 15:49

This branch was successfully deployed

1 active deployment
development — 9aab1f2a Deployed Sep 22, 2026 by liulanze via Check if we need to update the SDK #44
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