Initialize file logging to more early

[viceice]

What would you like Renovate to be able to do?

Currently we miss a lot of log messages on file logging here

renovate/lib/workers/global/config/parse/index.ts

Lines 58 to 70 in dca3418

	// Add file logger
	// istanbul ignore if
	if (config.logFile) {
	logger.debug(
	`Enabling ${config.logFileLevel} logging to ${config.logFile}`
	);
	await ensureDir(getSubDirectory(config.logFile));
	addStream({
	name: 'logfile',
	path: config.logFile,
	level: config.logFileLevel,
	});
	}

I like to remove the config oiptions and initialize it via env only, like we do for LOG_LEVEL.

If you have any ideas on how this should be implemented, please tell us here.

Sure, do it like we do for LOG_LEVEL. Deprecate old option and remove in a future major change.

Is this a feature you are interested in implementing yourself?

Yes

[rarkins]

Which options specifically? e.g. LOG_FILE ?

[viceice]

Yes LOG_FILE and LOG_FILE_LEVEL.

[viceice]

@zharinov @RahulGautamSingh do you have some time for it?

should be easy to implement

[RahulGautamSingh]

@RahulGautamSingh do you have some time for it?

I can try it this week

[rarkins]

@viceice should we drop support of logFile and logFileLevel and target this for v38?

[viceice]

sounds good to me

[RahulGautamSingh]

One more PR is needed to complete this one. logFile and logFileContext should be dropped in it.

[viceice]

yes, should be dropped in v38

[renovate-release]

🎉 This issue has been resolved in version 38.0.0 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)
• 38.0.0

Your semantic-release bot 📦🚀