Skip to content

Commit

Permalink
UPSTREAM: <carry>: Bug 1893156: include / prefix in instance ID output
Browse files Browse the repository at this point in the history
When we want to read an instance ID from the metadata service, cloud provider
doesn't include "/" prefix, which is required for successful parsing of
provider the ID later.
This commit adds the missing "/" prefix to the output.

Cherry-picked from openshift/kubernetes#343

Origin-commit: 65347c66667fe6f29819b65693221d3de56e9caa

Kubernetes-commit: eb7342883d232a9876390a0eab4344894a05b81c
  • Loading branch information
Fedosin authored and k8s-publishing-bot committed Nov 2, 2020
1 parent 6996040 commit 632cbd3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion openstack/openstack_instances.go
Expand Up @@ -221,7 +221,7 @@ func (i *Instances) InstanceID(ctx context.Context, name types.NodeName) (string
}
localName := types.NodeName(md.Name)
if localName == name {
return md.UUID, nil
return "/" + md.UUID, nil
}

srv, err := getServerByName(i.compute, name)
Expand Down

0 comments on commit 632cbd3

Please sign in to comment.