Closed Bug 1471299 Opened 4 years ago Closed 4 years ago
collapse fs crashstorage class hierarchy
53 bytes, text/x-github-pull-request
|Details | Review|
Socorro has a deep class hierarchy of filesystem crashstorage classes. This has evolved over time with an effort to maintain backwards compatibility and probably some other requirements that we don't have anymore. This bug covers figuring out which ones we're using in the processor and collapsing the hierarchy to those.
I did a pass on this already, so I'll resurrect that work and do it again. It helps to do now since we're doing the python 2 to 3 port and it's likely the filesystem classes will require python 2-to-3 work. If we don't use the classes, it'd be better to ditch the code so we don't have to port them.
Assignee: nobody → willkg
Status: NEW → ASSIGNED
Priority: -- → P2
This is a rehash of bug #1363550. I thought about duping this bug to that one, but it's worth redoing the analysis of what's in use since a lot of things have changed in the last year.
Looks like we have these two uses: socorro/app/fetch_transform_save_app.py: default='socorro.external.fs.crashstorage.FSPermanentStorage', socorro/app/fetch_transform_save_app.py: default='socorro.external.fs.crashstorage.FSPermanentStorage', We should keep that one class and nix everything else. While we're at it, we can rename it to FSCrashStorage. Also, while we're doing this, we should nix socorro/external/fs/fs_new_crash_source.py since that's not used anywhere.
Commits pushed to master at https://github.com/mozilla-services/socorro https://github.com/mozilla-services/socorro/commit/9385546dba98224f57db30debc1bc1d66680f87c fix bug 1471299: remove fs new_crash_source code We don't use this, so let's remove it. https://github.com/mozilla-services/socorro/commit/b4e93635c49f4c505f7175eba1632aaedbb4bb9c fix bug 1471299: remove orphaned code the submitter used We rewrote the stage submitter as an AWS Lambda job and no longer have the stage submitter code in the socorro repo. This removes some code that the submitter used that we no longer use and has no tests. https://github.com/mozilla-services/socorro/commit/0beccf29dc7d052a6323a07d65da9ccb3bec0495 fix bug 1471299: remove tar file related crashstorage classes https://github.com/mozilla-services/socorro/commit/ff5897b4d750e72023c7074e576353fa0fcf1d91 fix bug 1471299: remove FSDatedRadixTreeStorage and friends We don't use these classes anymore. This removes them and the tests. https://github.com/mozilla-services/socorro/commit/397625921b524b5d966aa4924c8cd6419074431a fix bug 1471299: remove unused fs crashstorage classes This collapses the last two fs crash storage classes into a single one, renames it to FSPermanentStorage, and fixes the tests for it. https://github.com/mozilla-services/socorro/commit/5e0c7ea74fe588f10a3674048db50d578326dce0 fix bug 1471299: remove unused filesystem utility functions https://github.com/mozilla-services/socorro/commit/a1100df887a077596539f1fc08c63754705ecd20 Merge pull request #4499 from willkg/1471299-collapse-fs fix bug 1471299: collapse fs crashstorage classes
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.