OutputFile logging option

[mbalamaruthu]

Add an option to log to a file using lumberjack in addition to stdout logging

[alexandrevilain]

Hi!
Thanks for the contribution @mbalamaruthu !

I'm not a big fan of such addition, because in the Kubernetes world, it's a up to the user to handle logs in a container-native way. Some users would use kubectl logs others (and most of them) would use a logging aggreation tools like fluentbit.

What's your use-case ?

[mbalamaruthu]

Hi! Thanks for the contribution @mbalamaruthu !

I'm not a big fan of such addition, because in the Kubernetes world, it's a up to the user to handle logs in a container-native way. Some users would use kubectl logs others (and most of them) would use a logging aggreation tools like fluentbit.

What's your use-case ?

Basically I'm looking for an option to log to an output file as well, which further could be forwarded or collected using a sidecar.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[alexandrevilain]

Basically I'm looking for an option to log to an output file as well, which further could be forwarded or collected using a sidecar.

Can't you just forward stdout logs to a file for this use-case ?

[mbalamaruthu]

Basically I'm looking for an option to log to an output file as well, which further could be forwarded or collected using a sidecar.

Can't you just forward stdout logs to a file for this use-case ?

Yes that is one approach but log rotation, archival and cleanups also needs to be handled somehow though.

[alexandrevilain]

Hi @mbalamaruthu !

After checking your PR another time, it may be ok to merge this PR but I would like to make this more configurable.
Could you please add more flags to configure the logs output feature?
Would would see something like:

• log-file
• log-file-max-size
• log-file-max-age-days
• And so one ..

WDYT ?