gerrit-hooks has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 4:
* Update Tracker::IGNORE, no bug url/s found
* Set MODIFIED::IGNORE, no Bug-Url found.
--
To view, visit ht
Dan Kenigsberg has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 3: Code-Review+2
raising
--
To view, visit https://gerrit.ovirt.org/64217
To unsubscribe, visit https://gerr
Dan Kenigsberg has submitted this change and it was merged.
Change subject: logging: Improve log level alignment
..
logging: Improve log level alignment
Shorten WARNING to WARN and CRITICAL to CRIT to make log messages align
ni
Edward Haas has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 3:
IMO the two char saving is not worth the small hack
(https://gerrit.ovirt.org/#/c/64217/3/vdsm/vdsm), it is a
Nir Soffer has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 3:
Edward, 2 spaces on each log line, millions of spaces per day :-)
--
To view, visit https://gerrit.ovirt.org/
Piotr Kliczewski has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/64217
To unsubscribe, visit https://gerrit.ovir
Edward Haas has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 3:
Does it really worth it for just saving 2 spaces?
If this is just for the output format, can it be cut there
Milan Zamazal has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 3: Code-Review+1
--
To view, visit https://gerrit.ovirt.org/64217
To unsubscribe, visit https://gerrit.ovirt.o
Nir Soffer has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 3: Verified+1
--
To view, visit https://gerrit.ovirt.org/64217
To unsubscribe, visit https://gerrit.ovirt.org/set
gerrit-hooks has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 3:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Piotr Kliczewski has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 2:
Looks good to me but please add example to commit message as Milan suggests
--
To view, visit https://g
Milan Zamazal has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 2:
(1 comment)
I like the change, let's just replace the reference in the commit message.
https://gerrit.ovir
gerrit-hooks has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 2:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Nir Soffer has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 2: Verified+1
--
To view, visit https://gerrit.ovirt.org/64217
To unsubscribe, visit https://gerrit.ovirt.org/set
Nir Soffer has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 1: Verified+1
--
To view, visit https://gerrit.ovirt.org/64217
To unsubscribe, visit https://gerrit.ovirt.org/set
gerrit-hooks has posted comments on this change.
Change subject: logging: Improve log level alignment
..
Patch Set 1:
* Update tracker: IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches
Nir Soffer has uploaded a new change for review.
Change subject: logging: Improve log level alignment
..
logging: Improve log level alignment
Shorten WARNING to WARN and CRITICAL to CRIT to make log messages align
nicer.
Here i
17 matches
Mail list logo