Re: Review Request 65476: Add PartitionPolicy to config summary when defined

2018-02-02 Thread David McLaughlin


> On Feb. 2, 2018, 10:06 a.m., Aurora ReviewBot wrote:
> > Master (21a371c) is red with this patch.
> >   ./build-support/jenkins/build.sh
> > 
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >   WARN] Reached consecutive failure limit.
> >  --- Captured log call 
> >  health_checker.py  167 INFO  INFO] Reached 
> > consecutive success limit.
> >  health_checker.py  143 WARNING   WARN] Health 
> > check failure: failure-2
> >  health_checker.py  184 WARNING   WARN] 
> > Ignoring failure of attempt: 2
> >  health_checker.py  143 WARNING   WARN] Health 
> > check failure: failure-3
> >  health_checker.py  159 WARNING   WARN] Reached 
> > consecutive failure limit.
> >   5 failed, 795 passed, 6 skipped in 368.28 seconds 
> >  
> > FAILURE
> > 
> > 
> >Waiting for background workers to finish.
> > 10:06:03 06:47   [complete]
> >FAILURE
> > 
> > 
> > I will refresh this build result if you post a review containing 
> > "@ReviewBot retry"

I'm randomly getting this failure locally too. Does anyone have an idea what 
change was introduced that caused this? Should we mark the test as skippable 
until it's triaged?


- David


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


On Feb. 2, 2018, 7:46 a.m., David McLaughlin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65476/
> ---
> 
> (Updated Feb. 2, 2018, 7:46 a.m.)
> 
> 
> Review request for Aurora, Jordan Ly and Santhosh Kumar Shanmugham.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add PartitionPolicy to config summary when defined
> 
> 
> Diffs
> -
> 
>   ui/src/main/js/components/TaskConfigSummary.js 
> 01902cdd90192113b93cdf5872cd560c431f2bf0 
> 
> 
> Diff: https://reviews.apache.org/r/65476/diff/1/
> 
> 
> Testing
> ---
> 
> Testing in vagrant (see screenshot).
> 
> 
> File Attachments
> 
> 
> Screen Shot 2018-02-01 at 11.46.17 PM.png
>   
> https://reviews.apache.org/media/uploaded/files/2018/02/02/1353e8ee-bb7b-4546-adf6-9f1ececb36b4__Screen_Shot_2018-02-01_at_11.46.17_PM.png
> 
> 
> Thanks,
> 
> David McLaughlin
> 
>



Re: Review Request 65476: Add PartitionPolicy to config summary when defined

2018-02-02 Thread Santhosh Kumar Shanmugham

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


Ship it!




Ship It!

- Santhosh Kumar Shanmugham


On Feb. 1, 2018, 11:46 p.m., David McLaughlin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65476/
> ---
> 
> (Updated Feb. 1, 2018, 11:46 p.m.)
> 
> 
> Review request for Aurora, Jordan Ly and Santhosh Kumar Shanmugham.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add PartitionPolicy to config summary when defined
> 
> 
> Diffs
> -
> 
>   ui/src/main/js/components/TaskConfigSummary.js 
> 01902cdd90192113b93cdf5872cd560c431f2bf0 
> 
> 
> Diff: https://reviews.apache.org/r/65476/diff/1/
> 
> 
> Testing
> ---
> 
> Testing in vagrant (see screenshot).
> 
> 
> File Attachments
> 
> 
> Screen Shot 2018-02-01 at 11.46.17 PM.png
>   
> https://reviews.apache.org/media/uploaded/files/2018/02/02/1353e8ee-bb7b-4546-adf6-9f1ececb36b4__Screen_Shot_2018-02-01_at_11.46.17_PM.png
> 
> 
> Thanks,
> 
> David McLaughlin
> 
>



Re: Review Request 65476: Add PartitionPolicy to config summary when defined

2018-02-02 Thread Aurora ReviewBot

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



Master (21a371c) is red with this patch.
  ./build-support/jenkins/build.sh

  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
 --- Captured log call 
 health_checker.py  167 INFO  INFO] Reached 
consecutive success limit.
 health_checker.py  143 WARNING   WARN] Health 
check failure: failure-2
 health_checker.py  184 WARNING   WARN] Ignoring 
failure of attempt: 2
 health_checker.py  143 WARNING   WARN] Health 
check failure: failure-3
 health_checker.py  159 WARNING   WARN] Reached 
consecutive failure limit.
  5 failed, 795 passed, 6 skipped in 368.28 seconds 
 
FAILURE


   Waiting for background workers to finish.
10:06:03 06:47   [complete]
   FAILURE


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

- Aurora ReviewBot


On Feb. 2, 2018, 7:46 a.m., David McLaughlin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65476/
> ---
> 
> (Updated Feb. 2, 2018, 7:46 a.m.)
> 
> 
> Review request for Aurora, Jordan Ly and Santhosh Kumar Shanmugham.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add PartitionPolicy to config summary when defined
> 
> 
> Diffs
> -
> 
>   ui/src/main/js/components/TaskConfigSummary.js 
> 01902cdd90192113b93cdf5872cd560c431f2bf0 
> 
> 
> Diff: https://reviews.apache.org/r/65476/diff/1/
> 
> 
> Testing
> ---
> 
> Testing in vagrant (see screenshot).
> 
> 
> File Attachments
> 
> 
> Screen Shot 2018-02-01 at 11.46.17 PM.png
>   
> https://reviews.apache.org/media/uploaded/files/2018/02/02/1353e8ee-bb7b-4546-adf6-9f1ececb36b4__Screen_Shot_2018-02-01_at_11.46.17_PM.png
> 
> 
> Thanks,
> 
> David McLaughlin
> 
>



Re: Review Request 65476: Add PartitionPolicy to config summary when defined

2018-02-02 Thread Stephan Erb

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



@ReviewBot retry

- Stephan Erb


On Feb. 2, 2018, 8:46 a.m., David McLaughlin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65476/
> ---
> 
> (Updated Feb. 2, 2018, 8:46 a.m.)
> 
> 
> Review request for Aurora, Jordan Ly and Santhosh Kumar Shanmugham.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add PartitionPolicy to config summary when defined
> 
> 
> Diffs
> -
> 
>   ui/src/main/js/components/TaskConfigSummary.js 
> 01902cdd90192113b93cdf5872cd560c431f2bf0 
> 
> 
> Diff: https://reviews.apache.org/r/65476/diff/1/
> 
> 
> Testing
> ---
> 
> Testing in vagrant (see screenshot).
> 
> 
> File Attachments
> 
> 
> Screen Shot 2018-02-01 at 11.46.17 PM.png
>   
> https://reviews.apache.org/media/uploaded/files/2018/02/02/1353e8ee-bb7b-4546-adf6-9f1ececb36b4__Screen_Shot_2018-02-01_at_11.46.17_PM.png
> 
> 
> Thanks,
> 
> David McLaughlin
> 
>



Re: Review Request 65476: Add PartitionPolicy to config summary when defined

2018-02-02 Thread Stephan Erb

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


Ship it!




Ship It!

- Stephan Erb


On Feb. 2, 2018, 8:46 a.m., David McLaughlin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65476/
> ---
> 
> (Updated Feb. 2, 2018, 8:46 a.m.)
> 
> 
> Review request for Aurora, Jordan Ly and Santhosh Kumar Shanmugham.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add PartitionPolicy to config summary when defined
> 
> 
> Diffs
> -
> 
>   ui/src/main/js/components/TaskConfigSummary.js 
> 01902cdd90192113b93cdf5872cd560c431f2bf0 
> 
> 
> Diff: https://reviews.apache.org/r/65476/diff/1/
> 
> 
> Testing
> ---
> 
> Testing in vagrant (see screenshot).
> 
> 
> File Attachments
> 
> 
> Screen Shot 2018-02-01 at 11.46.17 PM.png
>   
> https://reviews.apache.org/media/uploaded/files/2018/02/02/1353e8ee-bb7b-4546-adf6-9f1ececb36b4__Screen_Shot_2018-02-01_at_11.46.17_PM.png
> 
> 
> Thanks,
> 
> David McLaughlin
> 
>



Re: Review Request 65476: Add PartitionPolicy to config summary when defined

2018-02-02 Thread Aurora ReviewBot

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



Master (21a371c) is red with this patch.
  ./build-support/jenkins/build.sh

  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
  WARN] Reached consecutive failure limit.
 --- Captured log call 
 health_checker.py  143 WARNING   WARN] Health 
check failure: failure
 health_checker.py  184 WARNING   WARN] Ignoring 
failure of attempt: 1
 health_checker.py  143 WARNING   WARN] Health 
check failure: failure
 health_checker.py  184 WARNING   WARN] Ignoring 
failure of attempt: 2
 health_checker.py  143 WARNING   WARN] Health 
check failure: failure
 health_checker.py  159 WARNING   WARN] Reached 
consecutive failure limit.
  1 failed, 799 passed, 6 skipped in 258.85 seconds 
 
FAILURE


   Waiting for background workers to finish.
08:18:41 04:50   [complete]
   FAILURE


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

- Aurora ReviewBot


On Feb. 2, 2018, 7:46 a.m., David McLaughlin wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65476/
> ---
> 
> (Updated Feb. 2, 2018, 7:46 a.m.)
> 
> 
> Review request for Aurora, Jordan Ly and Santhosh Kumar Shanmugham.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add PartitionPolicy to config summary when defined
> 
> 
> Diffs
> -
> 
>   ui/src/main/js/components/TaskConfigSummary.js 
> 01902cdd90192113b93cdf5872cd560c431f2bf0 
> 
> 
> Diff: https://reviews.apache.org/r/65476/diff/1/
> 
> 
> Testing
> ---
> 
> Testing in vagrant (see screenshot).
> 
> 
> File Attachments
> 
> 
> Screen Shot 2018-02-01 at 11.46.17 PM.png
>   
> https://reviews.apache.org/media/uploaded/files/2018/02/02/1353e8ee-bb7b-4546-adf6-9f1ececb36b4__Screen_Shot_2018-02-01_at_11.46.17_PM.png
> 
> 
> Thanks,
> 
> David McLaughlin
> 
>