chore(golang): remove unused builddeps-only components - #18635
Open
reuben olinsky (reubeno) wants to merge 1 commit into
Open
chore(golang): remove unused builddeps-only components#18635reuben olinsky (reubeno) wants to merge 1 commit into
builddeps-only components#18635reuben olinsky (reubeno) wants to merge 1 commit into
Conversation
Drop unused Google Cloud SDK, Autorest, client-go, migrate, and ocagent components. Their remaining consumers are bundled or removed. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Removes obsolete Go SDK components whose consumers now bundle dependencies or no longer exist.
Changes:
- Removes 12 component definitions and related configuration.
- Deletes corresponding locks and rendered RPM artifacts.
- Cleans stale check-disablement entries.
Reviewed changes
Copilot reviewed 31 out of 43 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
base/comps/components.toml |
Removes inline component registrations. |
base/comps/component-check-disablement.toml |
Removes obsolete check exceptions. |
base/comps/migrate/migrate.comp.toml |
Removes migrate definition. |
base/comps/migrate/overlays/0001-add-missing-jmespath-buildrequires.overlay.toml |
Removes migrate overlay. |
base/comps/golang-cloud-google/golang-cloud-google.comp.toml |
Removes pinned Google Cloud definition. |
locks/migrate.lock |
Removes component lock. |
locks/golang-k8s-client.lock |
Removes component lock. |
locks/golang-google-api.lock |
Removes component lock. |
locks/golang-github-googleapis-gax.lock |
Removes component lock. |
locks/golang-github-azure-autorest.lock |
Removes component lock. |
locks/golang-contrib-opencensus-exporter-ocagent.lock |
Removes component lock. |
locks/golang-cloud-google.lock |
Removes component lock. |
locks/golang-cloud-google-storage.lock |
Removes component lock. |
locks/golang-cloud-google-longrunning.lock |
Removes component lock. |
locks/golang-cloud-google-iam.lock |
Removes component lock. |
locks/golang-cloud-google-cloudtasks.lock |
Removes component lock. |
locks/golang-cloud-google-auth.lock |
Removes component lock. |
specs/m/migrate/migrate.spec |
Removes rendered spec. |
specs/m/migrate/sources |
Removes source checksum. |
specs/m/migrate/0001-refactor-update-go-gitlab.patch |
Removes package patch. |
specs/g/golang-k8s-client/golang-k8s-client.spec |
Removes rendered spec. |
specs/g/golang-k8s-client/sources |
Removes source checksum. |
specs/g/golang-google-api/golang-google-api.spec |
Removes rendered spec. |
specs/g/golang-google-api/sources |
Removes source checksum. |
specs/g/golang-github-googleapis-gax/golang-github-googleapis-gax.spec |
Removes rendered spec. |
specs/g/golang-github-googleapis-gax/sources |
Removes source checksum. |
specs/g/golang-github-azure-autorest/golang-github-azure-autorest.spec |
Removes rendered spec. |
specs/g/golang-github-azure-autorest/sources |
Removes source checksum. |
specs/g/golang-github-azure-autorest/format-error.patch |
Removes package patch. |
specs/g/golang-contrib-opencensus-exporter-ocagent/golang-contrib-opencensus-exporter-ocagent.spec |
Removes rendered spec. |
specs/g/golang-contrib-opencensus-exporter-ocagent/sources |
Removes source checksum. |
specs/g/golang-cloud-google/golang-cloud-google.spec |
Removes rendered spec. |
specs/g/golang-cloud-google/sources |
Removes source checksum. |
specs/g/golang-cloud-google-storage/golang-cloud-google-storage.spec |
Removes rendered spec. |
specs/g/golang-cloud-google-storage/sources |
Removes source checksum. |
specs/g/golang-cloud-google-longrunning/golang-cloud-google-longrunning.spec |
Removes rendered spec. |
specs/g/golang-cloud-google-longrunning/sources |
Removes source checksum. |
specs/g/golang-cloud-google-iam/golang-cloud-google-iam.spec |
Removes rendered spec. |
specs/g/golang-cloud-google-iam/sources |
Removes source checksum. |
specs/g/golang-cloud-google-cloudtasks/golang-cloud-google-cloudtasks.spec |
Removes rendered spec. |
specs/g/golang-cloud-google-cloudtasks/sources |
Removes source checksum. |
specs/g/golang-cloud-google-auth/golang-cloud-google-auth.spec |
Removes rendered spec. |
specs/g/golang-cloud-google-auth/sources |
Removes source checksum. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Member
Author
|
/azp run |
|
Azure Pipelines: 2 pipeline(s) were filtered out due to trigger conditions. |
reuben olinsky (reubeno)
marked this pull request as ready for review
August 26, 2026 21:21
reuben olinsky (reubeno)
changed the title
builddeps-only components
Member
|
LGTM! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Drop unused Google Cloud SDK, Autorest, client-go, migrate, and ocagent components. Their remaining consumers are bundled or removed.
Justification
We're having build issues with many of these packages, due to dependencies, compatibility with newer golang versions, etc. Each of the removed packages (except for
migrate) has been abandoned/sunset in upstream Fedora. Theazure-autorestpackage is for an OSS Microsoft project that was archived ~18 months ago.migrateis alive and kicking in Fedora, but we do not have any dependencies on it within the distro, nor is it part of our supportedbaserepo.Verification
Copilot analysis +
dnfrepoquery analysis +pkgxanalysis (run against PME prod koji RPMs) to confirm that the only build or runtime dependencies on these packages are from within this set.