Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Identify VIC Appliance VM #1018

Closed
2 tasks
andrewtchin opened this issue Oct 31, 2017 · 9 comments
Closed
2 tasks

Identify VIC Appliance VM #1018

andrewtchin opened this issue Oct 31, 2017 · 9 comments
Labels
product/ova Related to the OVA packaging of vSphere Integrated Containers

Comments

@andrewtchin
Copy link
Contributor

andrewtchin commented Oct 31, 2017

User Statement:

As a UI I need to be able to identify which VMs are VIC Appliances.

Details:
Related to #920
http://pubs.vmware.com/vsphere-6-5/index.jsp#com.vmware.vspsdk.apiref.doc/vim.ExtensibleManagedObject.html?resultof=%2522%2573%2565%2574%2543%2575%2573%2574%256f%256d%2556%2561%256c%2575%2565%2522%2520%2522%2573%2565%2574%2563%2575%2573%2574%256f%256d%2576%2561%256c%2575%2522%2520

Get the VirtualMachine object and set a custom value on it
Tag should be on key identifier with value vic-appliance-<BUILD_NUMBER>
Figure out build number format with @jooskim and @mdharamadas1

Acceptance Criteria:

  • Use API to tag the VM during the appliance initialization process
  • Same process needs to apply during all cases of initialization - getting started page, upgrade, registration API
@andrewtchin andrewtchin added product/ova Related to the OVA packaging of vSphere Integrated Containers priority/high labels Oct 31, 2017
@zjs zjs added this to the Sprint 20 Lifecycle milestone Oct 31, 2017
@zjs
Copy link
Member

zjs commented Nov 2, 2017

Can we re-use the tagging used by VCHs to discover the OVA?

@ghost ghost self-assigned this Nov 2, 2017
@jooskim
Copy link
Contributor

jooskim commented Nov 3, 2017

@andrewtchin @morris-jason Another request here: is it possible to set another CustomFieldDef (see doc here) for IP address of the VM? After perusing the documentation on CustomFieldsManager (see doc here) I feel like it would be more efficient to expose the IP address directly as a CustomFieldStringValue (see doc here) object than to acquire the base64 encoded form and decode it out of an array of hundreds of entities in vm.config.extraConfig.

@zjs
Copy link
Member

zjs commented Nov 3, 2017

If we expose the IP address in a new place, we'll need to ensure it gets updated each time the IP address changes. How feasible is that (especially in the case of DHCP)?

@andrewtchin
Copy link
Contributor Author

Also how does the UI handle IP vs hostname? I don't think it's possible for us to update the tag value since we do not store the vSphere credentials used to tag the VM

@jooskim
Copy link
Contributor

jooskim commented Nov 3, 2017

Hmm, it makes sense that if we add another custom field for IP there will be another burden to update the value every time the IP changes. I'll then hold onto the current approach to retrieving the IP address of a VM.

@jooskim
Copy link
Contributor

jooskim commented Nov 3, 2017

@andrewtchin We will be using the information to construct, in the Angular service, a URL to the vic-machine-server endpoint. It doesn't really matter whether that's hostname or IP for our use case.

@andrewtchin
Copy link
Contributor Author

What about for TLS? Do you ignore cert errors if the user is using a self signed or one signed by a non public CA?

@jooskim
Copy link
Contributor

jooskim commented Nov 3, 2017

I think that issue was handled by Cristian. @cristianfalcone Could you let us know how we handled that?

@ghost ghost mentioned this issue Nov 6, 2017
jooskim added a commit to jooskim/vic-ui that referenced this issue Nov 6, 2017
jooskim added a commit to jooskim/vic-ui that referenced this issue Nov 6, 2017
@ghost ghost closed this as completed in #1040 Nov 7, 2017
ghost pushed a commit that referenced this issue Nov 7, 2017
Allows the vSphere H5 Client to identify the appliance vm
from a custom attribute called vic-ova-identifier. This key 
has a value of the appliance ova build version. Fixes #1018.
jooskim added a commit to vmware/vic-ui that referenced this issue Nov 8, 2017
* [WIP] Implement endpoint for vic appliances list

* Turn off sourcemaps in ng test

This saves us from headaches by debunking myterious
XMLHttpRequest errors in unit test failures which
oftentimes are actually not XMLHttpRequest errors.

* Use tee for outputting vic-ui build tasks

* Sync with Jason's work on vmware/vic-product#1018

* Sort desc appliances info by build number

* Update SummaryComponent to use lookup service

* Add more test report recipients

* Move vch appliance detection to general.component
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product/ova Related to the OVA packaging of vSphere Integrated Containers
Projects
None yet
Development

No branches or pull requests

3 participants