Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(53)

Unified Diff: infra_libs/logs/README.md

Issue 2213143002: Add infra_libs as a bootstrap dependency. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Removed the ugly import hack Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: infra_libs/logs/README.md
diff --git a/infra_libs/logs/README.md b/infra_libs/logs/README.md
deleted file mode 100644
index 6cc211ca0288b55a2598888e4278f94cd5430165..0000000000000000000000000000000000000000
--- a/infra_libs/logs/README.md
+++ /dev/null
@@ -1,59 +0,0 @@
-# Logging in infra.git
-
-## Features
-
-The `infra_libs.logs` package contains some code to simplify logging and
-make it consistent and easily configurable. Using it makes your code
-future-proof.
-
-Offered features:
-
-* log level can be changed from the command-line.
-* too verbose modules can be blacklisted for easier debugging.
-* ensures a consistent log format.
-
-A typical log line looks like:
-
- [I2014-06-27T11:42:32.418716-07:00 7082 logs:71] this is the message
-
-The first letter gives the severity of the message, followed by a
-timestamp with timezone information (iso8601), the process id, the
-current module name, and the thread id. After the closing square bracket
-comes the actual message.
-
-## Sample code
-
-This is the standard way to set up logging so as to take advantage of
-the goodness provided by `infra_libs.logs`.
-
-In top-level files (other example in
-[infra.services.sysmon.__main__](../../infra/services/sysmon/__main__.py)):
-
-```python
-import argparse
-import infra_libs.logs
-
-parser = argparse.ArgumentParser()
-infra_libs.logs.add_argparse_options(parser)
-
-options = parser.parse_args()
-infra_libs.logs.process_argparse_options(options)
-```
-
-Logging messages should be done this way (other example in
-`infra.libs.service_utils.outer_loop`):
-
-```python
-import logging
-LOGGER = logging.getLogger(__name__)
-
-LOGGER.info('great message')
-LOGGER.error('terrible error')
-```
-
-Using `logging.getLogger` is a good practice in general (not restricted to
-using infra_libs.logs) because it allows for module blacklisting and
-other goodness. It should be done at import time. See also the official
-[logging HOWTO](https://docs.python.org/2/howto/logging.html).
-`infra_libs.logs` also formats the output of the root logger, but using
-this logger is not recommended.

Powered by Google App Engine
This is Rietveld 408576698