You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@aurora.apache.org by Maxim Khutornenko <ma...@apache.org> on 2014/10/17 03:51:03 UTC
Review Request 26851: Adding storage recovery from backup doc.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/
-----------------------------------------------------------
Review request for Aurora, Kevin Sweeney and Bill Farner.
Bugs: AURORA-839
https://issues.apache.org/jira/browse/AURORA-839
Repository: aurora
Description
-------
Summarizing recovery from backup steps.
Diffs
-----
docs/storage_recovery.md PRE-CREATION
Diff: https://reviews.apache.org/r/26851/diff/
Testing
-------
https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage_recovery.md
Thanks,
Maxim Khutornenko
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Bill Farner <wf...@apache.org>.
> On Oct. 17, 2014, 5:07 p.m., Bill Farner wrote:
> > docs/storage_recovery.md, line 43
> > <https://reviews.apache.org/r/26851/diff/1/?file=723887#file723887line43>
> >
> > Avoid falling back to HTML if you can. In this case, triple backticks will do the same thing.
>
> Maxim Khutornenko wrote:
> That did not help in my case so I had to use <pre/> to ensure proper look.
I just forked your linked page, and used triple backticks for the same effect. Is there something i'm missing?
- Bill
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/#review57160
-----------------------------------------------------------
On Oct. 17, 2014, 9:47 p.m., Maxim Khutornenko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26851/
> -----------------------------------------------------------
>
> (Updated Oct. 17, 2014, 9:47 p.m.)
>
>
> Review request for Aurora, Kevin Sweeney and Bill Farner.
>
>
> Bugs: AURORA-839
> https://issues.apache.org/jira/browse/AURORA-839
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Summarizing recovery from backup steps.
>
>
> Diffs
> -----
>
> docs/storage-config.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/26851/diff/
>
>
> Testing
> -------
>
> https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage-config.md
>
>
> Thanks,
>
> Maxim Khutornenko
>
>
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Maxim Khutornenko <ma...@apache.org>.
> On Oct. 17, 2014, 5:07 p.m., Bill Farner wrote:
> > docs/storage_recovery.md, line 1
> > <https://reviews.apache.org/r/26851/diff/1/?file=723887#file723887line1>
> >
> > How about: "Recovering from a scheduler backup"
Done.
> On Oct. 17, 2014, 5:07 p.m., Bill Farner wrote:
> > docs/storage_recovery.md, line 9
> > <https://reviews.apache.org/r/26851/diff/1/?file=723887#file723887line9>
> >
> > This is a bit doom-and-gloom, and jarrin to the reader since it doesn't include background. What does 'prepare' mean? What does 'data loss' mean?
> >
> > Consider re-wording to something like:
> >
> > > Be sure to read the entire page before attempting to restore from a backup, as it may have unintendd consequences.
Done.
> On Oct. 17, 2014, 5:07 p.m., Bill Farner wrote:
> > docs/storage_recovery.md, line 18
> > <https://reviews.apache.org/r/26851/diff/1/?file=723887#file723887line18>
> >
> > I'd also add:
> >
> > > Usually, it is a bad idea to attempt to restore an update that is not extremely recent, usually less then a few hours old. This is because the scheduler will expect the cluster to look exactly as the backup does, so any tasks that have been rescheduled since the backup was taken will be killed.
Added.
> On Oct. 17, 2014, 5:07 p.m., Bill Farner wrote:
> > docs/storage_recovery.md, line 22
> > <https://reviews.apache.org/r/26851/diff/1/?file=723887#file723887line22>
> >
> > Is the "and disable..." part necessary? Presumably an operator knows to use the process manager to stop a scheduler.
Dropped.
> On Oct. 17, 2014, 5:07 p.m., Bill Farner wrote:
> > docs/storage_recovery.md, line 24
> > <https://reviews.apache.org/r/26851/diff/1/?file=723887#file723887line24>
> >
> > Please elaborate on "concurrent access"
Reworded.
> On Oct. 17, 2014, 5:07 p.m., Bill Farner wrote:
> > docs/storage_recovery.md, line 30
> > <https://reviews.apache.org/r/26851/diff/1/?file=723887#file723887line30>
> >
> > Please elaobrate on the purpose of "prevent[ing] accidental task GC".
Done.
> On Oct. 17, 2014, 5:07 p.m., Bill Farner wrote:
> > docs/storage_recovery.md, line 36
> > <https://reviews.apache.org/r/26851/diff/1/?file=723887#file723887line36>
> >
> > Some prose in each of these sections would help a lot to improve flow of the document. For example, at this point i'm not clear if i'm assessing the damage of the scheduler configuration changes i made in 'preparation'.
Sure, added.
> On Oct. 17, 2014, 5:07 p.m., Bill Farner wrote:
> > docs/storage_recovery.md, line 43
> > <https://reviews.apache.org/r/26851/diff/1/?file=723887#file723887line43>
> >
> > Avoid falling back to HTML if you can. In this case, triple backticks will do the same thing.
That did not help in my case so I had to use <pre/> to ensure proper look.
> On Oct. 17, 2014, 5:07 p.m., Bill Farner wrote:
> > docs/storage_recovery.md, line 52
> > <https://reviews.apache.org/r/26851/diff/1/?file=723887#file723887line52>
> >
> > How do i know the integrity of the replicated log?
Added a NOTE at the end.
- Maxim
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/#review57160
-----------------------------------------------------------
On Oct. 17, 2014, 1:51 a.m., Maxim Khutornenko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26851/
> -----------------------------------------------------------
>
> (Updated Oct. 17, 2014, 1:51 a.m.)
>
>
> Review request for Aurora, Kevin Sweeney and Bill Farner.
>
>
> Bugs: AURORA-839
> https://issues.apache.org/jira/browse/AURORA-839
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Summarizing recovery from backup steps.
>
>
> Diffs
> -----
>
> docs/storage_recovery.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/26851/diff/
>
>
> Testing
> -------
>
> https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage_recovery.md
>
>
> Thanks,
>
> Maxim Khutornenko
>
>
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Bill Farner <wf...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/#review57160
-----------------------------------------------------------
docs/storage_recovery.md
<https://reviews.apache.org/r/26851/#comment97635>
How about: "Recovering from a scheduler backup"
docs/storage_recovery.md
<https://reviews.apache.org/r/26851/#comment97640>
This is a bit doom-and-gloom, and jarrin to the reader since it doesn't include background. What does 'prepare' mean? What does 'data loss' mean?
Consider re-wording to something like:
> Be sure to read the entire page before attempting to restore from a backup, as it may have unintendd consequences.
docs/storage_recovery.md
<https://reviews.apache.org/r/26851/#comment97642>
"requires full scheduler outage" is ambiguous, and sounds scary. How about "requires all schedulers to be taken down temporarily while restoring."
docs/storage_recovery.md
<https://reviews.apache.org/r/26851/#comment97645>
I'd also add:
> Usually, it is a bad idea to attempt to restore an update that is not extremely recent, usually less then a few hours old. This is because the scheduler will expect the cluster to look exactly as the backup does, so any tasks that have been rescheduled since the backup was taken will be killed.
docs/storage_recovery.md
<https://reviews.apache.org/r/26851/#comment97647>
Is the "and disable..." part necessary? Presumably an operator knows to use the process manager to stop a scheduler.
docs/storage_recovery.md
<https://reviews.apache.org/r/26851/#comment97648>
Please elaborate on "concurrent access"
docs/storage_recovery.md
<https://reviews.apache.org/r/26851/#comment97649>
Please elaobrate on the purpose of "prevent[ing] accidental task GC".
docs/storage_recovery.md
<https://reviews.apache.org/r/26851/#comment97655>
Some prose in each of these sections would help a lot to improve flow of the document. For example, at this point i'm not clear if i'm assessing the damage of the scheduler configuration changes i made in 'preparation'.
docs/storage_recovery.md
<https://reviews.apache.org/r/26851/#comment97651>
Avoid falling back to HTML if you can. In this case, triple backticks will do the same thing.
docs/storage_recovery.md
<https://reviews.apache.org/r/26851/#comment97657>
How do i know the integrity of the replicated log?
- Bill Farner
On Oct. 17, 2014, 1:51 a.m., Maxim Khutornenko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26851/
> -----------------------------------------------------------
>
> (Updated Oct. 17, 2014, 1:51 a.m.)
>
>
> Review request for Aurora, Kevin Sweeney and Bill Farner.
>
>
> Bugs: AURORA-839
> https://issues.apache.org/jira/browse/AURORA-839
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Summarizing recovery from backup steps.
>
>
> Diffs
> -----
>
> docs/storage_recovery.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/26851/diff/
>
>
> Testing
> -------
>
> https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage_recovery.md
>
>
> Thanks,
>
> Maxim Khutornenko
>
>
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Maxim Khutornenko <ma...@apache.org>.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 22
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line22>
> >
> > s/is summarizing/summarizes/
Done.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 23
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line23>
> >
> > s/ in a datacenter//
Done.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 32
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line32>
> >
> > Consider consistently using the term "Replicated log" instead of "Mesos log" or "Mesos replicated log". The rest is implementation detail.
Done.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 34
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line34>
> >
> > Remove - redundant to the section title.
Dropped.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 42
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line42>
> >
> > s/drive/disk/
Done.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 46
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line46>
> >
> > ZooKeeper (capital K)
Done.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 56
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line56>
> >
> > "The interval on which the scheduler writes local storage backups. The default is every hour."
Done.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 60
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line60>
> >
> > Directory to write backups to.
Done.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 63
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line63>
> >
> > s/Max/Maximum/
> >
> > s/them oldest first/the oldest backup(s)/
Done.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 79
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line79>
> >
> > Comment not addressed in the last round:
> >
> > "requires full scheduler outage" is ambiguous, and sounds scary. How about "requires all schedulers to be taken down temporarily while restoring."
Done.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 78
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line78>
> >
> > s/Mesos //
Obviated by the decision to use "Mesos replicated log" as suggested above.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 90
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line90>
> >
> > s/from backup/from a backup/
Done.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 100
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line100>
> >
> > Before going into the configuration changes, can you offer a little context? Without any background, they don't offer any guidance about why we're doing these seemingly-strange things.
Added.
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 113
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line113>
> >
> > s/from backup/from a backup/
removed
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 118
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line118>
> >
> > This needs more detail, especially because the scheduler exhibits this behavior out of the box (every 24 hours).
removed
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 138
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line138>
> >
> > Ok - that was the first thing i thought when reading this: why not make the procedure completely uniform. I think this will be much easier to follow if there is only one path to follow, considering that the reader needing this doc is likely already in a panic.
removed
> On Oct. 20, 2014, 5:56 p.m., Bill Farner wrote:
> > docs/storage-config.md, line 144
> > <https://reviews.apache.org/r/26851/diff/3/?file=724940#file724940line144>
> >
> > s/ snapshot//
done.
- Maxim
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/#review57364
-----------------------------------------------------------
On Oct. 17, 2014, 9:47 p.m., Maxim Khutornenko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26851/
> -----------------------------------------------------------
>
> (Updated Oct. 17, 2014, 9:47 p.m.)
>
>
> Review request for Aurora, Kevin Sweeney and Bill Farner.
>
>
> Bugs: AURORA-839
> https://issues.apache.org/jira/browse/AURORA-839
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Summarizing recovery from backup steps.
>
>
> Diffs
> -----
>
> docs/storage-config.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/26851/diff/
>
>
> Testing
> -------
>
> https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage-config.md
>
>
> Thanks,
>
> Maxim Khutornenko
>
>
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Bill Farner <wf...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/#review57364
-----------------------------------------------------------
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98014>
s/is summarizing/summarizes/
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98015>
s/ in a datacenter//
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98017>
Consider consistently using the term "Replicated log" instead of "Mesos log" or "Mesos replicated log". The rest is implementation detail.
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98019>
Remove - redundant to the section title.
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98020>
s/drive/disk/
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98021>
ZooKeeper (capital K)
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98023>
"The interval on which the scheduler writes local storage backups. The default is every hour."
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98027>
Directory to write backups to.
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98028>
s/Max/Maximum/
s/them oldest first/the oldest backup(s)/
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98030>
s/Mesos //
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98029>
Comment not addressed in the last round:
"requires full scheduler outage" is ambiguous, and sounds scary. How about "requires all schedulers to be taken down temporarily while restoring."
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98031>
s/from backup/from a backup/
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98035>
Before going into the configuration changes, can you offer a little context? Without any background, they don't offer any guidance about why we're doing these seemingly-strange things.
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98036>
s/from backup/from a backup/
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98039>
This needs more detail, especially because the scheduler exhibits this behavior out of the box (every 24 hours).
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98043>
Ok - that was the first thing i thought when reading this: why not make the procedure completely uniform. I think this will be much easier to follow if there is only one path to follow, considering that the reader needing this doc is likely already in a panic.
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment98040>
s/ snapshot//
- Bill Farner
On Oct. 17, 2014, 9:47 p.m., Maxim Khutornenko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26851/
> -----------------------------------------------------------
>
> (Updated Oct. 17, 2014, 9:47 p.m.)
>
>
> Review request for Aurora, Kevin Sweeney and Bill Farner.
>
>
> Bugs: AURORA-839
> https://issues.apache.org/jira/browse/AURORA-839
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Summarizing recovery from backup steps.
>
>
> Diffs
> -----
>
> docs/storage-config.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/26851/diff/
>
>
> Testing
> -------
>
> https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage-config.md
>
>
> Thanks,
>
> Maxim Khutornenko
>
>
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Maxim Khutornenko <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/
-----------------------------------------------------------
(Updated Oct. 24, 2014, 12:18 a.m.)
Review request for Aurora, Kevin Sweeney and Bill Farner.
Changes
-------
Rebased.
Bugs: AURORA-839
https://issues.apache.org/jira/browse/AURORA-839
Repository: aurora
Description
-------
Summarizing recovery from backup steps.
Diffs (updated)
-----
docs/deploying-aurora-scheduler.md 5f957c74a03621107edf881a1dd1f75f0c7ac007
docs/storage-config.md PRE-CREATION
Diff: https://reviews.apache.org/r/26851/diff/
Testing
-------
https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage-config.md
Thanks,
Maxim Khutornenko
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Maxim Khutornenko <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/
-----------------------------------------------------------
(Updated Oct. 24, 2014, 12:13 a.m.)
Review request for Aurora, Kevin Sweeney and Bill Farner.
Changes
-------
CR comments.
Bugs: AURORA-839
https://issues.apache.org/jira/browse/AURORA-839
Repository: aurora
Description
-------
Summarizing recovery from backup steps.
Diffs (updated)
-----
docs/deploying-aurora-scheduler.md 380577e0f5a45dd0d521f3379d76febde841927b
docs/storage-config.md PRE-CREATION
Diff: https://reviews.apache.org/r/26851/diff/
Testing
-------
https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage-config.md
Thanks,
Maxim Khutornenko
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Maxim Khutornenko <ma...@apache.org>.
> On Oct. 24, 2014, 12:09 a.m., Kevin Sweeney wrote:
> > docs/storage-config.md, line 81
> > <https://reviews.apache.org/r/26851/diff/5/?file=731753#file731753line81>
> >
> > "restore a backup that is not extremely recent"
Good catch. Thanks.
- Maxim
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/#review58175
-----------------------------------------------------------
On Oct. 24, 2014, 12:05 a.m., Maxim Khutornenko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26851/
> -----------------------------------------------------------
>
> (Updated Oct. 24, 2014, 12:05 a.m.)
>
>
> Review request for Aurora, Kevin Sweeney and Bill Farner.
>
>
> Bugs: AURORA-839
> https://issues.apache.org/jira/browse/AURORA-839
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Summarizing recovery from backup steps.
>
>
> Diffs
> -----
>
> docs/deploying-aurora-scheduler.md 380577e0f5a45dd0d521f3379d76febde841927b
> docs/storage-config.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/26851/diff/
>
>
> Testing
> -------
>
> https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage-config.md
>
>
> Thanks,
>
> Maxim Khutornenko
>
>
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Kevin Sweeney <ke...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/#review58175
-----------------------------------------------------------
Ship it!
docs/storage-config.md
<https://reviews.apache.org/r/26851/#comment99108>
"restore a backup that is not extremely recent"
- Kevin Sweeney
On Oct. 23, 2014, 5:05 p.m., Maxim Khutornenko wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26851/
> -----------------------------------------------------------
>
> (Updated Oct. 23, 2014, 5:05 p.m.)
>
>
> Review request for Aurora, Kevin Sweeney and Bill Farner.
>
>
> Bugs: AURORA-839
> https://issues.apache.org/jira/browse/AURORA-839
>
>
> Repository: aurora
>
>
> Description
> -------
>
> Summarizing recovery from backup steps.
>
>
> Diffs
> -----
>
> docs/deploying-aurora-scheduler.md 380577e0f5a45dd0d521f3379d76febde841927b
> docs/storage-config.md PRE-CREATION
>
> Diff: https://reviews.apache.org/r/26851/diff/
>
>
> Testing
> -------
>
> https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage-config.md
>
>
> Thanks,
>
> Maxim Khutornenko
>
>
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Maxim Khutornenko <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/
-----------------------------------------------------------
(Updated Oct. 24, 2014, 12:05 a.m.)
Review request for Aurora, Kevin Sweeney and Bill Farner.
Changes
-------
Added link from deploying-aurora-scheduler.md
Bugs: AURORA-839
https://issues.apache.org/jira/browse/AURORA-839
Repository: aurora
Description
-------
Summarizing recovery from backup steps.
Diffs (updated)
-----
docs/deploying-aurora-scheduler.md 380577e0f5a45dd0d521f3379d76febde841927b
docs/storage-config.md PRE-CREATION
Diff: https://reviews.apache.org/r/26851/diff/
Testing
-------
https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage-config.md
Thanks,
Maxim Khutornenko
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Maxim Khutornenko <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/
-----------------------------------------------------------
(Updated Oct. 20, 2014, 11:58 p.m.)
Review request for Aurora, Kevin Sweeney and Bill Farner.
Changes
-------
CR comments.
Bugs: AURORA-839
https://issues.apache.org/jira/browse/AURORA-839
Repository: aurora
Description
-------
Summarizing recovery from backup steps.
Diffs (updated)
-----
docs/storage-config.md PRE-CREATION
Diff: https://reviews.apache.org/r/26851/diff/
Testing
-------
https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage-config.md
Thanks,
Maxim Khutornenko
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Maxim Khutornenko <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/
-----------------------------------------------------------
(Updated Oct. 17, 2014, 9:47 p.m.)
Review request for Aurora, Kevin Sweeney and Bill Farner.
Changes
-------
Merged storage config and recovery docs.
Bugs: AURORA-839
https://issues.apache.org/jira/browse/AURORA-839
Repository: aurora
Description
-------
Summarizing recovery from backup steps.
Diffs (updated)
-----
docs/storage-config.md PRE-CREATION
Diff: https://reviews.apache.org/r/26851/diff/
Testing (updated)
-------
https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage-config.md
Thanks,
Maxim Khutornenko
Re: Review Request 26851: Adding storage recovery from backup doc.
Posted by Maxim Khutornenko <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26851/
-----------------------------------------------------------
(Updated Oct. 17, 2014, 8:45 p.m.)
Review request for Aurora, Kevin Sweeney and Bill Farner.
Changes
-------
CR comments.
Bugs: AURORA-839
https://issues.apache.org/jira/browse/AURORA-839
Repository: aurora
Description
-------
Summarizing recovery from backup steps.
Diffs (updated)
-----
docs/storage_recovery.md PRE-CREATION
Diff: https://reviews.apache.org/r/26851/diff/
Testing
-------
https://github.com/maxim111333/incubator-aurora/blob/storage_config_doc/docs/storage_recovery.md
Thanks,
Maxim Khutornenko