Skip to content

Edu 51 privatelink nlb cleanup#37056

Open
kay-kim wants to merge 2 commits into
MaterializeInc:mainfrom
kay-kim:edu-51-privatelink-nlb-cleanup
Open

Edu 51 privatelink nlb cleanup#37056
kay-kim wants to merge 2 commits into
MaterializeInc:mainfrom
kay-kim:edu-51-privatelink-nlb-cleanup

Conversation

@kay-kim

@kay-kim kay-kim commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

The first patch just single-sources existing content where possible. <<-- can ignore/skip
The second patch 6b294d0 covers both (this is the one to review):

@kay-kim kay-kim requested a review from a team as a code owner June 16, 2026 01:02
You **do not need** to specify `AVAILABILITY ZONES` manually — these will
be optimally auto-assigned when none are provided.

- name: nlb-security-group-remarks

@kay-kim kay-kim Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

👉 This content is the single-sourced content to review.

---
headless: true
---
1. Get the IP address of your __INSTANCE__ instance. You'll need this address to register

@kay-kim kay-kim Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

can ignore. just single-sourcing with placeholder variables. Most of the changes in this PR can be ignored.

The actual content to review is in yaml https://github.com/MaterializeInc/materialize/pull/37056/changes#r3417506456


e. Click next, and register the respective __INSTANCE__ instance to the target group using its IP address.

1. #### Verify security groups and health checks

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Moved this down after creating NLB since we refer to NLBs before we create the NLBs. (this is now consistent with order in the privatelink kafka steps).

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