This is an archive of the discontinued Mercurial Phabricator instance.

Differential D6611

blackbox: disable extremely verbose logging (issue6110)
ClosedPublic

Authored by valentin.gatienbaron on Jul 7 2019, 12:13 PM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Commits
rHG56132ebd14c6: blackbox: disable extremely verbose logging (issue6110)
rHG341222d54bea: blackbox: disable extremely verbose logging (issue6110)
Summary

This is maybe not the best way to go about fixing this, but anything
is better than the status quo.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

durin42 added a subscriber: durin42.Jul 10 2019, 3:38 PM
durin42 added inline comments.
hgext/blackbox.py
97

Let's make this configurable, and this is the default. Maybe blackbox.ignore, with a default of the things you've specified?

valentin.gatienbaron marked an inline comment as done.Jul 10 2019, 6:21 PM
valentin.gatienbaron added inline comments.
hgext/blackbox.py
97

I made the ignore list configurable, although it seems to be of limited utility: you can replace the default, but extending seems like the more useful thing to do. Extending the list requires repeating the default, and the default may not be particularly stable across versions.

I changed the priority of tracked and ignored, so tracked takes predecence. This means that tracked = *, cmdserver would log cmdserver events, as that seems a bit better to me and maybe that's why you wanted the configurability.

valentin.gatienbaron added a commit: rHG341222d54bea: blackbox: disable extremely verbose logging (issue6110).Jul 17 2019, 7:05 PM
This revision was not accepted when it landed; it landed in state Needs Review.
Closed by commit rHG341222d54bea: blackbox: disable extremely verbose logging (issue6110) (authored by valentin.gatienbaron). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mhgext/blackbox.py (4 lines)

Diff 15787

hgext/blackbox.py

) )
_lastlogger = loggingutil.proxylogger() _lastlogger = loggingutil.proxylogger()
class blackboxlogger(object): class blackboxlogger(object):
def __init__(self, ui, repo): def __init__(self, ui, repo):
self._repo = repo self._repo = repo
self._trackedevents = set(ui.configlist('blackbox', 'track')) self._trackedevents = set(ui.configlist('blackbox', 'track'))
self._untrackedevents = {'chgserver', 'cmdserver', 'extension'}
durin42Unsubmitted
Not Done

Let's make this configurable, and this is the default. Maybe blackbox.ignore, with a default of the things you've specified?

durin42: Let's make this configurable, and this is the default. Maybe blackbox.ignore, with a default of…
valentin.gatienbaronAuthorUnsubmitted
Done

I made the ignore list configurable, although it seems to be of limited utility: you can replace the default, but extending seems like the more useful thing to do. Extending the list requires repeating the default, and the default may not be particularly stable across versions.

I changed the priority of tracked and ignored, so tracked takes predecence. This means that tracked = *, cmdserver would log cmdserver events, as that seems a bit better to me and maybe that's why you wanted the configurability.

valentin.gatienbaron: I made the ignore list configurable, although it seems to be of limited utility: you can…
self._maxfiles = ui.configint('blackbox', 'maxfiles') self._maxfiles = ui.configint('blackbox', 'maxfiles')
self._maxsize = ui.configbytes('blackbox', 'maxsize') self._maxsize = ui.configbytes('blackbox', 'maxsize')
self._inlog = False self._inlog = False
def tracked(self, event): def tracked(self, event):
return b'*' in self._trackedevents or event in self._trackedevents return ((b'*' in self._trackedevents or event in self._trackedevents)
and event not in self._untrackedevents)
def log(self, ui, event, msg, opts): def log(self, ui, event, msg, opts):
# self._log() -> ctx.dirty() may create new subrepo instance, which # self._log() -> ctx.dirty() may create new subrepo instance, which
# ui is derived from baseui. So the recursion guard in ui.log() # ui is derived from baseui. So the recursion guard in ui.log()
# doesn't work as it's local to the ui instance. # doesn't work as it's local to the ui instance.
if self._inlog: if self._inlog:
return return
self._inlog = True self._inlog = True