#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

The leasedb+cloud backend merge supports crawlers, so this ticket is unneeded.

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.