Skip to content

fix(golang-github-prometheus-common,golang-github-prometheus-client,golang-github-envoyproxy-control-plane): prune dependency cycles - #18508

Open
Mitch Zhu (miz060) wants to merge 1 commit into
4.0from
mitchzhu/golang-dependency-cycle-pruning
Open

fix(golang-github-prometheus-common,golang-github-prometheus-client,golang-github-envoyproxy-control-plane): prune dependency cycles#18508
Mitch Zhu (miz060) wants to merge 1 commit into
4.0from
mitchzhu/golang-dependency-cycle-pruning

Conversation

@miz060

@miz060 Mitch Zhu (miz060) commented Aug 19, 2026

Copy link
Copy Markdown
Member

Two legacy Go dependency cycles prevent clean Stage 2 rebuilds without existing package seeds. Prune unused Prometheus config packages and update Envoy control plane to upstream 0.13.4, which removes the deprecated OpenCensus APIs without a downstream source-pruning patch. This breaks the cycles while preserving the APIs required by Azure Linux consumers.

Validation:

  • Prometheus Common passes dynamic BuildRequires and %check.
  • Prometheus client passes dynamic BuildRequires and package build. Its pre-existing %check disablement remains because v1.19.0 lacks Go 1.26 runtime-metrics fixtures; upstream first ships them in v1.24.0.
  • Envoy control plane 0.13.4 builds, installs cleanly in mock, retains the core and discovery Go capabilities, and has no OpenCensus requirement.
  • A chained local-repository build compiles and packages gRPC 1.64.0 against the new control-plane RPM. Fedora disables the gRPC %check body in this package.

Copilot AI balanced review requested due to automatic review settings August 19, 2026 02: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

Prunes legacy Go dependency cycles and updates Envoy control plane to 0.13.4.

Changes:

  • Removes unused Prometheus config-related packages and tests.
  • Updates Envoy control plane source and version.
  • Refreshes generated specs and component locks.

Reviewed changes

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

Show a summary per file
File Description
specs/g/golang-github-prometheus-common/golang-github-prometheus-common.spec Removes config packages.
specs/g/golang-github-prometheus-client/golang-github-prometheus-client.spec Removes config-dependent example test.
specs/g/golang-github-envoyproxy-control-plane/sources Adds the 0.13.4 archive checksum.
specs/g/golang-github-envoyproxy-control-plane/golang-github-envoyproxy-control-plane.spec Updates the rendered version.
locks/golang-github-prometheus-common.lock Refreshes the input fingerprint.
locks/golang-github-prometheus-client.lock Refreshes the input fingerprint.
locks/golang-github-envoyproxy-control-plane.lock Refreshes the input fingerprint.
base/comps/golang-github-prometheus-common/overlays/0001-drop-unused-config-modules.overlay.toml Defines config-package pruning.
base/comps/golang-github-prometheus-common/overlays/0001-add-missing-jmespath-buildrequires.overlay.toml Removes the obsolete dependency workaround.
base/comps/golang-github-prometheus-client/overlays/0001-drop-pruned-config-example.overlay.toml Defines example-test pruning.
base/comps/golang-github-prometheus-client/golang-github-prometheus-client.comp.toml Moves the customized component to a dedicated definition.
base/comps/golang-github-envoyproxy-control-plane/overlays/0002-update-to-upstream-0.13.4.overlay.toml Defines the Envoy version update.
base/comps/golang-github-envoyproxy-control-plane/golang-github-envoyproxy-control-plane.comp.toml Registers the new source archive.
base/comps/components.toml Removes the now-customized client from the inline list.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

…olang-github-envoyproxy-control-plane): prune dependency cycles

Drop unused Prometheus config and sigv4 code, along with the client example that imports it, before dependency generation.

Backport Envoy removal of deprecated OpenCensus APIs so gRPC no longer pulls the legacy opencensus-proto and grpc-gateway cycle into its buildroot.

Signed-off-by: Mitch Zhu <mitchzhu@microsoft.com>
@miz060
Mitch Zhu (miz060) force-pushed the mitchzhu/golang-dependency-cycle-pruning branch from 3183c19 to b40da8c Compare August 19, 2026 17:46
Copilot AI review requested due to automatic review settings August 19, 2026 17:46
@miz060
Mitch Zhu (miz060) marked this pull request as ready for review August 19, 2026 17:50
@miz060
Mitch Zhu (miz060) requested a review from a team as a code owner August 19, 2026 17:50

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 11 out of 14 changed files in this pull request and generated no new comments.

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