If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

squash filesystem crashstorage classes



4 months ago
4 months ago


(Reporter: willkg, Assigned: willkg)


Firefox Tracking Flags

(Not tracked)



(1 attachment)

In Socorro, there are 7 filesystem crash storage classes, two of which are just renamed versions of other classes.

They're predominantly used by the collector and crashmover and FetchTransformSave apps of which the processor is one.

Between the Mozilla Socorro configuration and the codebase, the following classes are used:

* FSTemporaryStorage: collector and crashmover
* FSPermanentStorage (alias of FSLegacyRadixTreeStorage): FetchTransformSave apps
* FSLegacyRadixTreeStorage: middleware
* FSDatedPermanentStorage (alias of FSLegacyDatedRadixTreeStorage): Processor

We no longer use the collector and crashmover, so those uses will get removed in bug #1361832.

We no longer use the middleware, so that use will get removed in bug #1353371.

When I extracted the collector/crashmover code for the socorro-collector project, I squashed this family of classes down to two: FSRadixTreeStorage and FSTemporaryStorage.

I want to pull in the socorro-collector squashings and then change the Socorro users of filesystem crashstorage to use FSRadixTreeStorage and FSTemporaryStorage.

Why? There's about 2,000 lines of filesystem crashstorage code and tests. There are some 60 tests. I claim that the bulk of the work for this bug is already done, so this should be like a half day of work and save us a bunch of time as we dockerize, deprecate, rewrite, etc.

This bug covers doing that.
Grabbing this to work on this week. It'll make dockerizing easier.
Assignee: nobody → willkg
My notes.

All the instances of "external.fs.crashstorage" in the Socorro codebase except what's in unittest, config, and docs:

  104 :             'socorro.external.fs.crashstorage.FSPermanentStorage',
  106 :             'socorro.external.fs.crashstorage.FSPermanentStorage',
  100 :         default='socorro.external.fs.crashstorage.FSLegacyDatedRadixTreeStorage',
   53 :                 'socorro.external.fs.crashstorage.TarFileCrashStore',
  147 :         default='socorro.external.fs.crashstorage.FSLegacyDatedRadixTreeStorage'
   19 :         default='socorro.external.fs.crashstorage.FSLegacyDatedRadixTreeStorage'
  140 :         default='socorro.external.fs.crashstorage.FSLegacyRadixTreeStorage',
   22 : from socorro.external.fs.crashstorage import FSDatedPermanentStorage

All the instances of "external.fs.crashstorage" in the Socorro configuration:

    7 : socorro/collector/storage.crashstorage_class	socorro.external.fs.crashstorage.FSTemporaryStorage
   65 : socorro/crashmover/source.crashstorage_class	socorro.external.fs.crashstorage.FSTemporaryStorage

    8 : socorro/collector/storage.crashstorage_class	socorro.external.fs.crashstorage.FSTemporaryStorage
   53 : socorro/crashmover/source.crashstorage_class	socorro.external.fs.crashstorage.FSTemporaryStorage
Created attachment 8866391 [details] [review]
pr 3767
I nixed that PR for now. It'd need review time and it's probably not worth that investment right now. If we decide to do it later, we can resurrect the PR.

Given that, I think I'm going to close this out as WONTFIX.
Last Resolved: 4 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.