You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Ben Mahler <be...@gmail.com> on 2015/01/10 23:43:29 UTC

Review Request 29796: Added initial operational guide to document master quorum changes.

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

Review request for mesos and Jie Yu.


Repository: mesos-git


Description
-------

Please see here for the rendered markdown:

https://gist.github.com/bmahler/8c21de98471c9c55940b#file-operational-guide-md


Diffs
-----

  docs/home.md 33458cc9acf15b1cc8011bd5610c103dce7f2248 
  docs/operational-guide.md PRE-CREATION 

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


Testing
-------

N/A


Thanks,

Ben Mahler


Re: Review Request 29796: Added initial operational guide to document master quorum changes.

Posted by Jie Yu <yu...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29796/#review68069
-----------------------------------------------------------

Ship it!


Ship It!

- Jie Yu


On Jan. 14, 2015, 6:44 p.m., Ben Mahler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29796/
> -----------------------------------------------------------
> 
> (Updated Jan. 14, 2015, 6:44 p.m.)
> 
> 
> Review request for mesos and Jie Yu.
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> Please see here for the rendered markdown:
> 
> https://gist.github.com/bmahler/8c21de98471c9c55940b#file-operational-guide-md
> 
> 
> Diffs
> -----
> 
>   docs/home.md 33458cc9acf15b1cc8011bd5610c103dce7f2248 
>   docs/operational-guide.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/29796/diff/
> 
> 
> Testing
> -------
> 
> N/A
> 
> 
> Thanks,
> 
> Ben Mahler
> 
>


Re: Review Request 29796: Added initial operational guide to document master quorum changes.

Posted by Ben Mahler <be...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29796/
-----------------------------------------------------------

(Updated Jan. 14, 2015, 6:44 p.m.)


Review request for mesos and Jie Yu.


Changes
-------

Updates per Jie's review.


Repository: mesos-git


Description
-------

Please see here for the rendered markdown:

https://gist.github.com/bmahler/8c21de98471c9c55940b#file-operational-guide-md


Diffs (updated)
-----

  docs/home.md 33458cc9acf15b1cc8011bd5610c103dce7f2248 
  docs/operational-guide.md PRE-CREATION 

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


Testing
-------

N/A


Thanks,

Ben Mahler


Re: Review Request 29796: Added initial operational guide to document master quorum changes.

Posted by Ben Mahler <be...@gmail.com>.

> On Jan. 11, 2015, 1:42 a.m., Jie Yu wrote:
> > docs/operational-guide.md, line 34
> > <https://reviews.apache.org/r/29796/diff/1/?file=816101#file816101line34>
> >
> >     Maybe we should also say something about cases like:
> >     
> >     if changing from 7 masters to 3 masters are safe or not?

Commented that if an increase of N is desired, they should increment N times using these steps.


- Ben


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


On Jan. 10, 2015, 10:43 p.m., Ben Mahler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29796/
> -----------------------------------------------------------
> 
> (Updated Jan. 10, 2015, 10:43 p.m.)
> 
> 
> Review request for mesos and Jie Yu.
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> Please see here for the rendered markdown:
> 
> https://gist.github.com/bmahler/8c21de98471c9c55940b#file-operational-guide-md
> 
> 
> Diffs
> -----
> 
>   docs/home.md 33458cc9acf15b1cc8011bd5610c103dce7f2248 
>   docs/operational-guide.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/29796/diff/
> 
> 
> Testing
> -------
> 
> N/A
> 
> 
> Thanks,
> 
> Ben Mahler
> 
>


Re: Review Request 29796: Added initial operational guide to document master quorum changes.

Posted by Jie Yu <yu...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29796/#review67599
-----------------------------------------------------------



docs/operational-guide.md
<https://reviews.apache.org/r/29796/#comment111643>

    storage backend is configurable, right? Should we say something about the flag?



docs/operational-guide.md
<https://reviews.apache.org/r/29796/#comment111646>

    Is it safe to transit from 1 master to 3 masters using this method? What should we do in that case?



docs/operational-guide.md
<https://reviews.apache.org/r/29796/#comment111645>

    Maybe we should also say something about cases like:
    
    if changing from 7 masters to 3 masters are safe or not?



docs/operational-guide.md
<https://reviews.apache.org/r/29796/#comment111644>

    maybe we should emphasize that the new master has no log?


- Jie Yu


On Jan. 10, 2015, 10:43 p.m., Ben Mahler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29796/
> -----------------------------------------------------------
> 
> (Updated Jan. 10, 2015, 10:43 p.m.)
> 
> 
> Review request for mesos and Jie Yu.
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> Please see here for the rendered markdown:
> 
> https://gist.github.com/bmahler/8c21de98471c9c55940b#file-operational-guide-md
> 
> 
> Diffs
> -----
> 
>   docs/home.md 33458cc9acf15b1cc8011bd5610c103dce7f2248 
>   docs/operational-guide.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/29796/diff/
> 
> 
> Testing
> -------
> 
> N/A
> 
> 
> Thanks,
> 
> Ben Mahler
> 
>