-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MRR: test & execute #643
base: main
Are you sure you want to change the base?
MRR: test & execute #643
Conversation
In general, team docs want to have a new line after every sentence. But maybe we should apply that after reviewing the actual content. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some initial feedback, we can go through it together!
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
|
||
An instance can either be migrated to a brand-new instance or an instance that is already running the latest version of Aura and that fits the memory and storage configuration of the old instance. This means that if you choose the second option the instance you want to migrate to has to have at least the same amount of memory and storage as the old one. | ||
|
||
Note that cloning into an existing instance will overwrite all of its existing data and name. This action cannot be undone and may take longer than cloning to a new instance. It is advised to take a snapshot and download it before continuing if you still have data on that instance you want to keep. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Snapshot or backup, not sure what is correct? (2x in this doc)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From what I am seeing, the Aura console also plays fast and loose with it. So both seem to be okay and mean somewhat the same. The place where you would go to to download them is called Snapshots so I am gonna have it called that everywhere I think
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove the border around the image (2x in this PR)
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
Co-authored-by: Alexander Bouriakov <alexander.bouriakov@neo4j.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some comments, mainly editorial style suggestions.
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
modules/ROOT/pages/auradb/managing-databases/migration-readiness.adoc
Outdated
Show resolved
Hide resolved
|
||
Once you are done with testing, click the "Finalize live migration" button and the following dialog will be displayed. | ||
|
||
image::mrr-finalize-live.png[width=500] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this image is needed.
Co-authored-by: Jessica Wright <49636617+AlexicaWright@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you for addressing all my comments @SaschaPeukert !
This PR includes documentation updates Updated pages: |
No description provided.