Skip to content
This repository has been archived by the owner on Sep 18, 2020. It is now read-only.

locksmithctl: support for DNS SRV discovery mechanism #121

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

dmlambea
Copy link

@dmlambea dmlambea commented Jun 8, 2017

New commandline flag -discovery-srv which allows specifying a DNS domain to be queried for 'etcd-client' SRV records, both for HTTPS and HTTP schemas. This new flag is only honoured if no '-endpoint' flags are specified, for backwards compatibility with current configurations.

A successful DNS query supersedes any LOCKSMITHD_ENDPOINT or LOCKSMITHCTL_ENDPOINT envvars.

The DNS query function is derived from etcd's own function and then adapted to locksmith's requirements.

@dmlambea dmlambea changed the title Add support for DNS SRV discovery mechanism locksmith: support for DNS SRV discovery mechanism Aug 1, 2017
@dmlambea dmlambea changed the title locksmith: support for DNS SRV discovery mechanism locksmithctl: support for DNS SRV discovery mechanism Aug 1, 2017
@dmlambea
Copy link
Author

Bump

@dmlambea
Copy link
Author

@crawford, I see this PR has been quiet for several months. Is there anything else I can do to improve or fix the PR?

@crawford
Copy link
Contributor

@bgilbert can you have someone on the team take a look at this? It seems straightforward enough.

@dmlambea
Copy link
Author

@crawford @bgilbert Any news? Can we proceed or do you think something should be fixed/improved before continuing?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants