Marton Greber has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23611 )
Change subject: [test] Skip test if hostname has no DNS record ...................................................................... [test] Skip test if hostname has no DNS record With IPv6 support, some tests, that make use of DNS resolution, can end up with error if the ip_config_mode is set to a family for which 'hostname' has no corresponding DNS record. For example, if a hostname resolves to just an IPv4 address on a system, running the test with ip_config_mode as 'IPv6' fails because getaddrinfo doesn't find any DNS record for the provided hostname with desired family set to 'IPv6'. This patch avoids such scenarios by, proactively, checking whether DNS record exists or not and skips the test if it doesn't. Change-Id: Id377c73a752e18324dbd83aad7e613b3ba99ee6b Reviewed-on: http://gerrit.cloudera.org:8080/23611 Reviewed-by: Alexey Serbin <[email protected]> Reviewed-by: Marton Greber <[email protected]> Tested-by: Marton Greber <[email protected]> --- M src/kudu/master/mini_master-test.cc M src/kudu/rpc/rpc-test-base.h M src/kudu/util/net/net_util.cc 3 files changed, 27 insertions(+), 5 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Marton Greber: Looks good to me, but someone else must approve; Verified -- To view, visit http://gerrit.cloudera.org:8080/23611 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id377c73a752e18324dbd83aad7e613b3ba99ee6b Gerrit-Change-Number: 23611 Gerrit-PatchSet: 6 Gerrit-Owner: Ashwani Raina <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]>
