Skip to content

Azure Private Link Serverless - #7595

Open
shainaraskas wants to merge 9 commits into
mainfrom
azure-pl-serverless
Open

Azure Private Link Serverless#7595
shainaraskas wants to merge 9 commits into
mainfrom
azure-pl-serverless

Conversation

@shainaraskas

@shainaraskas shainaraskas commented Jul 27, 2026

Copy link
Copy Markdown
Member

Summary

Todo

  • azure native service caveats

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Elastic Docs AI PR menu

Check the box to run an AI review for this pull request.

Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team.

@github-actions

Copy link
Copy Markdown
Contributor

Elastic Docs Style Checker (Vale)

Summary: 8 warnings found

⚠️ Warnings (8): Fix when the suggestion improves clarity or correctness.
File Line Rule Message
deploy-manage/security/private-connectivity-azure.md 103 Elastic.Spelling 'australiaeast' is a possible misspelling.
deploy-manage/security/private-connectivity-azure.md 104 Elastic.Spelling 'brazilsouth' is a possible misspelling.
deploy-manage/security/private-connectivity-azure.md 105 Elastic.Spelling 'eastus' is a possible misspelling.
deploy-manage/security/private-connectivity-azure.md 107 Elastic.Spelling 'germanywestcentral' is a possible misspelling.
deploy-manage/security/private-connectivity-azure.md 108 Elastic.Spelling 'southeastasia' is a possible misspelling.
deploy-manage/security/private-connectivity-azure.md 109 Elastic.Spelling 'spaincentral' is a possible misspelling.
deploy-manage/security/private-connectivity-azure.md 110 Elastic.Spelling 'swedencentral' is a possible misspelling.
deploy-manage/security/private-connectivity-azure.md 111 Elastic.Spelling 'uaenorth' is a possible misspelling.

The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@shainaraskas
shainaraskas marked this pull request as ready for review July 27, 2026 21:50
@shainaraskas
shainaraskas requested a review from a team as a code owner July 27, 2026 21:50

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Docs review summary

Focus areas

  • Style and clarity: Mostly clean edits (fixed a duplicated "to to", removed a stray "please" is unrelated/pre-existing). Left a nit on an inline-code formatting inconsistency in the shared limitations snippet.
  • Jargon: No unexplained Elastic-internal jargon introduced by this diff; {{serverless-short}}/{{serverless-full}} substitutions are used consistently and expand to full terms.
  • Frontmatter and applies_to: Found one concrete bug — the new sub.example-phz-dn value in restrictions-known-problems.md ("vpce.us-east-1.aws.elastic-cloud.com in AWS") is malformed (has " in AWS" appended) and appears unused in the current diff; see inline comment. Other applies_to/products additions for serverless (azure, private-connectivity, ip-filtering) look consistent with existing patterns in the same files.
  • Content type fit: The expanded how-to/reference content (Requirements sections, region tables, API metadata retrieval) fits the existing page structure and content type for each file.
  • Parent issue satisfaction: Not applicable — no linked issue; PR body describes the doc scope directly and the diff appears to cover the described Azure Serverless expansion, AWS service names, and "AWS only" → "AWS and Azure" corrections.

Notes

  • Vale findings (Elastic.MappedPages warnings) are pre-existing mapped_pages entries, not introduced by this PR, so not flagged individually.
  • Did not flag broken links/anchors, which the docs build validates separately.

Generated by Docs review agent for #7595 · sonnet50 86.3 AIC · ⌖ 3.95 AIC · ⊞ 15.6K

Comment thread deploy-manage/deploy/elastic-cloud/restrictions-known-problems.md
Comment thread deploy-manage/security/_snippets/private-connectivity-limitations-ech.md Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant