Re: Review Request 19519: Allow override of default RB server in shipits tool.

2014-03-21 Thread Maxim Khutornenko

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

Ship it!


Ship It!

- Maxim Khutornenko


On March 21, 2014, 2:48 a.m., Bill Farner wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/19519/
 ---
 
 (Updated March 21, 2014, 2:48 a.m.)
 
 
 Review request for Aurora, Suman Karumuri and Maxim Khutornenko.
 
 
 Repository: aurora
 
 
 Description
 ---
 
 Switched to using review request ID field rather than URL, since that is 
 compatible across older reviewboard versions.
 
 
 Diffs
 -
 
   build-support/tools/list-missing-shipits 
 31aada9bd1d1b3c67f0397dc5c4c91cb32ed7e0f 
 
 Diff: https://reviews.apache.org/r/19519/diff/
 
 
 Testing
 ---
 
 Successfully queried a separate reviewboard server.  Sanity check by both 
 defaulting and manually setting apache's RB URL:
 
 $ ./build-support/tools/list-missing-shipits
 2014-01-15T03:12:01Z  https://reviews.apache.org/r/16710  kevints 
 Support multiple ...awaiting updated diff
 2014-02-25T23:49:37Z  https://reviews.apache.org/r/18490  kevints 
 Generate cron pre...ready to submit
 2014-02-27T23:17:03Z  https://reviews.apache.org/r/16062  drobinson   
 fixed structdump ...ready to submit
 2014-03-03T23:32:53Z  https://reviews.apache.org/r/18714  wfarner 
 Disable log and i...ready to submit
 2014-03-04T19:53:25Z  https://reviews.apache.org/r/18746  drobinson   
 fix rollback on f...need review from wickman
 2014-03-10T23:29:08Z  https://reviews.apache.org/r/18537  protochron  
 AURORA-227: Auror...need review from wickman
 2014-03-11T16:11:32Z  https://reviews.apache.org/r/18996  markcc  
 Fix errors in job...ready to submit
 2014-03-11T20:12:23Z  https://reviews.apache.org/r/19063  atollena
 Fix help usage fo...ready to submit
 2014-03-12T22:51:58Z  https://reviews.apache.org/r/18979  markcc  
 Add an updated ve...awaiting updated diff
 2014-03-12T22:59:36Z  https://reviews.apache.org/r/19143  markcc  
 Catch errors thro...ready to submit
 2014-03-13T22:45:34Z  https://reviews.apache.org/r/19207  tgalloway   
 Removed content a...need review from clambert, davelester, wickman
 2014-03-17T18:57:57Z  https://reviews.apache.org/r/19313  tgalloway   
 added killall com...need review from clambert, markcc, davelester
 2014-03-18T18:15:24Z  https://reviews.apache.org/r/19362  maxim   
 Changing from STA...need review from wickman
 2014-03-19T13:12:55Z  https://reviews.apache.org/r/19396  markcc  
 Implement client ...need review from kevints, wickman
 2014-03-19T21:37:33Z  https://reviews.apache.org/r/19243  maxim   
 Adding min retent...need review from kevints
 2014-03-19T22:54:43Z  https://reviews.apache.org/r/19426  wickman 
 Don't forget to r...ready to submit
 2014-03-20T00:20:06Z  https://reviews.apache.org/r/19436  maxim   
 Preserving sandbo...need review from wfarner
 2014-03-20T13:27:36Z  https://reviews.apache.org/r/19411  markcc  
 Add support for g...need review from maxim, wickman
 2014-03-20T15:37:08Z  https://reviews.apache.org/r/19466  markcc  
 Add batching to t...need review from maxim, wickman
 2014-03-20T16:19:12Z  https://reviews.apache.org/r/19462  dam_ned 
 [AURORA-282] add ...awaiting updated diff
 2014-03-20T18:20:38Z  https://reviews.apache.org/r/19078  maxim   
 Part 2 of convert...awaiting updated diff
 2014-03-20T18:29:24Z  https://reviews.apache.org/r/18658  kevints 
 GZIP HTTP thrift ...need review from mansu
 2014-03-20T18:47:00Z  https://reviews.apache.org/r/19431  markcc  
 Add instance spec...need review from wfarner, wickman
 2014-03-20T22:58:13Z  https://reviews.apache.org/r/19505  markcc  
 Die, underscores,...ready to submit
 2014-03-21T02:37:33Z  https://reviews.apache.org/r/19509  mansu   
 Now consuming non...need review from kevints, markcc, wickman
 
 Recently submitted: 3
 https://reviews.apache.org/r/19450mansu   Added nextCronRun...
 https://reviews.apache.org/r/19494wfarner Change missing sh...
 https://reviews.apache.org/r/18580kevints Upgrade mesos to ...
 
 
 $ ./build-support/tools/list-missing-shipits 
 --server=https://reviews.apache.org
 2014-01-15T03:12:01Z  https://reviews.apache.org/r/16710  kevints 
 Support multiple ...awaiting updated diff
 2014-02-25T23:49:37Z  https://reviews.apache.org/r/18490  kevints 
 Generate cron pre...ready to 

Review Request 19519: Allow override of default RB server in shipits tool.

2014-03-20 Thread Bill Farner

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

Review request for Aurora, Suman Karumuri and Maxim Khutornenko.


Repository: aurora


Description
---

Switched to using review request ID field rather than URL, since that is 
compatible across older reviewboard versions.


Diffs
-

  build-support/tools/list-missing-shipits 
31aada9bd1d1b3c67f0397dc5c4c91cb32ed7e0f 

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


Testing
---

Successfully queried a separate reviewboard server.  Sanity check by both 
defaulting and manually setting apache's RB URL:

$ ./build-support/tools/list-missing-shipits
2014-01-15T03:12:01Zhttps://reviews.apache.org/r/16710  kevints 
Support multiple ...awaiting updated diff
2014-02-25T23:49:37Zhttps://reviews.apache.org/r/18490  kevints 
Generate cron pre...ready to submit
2014-02-27T23:17:03Zhttps://reviews.apache.org/r/16062  drobinson   
fixed structdump ...ready to submit
2014-03-03T23:32:53Zhttps://reviews.apache.org/r/18714  wfarner 
Disable log and i...ready to submit
2014-03-04T19:53:25Zhttps://reviews.apache.org/r/18746  drobinson   
fix rollback on f...need review from wickman
2014-03-10T23:29:08Zhttps://reviews.apache.org/r/18537  protochron  
AURORA-227: Auror...need review from wickman
2014-03-11T16:11:32Zhttps://reviews.apache.org/r/18996  markcc  
Fix errors in job...ready to submit
2014-03-11T20:12:23Zhttps://reviews.apache.org/r/19063  atollena
Fix help usage fo...ready to submit
2014-03-12T22:51:58Zhttps://reviews.apache.org/r/18979  markcc  
Add an updated ve...awaiting updated diff
2014-03-12T22:59:36Zhttps://reviews.apache.org/r/19143  markcc  
Catch errors thro...ready to submit
2014-03-13T22:45:34Zhttps://reviews.apache.org/r/19207  tgalloway   
Removed content a...need review from clambert, davelester, wickman
2014-03-17T18:57:57Zhttps://reviews.apache.org/r/19313  tgalloway   
added killall com...need review from clambert, markcc, davelester
2014-03-18T18:15:24Zhttps://reviews.apache.org/r/19362  maxim   
Changing from STA...need review from wickman
2014-03-19T13:12:55Zhttps://reviews.apache.org/r/19396  markcc  
Implement client ...need review from kevints, wickman
2014-03-19T21:37:33Zhttps://reviews.apache.org/r/19243  maxim   
Adding min retent...need review from kevints
2014-03-19T22:54:43Zhttps://reviews.apache.org/r/19426  wickman 
Don't forget to r...ready to submit
2014-03-20T00:20:06Zhttps://reviews.apache.org/r/19436  maxim   
Preserving sandbo...need review from wfarner
2014-03-20T13:27:36Zhttps://reviews.apache.org/r/19411  markcc  
Add support for g...need review from maxim, wickman
2014-03-20T15:37:08Zhttps://reviews.apache.org/r/19466  markcc  
Add batching to t...need review from maxim, wickman
2014-03-20T16:19:12Zhttps://reviews.apache.org/r/19462  dam_ned 
[AURORA-282] add ...awaiting updated diff
2014-03-20T18:20:38Zhttps://reviews.apache.org/r/19078  maxim   
Part 2 of convert...awaiting updated diff
2014-03-20T18:29:24Zhttps://reviews.apache.org/r/18658  kevints 
GZIP HTTP thrift ...need review from mansu
2014-03-20T18:47:00Zhttps://reviews.apache.org/r/19431  markcc  
Add instance spec...need review from wfarner, wickman
2014-03-20T22:58:13Zhttps://reviews.apache.org/r/19505  markcc  
Die, underscores,...ready to submit
2014-03-21T02:37:33Zhttps://reviews.apache.org/r/19509  mansu   
Now consuming non...need review from kevints, markcc, wickman

Recently submitted: 3
https://reviews.apache.org/r/19450  mansu   Added nextCronRun...
https://reviews.apache.org/r/19494  wfarner Change missing sh...
https://reviews.apache.org/r/18580  kevints Upgrade mesos to ...


$ ./build-support/tools/list-missing-shipits --server=https://reviews.apache.org
2014-01-15T03:12:01Zhttps://reviews.apache.org/r/16710  kevints 
Support multiple ...awaiting updated diff
2014-02-25T23:49:37Zhttps://reviews.apache.org/r/18490  kevints 
Generate cron pre...ready to submit
2014-02-27T23:17:03Zhttps://reviews.apache.org/r/16062  drobinson   
fixed structdump ...ready to submit
2014-03-03T23:32:53Zhttps://reviews.apache.org/r/18714  wfarner 
Disable log and i...ready to submit
2014-03-04T19:53:25Zhttps://reviews.apache.org/r/18746  drobinson   
fix rollback on f...need review from wickman
2014-03-10T23:29:08Z