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

Info log level is too verbose #303

Closed
lubo opened this issue Aug 23, 2018 · 7 comments
Closed

Info log level is too verbose #303

lubo opened this issue Aug 23, 2018 · 7 comments
Assignees
Labels
kind/enhancement New feature or request

Comments

@lubo
Copy link

lubo commented Aug 23, 2018

I've been building an image containing many files and directories and my build log is pretty much just Not adding /foo/bar because it was added by a prior layer, Copying file /foo/bar to /bar/foo and Taking snapshot of files [<thousands of filenames>]... messages. How about dropping log level for these messages to debug?

@priyawadhwa priyawadhwa added the kind/enhancement New feature or request label Aug 31, 2018
priyawadhwa pushed a commit that referenced this issue Oct 12, 2018
Change loglevel for copying files to debug (#303)
@RafalSkolasinski
Copy link

This still seems to be a problem when building images using gcr.io/kaniko-project/executor:debug.
Any clue when the fix will propagate?

@jobec
Copy link

jobec commented Apr 11, 2019

Same here.
I'm also using the debug version (as specified here: https://docs.gitlab.com/ee/ci/docker/using_kaniko.html) and it is causing almost unusable build logs.

@slavoutich
Copy link

Kaniko executor has --verbosity flag, I assume setting it to warn will fix the issue for you (doing it now).

@jobec
Copy link

jobec commented Apr 15, 2019

This line is the problem:

logrus.Infof("Adding %s to layer, because it was changed.", path)

It was changed from debug to info between executor:latest (0.9.0) and executor:debug in this commit:
1bf4421#diff-736acbb393b480a7c2fa1ab622672582R181

@priyawadhwa can this be reopened?
If not fixed, this is going to explode in the next release.

@sharifelgamal sharifelgamal reopened this Apr 15, 2019
@sharifelgamal sharifelgamal self-assigned this Apr 15, 2019
@priyawadhwa
Copy link
Collaborator

@jobec thanks for pointing this out, looks like @sharifelgamal is on it!

@petervandenabeele
Copy link

petervandenabeele commented Apr 15, 2019

FYI, it's exploding now, I cannot build anymore: on gitlab, I get:
Job's log exceeded limit of 4194304 bytes

I will try the work-around like --verbosity warn (Ref. #638)

And that actually worked, the logs are useful again.

@sharifelgamal
Copy link
Contributor

This should now be fixed at HEAD.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

7 participants