Re: Review Request 44526: Fixed bug with LocalSchedulerMain not starting up due to missing Modules.

2016-03-08 Thread Aurora ReviewBot

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44526/#review122595
---


Ship it!




Master (26efe55) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot 
retry"

- Aurora ReviewBot


On March 8, 2016, 8:34 p.m., Amol Deshmukh wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/44526/
> ---
> 
> (Updated March 8, 2016, 8:34 p.m.)
> 
> 
> Review request for Aurora and Maxim Khutornenko.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Fixed bug with LocalSchedulerMain not starting up due to missing Modules.
> 
> 
> Diffs
> -
> 
>   src/test/java/org/apache/aurora/scheduler/app/local/LocalSchedulerMain.java 
> fb27657f5c15be3e9dd8b51fd57d0dd7f1a7121a 
> 
> Diff: https://reviews.apache.org/r/44526/diff/
> 
> 
> Testing
> ---
> 
> ```
> $ ./gradlew build -Pq
> ...
> BUILD SUCCESSFUL
> 
> Total time: 3 mins 37.928 secs
> ```
> 
> ```
> $ ./gradlew run
> ...
> I0308 12:26:25.344 [RedirectMonitor STARTING-EventThread, 
> LeaderRedirect$SchedulerMonitor:208] Found leader scheduler at 
> [ServiceInstance(serviceEndpoint:Endpoint(host:127.0.0.1, port:8081), 
> additionalEndpoints:{http=Endpoint(host:127.0.0.1, port:8081)}, status:ALIVE)]
> I0308 12:26:31.161 [pool-9-thread-1, FakeMaster:139] All offers consumed, 
> suppressing offer cycle.
> I0308 12:26:36.157 [pool-9-thread-1, FakeMaster:139] All offers consumed, 
> suppressing offer cycle.
> ...
> ```
> 
> 
> Thanks,
> 
> Amol Deshmukh
> 
>



Review Request 44526: Fixed bug with LocalSchedulerMain not starting up due to missing Modules.

2016-03-08 Thread Amol Deshmukh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44526/
---

Review request for Aurora and Maxim Khutornenko.


Repository: aurora


Description
---

Fixed bug with LocalSchedulerMain not starting up due to missing Modules.


Diffs
-

  src/test/java/org/apache/aurora/scheduler/app/local/LocalSchedulerMain.java 
fb27657f5c15be3e9dd8b51fd57d0dd7f1a7121a 

Diff: https://reviews.apache.org/r/44526/diff/


Testing
---

```
$ ./gradlew build -Pq
...
BUILD SUCCESSFUL

Total time: 3 mins 37.928 secs
```

```
$ ./gradlew run
...
I0308 12:26:25.344 [RedirectMonitor STARTING-EventThread, 
LeaderRedirect$SchedulerMonitor:208] Found leader scheduler at 
[ServiceInstance(serviceEndpoint:Endpoint(host:127.0.0.1, port:8081), 
additionalEndpoints:{http=Endpoint(host:127.0.0.1, port:8081)}, status:ALIVE)]
I0308 12:26:31.161 [pool-9-thread-1, FakeMaster:139] All offers consumed, 
suppressing offer cycle.
I0308 12:26:36.157 [pool-9-thread-1, FakeMaster:139] All offers consumed, 
suppressing offer cycle.
...
```


Thanks,

Amol Deshmukh