[STORM-2807] Integration test should shut down topologies immediately after the test

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.0.0, 1.1.1
• Fix Version/s: 2.0.0, 1.2.0, 1.1.2
• Labels: pull-request-available
• Assignee: Stig Rohde Døssing
• Reporter: Stig Rohde Døssing
• Created: 10/Nov/17 08:07
• Updated: 11/Nov/17 09:59
• Resolved: 11/Nov/17 09:59

Description

The integration test kills topologies with the default 30 second timeout. This is unnecessary and delays the following tests, because the killed topology is still occupying worker slots.

When the integration test kills topologies, it tries sending the kill message to Nimbus once, and may fail quietly. This breaks following tests, because the default Storm install has only 4 worker slots, and the test topologies each take up 3. When a topology is not shut down, it prevents the following topologies from being assigned.

Issue Links

• links to: GitHub Pull Request #2406