Skip to content

Conversation

@ahardin-rh
Copy link
Contributor

@ahardin-rh ahardin-rh commented Aug 19, 2016

This is a continuation of the work in #2140

Addresses https://bugzilla.redhat.com/show_bug.cgi?id=1259544

@ahardin-rh
Copy link
Contributor Author

@adellape Can you please eyeball this before we merge? This is verified by QE. I updated all links to xrefs, cleared out commenting, and squashed. Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe: s/Deployment/Deployment Type/

@adellape
Copy link
Contributor

@ahardin-rh Some comments inline, overall looks good tho.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@deads2k can you please take a look at this oc export all output? Is this correct? The formatting seems strange and I am not sure what was originally intended here. Thanks!

@ahardin-rh ahardin-rh force-pushed the tnguyen-PR2140 branch 2 times, most recently from 1b7b492 to 96d9901 Compare August 22, 2016 15:23
@ahardin-rh
Copy link
Contributor Author

@deads2k @adellape I made some updates. PTAL. Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the "yes" and "no's" before were indicating which of these are included. Some are and some aren't. I just didn't know why they had numbers.

@ahardin-rh
Copy link
Contributor Author

@deads2k Thanks for clarifying. This is updated.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not gathered.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not gathered.

list looks good otherwise.

* _build_cfg.yml: ...here.
* admin_guide/backup_restore.adoc: New file.
@ahardin-rh
Copy link
Contributor Author

@deads2k Fixed. Thanks. Also took the list out of code block format and moved to a bulleted list.

@ahardin-rh
Copy link
Contributor Author

[rev_history]
|xref:../admin_guide/backup_restore.adoc#admin-guide-backup-and-restore[Backup and Restore]
|New topic discussing back up (saving state to separate storage) and restore (recreating state from separate storage) at the cluster level.
%

@bfallonf bfallonf modified the milestones: Next Release, Staging Aug 23, 2016
@bfallonf bfallonf modified the milestones: Next Release, Staging, Published - 23/08/2016 Aug 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants