-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Set up check_stability logging at root level #7768
Conversation
978486b
to
6b00acb
Compare
Build PASSEDStarted: 2017-10-19 15:11:31 View more information about this build on: |
handler = logging.StreamHandler(sys.stdout) | ||
formatter = logging.Formatter(logging.BASIC_FORMAT, None) | ||
handler.setFormatter(formatter) | ||
logger.addHandler(handler) | ||
logger.setLevel(logging.DEBUG) | ||
logger.propagate = False |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unsure about this - Added to avoid dup'd messages if someone sets up root-level debugging elsewhere
@@ -10,7 +10,7 @@ | |||
here = os.path.dirname(__file__) | |||
wpt_root = os.path.abspath(os.path.join(here, os.pardir, os.pardir)) | |||
|
|||
logger = logging.getLogger() | |||
logger = logging.getLogger(wpt_root) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that in testfiles (where the lack of root-logging setup was the issue) and check_stability are both changed to wpt_root dir logging.
fc67efa
to
ae9cbe9
Compare
ae9cbe9
to
037e88b
Compare
So, I might have fixed this in the wpt tool with another change that I made recently. Can you verify that the problem still exists on current master? |
Yep, unable to repro the problem anymore - closing. |
When the command
git fetch https://github.com/w3c/web-platform-tests.git master:master
fails (because of local branch state), logging is attempted (with the default/root logger), which is not set up. This causes the output to be:instead of something like
cc/ @jgraham
This change is