Skip to content
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

vendor: Add #12

Merged
merged 3 commits into from
Apr 6, 2018
Merged

vendor: Add #12

merged 3 commits into from
Apr 6, 2018

Conversation

gaocegege
Copy link
Member

Close #5

Using dep to install all vendor dependencies.

/assign @YujiOshima

Gopkg.toml Outdated

[[constraint]]
name = "github.com/docker/docker"
version = "1.13.1"

Choose a reason for hiding this comment

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

Note: this version is no longer maintained. I'd suggest syncing with the master.

Copy link
Member Author

Choose a reason for hiding this comment

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

@AkihiroSuda Thanks for your advice, I will update it.

Signed-off-by: Ce Gao <[email protected]>
Signed-off-by: Ce Gao <[email protected]>
@gaocegege
Copy link
Member Author

@AkihiroSuda

Addressed. PTAL

Gopkg.toml Outdated

[[constraint]]
name = "github.com/docker/docker"
branch = "master"

Choose a reason for hiding this comment

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

nit: I guess you don't need to retain this constraint entry for the master branch

Copy link

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

LGTM (non-binding)

@YujiOshima
Copy link
Contributor

YujiOshima commented Apr 6, 2018

@gaocegege Thank you for great work!

Each docker image is still using go get packages.
We should update Dockerfiles to use the vendored packages.

@gaocegege
Copy link
Member Author

@YujiOshima Yeah, I think we could use multiple stage build to reuse the vendor to build binaries.

@YujiOshima
Copy link
Contributor

@gaocegege Absolutely, It can reduce image size so much and accelerate images build.

@gaocegege
Copy link
Member Author

I opened an issue to keep track of the process #14

@YujiOshima
Copy link
Contributor

Thanks!
/lgtm

@k8s-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: YujiOshima

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@gaocegege gaocegege merged commit f38278f into kubeflow:master Apr 6, 2018
@gaocegege gaocegege deleted the vendor branch April 6, 2018 07:54
kuikuikuizzZ pushed a commit to kuikuikuizzZ/katib that referenced this pull request Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants