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


Master (43f58bb) is red with this patch.
  ./build-support/jenkins/build.sh

                     
src/test/python/apache/thermos/observer/test_detector.py::test_observer_task_detector_standard_transitions
 PASSED
                     
src/test/python/apache/thermos/observer/test_detector.py::test_observer_task_detector_nonstandard_transitions
 PASSED
                     
src/test/python/apache/thermos/observer/test_task_observer.py::TaskObserverTest::test_run_loop
 FAILED
                     
                     ==================== FAILURES ====================
                     _________ TaskObserverTest.test_run_loop _________
                     
                     self = <test_task_observer.TaskObserverTest 
testMethod=test_run_loop>
                     
                         def test_run_loop(self):
                           """Test observer run loop."""
                           mock_task_detector = 
create_autospec(spec=ObserverTaskDetector)
                           with patch(
                               
"apache.thermos.observer.task_observer.ObserverTaskDetector",
                               return_value=mock_task_detector) as 
mock_detector:
                             with patch('threading._Event.wait') as 
mock_wait:
                         
                               run_count = 3
                               interval = 15
                               observer = TaskObserver(mock_detector, 
interval=Amount(interval, Time.SECONDS))
                               observer.start()
                               while len(mock_wait.mock_calls) < 
run_count:
                                 pass
                         
                               observer.stop()
                         
                     >         assert len(mock_task_detector.mock_calls) >= 
run_count
                     E         AssertionError: assert 2 >= 3
                     E          +  where 2 = len([call.refresh(), 
call.refresh()])
                     E          +    where [call.refresh(), 
call.refresh()] = <MagicMock spec='ObserverTaskDetector' 
id='140349969688336'>.mock_calls
                     
                     
src/test/python/apache/thermos/observer/test_task_observer.py:42: AssertionError
                      generated xml file: 
/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/test-results/src.test.python.apache.thermos.observer.observer.xml
 
                     ======= 1 failed, 3 passed in 0.26 seconds 
=======
                     
FAILURE


23:08:40 04:03   [complete]
               FAILURE


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

- Aurora ReviewBot


On Dec. 30, 2015, 10:51 p.m., Kunal Thakar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41809/
> -----------------------------------------------------------
> 
> (Updated Dec. 30, 2015, 10:51 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen and Bill Farner.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Allow custom announce path
> 
> Related ticket: https://issues.apache.org/jira/browse/AURORA-1569
> 
> 
> Diffs
> -----
> 
>   docs/configuration-reference.md cf63cfa487c531b56f5238a4768e1e9e9ac1c30b 
>   src/main/python/apache/aurora/config/schema/base.py 
> 69182c320579bf5100e5646904c8ce1336afdebb 
>   src/main/python/apache/aurora/executor/common/announcer.py 
> dda76f018f472d7d8228459eb89f4c5daf9df26d 
>   src/test/python/apache/aurora/executor/common/test_announcer.py 
> 46ad784987b626e10e235831707540b807158955 
> 
> Diff: https://reviews.apache.org/r/41809/diff/
> 
> 
> Testing
> -------
> 
> Ran unit tests ./pants test src/test/python/apache/aurora:all
> Manually tested the changes in vagrant.
> 
> 
> Thanks,
> 
> Kunal Thakar
> 
>

Reply via email to