Skip to content
This repository has been archived by the owner on Dec 5, 2017. It is now read-only.

Add external IP address to NodeSpec #725

Open
2 tasks
sttts opened this issue Jan 5, 2016 · 8 comments
Open
2 tasks

Add external IP address to NodeSpec #725

sttts opened this issue Jan 5, 2016 · 8 comments

Comments

@sttts
Copy link

sttts commented Jan 5, 2016

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.

@jdef
Copy link

jdef commented Jan 6, 2016

related conversation, though more concerned with determining the correct "internal IP" vs "external IP": d2iq-archive/mesos-dns#379

@jdef
Copy link

jdef commented Jan 6, 2016

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

@sttts
Copy link
Author

sttts commented Jan 6, 2016

Because they are blackbox tests from the internet.

@jdef
Copy link

jdef commented Jan 6, 2016

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
Copy link

jdef commented Jan 6, 2016

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

@sttts
Copy link
Author

sttts commented Jan 6, 2016

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

@jdef
Copy link

jdef commented Jan 6, 2016

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)

@sttts
Copy link
Author

sttts commented Jan 6, 2016

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

@jdef jdef modified the milestones: v0.8.0, v0.7.2 Jan 20, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants