-
Notifications
You must be signed in to change notification settings - Fork 103
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
Move the DNS/WAN discovery to use the new AuthService #413
Conversation
Hi @aleoli. Thanks for your PR! I am @adamjensenbot.
Make sure this PR appears in the liqo changelog, adding one of the following labels:
|
d03f84a
to
73c4110
Compare
5495e08
to
7fd0408
Compare
7fd0408
to
83bebc7
Compare
internal/discovery/search-domain-operator/search-domain-controller.go
Outdated
Show resolved
Hide resolved
83bebc7
to
46e454c
Compare
/rebase |
46e454c
to
683aa5d
Compare
pkg/discovery/const.go
Outdated
@@ -4,6 +4,7 @@ const ( | |||
ClusterIdLabel = "liqo.io/cluster-id" | |||
AuthTokenLabel = "liqo.io/auth-token" | |||
RemoteIdentityLabel = "liqo.io/remote-identity" | |||
DiscoveryTypeLabel = "liqo.io/discovery-type" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what do you think about adding the discovery
prefix to all these labels?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mean something like "discovery.liqo.io/..."?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if you think it is appropriate for these labels of course
683aa5d
to
032115c
Compare
/merge |
Description
This PR changes the way to do WAN discovery with DNS
Now, we have to discover the AuthService to get foreign cluster information. SRV records are enow pointing to the ingress/service that is exposing the AuthService on the foreign cluster
The registration of the home cluster is simpler than before, no TXT record is needed because those data are retrieved from the new service as for LAN discovery
ref. issue #371