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

[Jul 31] Add cloud region info #47

Merged
merged 7 commits into from
Aug 1, 2024
Merged

[Jul 31] Add cloud region info #47

merged 7 commits into from
Aug 1, 2024

Conversation

shainaraskas
Copy link
Collaborator

@shainaraskas shainaraskas commented Jul 24, 2024

Adds information about available cloud regions: cloud regions

links to cloud region docs in ES project creation docs: link

image

Added links in obs and sec project creation docs as well:
elastic/observability-docs#4093
elastic/security-docs#5604

@elasticdocs
Copy link
Collaborator

elasticdocs commented Jul 24, 2024

🚀 Built elastic-dot-co-docs-preview-docs successfully!

Issues? Visit #next-docs in Slack

@shainaraskas shainaraskas marked this pull request as ready for review July 24, 2024 19:51
@shainaraskas shainaraskas requested a review from a team as a code owner July 24, 2024 19:51
serverless/pages/cloud-regions.mdx Outdated Show resolved Hide resolved
serverless/pages/cloud-regions.mdx Outdated Show resolved Hide resolved
serverless/pages/cloud-regions.mdx Outdated Show resolved Hide resolved
Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

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

Nice addition!

Co-authored-by: Kaarina Tungseth <[email protected]>
@yuvielastic
Copy link

I also see a couple of references of region in the docs; would it make sense to link the new region page to that?

Also, for O11y & Security - Is there a reason why we added the following line:

The region should be as close as possible to the location of your data.

I don't see a reason for this to be mentioned as it's customer choice to choose the region which can be far away from the location of their data. I will recommend to remove this line.

Thoughts?

@shainaraskas
Copy link
Collaborator Author

I also see a couple of references of region in the docs; would it make sense to link the new region page to that?

Also, for O11y & Security - Is there a reason why we added the following line:

The region should be as close as possible to the location of your data.

I don't see a reason for this to be mentioned as it's customer choice to choose the region which can be far away from the location of their data. I will recommend to remove this line.

Thoughts?

hi @yuvielastic - I've taken care of the link additions in these PRs, which depend on this PR to merge before they can pass CI:

elastic/observability-docs#4093
elastic/security-docs#5604

I'll remove the guidance around proximity from those two pages in those PRs as well.

@shainaraskas shainaraskas changed the title Add cloud region info [Jul 31] Add cloud region info Jul 25, 2024
@KOTungseth
Copy link
Contributor

I don't see a reason for this to be mentioned as it's customer choice to choose the region which can be far away from the location of their data. I will recommend to remove this line.

Thoughts?

Yes, this is a customer choice, but we should provide guidance on the best possible region option they should choose. I think we should keep this in.

@yuvielastic
Copy link

Yes, this is a customer choice, but we should provide guidance on the best possible region option they should choose. I think we should keep this in.
@KOTungseth But this statement isn't correct as there are multiple factors that customers can look at to identify the best possible region.

Reasons could be:

  • Data residency requirements
  • Regulatory compliance
  • Latency
  • Cost
  • Disaster recovery needs
  • Proximity (location)

Some of these reasons are something that's not guided by us but chosen by customers (based on their preferences) that we are not privy to.

So either we add these bunch of possible reasons (which would confuse customers more as they may not be necessary thinking from this guidance POV) or to keep things simple, I would suggest simply not to mention. My vote is to keep things simple and not mention the guidance at all.

@shainaraskas
Copy link
Collaborator Author

shainaraskas commented Jul 25, 2024

@KOTungseth + @yuvielastic in the new region page in this PR, we give them limited guidance:

If you aren't sure which region to pick, choose one that is geographically close to you to reduce latency.

it's a little less prescriptive than the statements that appear in the observability and security docs. We can assume that people who need guidance will click through and read the less prescriptive recommendation there.

@KOTungseth
Copy link
Contributor

@KOTungseth + @yuvielastic in the new region page in this PR, we give them limited guidance:

If you aren't sure which region to pick, choose one that is geographically close to you to reduce latency.

it's a little less prescriptive than the statements that appear in the observability and security docs. We can assume that people who need guidance will click through and read the less prescriptive recommendation there.

I'm good with this if @yuvielastic is good with this.

@yuvielastic
Copy link

LGTM - Thanks for improvising on it 😊

yuvielastic
yuvielastic previously approved these changes Jul 29, 2024
Copy link

@yuvielastic yuvielastic left a comment

Choose a reason for hiding this comment

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

Added 2 small comments to remove confusion. Rest LGTM

serverless/pages/cloud-regions.mdx Outdated Show resolved Hide resolved
serverless/pages/cloud-regions.mdx Outdated Show resolved Hide resolved
@yuvielastic yuvielastic dismissed their stale review July 29, 2024 04:18

Added 2 small comments; to make changes before merging/approving.

Copy link
Contributor

@kilfoyle kilfoyle left a comment

Choose a reason for hiding this comment

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

LGTM! 🐎

@yuvielastic
Copy link

As the regions have been launched, can we please merge the documentation?

@shainaraskas shainaraskas merged commit edac08b into main Aug 1, 2024
2 checks passed
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.

6 participants