search

aws-samples / sagemaker-studio-sparkmagic-lib

MIT No Attribution
7 stars 8 forks source link

Allow surfacing of primary node public dns #10

Closed varun196 closed 2 years ago

varun196 commented 2 years ago

Description

Allow surfacing of primary node public dns

Motivation

Primary node public DNS is needed to connect to EMR cluster in public subnet.

Testing Done


============================================================================================================= short test summary info =============================================================================================================
PASSED sagemaker_studio_sparkmagic_lib/tests/emr_test.py::test_get_cluster_happy_case_non_kerberos
PASSED sagemaker_studio_sparkmagic_lib/tests/emr_test.py::test_get_cluster_happy_case_non_kerberos_with_pagination
PASSED sagemaker_studio_sparkmagic_lib/tests/emr_test.py::test_get_cluster_happy_case_kerberos
PASSED sagemaker_studio_sparkmagic_lib/tests/emr_test.py::test_get_bad_cluster
PASSED sagemaker_studio_sparkmagic_lib/tests/emr_test.py::test_cluster_dedicated_krb_cluster
PASSED sagemaker_studio_sparkmagic_lib/tests/emr_test.py::test_cross_realm_krb_cluster
PASSED sagemaker_studio_sparkmagic_lib/tests/emr_test.py::test_external_kdc_cluster
PASSED sagemaker_studio_sparkmagic_lib/tests/kerberos_test.py::test_generated_krb_conf
PASSED sagemaker_studio_sparkmagic_lib/tests/utils_test.py::test_get_domain_search_without_search_line
PASSED sagemaker_studio_sparkmagic_lib/tests/utils_test.py::test_get_domain_search_happy_case
PASSED sagemaker_studio_sparkmagic_lib/tests/utils_test.py::test_get_domain_search_resolv_does_not_exist
PASSED sagemaker_studio_sparkmagic_lib/tests/utils_test.py::test_get_emr_endpoint_url
=============================================================================================================== 12 passed in 0.63s ================================================================================================================```

**Backwards Compatibility Criteria (if any)**

N/A

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.