Re: Review Request 65476: Add PartitionPolicy to config summary when defined
> 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
--- 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
--- 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
--- 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
--- 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
--- 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 > >
Review Request 65476: Add PartitionPolicy to config summary when defined
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65476/ --- 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