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

RC: RDI In the Cloud #1066

Draft
wants to merge 68 commits into
base: main
Choose a base branch
from
Draft

RC: RDI In the Cloud #1066

wants to merge 68 commits into from

Conversation

cmilesb
Copy link
Contributor

@cmilesb cmilesb commented Jan 16, 2025

@cmilesb cmilesb added do not merge yet rc Redis Cloud docs labels Jan 16, 2025
@cmilesb cmilesb self-assigned this Jan 16, 2025
Copy link
Contributor

github-actions bot commented Jan 16, 2025

Staging links:
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/define
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/setup
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/view-edit
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/view-edit
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/define
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/view-edit
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/define
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/view-edit
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/define
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/setup
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/define
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/setup
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/view-edit
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/define
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/view-edit
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/define
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/view-edit
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/define
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/view-edit
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/define
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/view-edit
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/define
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/setup
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/define
https://redis.io/docs/staging/preview-rc-rdi/operate/rc/databases/rdi/view-edit

@cmilesb cmilesb mentioned this pull request Jan 16, 2025
@cmilesb cmilesb requested a review from yaronp68 February 3, 2025 18:49
@yaronp68
Copy link
Collaborator

yaronp68 commented Feb 4, 2025

this is great! Now sure if during private preview we should publish it online but more for internal use (staging or similar)
I think we should only expose when public preview with required changes

@cmilesb
Copy link
Contributor Author

cmilesb commented Feb 4, 2025

@yaronp68 It's on staging, so that's what I've been giving people. I won't be merging this PR until we get to a public preview or something similar.

@ZdravkoDonev-redis
Copy link
Collaborator

Just for visibility - this is the Cloud Public Preview initiative - https://redislabs.atlassian.net/issues/RED-147332?filter=13199

Once it's released, we can merge this PR.
Most likely we'll be doing updates to this.

@ZdravkoDonev-redis
Copy link
Collaborator

@cmilesb Can we also create a FAQ page for the RDI Cloud somewhere? Maybe as as subsection?

The first question that we can add is:

Q: Does RDI need to have an AWS PrivateLink or it can also be configured to have VPC peering instead?
A: RDI only works with AWS private link. This is the safest way to protect your source database.

1. [Create an encryption key](#create-encryption-key) using AWS Key Management Service with the right permissions.
1. [Create a secret](#create-database-credentials-secret) containing the source database credentials encrypted using that key.

### Create encryption key
Copy link
Collaborator

Choose a reason for hiding this comment

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

@cmilesb Lets put a big note here. 2 people from Redis already hit this issue when they are using Amazon RDS for the source database. And the issue they had was that they were using the AWS KMS managed key provided while creating the RDS instance.

But, You can't access an AWS KMS managed key-encrypted secret from another account.

The error that our team has seen was:

You can't access a secret from a different AWS account if you encrypt the secret with the default KMS service key.

The issue is described here - https://repost.aws/knowledge-center/secrets-manager-cross-account-key

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ZdravkoDonev-redis Just to clarify - this is a source database requirement? So should this be located under Prerequisites?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge yet rc Redis Cloud docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants