Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15488 )

Change subject: [util] KUDU-3067 add OpenStack metadata detector
......................................................................

[util] KUDU-3067 add OpenStack metadata detector

This patch adds OpenStack metadata detector that works with OpenStack
Nova metadata server (see [1] for details).  In addition, this patch
fixes the existing AWS detector to tell apart a true EC2 instance
from a masquerading OpenStack one [2].

I couldn't get access to an OpenStack instance, but I asked the reporter
of KUDU-3067 to test how it works and report back.

1. https://docs.openstack.org/nova/latest/user/metadata.html#metadata-service
2. https://docs.openstack.org/nova/latest/user/metadata.html#metadata-ec2-format

Change-Id: I84cc6d155ab1fbd7b401f5349d292f46fcac3a34
Reviewed-on: http://gerrit.cloudera.org:8080/15488
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <[email protected]>
Reviewed-by: liusheng <[email protected]>
---
M src/kudu/util/cloud/instance_detector-test.cc
M src/kudu/util/cloud/instance_detector.cc
M src/kudu/util/cloud/instance_metadata.cc
M src/kudu/util/cloud/instance_metadata.h
4 files changed, 89 insertions(+), 10 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Adar Dembo: Looks good to me, approved
  liusheng: Looks good to me, but someone else must approve

--
To view, visit http://gerrit.cloudera.org:8080/15488
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I84cc6d155ab1fbd7b401f5349d292f46fcac3a34
Gerrit-Change-Number: 15488
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: liusheng <[email protected]>

Reply via email to