kubernetes-mesos icon indicating copy to clipboard operation
kubernetes-mesos copied to clipboard

Add external IP address to NodeSpec

Open sttts opened this issue 10 years ago • 8 comments

In order to run conformance tests, it is necessary to have IP addresses which are reachable from the internet. While the public slaves have those IPs, they are not exposed in the NodeSpec yet.

TODO:

  • [ ] investigate where we get this information from. Possibilities: state.json, network interface
  • [ ] on node creation and/or on update in the executor, add the address

HishRisk: it is not clear whether we have this information in the scheduler or controller manager. We have to check whether the state.json includes this information.

NOTE: this ticket may or may not actually be an issue when running tests - needs investigation to determine whether this is really a problem at all for us.

sttts avatar Jan 05 '16 14:01 sttts

related conversation, though more concerned with determining the correct "internal IP" vs "external IP": https://github.com/mesosphere/mesos-dns/issues/379

jdef avatar Jan 06 '16 16:01 jdef

why do we need public IP's to run conformance tests?

jdef avatar Jan 06 '16 16:01 jdef

Because they are blackbox tests from the internet.

sttts avatar Jan 06 '16 17:01 sttts

it's unlikely that state.json includes public IP addresses. I asked around today and was told that there is no good way to get public IPs on DCOS.

jdef avatar Jan 06 '16 18:01 jdef

@sttts can you list one or two conformance tests that depend on this?

jdef avatar Jan 06 '16 18:01 jdef

Nearly all. Guestbook as an example, our smoke test.

sttts avatar Jan 06 '16 18:01 sttts

for guestbook we can use a DNS name to access the frontend service via the api proxy URI, so I'm still unclear why, specifically, we need public IPs. can you elaborate? (I'm probably missing something obvious here)

jdef avatar Jan 06 '16 18:01 jdef

In fact, the guestbook test uses the kubeclient proxy already. Maybe my assumption was wrong.

sttts avatar Jan 06 '16 19:01 sttts