Skip to content

Support ID-based selection of host via create API#8181

Merged
zhoumeina merged 2 commits intovmware:masterfrom
zjs:topic/ref-host-by-id
Aug 17, 2018
Merged

Support ID-based selection of host via create API#8181
zhoumeina merged 2 commits intovmware:masterfrom
zjs:topic/ref-host-by-id

Conversation

@zjs
Copy link
Member

@zjs zjs commented Aug 7, 2018

Allow a stand-alone host (a host not in a cluster) to be referenced
by ID during API-based VCH creation.

Fixes #6711

[specific ci=Group23-VIC-Machine-Service]

Allow a stand-alone host (a host not in a cluster) to be referenced
by ID during API-based VCH creation.
@zjs zjs self-assigned this Aug 7, 2018
@zjs zjs requested a review from a team as a code owner August 7, 2018 22:49
Copy link
Contributor

@hickeng hickeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. Approved conditional on fixing the unit tests.

@zhoumeina zhoumeina merged commit 67f248c into vmware:master Aug 17, 2018
zjs added a commit to zjs/vic that referenced this pull request Aug 28, 2018
* Support ID-based selection of host via create API

Allow a stand-alone host (a host not in a cluster) to be referenced
by ID during API-based VCH creation.

* fixup! Support ID-based selection of host via create API

(cherry picked from commit 67f248c)

Conflicts:
	lib/apiservers/service/restapi/handlers/decode/references.go
	lib/apiservers/service/restapi/handlers/decode/references_test.go
	lib/apiservers/service/restapi/handlers/vch_create.go
zjs added a commit to zjs/vic that referenced this pull request Aug 29, 2018
* Support ID-based selection of host via create API

Allow a stand-alone host (a host not in a cluster) to be referenced
by ID during API-based VCH creation.

* fixup! Support ID-based selection of host via create API

(cherry picked from commit 67f248c)

Conflicts:
	lib/apiservers/service/restapi/handlers/decode/references.go
	lib/apiservers/service/restapi/handlers/decode/references_test.go
	lib/apiservers/service/restapi/handlers/vch_create.go
zjs added a commit to zjs/vic that referenced this pull request Aug 30, 2018
…re#8239)

* Support ID-based selection of host via create API

Allow a stand-alone host (a host not in a cluster) to be referenced
by ID during API-based VCH creation.

* fixup! Support ID-based selection of host via create API

(cherry picked from commit 67f248c)

Conflicts:
	lib/apiservers/service/restapi/handlers/decode/references.go
	lib/apiservers/service/restapi/handlers/decode/references_test.go
	lib/apiservers/service/restapi/handlers/vch_create.go
zjs added a commit that referenced this pull request Aug 30, 2018
* Support ID-based selection of host via create API

Allow a stand-alone host (a host not in a cluster) to be referenced
by ID during API-based VCH creation.

* fixup! Support ID-based selection of host via create API

(cherry picked from commit 67f248c)

Conflicts:
	lib/apiservers/service/restapi/handlers/decode/references.go
	lib/apiservers/service/restapi/handlers/decode/references_test.go
	lib/apiservers/service/restapi/handlers/vch_create.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants