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

Add inventory folder support to vic-machine and VCH #773

Closed
hickeng opened this issue May 23, 2016 · 9 comments
Closed

Add inventory folder support to vic-machine and VCH #773

hickeng opened this issue May 23, 2016 · 9 comments
Labels
component/install Epic Represents a ZenHub Epic impact/doc priority/p1 source/customer Reported by a customer, directly or via an intermediary team/foundation

Comments

@hickeng
Copy link
Member

hickeng commented May 23, 2016

Add folder and naming pattern options to vic-machine design doc and implementation.

Various customers use folder and naming conventions as organisation mechanisms within VC, and if we're going to allow tight integration we need to support continued usage of those models.

There are two variants of folders that we must concern ourselves with:

  1. datastore folders - only relevant on non-vSAN datastores
  2. datacenter folders - only relevant on vCenter

(1) is expected to be addressed via container-store options to vic-machine.

This issue addresses (2), and it's expected that be done via a name pattern that includes a datacenter folder path element.

@hickeng hickeng self-assigned this May 23, 2016
@stuclem stuclem added the impact/doc/user Requires changes to official user documentation label May 24, 2016
@stuclem stuclem removed the impact/doc/user Requires changes to official user documentation label May 30, 2016
@hickeng hickeng added this to the VIC GA Release milestone Jul 1, 2016
@hickeng hickeng changed the title Add folder options to vic-machine Add VM folder options to vic-machine Jul 5, 2016
@hickeng hickeng changed the title Add VM folder options to vic-machine Add folder options to vic-machine Jul 5, 2016
@karthik-narayan
Copy link

@andrewtchin @emlin I assume that this is the issue we hit with the customer. Could you confirm?

Also @hickeng the customer was using folders in VC to organize their switches. Besides datastores and datacenters, we might also need to account for folders in networking.

@emlin
Copy link
Contributor

emlin commented Aug 8, 2016

@hickeng @karthik-narayan This customer also has network folder
And also one question for the datastore folders. As George mentioned, this issue should be solved in container-datastore, don't we want to have folder support for image datastore and volume datastore?
This customer has pretty everything under folder.

As we already had hierarchy structure to configure datastore path in the parameter for image datastore, volume datastore, I'm wondering if together with folder hierarchy, we'll need more complicated validation.

@karthik-narayan
Copy link

I'll let you figure out the implementation details, but this is a use case that I've hit more than once and is a very common one. Asking customers to move objects out of the folder to get VIC working is a non-starter and we should get to this at the earliest.

@mdubya66 mdubya66 removed this from the VIC GA Release milestone Sep 19, 2016
@mdubya66 mdubya66 added the impact/doc/note Requires creation of or changes to an official release note label Sep 20, 2016
@stuclem
Copy link
Contributor

stuclem commented Nov 7, 2016

Already in the release notes, so removing the kind/note tag.

@hickeng hickeng removed the impact/doc/note Requires creation of or changes to an official release note label Nov 7, 2016
@mdubya66 mdubya66 added the source/customer Reported by a customer, directly or via an intermediary label Jan 12, 2017
@mdubya66 mdubya66 added this to the v0.9.0 milestone Jan 12, 2017
@mdubya66 mdubya66 assigned mdubya66 and dougm and unassigned hickeng and mdubya66 Jan 12, 2017
@mdubya66
Copy link
Contributor

assigning to dougm to investigate the govmomi requirements

@dougm
Copy link
Member

dougm commented Jan 17, 2017

There are 2 high level issues related to folder support. Spoke with @hickeng and this issue is related to inventory support for creating new resources. For example, users should be able to specify which inventory folder container VMs are created in. A separate issue is finding existing resources within inventory folders when the full path is not specified.

I'm going to convert this issue to an epic and create 2 separate issues for create + find.

@dougm dougm added the Epic Represents a ZenHub Epic label Jan 17, 2017
@dougm dougm changed the title Add folder options to vic-machine Add inventory folder support to vic-machine and VCH Jan 17, 2017
@dougm dougm removed their assignment Jan 17, 2017
@mdubya66 mdubya66 removed this from the v0.9.0 milestone Jan 19, 2017
AngieCris pushed a commit to AngieCris/vic that referenced this issue Oct 9, 2017
* First mention of dch-photon

* More dch-photon edits

* Adding new file for VCH deployment with dchphoton

* Documenting dch-photon

* Committing so I can swap branches

* More dch-photon updates

* More dch-photon updates

* Typo
@mdubya66
Copy link
Contributor

why is this now a ROBO requirement?

@cgtexmex
Copy link
Contributor

ROBO Licensing doesn't include resource pools (RP). RPs are the mechanism we use to group a VCH with it's containers -- essentially we identify that the containerVMs belong to a given VCH because they are in the same RP. That grouping / identification is the used in many areas of vic-machine and the VCH appliance. Inventory Folders provide a similar grouping mechanism so it's a natural fit as a ROBO alternative.

@mdubya66
Copy link
Contributor

All issues closed. Closing the Epic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/install Epic Represents a ZenHub Epic impact/doc priority/p1 source/customer Reported by a customer, directly or via an intermediary team/foundation
Projects
None yet
Development

No branches or pull requests

8 participants