Skip to content

NO-JIRA: Hermeto: go mod vendor#17

Merged
openshift-merge-bot[bot] merged 3 commits intoopenshift:mainfrom
rayfordj:hermeto/go-mod-vendor
Apr 22, 2025
Merged

NO-JIRA: Hermeto: go mod vendor#17
openshift-merge-bot[bot] merged 3 commits intoopenshift:mainfrom
rayfordj:hermeto/go-mod-vendor

Conversation

@rayfordj
Copy link
Copy Markdown
Contributor

https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=67393988

https://download-01.beak-001.prod.iad2.dc.redhat.com/brewroot/work/tasks/3988/67393988/remote-sources.log

2025-04-22 09:02:11,555 INFO Vendoring the gomod dependencies
2025-04-22 09:02:22,047 ERROR vendor/modules.txt changed after vendoring:
diff --git a/vendor/modules.txt b/vendor/modules.txt
index c6c9b64..974648e 100644
--- a/vendor/modules.txt
+++ b/vendor/modules.txt
@@ -186,7 +186,6 @@ github.com/prometheus/procfs/internal/fs
 github.com/prometheus/procfs/internal/util
 # github.com/robfig/cron/v3 v3.0.1
 ## explicit; go 1.12
-github.com/robfig/cron/v3
 # github.com/sirupsen/logrus v1.9.3
 ## explicit; go 1.13
 github.com/sirupsen/logrus
2025-04-22 09:02:23,690 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.
  Note that you may need to `git add --force` ignored files in the vendor/ dir.
  Docs: https://github.com/hermetoproject/hermeto/blob/main/docs/gomod.md#vendoring
2025/04/22 09:02:24 Skipping step because a previous step failed

https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=67393988

https://download-01.beak-001.prod.iad2.dc.redhat.com/brewroot/work/tasks/3988/67393988/remote-sources.log

```
2025-04-22 09:02:11,555 INFO Vendoring the gomod dependencies
2025-04-22 09:02:22,047 ERROR vendor/modules.txt changed after vendoring:
diff --git a/vendor/modules.txt b/vendor/modules.txt
index c6c9b64..974648e 100644
--- a/vendor/modules.txt
+++ b/vendor/modules.txt
@@ -186,7 +186,6 @@ github.com/prometheus/procfs/internal/fs
 github.com/prometheus/procfs/internal/util
 # github.com/robfig/cron/v3 v3.0.1
 ## explicit; go 1.12
-github.com/robfig/cron/v3
 # github.com/sirupsen/logrus v1.9.3
 ## explicit; go 1.13
 github.com/sirupsen/logrus
2025-04-22 09:02:23,690 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.
  Note that you may need to `git add --force` ignored files in the vendor/ dir.
  Docs: https://github.com/hermetoproject/hermeto/blob/main/docs/gomod.md#vendoring
2025/04/22 09:02:24 Skipping step because a previous step failed
```
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Apr 22, 2025
@openshift-ci-robot
Copy link
Copy Markdown

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

Details

In response to this:

https://brewweb.engineering.redhat.com/brew/taskinfo?taskID=67393988

https://download-01.beak-001.prod.iad2.dc.redhat.com/brewroot/work/tasks/3988/67393988/remote-sources.log

2025-04-22 09:02:11,555 INFO Vendoring the gomod dependencies
2025-04-22 09:02:22,047 ERROR vendor/modules.txt changed after vendoring:
diff --git a/vendor/modules.txt b/vendor/modules.txt
index c6c9b64..974648e 100644
--- a/vendor/modules.txt
+++ b/vendor/modules.txt
@@ -186,7 +186,6 @@ github.com/prometheus/procfs/internal/fs
github.com/prometheus/procfs/internal/util
# github.com/robfig/cron/v3 v3.0.1
## explicit; go 1.12
-github.com/robfig/cron/v3
# github.com/sirupsen/logrus v1.9.3
## explicit; go 1.13
github.com/sirupsen/logrus
2025-04-22 09:02:23,690 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.
 Note that you may need to `git add --force` ignored files in the vendor/ dir.
 Docs: https://github.com/hermetoproject/hermeto/blob/main/docs/gomod.md#vendoring
2025/04/22 09:02:24 Skipping step because a previous step failed

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.

@rayfordj
Copy link
Copy Markdown
Contributor Author

/cc @kaovilai
/cc @jparrill

PR to unblock downstream OADP builds.

Copy link
Copy Markdown
Member

@kaovilai kaovilai left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 22, 2025
@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Apr 22, 2025
@openshift-ci openshift-ci Bot removed the lgtm Indicates that a PR is ready to be merged. label Apr 22, 2025
@rayfordj
Copy link
Copy Markdown
Contributor Author

/label tide/merge-method-squash

@openshift-ci openshift-ci Bot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Apr 22, 2025
@rayfordj rayfordj requested a review from kaovilai April 22, 2025 16:13
Copy link
Copy Markdown
Member

@kaovilai kaovilai left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Apr 22, 2025
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 22, 2025

@rayfordj: 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.

@jparrill
Copy link
Copy Markdown
Contributor

/approve

@openshift-merge-bot openshift-merge-bot Bot merged commit bcdcf10 into openshift:main Apr 22, 2025
5 checks passed
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 22, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jparrill, kaovilai, rayfordj

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

@rayfordj rayfordj deleted the hermeto/go-mod-vendor branch April 22, 2025 16:24
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. 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. tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants