| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | None | T252751 ☂Use system loggers | |||
| Resolved | • kalle | T253852 Patch Mary to use system logger |
Event Timeline
Comment Actions
There is quite a bit of logging to stdout and stderr. This is not a problem for WMF, but it might be a problem if we in the future need to inspect the logs due to some issue in production. We should have a chat about this at some point, whether or not we should port all of these console output to use the Log4J framework used by Mary.
Comment Actions
Not a problem for us if you want to reduce the logging.
On the other hand, we have discussed the possibility of removing Marytts from the default configuration for Wikispeech in the future. So perhaps it's not worth spending too much effort on tweaking the logging.
Comment Actions
Merged to Gerrit
kalle@musa:~/projekt/wikimedia/wikispeech-services/mary-tts$ tmp=$(mktemp)
kalle@musa:~/projekt/wikimedia/wikispeech-services/mary-tts$ hook=$(readlink -f $(git rev-parse --git-dir))/hooks/commit-msg
kalle@musa:~/projekt/wikimedia/wikispeech-services/mary-tts$ git filter-branch -f --msg-filter "cat > $tmp; \"$hook\" $tmp; cat $tmp" @{u}..HEAD
WARNING: git-filter-branch has a glut of gotchas generating mangled history
rewrites. Hit Ctrl-C before proceeding to abort, then use an
alternative filtering tool such as 'git filter-repo'
(https://github.com/newren/git-filter-repo/) instead. See the
filter-branch manual page for more details; to squelch this warning,
set FILTER_BRANCH_SQUELCH_WARNING=1.
Proceeding with filter-branch...
Rewrite 97eebf4e2fa7efb4f97f7253d5208ccd72005ed2 (4/4) (0 seconds passed, remaining 0 predicted)
Ref 'refs/heads/master' was rewritten
kalle@musa:~/projekt/wikimedia/wikispeech-services/mary-tts$ git review -R
You are about to submit multiple commits. This is expected if you are
submitting a commit that is dependent on one or more in-review
commits, or if you are submitting multiple self-contained but
dependent changes. Otherwise you should consider squashing your
changes into one commit before submitting (for indivisible changes) or
submitting from separate branches (for independent changes).
The outstanding commits are:
e7447077 (HEAD -> master) Merge remote-tracking branch 'github/master'
9ed911be Removed docker build from travis build tests
6f2f63ed Merge pull request #5 from karlwettin/master
7ffebe54 Removed logging to file and added logging to syslog at WARNing threshold.
Do you really want to submit the above commits?
Type 'yes' to confirm, other to cancel: yes
remote:
remote: Processing changes: new: 4 (\)
remote: Processing changes: new: 4 (|)
remote: Processing changes: new: 4 (/)
remote: Processing changes: new: 4 (-)
remote: Processing changes: new: 4 (\)
remote: Processing changes: new: 4 (\)
remote: Processing changes: new: 4, done
remote: commit 6f2f63e: warning: 6f2f63e: too many message lines longer than 120 characters; manually wrap lines
remote:
remote: New Changes:
remote: https://gerrit.wikimedia.org/r/#/c/mediawiki/services/wikispeech/mary-tts/+/604092 Removed logging to file and added logging to syslog at WARNing threshold.
remote: https://gerrit.wikimedia.org/r/#/c/mediawiki/services/wikispeech/mary-tts/+/604093 Merge pull request #5 from karlwettin/master
remote: https://gerrit.wikimedia.org/r/#/c/mediawiki/services/wikispeech/mary-tts/+/604094 Removed docker build from travis build tests
remote: https://gerrit.wikimedia.org/r/#/c/mediawiki/services/wikispeech/mary-tts/+/604095 Merge remote-tracking branch 'github/master'
remote:
To ssh://gerrit.wikimedia.org:29418/mediawiki/services/wikispeech/mary-tts
* [new branch] HEAD -> refs/for/master
kalle@musa:~/projekt/wikimedia/wikispeech-services/mary-tts$This comment was removed by HaraldBerthelsen.