1361832 - remove collector code

Status: RESOLVED FIXED

(Socorro :: General, task)

Description

[Will Kahn-Greene [:willkg] ET needinfo? me]

We switched to Antenna for crash collection and no longer need the Socorro collector. Now that we're not using it, it's just sitting around and taking up maintenance resources. Best to remove it.

This bug covers removing the collector and crashmover from the repo, tests, docs, and the integration tests.

Comment 1

[Will Kahn-Greene [:willkg] ET needinfo? me]

I brought up some concern that there are still users of Socorro that continue to use it even though we're not supporting non-Mozilla users anymore.

To alleviate that a little, I threw this PR together:

https://github.com/mozilla/socorro/pull/3750

That updates the README and the index page of the docs adding a note about how we're not supporting Socorro and we are making non-backwards-compatible changes. It also includes a link to the last "known good commit" and tarball.

Is that perfect? No. But it's probably ok for now.

If we need to go another step further, we should fork socorro into a socorro-orig repository at that commit and encourage users to use that one. Maybe give that project a different name so as to reduce confusion?

Anyhow, we can cross that bridge if we need to. Alternatively, maybe users can band together and fork it and maintain it themselves.

Comment 2

[Will Kahn-Greene [:willkg] ET needinfo? me]

This PR removes some of the 2015 versions of code as well as the integration tests: https://github.com/mozilla/socorro/pull/3759

That's covered in bug #1363130.

Comment 3

[Will Kahn-Greene [:willkg] ET needinfo? me]

Before I do this work, I want to wait for another -prod deploy and verify the collector nodes are gone. Then we can do this.

Comment 4

[Will Kahn-Greene [:willkg] ET needinfo? me]

Verified there are no more collector nodes and we can remove the code now.

Comment 5

[Will Kahn-Greene [:willkg] ET needinfo? me]

Attachment: pr 3802

Comment 6

[Will Kahn-Greene [:willkg] ET needinfo? me]

After that PR, I'll need to update the stage submitter node. I might have to tweak the scripts, too.

Comment 7

[[github robot]]

Commit pushed to master at https://github.com/mozilla-services/socorro

https://github.com/mozilla-services/socorro/commit/b2f53d61220d20284a8edb1b75443c8c0a308b0f
Fixes bug 1361832 - remove socorro collector and crashmover (#3802)

* Fixes bug 1361832 - remove socorro collector and crashmover

This removes the socorro collector and crashmover, some of the things that used
it, tests, scripts, configuration, and various related errata.

This moves the submitter (which we're still using) to it's own Python module.

* Remove unused file

* Cosmetic docstring things to bump Travis CI