Skip to content

Remove Dependabot auto-merge workflow#1140

Merged
erain merged 1 commit intoGoogleCloudPlatform:masterfrom
erain:disable-dependabot-auto-merge
Mar 24, 2026
Merged

Remove Dependabot auto-merge workflow#1140
erain merged 1 commit intoGoogleCloudPlatform:masterfrom
erain:disable-dependabot-auto-merge

Conversation

@erain
Copy link
Contributor

@erain erain commented Mar 24, 2026

Summary

This PR removes the auto-merge functionality for Dependabot PRs that was introduced in #1035.

Rationale

While the original intent of auto-merge was to reduce manual toil for routine dependency updates, automatic merging has proven to be risky for this repository.

The Problem

The auto-merge workflow only considered semantic versioning classifications (patch and minor updates) when deciding whether to auto-merge:

if: ${{ steps.metadata.outputs.update-type == 'version-update:semver-patch' || steps.metadata.outputs.update-type == 'version-update:semver-minor' }}

However, this is insufficient for Kubernetes-related dependencies where "minor" version bumps can introduce breaking API changes.

Real-world Example

A recent auto-merged commit (13a30c12) updated k8s.io/apiserver from 0.33.1 to 0.35.2.

This was classified as a "version-update:semver-minor" update, but:

  • The jump spans two minor versions (0.33 → 0.35)
  • Kubernetes Go module minor versions often contain API deprecations and behavioral changes
  • Such changes require human review to assess impact on the custom metrics adapter and other components

Proposed Change

Remove the .github/workflows/dependabot-auto-merge.yaml workflow entirely. After this change:

  • All Dependabot PRs will require manual review and approval
  • Maintainers can properly assess the risk of each dependency update
  • Safety is prioritized over automation convenience

Related

@google-cla
Copy link

google-cla bot commented Mar 24, 2026

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

This change removes the auto-merge functionality for Dependabot PRs that was
introduced in PR GoogleCloudPlatform#1035. While the original intent was to reduce manual toil
for routine dependency updates, automatic merging has proven to be risky for
this repository.

The auto-merge workflow only considered semantic versioning (patch and minor
updates), but this is insufficient for Kubernetes-related dependencies where
minor version bumps can introduce breaking API changes.

For example, commit 13a30c1 updated
k8s.io/apiserver from 0.33.1 to 0.35.2, which was classified as a
minor update but actually contained breaking changes that could affect
compatibility.

After this change, all Dependabot PRs will require manual review and
approval before merging, ensuring that maintainers can properly assess
the risk of each dependency update.
@erain erain force-pushed the disable-dependabot-auto-merge branch from 412835b to 5e62b96 Compare March 24, 2026 14:38
@erain erain requested a review from juli4n March 24, 2026 14:41
Copy link
Contributor

@juli4n juli4n left a comment

Choose a reason for hiding this comment

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

Thanks!

@erain erain merged commit aff1dd3 into GoogleCloudPlatform:master Mar 24, 2026
15 checks passed
@erain erain deleted the disable-dependabot-auto-merge branch March 24, 2026 14:43
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