Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

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

Ship it!


Joshua, Zameer - i'm going to push forward with this in lieu of your reviews.  
I think we've gotten lots of useful feedback and this doc is a net win in its 
present form.  Happy to iterate on follow-up comments.

- Bill Farner


On Dec. 21, 2015, 10:41 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 21, 2015, 10:41 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Bugs: AURORA-1483
> https://issues.apache.org/jira/browse/AURORA-1483
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

> On Dec. 21, 2015, 8:43 a.m., John Sirois wrote:
> > docs/installing.md, line 263
> > 
> >
> > I used `openjdk-8-jre-headless` here.

Thanks, changed.


- Bill


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


On Dec. 20, 2015, 9:32 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 20, 2015, 9:32 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Bugs: AURORA-1483
> https://issues.apache.org/jira/browse/AURORA-1483
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

> On Dec. 21, 2015, 9:16 a.m., John Sirois wrote:
> > 1 small issue for CentOS noted below, the larger issue blocks example job 
> > submission and involves the scheduler failing to open the distributed log - 
> > looking into that currently.
> 
> John Sirois wrote:
> Also discovered the aurora-scheduler rpm install pulling in fonts - 
> spawned: https://reviews.apache.org/r/41610/
> 
> John Sirois wrote:
> I needed the following to get aurora reading the log initialized in the 
> instructions and connected to the master installed via the instructions:
> ```
> [vagrant@localhost ~]$ aurora job create main/aurora/prod/hello 
> hello_world.aurora 
>  INFO] Creating job hello
>  INFO] Checking status of main/aurora/prod/hello
> Job create succeeded: job 
> url=http://127.0.0.1:8081/scheduler/aurora/prod/hello
> [vagrant@localhost ~]$ diff /etc/sysconfig/aurora.orig 
> /etc/sysconfig/aurora
> 41c41
> <   -mesos_master_address='zk://127.0.0.1:2181/mesos/master'
> ---
> >   -mesos_master_address='zk://127.0.0.1:2181/mesos'
> 69c69
> <   -native_log_file_path='/var/lib/aurora/db'
> ---
> >   -native_log_file_path='/var/lib/aurora/scheduler/db'
> ```

Yeah, this is another bit of drift between the RPM and deb packages.  Added a 
note.


> On Dec. 21, 2015, 9:16 a.m., John Sirois wrote:
> > docs/installing.md, line 236
> > 
> >
> > # Note: for older Aurora RPM versions, this may be called 
> > 'aurora-client'.

Done.


- Bill


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


On Dec. 20, 2015, 9:32 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 20, 2015, 9:32 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Bugs: AURORA-1483
> https://issues.apache.org/jira/browse/AURORA-1483
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Aurora ReviewBot]

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

Ship it!


Master (d3dae19) is green with this patch.
  ./build-support/jenkins/build.sh

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

- Aurora ReviewBot


On Dec. 21, 2015, 6:41 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 21, 2015, 6:41 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Bugs: AURORA-1483
> https://issues.apache.org/jira/browse/AURORA-1483
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[John Sirois]

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

Ship it!


Ship It!

- John Sirois


On Dec. 21, 2015, 11:41 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 21, 2015, 11:41 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Bugs: AURORA-1483
> https://issues.apache.org/jira/browse/AURORA-1483
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[John Sirois]

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


Re-ran through the doc for Ubuntu, all worked exactly as-is to be able to 
submit a hello_world.aurora successfully.  I used 1 mod noted below:


docs/installing.md (line 263)


I used `openjdk-8-jre-headless` here.


- John Sirois


On Dec. 20, 2015, 10:32 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 20, 2015, 10:32 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Bugs: AURORA-1483
> https://issues.apache.org/jira/browse/AURORA-1483
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[John Sirois]

> On Dec. 21, 2015, 10:16 a.m., John Sirois wrote:
> > 1 small issue for CentOS noted below, the larger issue blocks example job 
> > submission and involves the scheduler failing to open the distributed log - 
> > looking into that currently.

Also discovered the aurora-scheduler rpm install pulling in fonts - spawned: 
https://reviews.apache.org/r/41610/


- John


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


On Dec. 20, 2015, 10:32 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 20, 2015, 10:32 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Bugs: AURORA-1483
> https://issues.apache.org/jira/browse/AURORA-1483
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[John Sirois]

> On Dec. 21, 2015, 10:16 a.m., John Sirois wrote:
> > 1 small issue for CentOS noted below, the larger issue blocks example job 
> > submission and involves the scheduler failing to open the distributed log - 
> > looking into that currently.
> 
> John Sirois wrote:
> Also discovered the aurora-scheduler rpm install pulling in fonts - 
> spawned: https://reviews.apache.org/r/41610/

I needed the following to get aurora reading the log initialized in the 
instructions and connected to the master installed via the instructions:
```
[vagrant@localhost ~]$ aurora job create main/aurora/prod/hello 
hello_world.aurora 
 INFO] Creating job hello
 INFO] Checking status of main/aurora/prod/hello
Job create succeeded: job url=http://127.0.0.1:8081/scheduler/aurora/prod/hello
[vagrant@localhost ~]$ diff /etc/sysconfig/aurora.orig /etc/sysconfig/aurora
41c41
<   -mesos_master_address='zk://127.0.0.1:2181/mesos/master'
---
>   -mesos_master_address='zk://127.0.0.1:2181/mesos'
69c69
<   -native_log_file_path='/var/lib/aurora/db'
---
>   -native_log_file_path='/var/lib/aurora/scheduler/db'
```


- John


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


On Dec. 20, 2015, 10:32 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 20, 2015, 10:32 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Bugs: AURORA-1483
> https://issues.apache.org/jira/browse/AURORA-1483
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[John Sirois]

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

Ship it!


Besides final install detail issues noted above, this LGTM.  More work later to 
make this process stable and easily testable / automated.

- John Sirois


On Dec. 20, 2015, 10:32 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 20, 2015, 10:32 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Bugs: AURORA-1483
> https://issues.apache.org/jira/browse/AURORA-1483
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Stephan Erb]

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


Please update the PR to add the link to the relvant ticket: 
https://issues.apache.org/jira/browse/AURORA-1483

- Stephan Erb


On Dec. 20, 2015, 3:24 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 20, 2015, 3:24 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

> On Dec. 20, 2015, 6 a.m., Stephan Erb wrote:
> > Please update the PR to add the link to the relvant ticket: 
> > https://issues.apache.org/jira/browse/AURORA-1483

Forgot about that one, thanks!


- Bill


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


On Dec. 19, 2015, 6:24 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 19, 2015, 6:24 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

> On Dec. 18, 2015, 2:16 p.m., Stephan Erb wrote:
> > docs/installing.md, line 146
> > 
> >
> > Isn't that outdated?
> 
> John Sirois wrote:
> I think so - I used `sudo yum install -y mesos-0.23` for this step.
> 
> Bill Farner wrote:
> I left this at 0.22.0 because that's what the 0.9.0 rpm currently relies 
> on:
> 
> ```
> Error: Package: aurora-0.9.0-1.el7.centos.aurora.x86_64 
> (apache-aurora-wfarner)
>Requires: mesos = 0.22.0
> ```
> 
> John Sirois wrote:
> OK.  I had actually switched the new yum repo instructions to use 
> http://people.apache.org/~wfarner/aurora/distributions/0.10.0-1/rpm/centos-7/ 
> in my testing which got rpm / deb on the same page.

Yeah, sorry for the confusion.  There will be better alignment once 
https://reviews.apache.org/r/40877/ lands and we can release an RPM from master 
of the packaging repo.


- Bill


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


On Dec. 20, 2015, 9:32 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 20, 2015, 9:32 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Bugs: AURORA-1483
> https://issues.apache.org/jira/browse/AURORA-1483
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

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


Question for the reviewers - what should we target as the base for these 
install instructions?  For example, the vanilla Ubuntu Trusty server image from 
ubuntu.com needs fewer base deps than the `ubuntu:trusty` Docker image.

- Bill Farner


On Dec. 18, 2015, 7:03 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 7:03 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Stephan Erb]

> On Dec. 19, 2015, 8:26 p.m., Bill Farner wrote:
> > Question for the reviewers - what should we target as the base for these 
> > install instructions?  For example, the vanilla Ubuntu Trusty server image 
> > from ubuntu.com needs fewer base deps than the `ubuntu:trusty` Docker image.

We should target the more minimal image, so that we have all requirements 
listed in the install instructions. Otherwise there might be surprises for new 
users that suddenly have to track down which dependency is missing.


- Stephan


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


On Dec. 19, 2015, 4:03 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 19, 2015, 4:03 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Stephan Erb]

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

Ship it!


Overall document looks fine. One minor nitpick below. 

Please note that I did not check the exact install intructions as I am on the 
road right now.


docs/installing.md (line 241)


You seemed to have stopped mid-sentance.



docs/installing.md (line 241)


You seemed to have stopped mid-sentance.


- Stephan Erb


On Dec. 19, 2015, 4:03 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 19, 2015, 4:03 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Aurora ReviewBot]

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

Ship it!


Master (5183fe0) is green with this patch.
  ./build-support/jenkins/build.sh

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

- Aurora ReviewBot


On Dec. 19, 2015, 8 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 19, 2015, 8 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

> On Dec. 19, 2015, 11:26 a.m., Bill Farner wrote:
> > Question for the reviewers - what should we target as the base for these 
> > install instructions?  For example, the vanilla Ubuntu Trusty server image 
> > from ubuntu.com needs fewer base deps than the `ubuntu:trusty` Docker image.
> 
> Stephan Erb wrote:
> We should target the more minimal image, so that we have all requirements 
> listed in the install instructions. Otherwise there might be surprises for 
> new users that suddenly have to track down which dependency is missing.

Thanks, that's the way i was leaning.  In that case, let's use `ubuntu:trusty` 
and `centos:7` docker images as a test environment.  The instructions will 
certainly currently fail those tests.


- Bill


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


On Dec. 18, 2015, 7:03 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 7:03 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Aurora ReviewBot]

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

Ship it!


Master (5183fe0) is green with this patch.
  ./build-support/jenkins/build.sh

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

- Aurora ReviewBot


On Dec. 20, 2015, 2:24 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 20, 2015, 2:24 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

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

(Updated Dec. 19, 2015, 6:24 p.m.)


Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.


Changes
---

Deps should be in better shape now.  I shuffled things a bit to avoid 
duplication of instructions for installing mesos (since master and slave share 
a package).


Repository: aurora


Description
---

Add a guide for installing Aurora.


Diffs (updated)
-

  README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
  docs/installing.md PRE-CREATION 

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


Testing
---

Rendered here: 
https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md

I could use more eyes and independent trials on the installation steps.


Thanks,

Bill Farner

Re: Review Request 41564: Add a guide for installing Aurora.

[Maxim Khutornenko]

> On Dec. 18, 2015, 8:11 p.m., Maxim Khutornenko wrote:
> > docs/installing.md, line 58
> > 
> >
> > This is very important part of our infrastructure. I'd add a bit more 
> > details here highlighting its importance (including scheduler->master 
> > discovery).
> 
> Bill Farner wrote:
> I don't think explaining the importance of a component is valuable or 
> objective detail in this document.  IMHO this document is best as a recipe 
> for getting something working.

Wasn't suggesting an elaborate explanation, just a bit of additional info 
around discovery and leader election, e.g.: "...used for discovering Mesos 
master and scheduler leader election".


- Maxim


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


On Dec. 18, 2015, 7:58 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 7:58 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

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

(Updated Dec. 18, 2015, 12:38 p.m.)


Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.


Repository: aurora


Description
---

Add a guide for installing Aurora.


Diffs (updated)
-

  README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
  docs/installing.md PRE-CREATION 

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


Testing
---

Rendered here: 
https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md

I could use more eyes and independent trials on the installation steps.


Thanks,

Bill Farner

Re: Review Request 41564: Add a guide for installing Aurora.

[Maxim Khutornenko]

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



docs/installing.md (line 58)


This is very important part of our infrastructure. I'd add a bit more 
details here highlighting its importance (including scheduler->master 
discovery).


- Maxim Khutornenko


On Dec. 18, 2015, 7:58 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 7:58 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[John Sirois]

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



docs/installing.md (line 155)


believing



docs/installing.md (line 157)


Should there be a service stop issued 1st?
Also Ubuntu 14.04 uses upstart, so you might either suggest the below is a 
Centos 7 example and the start command may need to be adjusted - or else give 
and example for each OS.


- John Sirois


On Dec. 18, 2015, 12:58 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 12:58 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

> On Dec. 18, 2015, 12:11 p.m., Maxim Khutornenko wrote:
> > docs/installing.md, line 58
> > 
> >
> > This is very important part of our infrastructure. I'd add a bit more 
> > details here highlighting its importance (including scheduler->master 
> > discovery).

I don't think explaining the importance of a component is valuable or objective 
detail in this document.  IMHO this document is best as a recipe for getting 
something working.


- Bill


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


On Dec. 18, 2015, 11:58 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 11:58 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

> On Dec. 18, 2015, 2:16 p.m., Stephan Erb wrote:
> > docs/installing.md, line 29
> > 
> >
> > You are happily mixing the terms jobs and services here. Other sections 
> > of this document also use the word task. It is obvious for us that these 
> > are basically the same thing. However, for people new to Aurora this might 
> > be confusing.

Thanks, reworded.


> On Dec. 18, 2015, 2:16 p.m., Stephan Erb wrote:
> > docs/installing.md, line 35
> > 
> >
> > Have you thought about adding an additional section regarding the 
> > aurora config files? Even a simple guideline can be quite helpful for 
> > starters,  like you have one config file per service/job and that these 
> > files are commonly tracked in a git repository.

I see what you mean.  I'd like to avoid a slippery slope wherein this becomes 
encyclopedic, but how about a brief mention and a link to the configuration 
guide? (Added in latest draft.)


> On Dec. 18, 2015, 2:16 p.m., Stephan Erb wrote:
> > docs/installing.md, line 44
> > 
> >
> > I think we should extend this section slightly: 
> > 
> > * it should somehow contain the word `thermos` as it is used in the 
> > rest of the Aurora documentation quite often
> > * it could stress a little bit more that this executes the user 
> > processes

Great suggestions, done.


> On Dec. 18, 2015, 2:16 p.m., Stephan Erb wrote:
> > docs/installing.md, line 66
> > 
> >
> > A common confusion for Mesos adopters seems to be the relation between 
> > the agent and an executor. We could state here that the agent spaws 
> > executors which in turn runs the user code (either on the physical host or 
> > within a docker container).

Done.


> On Dec. 18, 2015, 2:16 p.m., Stephan Erb wrote:
> > docs/installing.md, line 77
> > 
> >
> > When getting started, it is quite difficult to decide whether to 
> > co-locate all those services for simplicity or to keep them seperated for 
> > isolation, security, performance reasons etc. Is there any further advice 
> > we could give here?

Done.


> On Dec. 18, 2015, 2:16 p.m., Stephan Erb wrote:
> > docs/installing.md, line 146
> > 
> >
> > Isn't that outdated?
> 
> John Sirois wrote:
> I think so - I used `sudo yum install -y mesos-0.23` for this step.

I left this at 0.22.0 because that's what the 0.9.0 rpm currently relies on:

```
Error: Package: aurora-0.9.0-1.el7.centos.aurora.x86_64 (apache-aurora-wfarner)
   Requires: mesos = 0.22.0
```


- Bill


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


On Dec. 18, 2015, 12:39 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 12:39 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[John Sirois]

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


Not as thorough a check of Ubuntu, but so far:


docs/installing.md (line 126)


Had to s/ mesos// and then:
```
wget -c 
http://downloads.mesosphere.io/master/ubuntu/14.04/mesos_0.23.0-1.0.ubuntu1404_amd64.deb
sudo dpkg -i mesos_0.23.0-1.0.ubuntu1404_amd64.deb
sudo start mesos-master
```



docs/installing.md (line 130)


This looks fine although I used:
```
wget -c http://apache.bintray.com/aurora/aurora-scheduler_0.10.0-1_amd64.deb
sudo dpkg -i aurora-scheduler_0.10.0-1_amd64.deb
```



docs/installing.md (line 189)


Missing equivalent using 
https://apache.bintray.com/aurora/:aurora-executor_0.10.0-1_amd64.deb for 
Ubuntu.



docs/installing.md (line 200)


Looks fine although I used:
```
wget -c https://apache.bintray.com/aurora/aurora-tools_0.10.0-1_amd64.deb
sudo dpkg -i aurora-tools_0.10.0-1_amd64.deb
```


- John Sirois


On Dec. 18, 2015, 1:39 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 1:39 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[John Sirois]

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


The comments below are from a test of CentOS 7 instructions - using the aurora 
0.10.0-1 packages.


docs/installing.md (line 111)


Invalid URL - I updated to 
http://people.apache.org/~wfarner/aurora/distributions/0.10.0-1/rpm/centos-7/ 
to get things working



docs/installing.md (line 141)


s/zookeeper-serve/zookeeper-server/ - also this requires java (8 for aurora 
later).  It would be good to call out the need to have java 8 installed or else 
just re-order the steps since the aurora package properly depends on java (8) 
and the zk package does not properly depend on java.

I used this fwiw:
```
sudo yum install -y java-1.8.0-openjdk
```



docs/installing.md (line 150)


s/aurora/aurora-scheduler/



docs/installing.md (line 189)


s/aurora-thermos/aurora-executor/


- John Sirois


On Dec. 18, 2015, 1:39 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 1:39 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[John Sirois]

> On Dec. 18, 2015, 3:16 p.m., Stephan Erb wrote:
> > docs/installing.md, line 146
> > 
> >
> > Isn't that outdated?

I think so - I used `sudo yum install -y mesos-0.23` for this step.


- John


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


On Dec. 18, 2015, 1:39 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 1:39 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Aurora ReviewBot]

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

Ship it!


Master (5183fe0) is green with this patch.
  ./build-support/jenkins/build.sh

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

- Aurora ReviewBot


On Dec. 18, 2015, 8:39 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 8:39 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Stephan Erb]

> On Dez. 18, 2015, 9:12 nachm., John Sirois wrote:
> >
> 
> John Sirois wrote:
> I won't be able to get to trials until later this evening.  Ideally 
> though this doc could be tied to the test/ provision.sh's somehow for more 
> assured working-ness.  I have no useful suggestion though atm of how to do 
> that reasonably.

That would be ideal. The install instructions might get outdated rather easily.


- Stephan


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


On Dez. 18, 2015, 9:39 nachm., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dez. 18, 2015, 9:39 nachm.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

> On Dec. 18, 2015, 4:43 p.m., John Sirois wrote:
> > docs/installing.md, line 126
> > 
> >
> > Had to s/ mesos// and then:
> > ```
> > wget -c 
> > http://downloads.mesosphere.io/master/ubuntu/14.04/mesos_0.23.0-1.0.ubuntu1404_amd64.deb
> > sudo dpkg -i mesos_0.23.0-1.0.ubuntu1404_amd64.deb
> > sudo start mesos-master
> > ```

Wow, yeah...i must have stopped mid-thought here.  Thanks.


> On Dec. 18, 2015, 4:43 p.m., John Sirois wrote:
> > docs/installing.md, line 130
> > 
> >
> > This looks fine although I used:
> > ```
> > wget -c 
> > http://apache.bintray.com/aurora/aurora-scheduler_0.10.0-1_amd64.deb
> > sudo dpkg -i aurora-scheduler_0.10.0-1_amd64.deb
> > ```

Heh, thanks.  Hot off the presses :-)

Didn't realize bintray had those shorter URLs, sweet!

Updated.


> On Dec. 18, 2015, 4:43 p.m., John Sirois wrote:
> > docs/installing.md, line 189
> > 
> >
> > Missing equivalent using 
> > https://apache.bintray.com/aurora/:aurora-executor_0.10.0-1_amd64.deb for 
> > Ubuntu.

Thanks, fixed.


> On Dec. 18, 2015, 4:43 p.m., John Sirois wrote:
> > docs/installing.md, line 200
> > 
> >
> > Looks fine although I used:
> > ```
> > wget -c 
> > https://apache.bintray.com/aurora/aurora-tools_0.10.0-1_amd64.deb
> > sudo dpkg -i aurora-tools_0.10.0-1_amd64.deb
> > ```

Done.


- Bill


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


On Dec. 18, 2015, 12:39 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 12:39 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Aurora ReviewBot]

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

Ship it!


Master (5183fe0) is green with this patch.
  ./build-support/jenkins/build.sh

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

- Aurora ReviewBot


On Dec. 19, 2015, 3:03 a.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 19, 2015, 3:03 a.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

> On Dec. 18, 2015, 3:45 p.m., John Sirois wrote:
> > docs/installing.md, line 111
> > 
> >
> > Invalid URL - I updated to 
> > http://people.apache.org/~wfarner/aurora/distributions/0.10.0-1/rpm/centos-7/
> >  to get things working

Doh, thanks.  This was a bad find/replace.


> On Dec. 18, 2015, 3:45 p.m., John Sirois wrote:
> > docs/installing.md, line 141
> > 
> >
> > s/zookeeper-serve/zookeeper-server/ - also this requires java (8 for 
> > aurora later).  It would be good to call out the need to have java 8 
> > installed or else just re-order the steps since the aurora package properly 
> > depends on java (8) and the zk package does not properly depend on java.
> > 
> > I used this fwiw:
> > ```
> > sudo yum install -y java-1.8.0-openjdk
> > ```

Thanks, done.  Went with `java-1.8.0-openjdk-headless`.


> On Dec. 18, 2015, 3:45 p.m., John Sirois wrote:
> > docs/installing.md, line 150
> > 
> >
> > s/aurora/aurora-scheduler/

Done.  Since the 0.9.0 RPM mentioned earlier uses the `aurora` name, i've left 
a note so people don't get tripped up.


> On Dec. 18, 2015, 3:45 p.m., John Sirois wrote:
> > docs/installing.md, line 189
> > 
> >
> > s/aurora-thermos/aurora-executor/

Thanks.  Ditto re: naming.


- Bill


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


On Dec. 18, 2015, 12:39 p.m., Bill Farner wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/41564/
> ---
> 
> (Updated Dec. 18, 2015, 12:39 p.m.)
> 
> 
> Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.
> 
> 
> Repository: aurora
> 
> 
> Description
> ---
> 
> Add a guide for installing Aurora.
> 
> 
> Diffs
> -
> 
>   README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
>   docs/installing.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/41564/diff/
> 
> 
> Testing
> ---
> 
> Rendered here: 
> https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md
> 
> I could use more eyes and independent trials on the installation steps.
> 
> 
> Thanks,
> 
> Bill Farner
> 
>

Re: Review Request 41564: Add a guide for installing Aurora.

[Bill Farner]

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

(Updated Dec. 18, 2015, 7:03 p.m.)


Review request for Aurora, Joshua Cohen, John Sirois, and Zameer Manji.


Changes
---

Addressed Stephan and John's comments.

I'm pretty sure i still don't have the deps correct for the worker machine at 
least, will revisit before submitting unless somebody beats me to it.


Repository: aurora


Description
---

Add a guide for installing Aurora.


Diffs (updated)
-

  README.md e88e1beddbea1be76e6ea6cabee0bd105ae5360e 
  docs/installing.md PRE-CREATION 

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


Testing
---

Rendered here: 
https://github.com/wfarner/aurora/blob/wfarner/installing_aurora_doc/docs/installing.md

I could use more eyes and independent trials on the installation steps.


Thanks,

Bill Farner