openSUSE / obs-build

OBS build script, can be used with OBS or stand alone
GNU General Public License v2.0
132 stars 183 forks source link

Z vm fixes #916

Open azouhr opened 1 year ago

azouhr commented 1 year ago

This is the changes, that I currently use to operate the z/VM worker on OBS. In order to simplify the debugging and logging of the worker, I also added a function ".log" that allows to specify a certain log level and to print out the affected file and function within that file.

.log 5 "This is a notice"

The severity levels are used from https://en.wikipedia.org/wiki/Syslog#Severity_level The default severity level is set to 4, to keep the default output clean.

Feel free to do changes if needed.

adrianschroeter commented 1 year ago

This way of logging is conflicting with our current idea of debug logging, we do not want to increase or decrease all kinds of logs, but we want to enable logging for specific areas. We want to enable logging by a specified area (eg. zvm, kvm or all vm handling) and enable debug logging only for these kind of operations.

Could you adapt your zvm worker fixes just to check for BUILD_VERBOSE_VM=true for example?