Skip to content

Conversation

@thaJeztah
Copy link
Member

relates to:

Commit 00a11e9 added these exclude rules as a temporary workaround until these transitive dependency versions would be gone;

downgrade go-difflib and go-spew to tagged releases

These dependencies were updated to "master" in some modules we depend on,
but have no code-changes since their last release. Unfortunately, this also
causes a ripple effect, forcing all users of the containerd module to also
update these dependencies to an unrelease / un-tagged version.

Both these dependencies will unlikely do a new release in the near future,
so exclude these versions so that we can downgrade to the current release.

Commit fb8c01d updated the containerd/zfs module to v2.0.0, which was the remaining dependency using these untagged versions, so we can remove these exclude rules again.

This reverts commit 00a11e9.

Commit 00a11e9 added these exclude
rules as a temporary workaround until these transitive dependency
versions would be gone;

> downgrade go-difflib and go-spew to tagged releases
>
> These dependencies were updated to "master" in some modules we depend on,
> but have no code-changes since their last release. Unfortunately, this also
> causes a ripple effect, forcing all users of the containerd module to also
> update these dependencies to an unrelease / un-tagged version.
>
> Both these dependencies will unlikely do a new release in the near future,
> so exclude these versions so that we can downgrade to the current release.

Commit fb8c01d updated the containerd/zfs
module to v2.0.0, which was the remaining dependency using these untagged
versions, so we can remove these exclude rules again.

This reverts commit 00a11e9.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@github-project-automation github-project-automation bot moved this to Needs Triage in Pull Request Review Dec 9, 2025
@dosubot dosubot bot added the dependencies Pull requests that update a dependency file label Dec 9, 2025
@austinvazquez austinvazquez moved this from Needs Triage to Needs Reviewers in Pull Request Review Dec 9, 2025
@github-project-automation github-project-automation bot moved this from Needs Reviewers to Review In Progress in Pull Request Review Dec 9, 2025
@AkihiroSuda AkihiroSuda added this pull request to the merge queue Dec 9, 2025
@AkihiroSuda AkihiroSuda added cherry-pick/2.1.x Change to be cherry picked to release/2.1 branch cherry-pick/2.2.x Change to be cherry picked to release/2.2 branch labels Dec 9, 2025
@thaJeztah
Copy link
Member Author

@AkihiroSuda for cherry pick; this will require both this one and #12641 to be cherry-picked (probably fine though)

Merged via the queue into containerd:main with commit 640f0ee Dec 9, 2025
89 of 92 checks passed
@github-project-automation github-project-automation bot moved this from Review In Progress to Done in Pull Request Review Dec 9, 2025
@thaJeztah thaJeztah deleted the no_exclude branch December 9, 2025 17:10
@AkihiroSuda AkihiroSuda added cherry-picked/2.1.x PR commits are cherry picked into the release/2.1 branch cherry-picked/2.2.x PR commits are cherry-picked into release/2.2 branch cherry-pick/2.1.x Change to be cherry picked to release/2.1 branch and removed cherry-pick/2.1.x Change to be cherry picked to release/2.1 branch cherry-pick/2.2.x Change to be cherry picked to release/2.2 branch cherry-picked/2.1.x PR commits are cherry picked into the release/2.1 branch labels Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick/2.1.x Change to be cherry picked to release/2.1 branch cherry-picked/2.2.x PR commits are cherry-picked into release/2.2 branch dependencies Pull requests that update a dependency file size/S

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants