Skip to content

Fix the direct VPCE error about HA#4817

Open
lukeknep wants to merge 4 commits into
mainfrom
direct-vpce-ha-fix
Open

Fix the direct VPCE error about HA#4817
lukeknep wants to merge 4 commits into
mainfrom
direct-vpce-ha-fix

Conversation

@lukeknep

@lukeknep lukeknep commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Fix an error (made by me, recently) in the Networking docs and explain how Workers can use the "VPC Endpoint" in their config settings, even if they use HA, but the Worker should use the corresponding VPC Endpoint for the region it is in.

┆Attachments: EDU-6646 Fix the direct VPCE error about HA

@lukeknep lukeknep requested a review from a team as a code owner July 3, 2026 03:58
@vercel

vercel Bot commented Jul 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
temporal-documentation Ready Ready Preview, Comment Jul 6, 2026 4:08am

Request Review

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

📖 Docs PR preview links

@@ -202,24 +212,82 @@ The complete guidance — including single-cloud (AWS-only) HA, multi-cloud HA (

## Direct VPCE targeting without per-Namespace DNS {/* #direct-vpce */}

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.

📝 [vale] reported by reviewdog 🐶
[Temporal.Headings] 'Direct VPCE targeting without per-Namespace DNS {/* #direct-vpce */}' should use sentence-style capitalization.

class HPRIM,HSEC region
```

## Adding PrivateLink from additional AWS accounts

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.

📝 [vale] reported by reviewdog 🐶
[Temporal.Headings] 'Adding PrivateLink from additional AWS accounts' should use sentence-style capitalization.

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