Skip to content

{Packaging} Update Expat to be compatible with current Python - #33986

Closed
mansoor sarfraz (msarfraz) wants to merge 1 commit into
Azure:devfrom
msarfraz:fix-expat-mismatch
Closed

{Packaging} Update Expat to be compatible with current Python#33986
mansoor sarfraz (msarfraz) wants to merge 1 commit into
Azure:devfrom
msarfraz:fix-expat-mismatch

Conversation

@msarfraz

Copy link
Copy Markdown
Contributor

Summary
Adds a dnf update -y expat step to the RPM Docker test script (scripts/release/rpm/test_rpm_in_docker.sh) before installing the built RPM.
Background
The pinned UBI 8/9 base images ship with an older version of the Expat XML library. This version is incompatible with the pyexpat module in the current Python 3.12 runtime, causing failures when the RPM is exercised inside the container.
Changes

  • Update the expat package via dnf prior to installing the Azure CLI RPM and its dependencies.

Impact

  • Scope: CI/release test tooling only — no changes to Azure CLI source or shipped artifacts.
  • Ensures the RPM smoke test in Docker runs against an Expat version compatible with Python 3.12, preventing spurious pyexpat errors.
    Testing
  • Runs as part of the existing RPM-in-Docker test flow (set -exv ensures the new step is logged and any failure aborts the run).

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 a package update step to the UBI8/UBI9 RPM-in-Docker smoke test to avoid pyexpat failures with the Python 3.12 runtime by ensuring the container has a compatible Expat library version.

Changes:

  • Update Expat via dnf before installing and exercising the built Azure CLI RPM inside the container.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines +7 to +8
# Pinned UBI 8/9 images contain an older Expat incompatible with current Python 3.12 pyexpat.
dnf update -y expat
@yonzhan

Copy link
Copy Markdown
Collaborator

Packaging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants