Introduce InventoryTemplates and NestedClusters#80
Open
ibrokethecloud wants to merge 1 commit intoharvester:mainfrom
Open
Introduce InventoryTemplates and NestedClusters#80ibrokethecloud wants to merge 1 commit intoharvester:mainfrom
ibrokethecloud wants to merge 1 commit intoharvester:mainfrom
Conversation
provisoining of VM based Harvester clusters via seeder Signed-off-by: Gaurav Mehta <gaurav.mehta@suse.com>
|
This looks great! |
|
@starbops @Vicente-Cheng @ihcsim Let's make progress on this PR, since QA is waiting for it to start working on running tests on the nested Harvester. @ibrokethecloud please fix the conflict first. cc @harvester/qa |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem:
Improve hardware utilisation by allowing nested harvester clusters to be provisioned by seeder by using VM's and kubevirt BMC to allow VM's to be managed as inventory
Solution:
The PR introduces two new types
Related Issue(s):
Test plan:
Additional documentation or context