Skip to content

docs(clickpipes): correct ap-southeast-3 static NAT IPs - #6610

Closed
whites11 wants to merge 1 commit into
mainfrom
fix/clickpipes-ap-southeast-3-egress-ips
Closed

docs(clickpipes): correct ap-southeast-3 static NAT IPs#6610
whites11 wants to merge 1 commit into
mainfrom
fix/clickpipes-ap-southeast-3-egress-ips

Conversation

@whites11

@whites11 whites11 commented Aug 3, 2026

Copy link
Copy Markdown
Member

Summary

The ap-southeast-3 (Jakarta) row in the ClickPipes static NAT IPs table lists three
addresses ClickPipes never egresses from, and omits three that it does.

IPs What they actually are
✅ correct today 16.78.195.195, 16.79.88.54, 43.218.184.235 the three NAT gateway EIPs (…-public-{a,b,c}-nat-ip) — unchanged by this PR
❌ listed but wrong 16.78.153.162, 16.79.6.125, 108.137.52.155 public proxy load-balancer EIPs (…-public-{a,b,c}-proxy-ip) — ingress addresses, never a source of outbound ClickPipes traffic
➕ missing 15.232.6.147, 15.232.46.145, 108.137.199.68 the alterNAT instance EIPs (ap-southeast-3-alternat-0/1/2) — where ClickPipes egress traffic in this region actually originates

So a customer in ap-southeast-3 who allowlisted exactly what the docs said would have
three useless entries in their firewall and would be missing the alterNAT egress path.

Only the Jakarta row changes; no other region is touched.

How this was verified

  • aws ec2 describe-addresses in the ClickPipes production account, ap-southeast-3:
    the Name tags cleanly separate …-nat-ip (NAT gateway), ap-southeast-3-alternat-*
    (alterNAT instances) and …-proxy-ip (proxy LB).
  • Cross-checked against the Terraform plan for
    data-plane-infrastructure#23729,
    which starts publishing these same IPs as clickpipes_egress_ips in
    static-ips.json. That plan agrees byte-for-byte with this table for 28 of 30
    documented regions — ap-southeast-3 was one of the two that did not, and this PR
    closes it.

Related: CLP-1115, #6596 (documents the new clickpipes_egress_ips field),
#6605 (adds the missing ap-east-2 row).

🤖 Generated with Claude Code

The ap-southeast-3 row listed the three public proxy load-balancer EIPs
(tagged ...public-{a,b,c}-proxy-ip), which are ingress addresses, and
omitted the three alterNAT instance EIPs that ClickPipes actually egresses
from. Customers allowlisting this row were allowlisting three addresses
ClickPipes never connects out from, while missing the real ones.

Replaces 16.78.153.162 / 16.79.6.125 / 108.137.52.155 with
15.232.6.147 / 15.232.46.145 / 108.137.199.68 (ap-southeast-3-alternat-0/1/2).
The three NAT gateway EIPs already listed are unchanged.

Verified against the live EIP allocations in the ClickPipes production
account and against the static-ips.json Terraform plan (CLP-1115).
@vercel

vercel Bot commented Aug 3, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
clickhouse-docs Error Error Aug 3, 2026 1:17pm
4 Skipped Deployments
Project Deployment Actions Updated (UTC)
clickhouse-docs-jp Ignored Ignored Aug 3, 2026 1:17pm
clickhouse-docs-ko Ignored Ignored Aug 3, 2026 1:17pm
clickhouse-docs-ru Ignored Ignored Aug 3, 2026 1:17pm
clickhouse-docs-zh Ignored Ignored Aug 3, 2026 1:17pm

Request Review

@whites11
whites11 requested review from a team as code owners August 3, 2026 13:10
@whites11 whites11 self-assigned this Aug 3, 2026
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

Thanks for contributing to the ClickHouse documentation!

We’ve moved documentation development to the main ClickHouse/ClickHouse repository. For future documentation changes, please open your pull request there, with changes under the docs/ directory.

We’ll continue reviewing this pull request here during the transition, and it will be transferred as part of the migration. You don’t need to reopen it.

See the documentation README for more information.

@Blargian

Blargian commented Aug 3, 2026

Copy link
Copy Markdown
Member

Closing in favour of ClickHouse/ClickHouse#113109

@Blargian Blargian closed this Aug 3, 2026
govtech42 pushed a commit to mirror81/ClickHouse that referenced this pull request Aug 3, 2026
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.

3 participants