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

Dockerized Development Misses The Vendor Folder #1259

Closed
marwan-at-work opened this issue May 30, 2019 · 8 comments
Closed

Dockerized Development Misses The Vendor Folder #1259

marwan-at-work opened this issue May 30, 2019 · 8 comments
Labels
bug Something isn't working ci/cd Work to do to improve our CI/CD system good first issue Great issues for new Athenians to work on!

Comments

@marwan-at-work
Copy link
Contributor

Our docker-compose up dev command uses the same Dockerfile we use in our deployment pipeline which includes -mod=vendor...However, our deployment pipeline runs go mod vendor on the fly while our development environment doesn't

We therefore should expose the local cache to our Dockerfile build in our pipeline so that we can remove the -mod=vendor and also ensure that we don't re-download everything when building docker.

@marwan-at-work marwan-at-work added bug Something isn't working ci/cd Work to do to improve our CI/CD system good first issue Great issues for new Athenians to work on! labels May 30, 2019
@paivagustavo
Copy link

Seems to be duplicated from #1242

@marwan-at-work
Copy link
Contributor Author

Potentially :) this one is more concerned with making sure our CI/CD pipeline caches the GOPATH (or rebuilds with a public proxy)

@keremgocen
Copy link

Hey, I'm new here and still getting familiar with the system. This one looks like something I can start working on.

@marwan-at-work
Copy link
Contributor Author

@keremgocen sounds good ✌️

@arschles
Copy link
Member

arschles commented Aug 5, 2019

@keremgocen would you still be interested in taking this one? I'm happy to help out if you are. If not, absolutely no worries. I just wanted to check in.

@keremgocen
Copy link

@arschles feel free to take it, I recently started a new project so it's a bit hard for me to create extra time right now. I'm sure I can find something else to work on when I have a better schedule.

@chriscoffee
Copy link
Member

chriscoffee commented Aug 6, 2019

Just so I understand you right with this one @marwan-at-work (it's rather late here):

Are you wanting the developer environment in docker-compose to be able to depend_on a second container, which does the go mod download && go mod vendor and uses it as a mounted volume shared between the two. i.e https://gist.github.com/chriscoffee/f7e6d7052c0a426eebaceaf1ab2c8218#file-docker-compose-yml

Or

You want to migrate cmd/proxy/Dockerfile to build with download and vendor and then pass from the builder so it can be used elsewhere? i.e https://gist.github.com/chriscoffee/f7e6d7052c0a426eebaceaf1ab2c8218#file-dockerfile

Or something completely different? 🤷‍♂😅

P.S Excuse the crude examples

@DrPsychick
Copy link
Member

This seems obsolete by now, no -mod=vendor in the Dockerfile. Please reopen if I'm mistaken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ci/cd Work to do to improve our CI/CD system good first issue Great issues for new Athenians to work on!
Projects
None yet
Development

No branches or pull requests

6 participants