Skip to content

Find NLB IPs the documented way#605

Closed
daxelrod-rh wants to merge 1 commit intorh-mobb:mainfrom
daxelrod-rh:private-nlb-improve
Closed

Find NLB IPs the documented way#605
daxelrod-rh wants to merge 1 commit intorh-mobb:mainfrom
daxelrod-rh:private-nlb-improve

Conversation

@daxelrod-rh
Copy link
Copy Markdown
Contributor

DNS lookups are not a reliable way to find all of an NLB's listener IPs. There is no guarantee that the DNS server will give your machine all valid listeners, and there may be more than two.

Instead, use an AWS documented technique to find all network interfaces associated with an NLB to get those network interfaces' IPs. Use those IPs for the target group for the public NLB.

DNS lookups are not a reliable way to find all of an NLB's listener IPs.
There is no guarantee that the DNS server will give your machine all
valid listeners, and there may be more than two.

Instead, use an AWS documented technique to find all network interfaces
associated with an NLB to get those network interfaces' IPs. Use those
IPs for the target group for the public NLB.
@netlify
Copy link
Copy Markdown

netlify Bot commented Dec 3, 2024

Deploy Preview for rh-cloud-experts ready!

Name Link
🔨 Latest commit e1e3e9f
🔍 Latest deploy log https://app.netlify.com/sites/rh-cloud-experts/deploys/674f6e30202b4800087b9896
😎 Deploy Preview https://deploy-preview-605--rh-cloud-experts.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@michaelryanmcneill
Copy link
Copy Markdown
Contributor

@daxelrod-rh this likely needs both a rebase, as well as a review. Can you please confirm if this change is still needed and if so, can you update the PR for review?

@paulczar
Copy link
Copy Markdown
Contributor

paulczar commented May 5, 2026

Hey @daxelrod-rh,

The fix here is solid—AWS-documented approach is definitely the way to go for NLB IPs instead of DNS lookups.

Closing this one though since it's been sitting in draft for 154 days and no word back when @michaelryanmcneill asked about rebasing and confirming it's still needed. Just want to keep things tidy.

If you want to pick this back up sometime, just rebase it, flip it out of draft, and we'll get it merged. Easy enough to resurrect whenever.

Cheers!

@daxelrod-rh
Copy link
Copy Markdown
Contributor Author

Sorry about that, I must have missed this email in the flurry of notifications after my absence. I wasn't able to reopen this PR, so I created a new one #926

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