Re: Review Request 34661: Adds new documentation page listing Apache Aurora presentations (both videos and slides).

2015-06-09 Thread Kevin Sweeney 

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

Ship it!


Ship It!

- Kevin Sweeney


On June 8, 2015, 11:40 a.m., Dave Lester wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34661/
> ---
> 
> (Updated June 8, 2015, 11:40 a.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Adds new documentation page listing Apache Aurora presentations (both videos 
> and slides).
> 
> 
> Diffs
> -
> 
>   docs/README.md 508521fbd4fed8376e56b8c7e7eeabc301d1ef9e 
>   docs/images/presentations/02_19_2015_aurora_adopters_panel_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_tellapart_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_twitter_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_28_2015_apache_aurora_thumb.png PRE-CREATION 
>   docs/images/presentations/03_25_2014_introduction_to_aurora_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/04_30_2015_monolith_to_microservices_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/08_21_2014_past_present_future_thumb.png 
> PRE-CREATION 
>   docs/presentations.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/34661/diff/
> 
> 
> Testing
> ---
> 
> Markdown originally verified using Mou markdown editor/browser.
> 
> Also viewable on github: 
> https://github.com/davelester/incubator-aurora/blob/aurora-presentations-documentation/docs/presentations.md
> 
> 
> Thanks,
> 
> Dave Lester
> 
>

Re: Review Request 34661: Adds new documentation page listing Apache Aurora presentations (both videos and slides).

2015-06-08 Thread Dave Lester 

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

(Updated June 8, 2015, 6:40 p.m.)


Review request for Aurora, Kevin Sweeney and Bill Farner.


Repository: aurora


Description
---

Adds new documentation page listing Apache Aurora presentations (both videos 
and slides).


Diffs
-

  docs/README.md 508521fbd4fed8376e56b8c7e7eeabc301d1ef9e 
  docs/images/presentations/02_19_2015_aurora_adopters_panel_thumb.png 
PRE-CREATION 
  docs/images/presentations/02_19_2015_aurora_at_tellapart_thumb.png 
PRE-CREATION 
  docs/images/presentations/02_19_2015_aurora_at_twitter_thumb.png PRE-CREATION 
  docs/images/presentations/02_28_2015_apache_aurora_thumb.png PRE-CREATION 
  docs/images/presentations/03_25_2014_introduction_to_aurora_thumb.png 
PRE-CREATION 
  docs/images/presentations/04_30_2015_monolith_to_microservices_thumb.png 
PRE-CREATION 
  docs/images/presentations/08_21_2014_past_present_future_thumb.png 
PRE-CREATION 
  docs/presentations.md PRE-CREATION 

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


Testing (updated)
---

Markdown originally verified using Mou markdown editor/browser.

Also viewable on github: 
https://github.com/davelester/incubator-aurora/blob/aurora-presentations-documentation/docs/presentations.md


Thanks,

Dave Lester

Re: Review Request 34661: Adds new documentation page listing Apache Aurora presentations (both videos and slides).

2015-06-08 Thread Dave Lester 


> On May 26, 2015, 5:51 a.m., Aurora ReviewBot wrote:
> > This patch does not apply cleanly on master (6db13ba), do you need to 
> > rebase?
> > 
> > I will refresh this build result if you post a review containing 
> > "@ReviewBot retry"
> 
> Dave Lester wrote:
> @ReviewBot retry
> 
> Kevin Sweeney wrote:
> To get reviewbot to retry, you need to submit a new review rather than a 
> comment. But I've never seen it accept a patch on a retry. It's probably just 
> not able to accept a binary patch - can you push a branch somewhere for 
> review?

Sure, I pushed this to a branch for viewing: 
https://github.com/davelester/incubator-aurora/blob/aurora-presentations-documentation/docs/presentations.md


- Dave


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


On May 27, 2015, 10:42 p.m., Dave Lester wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34661/
> ---
> 
> (Updated May 27, 2015, 10:42 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Adds new documentation page listing Apache Aurora presentations (both videos 
> and slides).
> 
> 
> Diffs
> -
> 
>   docs/README.md 508521fbd4fed8376e56b8c7e7eeabc301d1ef9e 
>   docs/images/presentations/02_19_2015_aurora_adopters_panel_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_tellapart_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_twitter_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_28_2015_apache_aurora_thumb.png PRE-CREATION 
>   docs/images/presentations/03_25_2014_introduction_to_aurora_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/04_30_2015_monolith_to_microservices_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/08_21_2014_past_present_future_thumb.png 
> PRE-CREATION 
>   docs/presentations.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/34661/diff/
> 
> 
> Testing
> ---
> 
> Markdown originally verified using Mou markdown editor/browser.
> 
> 
> Thanks,
> 
> Dave Lester
> 
>

Re: Review Request 34661: Adds new documentation page listing Apache Aurora presentations (both videos and slides).

2015-06-05 Thread Joe Smith 

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

Ship it!


Validated the links point to working videos or slide decks, so LGTM


docs/README.md


dat whitespace


- Joe Smith


On May 27, 2015, 3:42 p.m., Dave Lester wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34661/
> ---
> 
> (Updated May 27, 2015, 3:42 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Adds new documentation page listing Apache Aurora presentations (both videos 
> and slides).
> 
> 
> Diffs
> -
> 
>   docs/README.md 508521fbd4fed8376e56b8c7e7eeabc301d1ef9e 
>   docs/images/presentations/02_19_2015_aurora_adopters_panel_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_tellapart_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_twitter_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_28_2015_apache_aurora_thumb.png PRE-CREATION 
>   docs/images/presentations/03_25_2014_introduction_to_aurora_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/04_30_2015_monolith_to_microservices_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/08_21_2014_past_present_future_thumb.png 
> PRE-CREATION 
>   docs/presentations.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/34661/diff/
> 
> 
> Testing
> ---
> 
> Markdown originally verified using Mou markdown editor/browser.
> 
> 
> Thanks,
> 
> Dave Lester
> 
>

Re: Review Request 34661: Adds new documentation page listing Apache Aurora presentations (both videos and slides).

2015-05-27 Thread Kevin Sweeney 


> On May 25, 2015, 10:51 p.m., Aurora ReviewBot wrote:
> > This patch does not apply cleanly on master (6db13ba), do you need to 
> > rebase?
> > 
> > I will refresh this build result if you post a review containing 
> > "@ReviewBot retry"
> 
> Dave Lester wrote:
> @ReviewBot retry

To get reviewbot to retry, you need to submit a new review rather than a 
comment. But I've never seen it accept a patch on a retry. It's probably just 
not able to accept a binary patch - can you push a branch somewhere for review?


- Kevin


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


On May 27, 2015, 3:42 p.m., Dave Lester wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34661/
> ---
> 
> (Updated May 27, 2015, 3:42 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Adds new documentation page listing Apache Aurora presentations (both videos 
> and slides).
> 
> 
> Diffs
> -
> 
>   docs/README.md 508521fbd4fed8376e56b8c7e7eeabc301d1ef9e 
>   docs/images/presentations/02_19_2015_aurora_adopters_panel_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_tellapart_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_twitter_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_28_2015_apache_aurora_thumb.png PRE-CREATION 
>   docs/images/presentations/03_25_2014_introduction_to_aurora_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/04_30_2015_monolith_to_microservices_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/08_21_2014_past_present_future_thumb.png 
> PRE-CREATION 
>   docs/presentations.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/34661/diff/
> 
> 
> Testing
> ---
> 
> Markdown originally verified using Mou markdown editor/browser.
> 
> 
> Thanks,
> 
> Dave Lester
> 
>

Re: Review Request 34661: Adds new documentation page listing Apache Aurora presentations (both videos and slides).

2015-05-25 Thread Dave Lester 


> On May 26, 2015, 5:51 a.m., Aurora ReviewBot wrote:
> > This patch does not apply cleanly on master (6db13ba), do you need to 
> > rebase?
> > 
> > I will refresh this build result if you post a review containing 
> > "@ReviewBot retry"

@ReviewBot retry


- Dave


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


On May 26, 2015, 5:41 a.m., Dave Lester wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34661/
> ---
> 
> (Updated May 26, 2015, 5:41 a.m.)
> 
> 
> Review request for Aurora.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Adds new documentation page listing Apache Aurora presentations (both videos 
> and slides).
> 
> 
> Diffs
> -
> 
>   docs/README.md 508521fbd4fed8376e56b8c7e7eeabc301d1ef9e 
>   docs/images/presentations/02_19_2015_aurora_adopters_panel_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_tellapart_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_twitter_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_28_2015_apache_aurora_thumb.png PRE-CREATION 
>   docs/images/presentations/03_25_2014_introduction_to_aurora_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/04_30_2015_monolith_to_microservices_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/08_21_2014_past_present_future_thumb.png 
> PRE-CREATION 
>   docs/presentations.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/34661/diff/
> 
> 
> Testing
> ---
> 
> Markdown originally verified using Mou markdown editor/browser.
> 
> 
> Thanks,
> 
> Dave Lester
> 
>

Re: Review Request 34661: Adds new documentation page listing Apache Aurora presentations (both videos and slides).

2015-05-25 Thread Aurora ReviewBot 

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


This patch does not apply cleanly on master (6db13ba), do you need to rebase?

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

- Aurora ReviewBot


On May 26, 2015, 5:41 a.m., Dave Lester wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34661/
> ---
> 
> (Updated May 26, 2015, 5:41 a.m.)
> 
> 
> Review request for Aurora.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Adds new documentation page listing Apache Aurora presentations (both videos 
> and slides).
> 
> 
> Diffs
> -
> 
>   docs/README.md 508521fbd4fed8376e56b8c7e7eeabc301d1ef9e 
>   docs/images/presentations/02_19_2015_aurora_adopters_panel_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_tellapart_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_twitter_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_28_2015_apache_aurora_thumb.png PRE-CREATION 
>   docs/images/presentations/03_25_2014_introduction_to_aurora_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/04_30_2015_monolith_to_microservices_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/08_21_2014_past_present_future_thumb.png 
> PRE-CREATION 
>   docs/presentations.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/34661/diff/
> 
> 
> Testing
> ---
> 
> Markdown originally verified using Mou markdown editor/browser.
> 
> 
> Thanks,
> 
> Dave Lester
> 
>

Re: Review Request 34661: Adds new documentation page listing Apache Aurora presentations (both videos and slides).

2015-05-25 Thread Dave Lester 

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

(Updated May 26, 2015, 5:41 a.m.)


Review request for Aurora.


Repository: aurora


Description
---

Adds new documentation page listing Apache Aurora presentations (both videos 
and slides).


Diffs (updated)
-

  docs/README.md 508521fbd4fed8376e56b8c7e7eeabc301d1ef9e 
  docs/images/presentations/02_19_2015_aurora_adopters_panel_thumb.png 
PRE-CREATION 
  docs/images/presentations/02_19_2015_aurora_at_tellapart_thumb.png 
PRE-CREATION 
  docs/images/presentations/02_19_2015_aurora_at_twitter_thumb.png PRE-CREATION 
  docs/images/presentations/02_28_2015_apache_aurora_thumb.png PRE-CREATION 
  docs/images/presentations/03_25_2014_introduction_to_aurora_thumb.png 
PRE-CREATION 
  docs/images/presentations/04_30_2015_monolith_to_microservices_thumb.png 
PRE-CREATION 
  docs/images/presentations/08_21_2014_past_present_future_thumb.png 
PRE-CREATION 
  docs/presentations.md PRE-CREATION 

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


Testing
---

Markdown originally verified using Mou markdown editor/browser.


Thanks,

Dave Lester

Re: Review Request 34661: Adds new documentation page listing Apache Aurora presentations (both videos and slides).

2015-05-25 Thread Aurora ReviewBot 

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


This patch does not apply cleanly on master (6db13ba), do you need to rebase?

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

- Aurora ReviewBot


On May 26, 2015, 12:45 a.m., Dave Lester wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34661/
> ---
> 
> (Updated May 26, 2015, 12:45 a.m.)
> 
> 
> Review request for Aurora.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Adds new documentation page listing Apache Aurora presentations (both videos 
> and slides).
> 
> 
> Diffs
> -
> 
>   docs/images/presentations/02_19_2015_aurora_adopters_panel_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_tellapart_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_19_2015_aurora_at_twitter_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/02_28_2015_apache_aurora_thumb.png PRE-CREATION 
>   docs/images/presentations/03_25_2014_introduction_to_aurora_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/04_30_2015_monolith_to_microservices_thumb.png 
> PRE-CREATION 
>   docs/images/presentations/08_21_2014_past_present_future_thumb.png 
> PRE-CREATION 
>   docs/presentations.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/34661/diff/
> 
> 
> Testing
> ---
> 
> Markdown originally verified using Mou markdown editor/browser.
> 
> 
> Thanks,
> 
> Dave Lester
> 
>