Skip to content

Run the same go list command the publishing bot runs#4131

Merged
kcp-ci-bot merged 4 commits into
kcp-dev:mainfrom
ntnn:prevent-publishing-bot-fail
May 19, 2026
Merged

Run the same go list command the publishing bot runs#4131
kcp-ci-bot merged 4 commits into
kcp-dev:mainfrom
ntnn:prevent-publishing-bot-fail

Conversation

@ntnn
Copy link
Copy Markdown
Member

@ntnn ntnn commented May 18, 2026

Summary

The publishing bot fails to resolve dependencies where go mod tidy happily succeeds.
go mod tidy only checks/updates what should go into the go.mod/go.sum.
go list ... builds the entire dependency graph.

What Type of PR Is This?

/kind bug

Related Issue(s)

Fixes #

Release Notes

NONE

@kcp-ci-bot kcp-ci-bot added release-note-none Denotes a PR that doesn't merit a release note. dco-signoff: yes Indicates the PR's author has signed the DCO. kind/bug Categorizes issue or PR as related to a bug. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 18, 2026
@ntnn
Copy link
Copy Markdown
Member Author

ntnn commented May 18, 2026

It's breaking ofc because it's missing the fix from #4130

ntnn added 3 commits May 18, 2026 16:49
Signed-off-by: Nelo-T. Wallus <red.brush9525@fastmail.com>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>
Signed-off-by: Nelo-T. Wallus <red.brush9525@fastmail.com>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>
Signed-off-by: Nelo-T. Wallus <red.brush9525@fastmail.com>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>
@ntnn ntnn force-pushed the prevent-publishing-bot-fail branch from 8db3eb3 to c4185cc Compare May 18, 2026 14:51
@kcp-ci-bot kcp-ci-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 18, 2026
Signed-off-by: Nelo-T. Wallus <red.brush9525@fastmail.com>
Signed-off-by: Nelo-T. Wallus <n.wallus@sap.com>
@ntnn ntnn force-pushed the prevent-publishing-bot-fail branch from c4185cc to 4243fe9 Compare May 18, 2026 14:56
@ntnn ntnn moved this to In review in tbd May 18, 2026
@ntnn ntnn added this to tbd May 18, 2026
Copy link
Copy Markdown
Contributor

@mjudeikis mjudeikis left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label May 19, 2026
@kcp-ci-bot
Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: f2e0ff10453d1cac734c43d61082df1c91870217

@kcp-ci-bot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mjudeikis

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kcp-ci-bot kcp-ci-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label May 19, 2026
@kcp-ci-bot kcp-ci-bot merged commit 2d38b47 into kcp-dev:main May 19, 2026
14 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in tbd May 19, 2026
@ntnn ntnn deleted the prevent-publishing-bot-fail branch May 19, 2026 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has signed the DCO. kind/bug Categorizes issue or PR as related to a bug. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants