Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use SRV records instead? #1

Open
OJFord opened this issue Oct 19, 2021 · 1 comment
Open

Use SRV records instead? #1

OJFord opened this issue Oct 19, 2021 · 1 comment

Comments

@OJFord
Copy link
Owner

OJFord commented Oct 19, 2021

It was pointed out to me that a SRV record would be appropriate here, and perhaps better suited than TXT if for nothing more than being well-defined and having pretty much the right semantics.

It's.. unfortunate then that I named this ipvs-txt-lb! Well, I thought I'd leave an issue here to say I'm aware of it, and for any strong feelings to be noted below. Especially interested if anyone's using this and has a particular use for them being in SRV as opposed to TXT, (currently necessitating ~duplicates) or a problem caused by them being TXT, etc., beyond aesthetic tidiness.

(I suppose checking both, or having it configurable, would also be an option.)

@OJFord
Copy link
Owner Author

OJFord commented Jul 2, 2022

Add this as an option.

Infer it from _.+\._tcp\..+ addresses, or else a --srv flag (and disable inference with --no-srv).

Should pick the highest priority results from dig ... SRV +short, and add to ipvsadm with their --weights.

I already implemented in python with srvlookup for another purpose - but don't want to add the dependency (on python and a package) here; since ipvsadm itself handles the weighting anyway it should be pretty trivial to sort on priority, select the top, and feed in.

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

No branches or pull requests

1 participant