Skip to content

Conversation

@adobes1
Copy link

@adobes1 adobes1 commented Nov 28, 2025

Hermetic builds require that running go mod vendor produces no diff. Without this PR, multus-networkpolicy can't be built hermetically.

  2025-11-28 13:52:18,370 ERROR [mode:STRICT] vendor directory changed after vendoring:
  M	vendor/github.com/spf13/cobra/command.go
  2025-11-28 13:52:18,384 DEBUG Running `/usr/local/go/go1.20/bin/go clean -modcache`
  2025-11-28 13:52:19,669 ERROR PackageRejected: The content of the vendor directory is not consistent with go.mod. Please check the logs for more details.
  Error: PackageRejected: The content of the vendor directory is not consistent with go.mod. Please check the logs for more details.
    Please try running `go mod vendor` and committing the changes.

Changes in this PR are the result of running go mod tidy and go mod vendor

Hermetic builds require that there be no diff
after running go mod tidy and go mod vendor.
Without these changes, multus-networkpolicy
can't be built hermetically.
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Nov 28, 2025
@openshift-ci-robot
Copy link
Contributor

@adobes1: This pull request explicitly references no jira issue.

Details

In response to this:

Hermetic builds require that running go mod vendor produces no diff. Without this PR, multus-networkpolicy can't be built hermetically.

 2025-11-28 13:52:18,370 ERROR [mode:STRICT] vendor directory changed after vendoring:
 M	vendor/github.com/spf13/cobra/command.go
 2025-11-28 13:52:18,384 DEBUG Running `/usr/local/go/go1.20/bin/go clean -modcache`
 2025-11-28 13:52:19,669 ERROR PackageRejected: The content of the vendor directory is not consistent with go.mod. Please check the logs for more details.
 Error: PackageRejected: The content of the vendor directory is not consistent with go.mod. Please check the logs for more details.
   Please try running `go mod vendor` and committing the changes.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot requested review from bpickard22 and dougbtv November 28, 2025 13:58
@zeeke
Copy link
Contributor

zeeke commented Nov 28, 2025

/lgtm
/approve
/label backport-risk-assessed

@openshift-ci openshift-ci bot added the backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. label Nov 28, 2025
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Nov 28, 2025
@openshift-ci
Copy link

openshift-ci bot commented Nov 28, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adobes1, zeeke

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 28, 2025
@adobes1
Copy link
Author

adobes1 commented Dec 1, 2025

/test okd-scos-images

@openshift-ci
Copy link

openshift-ci bot commented Dec 1, 2025

@adobes1: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Copy link

@SchSeba SchSeba left a comment

Choose a reason for hiding this comment

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

/hold

just a small comment. feel free to remove the hold

github.com/spf13/cobra v1.0.0/go.mod h1:/6GTrnGXV9HjY+aR4k0oJ5tcvakLuG6EuKReYlHNrgE=
github.com/spf13/cobra v1.1.3/go.mod h1:pGADOWyqRD/YMrPZigI/zbliZ2wVD/23d+is3pSWzOo=
github.com/spf13/cobra v1.4.0/go.mod h1:Wo4iy3BUC+X2Fybo0PDqwJIv3dNRiZLHQymsfxlB84g=
github.com/spf13/cobra v1.7.0 h1:hyqWnYt1ZQShIddO5kBpj3vu05/++x6tJ6dg8EC572I=
Copy link

Choose a reason for hiding this comment

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

I think this change is expected from u/s no?

Copy link
Author

Choose a reason for hiding this comment

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

Thank you! I'm unfortunately not too familiar with upstream work. If I remember correctly, changes to go.sum (produced by go mod tidy) don't block hermetic builds, so I can remove it from the PR if you'd like. However, this would cause verify-deps to fail.

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 2, 2025
@adobes1 adobes1 requested a review from SchSeba December 10, 2025 11:37
zeeke added a commit to zeeke/multus-networkpolicy that referenced this pull request Dec 11, 2025
…ebase

Bump mutli-networkpolicy-iptables to Kube 1.34
@SchSeba
Copy link

SchSeba commented Dec 24, 2025

please run this on https://github.com/k8snetworkplumbingwg/multi-networkpolicy and open the PR.
after that we can cherry-pick the changes.

btw that is the case also for 4.20 and 4.21 no?

@adobes1
Copy link
Author

adobes1 commented Jan 13, 2026

@SchSeba Apologies for the delay. From what I can see, this issue affects only 4.19. 4.20 and 4.21 are OK.

I ran go mod tidy and go mod vendor on https://github.com/k8snetworkplumbingwg/multi-networkpolicy-iptables (I assume you meant this repo instead of https://github.com/k8snetworkplumbingwg/multi-networkpolicy, as that one doesn't appear to be actively used - though I tested both and neither had vendor changes). The only changes produced were to go.mod, not to vendor/.

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. backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.