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

Build time optimization #257

Open
jotak opened this issue Jul 18, 2022 · 2 comments
Open

Build time optimization #257

jotak opened this issue Jul 18, 2022 · 2 comments

Comments

@jotak
Copy link
Member

jotak commented Jul 18, 2022

FLP takes quite a long time to build and it seems like there could be some optimizations.

One thing that seems not optimal and taking quite some time is, for every code change, re-downloading/installing golangci-lint. As it's part of make build_code, it's in the same docker cache layer than code build, hence is retriggered for every code change. We should see how we could decouple them.

Other possible improvements to consider:

  • check whether all dependencies we have are actually necessary
  • add vendors in git?
@ronensc
Copy link
Collaborator

ronensc commented Jul 19, 2022

I found that this line is run before make build_code

RUN go mod download -modfile=.bingo/golangci-lint.mod

Doesn't it download golangci-lint?

@jotak
Copy link
Member Author

jotak commented Jul 19, 2022

Doesn't it download golangci-lint?

Despite this download, I often see the output stuck at that point:

(re)installing /app/bin/golangci-lint-v1.43.0

But maybe I am mislead by lacking output at different steps, it's possible that this "(re)installing golangci-lint" is actually finished but there's another hidden step behind .. like linting

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants