Skip to content

feat(grub2): add AZL SBAT line, keep upstream provenance - #18243

Open
corvus-callidus wants to merge 1 commit into
4.0from
lyrydber/grub2_sbat
Open

feat(grub2): add AZL SBAT line, keep upstream provenance#18243
corvus-callidus wants to merge 1 commit into
4.0from
lyrydber/grub2_sbat

Conversation

@corvus-callidus

Copy link
Copy Markdown
Contributor

Summary

Adds Azure Linux SBAT metadata to GRUB EFI binaries.

  • Adds grub.azurelinux,3 for the Azure Linux distributor identity. Generation 3 matches the existing Azure Linux 3.0 SBAT record.
  • Preserves the imported Fedora provenance on grub.rh with its upstream EVR, 2.12-40.fc43, rather than substituting the Azure Linux release.
  • Points the Azure Linux contact URL to the GitHub issues page.
  • Corrects the inherited GNU GRUB SBAT URL typo (https//https://).

Implementation notes

The Fedora provenance EVR is temporarily defined in the component overlay. Azure Linux 4.0 does not yet include the azldev upstream-EVR macros from azure-linux-dev-tools#290. Replace the local macro once that work is available in 4.0.

Validation

  • Local and koji builds
  • Extracted the .sbat section from grubx64.efi:
$ objcopy --dump-section .sbat=/dev/stdout    boot/efi/EFI/azurelinux/grubx64.efi
sbat,1,SBAT Version,sbat,1,https://github.com/rhboot/shim/blob/main/SBAT.md
grub,5,Free Software Foundation,grub,2.12,https://www.gnu.org/software/grub/
grub.rh,2,Red Hat,grub2,2.12-40.fc43,mailto:secalert@redhat.com
grub.azurelinux,3,Microsoft,grub2,2.12-46.azl4,https://github.com/microsoft/azurelinux/issues

Copilot AI review requested due to automatic review settings July 28, 2026 20:33
@corvus-callidus
corvus-callidus requested a review from a team as a code owner July 28, 2026 20:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds Azure Linux SBAT metadata to GRUB EFI binaries while preserving Fedora provenance.

Changes:

  • Adds the grub.azurelinux,3 SBAT entry.
  • Preserves Fedora’s version-release in grub.rh.
  • Corrects the GNU GRUB URL.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
base/comps/grub2/grub2.comp.toml Adds SBAT overlays and interim provenance macro.
specs/g/grub2/grub2.spec Renders provenance substitution and release bump.
specs/g/grub2/sbat.csv.in Adds Azure Linux metadata and URL correction.
locks/grub2.lock Refreshes the component fingerprint.

Comment thread base/comps/grub2/grub2.comp.toml Outdated
Add Azure Linux's GRUB SBAT identity while retaining the imported Fedora
provenance record. This supports Azure Linux-specific revocation without losing
upstream lineage.

The Fedora EVR remains temporarily pinned until azldev provides the upstream-EVR
macros tracked by AB#21798.
Copilot AI review requested due to automatic review settings July 28, 2026 23:32
@corvus-callidus

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
3 pipeline(s) were filtered out due to trigger conditions.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

base/comps/grub2/grub2.comp.toml:71

  • The new SBAT overlays are non-trivial but have no overlay metadata. Current component guidance requires category and upstream-status so temporary workarounds and their removal triggers remain discoverable (see base/comps/qt5-qtmultimedia/qt5-qtmultimedia.comp.toml:9-11). Please add per-overlay metadata—or group overlays with shared provenance in an overlay file—including the linked azldev work for the temporary macro.
# SBAT: edit the upstream sbat.csv.in in place. Swap grub.rh to the
# @@UPSTREAM_VERSION_RELEASE@@ placeholder and append the grub.azurelinux vendor line.
# The placeholder is substituted at build time from an interim spec macro pinned to
# the Fedora grub2 NEVR; swap that macro for the azldev EVR macro when bug 21798 lands.

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