{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/5244","labels_url":"https://api.github.com/repos/apache/spark/issues/5244/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/5244/comments","events_url":"https://api.github.com/repos/apache/spark/issues/5244/events","html_url":"https://github.com/apache/spark/pull/5244","id":64969891,"number":5244,"title":"SPARK-6588: Add --private-ips flag to EC2 script","user":{"login":"mdagost","id":1334817,"avatar_url":"https://avatars.githubusercontent.com/u/1334817?v=3","gravatar_id":"","url":"https://api.github.com/users/mdagost","html_url":"https://github.com/mdagost","followers_url":"https://api.github.com/users/mdagost/followers","following_url":"https://api.github.com/users/mdagost/following{/other_user}","gists_url":"https://api.github.com/users/mdagost/gists{/gist_id}","starred_url":"https://api.github.com/users/mdagost/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mdagost/subscriptions","organizations_url":"https://api.github.com/users/mdagost/orgs","repos_url":"https://api.github.com/users/mdagost/repos","events_url":"https://api.github.com/users/mdagost/events{/privacy}","received_events_url":"https://api.github.com/users/mdagost/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-03-28T17:51:46Z","updated_at":"2015-03-28T17:54:58Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/5244","html_url":"https://github.com/apache/spark/pull/5244","diff_url":"https://github.com/apache/spark/pull/5244.diff","patch_url":"https://github.com/apache/spark/pull/5244.patch"},"body":"The `spark_ec2.py` script currently references the `ip_address` and `public_dns_name` attributes of an instance. On private networks, these fields aren't set, so we have problems.\r\n\r\nThis PR introduces a `--private-ips` flag that instead refers to the `private_ip_address` attribute in both cases."},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/87273976","html_url":"https://github.com/apache/spark/pull/5244#issuecomment-87273976","issue_url":"https://api.github.com/repos/apache/spark/issues/5244","id":87273976,"user":{"login":"srowen","id":822522,"avatar_url":"https://avatars.githubusercontent.com/u/822522?v=3","gravatar_id":"","url":"https://api.github.com/users/srowen","html_url":"https://github.com/srowen","followers_url":"https://api.github.com/users/srowen/followers","following_url":"https://api.github.com/users/srowen/following{/other_user}","gists_url":"https://api.github.com/users/srowen/gists{/gist_id}","starred_url":"https://api.github.com/users/srowen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/srowen/subscriptions","organizations_url":"https://api.github.com/users/srowen/orgs","repos_url":"https://api.github.com/users/srowen/repos","events_url":"https://api.github.com/users/srowen/events{/privacy}","received_events_url":"https://api.github.com/users/srowen/received_events","type":"User","site_admin":false},"created_at":"2015-03-28T17:54:58Z","updated_at":"2015-03-28T17:54:58Z","body":"You could take this all the way to implement what's described in SPARK-6220 as a more general version of this. And I think it would also be considered a resolution to SPARK-5242 and SPARK-5246. This overlaps with existing work at https://github.com/apache/spark/pull/4038 (see also https://github.com/mesos/spark-ec2/pull/91 and https://github.com/mesos/spark-ec2/pull/92) It'd be great to get a resolution to all of this since it keeps coming up in duplicate."}}