-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64482/#review193350
-----------------------------------------------------------
Looks like this did not work: Build on master failed with
```
org.apache.aurora.scheduler.events.WebhookTest >
testTaskChangedWithOldStateError FAILED
java.lang.AssertionError at WebhookTest.java:193
I1210 11:12:13.954 [ShutdownHook, SchedulerMain] Stopping scheduler services.
```
- Stephan Erb
On Dec. 10, 2017, 1:08 a.m., Stephan Erb wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64482/
> -----------------------------------------------------------
>
> (Updated Dec. 10, 2017, 1:08 a.m.)
>
>
> Review request for Aurora and John Sirois.
>
>
> Repository: aurora
>
>
> Description
> -------
>
> I opted for a minimal patch that stays close to the style of the other
> tests. There is a remaining risk of a race condition if the OS re-uses
> the same port between stopping jetty and triggering the webhooks, but
> it should be very unlikely.
>
>
> Diffs
> -----
>
> src/test/java/org/apache/aurora/scheduler/events/WebhookTest.java
> adeff8939a4c4a225444dc5ca2575f9eb705a92b
>
>
> Diff: https://reviews.apache.org/r/64482/diff/2/
>
>
> Testing
> -------
>
> Multiple runs of ./gradlew test
>
>
> Thanks,
>
> Stephan Erb
>
>