txlongpoll oopses are recorded with the wrong permission, causing oops loader script to fail with a permission denied error

Bug #891251 reported by Diogo Matsubara
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Gavin Panella
Python OOPS Date-dir repository
Fix Released
Critical
Robert Collins
txlongpoll
Fix Released
Critical
Gavin Panella

Bug Description

update_db script from oops-tools is failing with permission denied error. The script can't access the oops directoris under txlongpoll-oopses. The permission for that dir is:

drwx------ 2 archvsync archvsync 4.0K 2011-11-13 22:22 2011-11-13

One easy workaround is to make the directory r-x by everyone. This bug report is to fix the LP code that's recording those oops directories with the wrong permission

Traceback (most recent call last):
 File "bin/update_db", line 49, in <module>
   oopstools.scripts.update_db.main()
 File "/srv/lp-oops.canonical.com/cgi-bin/lpoops/src/oopstools/scripts/update_db.py", line 32, in main
   for oops in oops_store.find_oopses(start_date):
 File "/srv/lp-oops.canonical.com/cgi-bin/lpoops/src/oopstools/oops/dboopsloader.py", line 130, in find_oopses
   oops_files = set(os.listdir(datedir))
OSError: [Errno 13] Permission denied: '/srv/launchpad.net-logs/scripts/ackee/txlongpoll-oopses/2011-11-13'
make: *** [update_db] Error 1

Related branches

summary: txlongpoll oopses are recorded with the wrong permission, causing oops
- loader script to fail with an invalid permission
+ loader script to fail with a permission denied error
Revision history for this message
Robert Collins (lifeless) wrote :

txlongpoll will need a dependency bump to require oops-datedir-repo 0.0.13 (the next release) once thats put together.

affects: launchpad → python-oops-datedir-repo
Changed in txlongpoll:
status: New → Triaged
importance: Undecided → Critical
Changed in python-oops-datedir-repo:
status: Triaged → In Progress
assignee: nobody → Robert Collins (lifeless)
Revision history for this message
Robert Collins (lifeless) wrote :

oops-datedir-repo 0.0.13 released

Changed in python-oops-datedir-repo:
status: In Progress → Fix Released
Revision history for this message
Diogo Matsubara (matsubara) wrote :

Rob, does your fix going to fix old directories that were recorded with the wrong permission? If not, as part of the deployment for this bug we need to ask losas to change the permission of '/srv/launchpad.net-logs/scripts/ackee/txlongpoll-oopses/2011-11-13'. I'll file an RT requesting that depending on what you reply.

Thanks!

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 891251] Re: txlongpoll oopses are recorded with the wrong permission, causing oops loader script to fail with a permission denied error

Existing directories won't be retroactively fixed.

Revision history for this message
Diogo Matsubara (matsubara) wrote :

Filed https://rt.admin.canonical.com/Ticket/Display.html?id=49344 to have the dir permission fixed.

Gavin Panella (allenap)
Changed in txlongpoll:
status: Triaged → In Progress
assignee: nobody → Gavin Panella (allenap)
Gavin Panella (allenap)
Changed in launchpad:
status: New → In Progress
importance: Undecided → High
assignee: nobody → Gavin Panella (allenap)
Gavin Panella (allenap)
Changed in txlongpoll:
status: In Progress → Fix Committed
Gavin Panella (allenap)
Changed in txlongpoll:
status: Fix Committed → Fix Released
Changed in launchpad:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.