Opened at 2012-04-28T21:15:03Z
Closed at 2013-05-01T15:00:15Z
#1724 closed defect (fixed)
S3 backend: /storage page gives AttributeError: 'NoneType' object has no attribute 'get_state'
Reported by: | davidsarah | Owned by: | davidsarah |
---|---|---|---|
Priority: | minor | Milestone: | undecided |
Component: | code-storage | Version: | 1.9.0-s3branch |
Keywords: | s3-backend LeastAuthority.com crawlers wui test-needed | Cc: | |
Launchpad Bug: |
Description
The 'Storage Server' page (/storage) of a node running the S3 backend gives a nevow traceback with the following exception:
/home/customer/LAFS_source/src/allmydata/web/storage.py, line 90 in data_last_complete_bucket_count 88 89 def data_last_complete_bucket_count(self, ctx, data): 90 s = self.storage.bucket_counter.get_state() 91 count = s.get("last-complete-bucket-count") <type 'exceptions.AttributeError'>: 'NoneType' object has no attribute 'get_state'
This happens because the S3 backend doesn't support crawlers, so it doesn't initialize the bucket_counter state. This doesn't affect the LAE servers because they don't run a web interface, but it should be fixed in order to merge the S3 backend.
Change History (5)
comment:1 Changed at 2012-04-28T21:15:23Z by davidsarah
- Owner set to davidsarah
- Status changed from new to assigned
comment:2 Changed at 2012-11-29T21:05:17Z by davidsarah
- Resolution set to wontfix
- Status changed from assigned to closed
comment:3 Changed at 2012-11-29T21:06:04Z by davidsarah
- Keywords test-needed added
- Resolution wontfix deleted
- Status changed from closed to reopened
Oh but wait, why didn't this cause a failing test?
comment:4 Changed at 2013-01-04T20:36:56Z by zooko
- Keywords LeastAuthority.com added; lae removed
comment:5 Changed at 2013-05-01T15:00:15Z by daira
- Resolution set to fixed
- Status changed from reopened to closed
The WebStatusWithMockCloudBackend class in test_storage.py now tests this (on the cloud-rebased2 branch).
Note: See
TracTickets for help on using
tickets.
The leasedb+cloud backend merge supports crawlers, so this ticket is unneeded.