neoave / mrack

Multicloud use-case based multihost async provisioner for CIs and testing during development
Apache License 2.0
11 stars 14 forks source link

style: Increase readability of logs by using host #172

Closed Tiboris closed 2 years ago

Tiboris commented 2 years ago

Adding a host name from metadata or host object where it is possible to increase readability of the log messages and better debugging with only using the log content. Before this patch there were logging messages that were describing the error but from the log message we could not tell which host was causing that.

Signed-off-by: Tibor Dudlák tdudlak@redhat.com

Tiboris commented 2 years ago

Yeah the thing is that we can not use it at every single place somewhere host object is ready somewhere only dict from requirement.

dav-pascual commented 2 years ago

We could pass the parameters just to build the string, but i think this is good enough!