search

pwaller / docker-show-context

Show where time is wasted during the context upload of `docker build`
MIT License
335 stars 16 forks source link

It seems it would be very useful if this took the current .dockerignore into account #3

Closed richtera closed 6 years ago

richtera commented 6 years ago

Otherwise you will always get the top 10 offenders to be things you've already excluded using dockerignore. I would like to hone in on the best choices in dockerignore.

pwaller commented 6 years ago

Ah, interesting if this is not already the case! Patches very welcome if you want to work on it, please announce on this thread.

yciabaud commented 6 years ago

I confirm this is not using .dockerignore and this less useful than I expected it to be :)

pwaller commented 6 years ago

I see the problem here. .dockerignore is taken into account, but the messaging is confusing. I have a fix inbound.