-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
re-introduce vendor to satisfy reproducible builds requirement #3518
Conversation
/run-e2e internal* |
Can't approve from mobile, > 10k files changed and more than 3kk rows. Is it okey? |
Didn't you read all of them? |
I am not able to convince Hava tried several things, something like this:
ends with files like this (note the change in import):
the change ulitmately causes failed docker build:
💩 💩 💩 💩 💩 💩 💩 💩 |
9c6cd8a
to
cfa7bb9
Compare
794bed9
to
b61866a
Compare
/run-e2e internal* |
Signed-off-by: Zbynek Roubalik <[email protected]>
Signed-off-by: Zbynek Roubalik <[email protected]>
Signed-off-by: Zbynek Roubalik <[email protected]>
Signed-off-by: Zbynek Roubalik <[email protected]>
Signed-off-by: Zbynek Roubalik <[email protected]>
Signed-off-by: Zbynek Roubalik <[email protected]>
Signed-off-by: Zbynek Roubalik <[email protected]>
/run-e2e internal* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
re-introduce vendor to satisfy reproducible builds requirement
Checklist
Fixes #3509