Closed Bug 1425247 Opened 8 years ago Closed 7 years ago

integration test to make sure the entire pipeline is working

Categories

(Socorro :: General, task, P3)

Product:
Socorro
Component:
General
Type:
task

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: willkg, Unassigned)

References

Details

Before we split out the collector from Socorro, we used to have an integration test that brought up all the parts of the pipeline, posted a crash to the collector, and then verified that the crash made it to crash storage and was viewable in the webapp. I removed all that code when we switched to Antenna and removed all the collector code. This bug covers rebuilding that using the Docker infrastructure that we have. The test would: 1. bring up antenna, processor, and webapp 2. post a crash to antenna and get back a crashid 3. check the webapp api view from bug #1425245 and verify the crash made it through the pipeline correctly This integration test will run in Circle CI on pull requests. We also want to run this integration test against the -stage and -prod environments. We might want to create a special TestProduct product in all of the environments that we can use for this.
We want to do this in 2018q1, so making this a P2 so it's on the radar.
Priority: -- → P2
QA Contact: chartjes
Assignee: nobody → mkelly
Depends on: 1426739
Depends on: 1399305
Status on this: I threw a WIP PR for bug #1425245, but it was so complex and over-involved we abandoned that approach. Mike worked on reducing some of the complexity in processor configuration that led to that overly complex PR. Next step is to attempt bug #1425245 again, then work on this. Having said that, I said 2018q1 in comment #1 because this is a helpful thing to have to verify correctness post-deploy. We have other signals that aren't as clear as a pass/fail test, but we've been using them for a while and they've been ok. There is no actual deadline for this work, but there are higher priority projects competing for dev time. So unless someone else pitches in here, we'll get to it when we get to it.
Assignee: mkelly → nobody
Bumping this down to P3. It'd be nice to have maybe, but we have other signals that are good enough for now.
Priority: P2 → P3
I'm not going to do an integration test at this time. If we decide to revisit this, we can either reopen or write up a new bug.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.