PPA Apache log parser needs to exclude error logs
Bug #694004 reported by
William Grant
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Critical
|
William Grant | ||
Bug Description
As seen in OOPS-1818PPA10372, the PPA Apache log parser is trying to parse error logs, not just the access logs that we intended it to examine. The librarian case is fine, since only the access logs are synced to loganberry.
We probably want a configurable glob.
Related branches
lp:~wgrant/launchpad/bug-694004-log-parser-glob
- Tim Penhey (community): Approve (code)
- Steve Kowalik (community): Approve (code*)
-
Diff: 233 lines (+41/-25)8 files modifiedcronscripts/parse-ppa-apache-access-logs.py (+4/-0)
lib/canonical/config/schema-lazr.conf (+1/-0)
lib/lp/services/apachelogparser/base.py (+3/-5)
lib/lp/services/apachelogparser/script.py (+6/-1)
lib/lp/services/apachelogparser/tests/test_apachelogparser.py (+17/-17)
lib/lp/soyuz/scripts/ppa_apache_log_parser.py (+0/-2)
lib/lp/soyuz/scripts/tests/ppa-apache-log-files/ppa.launchpad.net-error.log (+1/-0)
lib/lp/soyuz/scripts/tests/test_ppa_apache_log_parser.py (+9/-0)
| Changed in launchpad: | |
| assignee: | nobody → William Grant (wgrant) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #1 |
| Changed in launchpad: | |
| milestone: | none → 11.01 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| importance: | High → Critical |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r12164 <http://