The orchestrator.log file prints the reactor configuration like this:
atomic_reactor.plugins.reactor_config - INFO - reading config content ...
It simply prints a string representation of very large dict. This is hard for human users to read when trying to understand the current settings. It is also hard for developers to compare log files with tools like "diff".
By contrast, the ContainerTasker class uses json.dumps() when logging, for example:
logger.debug(json.dumps(info, indent=2))
logger.info(json.dumps(version, indent=2))
It would be great to pretty-print reactor_conf.conf as well.
This change would make OSBS easier to understand.
The
orchestrator.logfile prints the reactor configuration like this:It simply prints a string representation of very large dict. This is hard for human users to read when trying to understand the current settings. It is also hard for developers to compare log files with tools like "diff".
By contrast, the ContainerTasker class uses
json.dumps()when logging, for example:It would be great to pretty-print
reactor_conf.confas well.This change would make OSBS easier to understand.