Skip to content
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

[PLAT-14912] docs warning for install root as subdirectory #23470

Merged
merged 4 commits into from
Aug 12, 2024

Conversation

chidmuthu
Copy link
Contributor

@chidmuthu chidmuthu commented Aug 12, 2024

PLAT-14913

@chidmuthu chidmuthu changed the title [PLAT-14912[ docs warning for install root as subdirectory [PLAT-14912] docs warning for install root as subdirectory Aug 12, 2024
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link

netlify bot commented Aug 12, 2024

Deploy Preview for infallible-bardeen-164bc9 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit cd19faa
🔍 Latest deploy log https://app.netlify.com/sites/infallible-bardeen-164bc9/deploys/66ba37cdad1e3000084ad5b1
😎 Deploy Preview https://deploy-preview-23470--infallible-bardeen-164bc9.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ddhodge ddhodge added the area/documentation Documentation needed label Aug 12, 2024
@@ -48,6 +48,10 @@ To migrate in place, do the following:

Note that the `installRoot` (by default `/opt/ybanywhere`) in `yba-ctl.yml` needs to be different from the Replicated Storage Path (by default `/opt/yugabyte`). If they are set to the same value, the migration will fail. You can delete `yba-ctl.yml` and try again.

{{< warning title="Install Root" >}}
The `installRoot` must **_not_** be a subdirectory of the Replicated Storage Path (`/opt/yugabyte` and `/opt/yugabyte/ybanywhere`) or upon completion of `replicated-migrate finish` all data will be lost while cleaning up Replicated.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The `installRoot` must **_not_** be a subdirectory of the Replicated Storage Path (`/opt/yugabyte` and `/opt/yugabyte/ybanywhere`) or upon completion of `replicated-migrate finish` all data will be lost while cleaning up Replicated.
The `installRoot` must _not_ be a subdirectory of the Replicated Storage Path (`/opt/yugabyte` and `/opt/yugabyte/ybanywhere`). If `installRoot` is in the Replicated Storage Path, `replicated-migrate finish` will delete all your data while cleaning up Replicated.

@@ -48,6 +48,10 @@ To migrate in place, do the following:

Note that the `installRoot` (by default `/opt/ybanywhere`) in `yba-ctl.yml` needs to be different from the Replicated Storage Path (by default `/opt/yugabyte`). If they are set to the same value, the migration will fail. You can delete `yba-ctl.yml` and try again.

{{< warning title="Install Root" >}}
The `installRoot` must **_not_** be a subdirectory of the Replicated Storage Path (`/opt/yugabyte` and `/opt/yugabyte/ybanywhere`) or upon completion of `replicated-migrate finish` all data will be lost while cleaning up Replicated.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The `installRoot` must **_not_** be a subdirectory of the Replicated Storage Path (`/opt/yugabyte` and `/opt/yugabyte/ybanywhere`) or upon completion of `replicated-migrate finish` all data will be lost while cleaning up Replicated.
The `installRoot` must _not_ be a subdirectory of the Replicated Storage Path (`/opt/yugabyte` and `/opt/yugabyte/ybanywhere`). If `installRoot` is in the Replicated Storage Path, `replicated-migrate finish` will delete all your data while cleaning up Replicated.

@chidmuthu chidmuthu merged commit a2b5495 into yugabyte:master Aug 12, 2024
0 of 2 checks passed
jasonyb pushed a commit that referenced this pull request Aug 13, 2024
Summary:
 bd1e19e [PLAT-14835]: Add extra transient YCQL index tables in xClusterTableConfig during GET calls and metrics.
 2715c58 [docs] updated yb version with shortcode (#23456)
 a2b5495 [PLAT-14912] docs warning for install root as subdirectory (#23470)
 53365b1 [#23422] YSQL: Disable random warmup feature by default for connection manager tests
 09d6e96 [#22876][#22835][#22773] CDCSDK: Add null checks & remove optimisation in table removal from CDC stream
 69d4052 [#22862] XCluster: Improving XCluster Index Base WAL Retention Policy
 706e97d [#23460] DocDB: Read vector index data
 b1a90b9 [#23428] docdb: Remove non-transactional snapshot code
 581648f [PLAT-13957] Update RBAC wrapper for xCluster DR
 fbaf945 Whitepaper on migration (#23468)
 f6af2f5 [PLAT-13936] Upgrade Grpc and its dependencies to fix CVEs
 d7027fe [PLAT-14892] Update PITR configuration step text
 92804ac [PLAT-14760] Use new xCluster sync API
 3cb8faf [PLAT-11243] Upgrade python requests to latest version
 Excluded: a036313 [#23070] YSQL, ASH: Replace ysql_session_id with pid
 4d2f71f [PLAT-14882] Retrieve userName from the attribute lists in case not found in dn
 99489c0 [PLAT-14909] Upgrade YBC version to 2.2.0.0-b4

Test Plan: Jenkins: rebase: pg15-cherrypicks

Reviewers: jason, tfoucher

Tags: #jenkins-ready

Differential Revision: https://phorge.dev.yugabyte.com/D37272
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Documentation needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants