-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
godeps: run go mod tidy and vendor for dependencies #122
godeps: run go mod tidy and vendor for dependencies #122
Conversation
PR red-hat-storage#121 bumped the ocs-op dependency but didn't update the vendor, we had a sub target of go mod tidy and vendor before building image and so CI passed silently and failed d/s where only vendor folder is used. Signed-off-by: Leela Venkaiah G <[email protected]>
Local build of image w/ only cached vendor: Removed the sub target which updates vendor for testing locally # gd
diff --git a/Makefile b/Makefile
index c5233a1..c65b100 100644
--- a/Makefile
+++ b/Makefile
@@ -82,7 +82,7 @@ go-build: ## Run go build against code.
run: manifests generate fmt vet ## Run a controller from your host.
go run ./main.go
-container-build: test-setup ## Build container image with the manager.
+container-build: ## Build container image with the manager.
docker build -t ${IMG} .
container-push: ## Push container image with the manager.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: leelavg, Madhu-1 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
a79f241
into
red-hat-storage:main
PR #121 bumped the ocs-op dependency but didn't update the vendor, we had a sub target of go mod tidy and vendor before building image and so CI passed silently and failed d/s where only vendor folder is used.