Skip to content

Conversation

@mikeharder
Copy link
Member

@mikeharder mikeharder commented Nov 6, 2025

@mikeharder mikeharder changed the title Revert "Resolve misconfigured dpkg (#43777)" [TEST] Revert "Resolve misconfigured dpkg (#43777)" Nov 6, 2025
@mikeharder mikeharder marked this pull request as ready for review November 12, 2025 19:09
Copilot AI review requested due to automatic review settings November 12, 2025 19:09
@mikeharder mikeharder self-assigned this Nov 12, 2025
@mikeharder mikeharder moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 🍔🌭 Nov 12, 2025
@mikeharder
Copy link
Member Author

@scbedd: 1ES has fixed the root cause, so we should be able to revert this workaround. If we want to keep it, we should add a comment explaining it's defense-in-depth.

Copilot finished reviewing on behalf of mikeharder November 12, 2025 19:11
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR reverts a previous workaround that addressed misconfigured dpkg issues in CI/CD runners. The change removes sudo dpkg --configure -a commands from pipeline templates to test whether the underlying runner/image bug has been resolved.

Key Changes:

  • Removed dpkg configuration workaround from PortAudio installation pipeline
  • Removed dpkg configuration workaround from package artifact build pipeline

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
eng/pipelines/templates/steps/install-portaudio.yml Removes dpkg --configure -a command before apt-get operations for Linux
eng/pipelines/templates/steps/build-package-artifacts.yml Removes dpkg --configure -a command before installing QEMU dependencies

@mikeharder mikeharder changed the title [TEST] Revert "Resolve misconfigured dpkg (#43777)" Revert "Resolve misconfigured dpkg (#43777)" Nov 12, 2025
Copy link
Member

@scbedd scbedd left a comment

Choose a reason for hiding this comment

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

Agreed. Thank you for pursuing this.

@scbedd scbedd merged commit 8ad14f3 into Azure:main Nov 12, 2025
29 checks passed
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 🍔🌭 Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎊 Closed

Development

Successfully merging this pull request may close these issues.

2 participants