← Back to issue list

chore(deps): update dependency protobuf to v4

View original Github issue

Metadata

Project
charmcraft
Number
#980
Type
pull request
State
closed
Author
renovate[bot]
Labels
—
Created
Updated
Closed

Current evaluation

Renovate PR to bump protobuf ==3.20.3 -> ==4.21.12 was closed unmerged on 2023-01-20; lengau denied it because macaroonbakery required protobuf 3. Renovate ignored4.x afterwards; the bump only landed later via #1639 (v5, commit c3805214).

Resolution: closed not a bug

Reason: Maintainer lengau closed the PR with "Denying this because macaroonbakery requires protobuf 3", and Renovate confirmed the close was intentional (ignore notification for all 4.x releases). The upgrade was therefore invalid at the time due to an upstream dependency constraint, not merged; it was only performed years later in #1639 (commit c3805214, protobuf v5) once macaroonbakery gained protobuf 4/5 support.

Evaluated by: mr-cal-bot

No scores available.

Issue body

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [protobuf](https://developers.google.com/protocol-buffers/) | `==3.20.3` -> `==4.21.12` | [![age](https://badges.renovateapi.com/packages/pypi/protobuf/4.21.12/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/pypi/protobuf/4.21.12/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/pypi/protobuf/4.21.12/compatibility-slim/3.20.3)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/pypi/protobuf/4.21.12/confidence-slim/3.20.3)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - "every weekend" in timezone Etc/UTC, Automerge - "after 1 am and before 7 am" in timezone Etc/UTC. 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://app.renovatebot.com/dashboard#github/canonical/charmcraft). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNC44NC4yIiwidXBkYXRlZEluVmVyIjoiMzQuODQuMiJ9-->

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed not a bug Renovate PR to bump protobuf ==3.20.3 -> ==4.21.12 was closed unmerged on 2023-01-20; lengau denied it because macaroonbakery required protobuf 3. Renovate ignored4.x afterwards; the bump only landed later via #1639 (v5, commit c3805214).
stealth/space-bunny-alpha — closed not a bug Renovate PR to bump protobuf 3.20.3 -> 4.21.12 was closed unmerged: maintainer lengau denied it because macaroonbakery required protobuf 3, and Renovate ignored future 4.x updates.
stealth/space-bunny-alpha — closed not a bug Rejected protobuf 4.21.12 upgrade in PR #980; it was closed unmerged, and Renovate was configured to ignore future 4.x releases.
stealth/space-bunny-alpha — closed not a bug Renovate PR to bump protobuf ==3.20.3 -> ==4.21.12 was closed unmerged on 2023-01-20 after maintainer @lengau denied it, citing macaroonbakery's protobuf 3 requirement; Renovate then ignored future 4.x updates.
stealth/space-bunny-alpha — closed not a bug Protobuf 4.21.12 upgrade was rejected and PR #980 closed unmerged because macaroonbakery requires protobuf 3; Renovate will ignore future 4.x updates.
qwen/qwen3.6-35b-a3b — — Rejected and closed without merging due to a compatibility conflict; the upstream dependency macaroonbakery requires protobuf version 3, preventing the v4 update. Renovate will ignore future v4 updates for this package.
qwen3.6-35b-a3b-mtp-q6 — — Rejected because macaroonbakery requires protobuf v3. The update was closed without merging, and Renovate will ignore future v4 releases.

Update history

No update history recorded yet.

Related work

  • Superseded By: canonical/charmcraft#1639 (confidence 85%)

    Merged protobuf bump to v5 (commit c3805214) after macaroonbakery added protobuf 4/5 support, effectively accomplishing what #980 could not.

  • Related To: canonical/charmcraft#1946 (confidence 45%)

    Later Renovate bump of protobuf to v5.28.2 (commit f2fe9c79), showing repeated pin churn around this dependency.

Related issues

Issue Project State Summary Similarity
#1639 chore(deps): update dependency protobuf to v5 charmcraft merged Renovate bumped protobuf from ==3.20.3 to ==5.26.1 in requirements files; merged via commit c3805214. Safe since protobuf is only an indirect dep via macaroonbakery, which gained protobuf 4/5 support upstream.
88%
#131 chore(deps): freeze protobuf craft-store merged Merged #131 as commit 317166c, freezing protobuf to avoid incompatibilities tracked by py-macaroon-bakery#94.
74%
#4484 chore(deps): pin protobuf in docs snapcraft closed Closed unmerged after prolonged inactivity; the one-line protobuf documentation-dependency pin intended to fix build-docs was never merged.
71%
#1028 chore(deps): update dependency humanize to v4.6.0 charmcraft merged Renovate dependency bump PR #1028 was merged (merge commit d011d922, branch commit b4964336), pinning humanize from ==4.5.0 to ==4.6.0 in the 2.3/3.x era requirements; no code changes needed.
71%
#132 Remove protobuf requirement craft-store closed Removed the temporary protobuf constraint after macaroonbakery 1.3.4 fixed upstream #94; PR #156 merged as b67cf49 and shipped in craft-store 2.6.1.
71%
#785 chore(deps): update dependency myst-parser to v3 craft-parts merged Renovate's myst-parser 2.0.0 -> 3.0.1 bump was approved and merged via commit 44e43e43 ("chore(deps): update dependency myst-parser to v3"), changing the pin in a single file; later superseded by the v4 bump in #871.
70%