CLOUDSTACK-9566 instance-id metadata for baremetal VM returns ID #1738
Conversation
There was a problem hiding this comment.
@SudharmaJain please either update an existing Marvin test case or add one to verify this fix.
|
@blueorangutan package |
|
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-182 |
|
The changes are in baremetal subsystem, which cannot be tested by current test environments. |
|
Code changes LGTM. |
|
LGTM as well |
|
Thanks all, I'll proceed with the merging as build, travis and packaging works, without an explicit Trillian test or BVT as both don't run tests against baremetal and the change is a single line change in the baremetal hypervisor plugin. |
CLOUDSTACK-9566 instance-id metadata for baremetal VM returns IDThere is difference in instance-id metadata across baremetal and other hypervisors. On Baremetal [root@ip-172-17-0-144 ~]# curl http://8.37.203.221/latest/meta-data/instance-id 6021 on Xen [root@ip-172-17-2-103 ~]# curl http://172.17.0.252/latest/meta-data/instance-id cbeb517a-e833-4a0c-b1e8-9ed70200fbbf In both cases it should be vm's uuid. * pr/1738: CLOUDSTACK-9566 instance-id metadata for baremetal VM returns ID Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
…scclouds' Atualizar tarifas do Quota com unidade incorreta Closes apache#1738 See merge request scclouds/scclouds!727
There is difference in instance-id metadata across baremetal and other hypervisors.
On Baremetal
[root@ip-172-17-0-144 ~]# curl http://8.37.203.221/latest/meta-data/instance-id
6021
on Xen
[root@ip-172-17-2-103 ~]# curl http://172.17.0.252/latest/meta-data/instance-id
cbeb517a-e833-4a0c-b1e8-9ed70200fbbf
In both cases it should be vm's uuid.