Skip to content

Conversation

shwstppr
Copy link
Contributor

Documentation PR for VM Ingestion feature.
Feature PR: apache/cloudstack#3606

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
- listUnmanagedInstances - to list unmanaged virtual machines for a cluster.

- importUnmanagedInstance - to import an unmanaged virtual machine into CloudStack.

Copy link
Member

Choose a reason for hiding this comment

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

@shwstppr add details on how to use the .py script etc?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@rhtyd yes, will be adding that. This was just initial commit to create PR, working on rest of the documentation

@shwstppr shwstppr mentioned this pull request Nov 18, 2019
5 tasks
@shwstppr shwstppr changed the title [WIP DO NOT MERGE] VM Ingestion [WIP DO NOT MERGE] VM ingestion Nov 18, 2019
@shwstppr
Copy link
Contributor Author

shwstppr commented Nov 18, 2019

@andrijapanicsb @PaulAngus Can you please review changes
I'm not sure if it is the correct location in documentation to add vm ingestion notes so please advise changes needed.
Also, advise on the format. Since this is my first documentation PR I'm not sure if added notes, API details correctly.

@PaulAngus
Copy link
Member

requesting docbuild

@ACSGitBot
Copy link

Your request had been received, i'll go and build the documentation and check the output log for errors.

This shouldn't take long.

@ACSGitBot
Copy link

Build finished. You can review it at: https://acs-www.shapeblue.com/docs/pr84

Build Log Output:

No log errors found to report.

@shwstppr
Copy link
Contributor Author

shwstppr commented Dec 2, 2019

requesting docbuild

@ACSGitBot
Copy link

Your request had been received, i'll go and build the documentation and check the output log for errors.

This shouldn't take long.

@ACSGitBot
Copy link

Build finished. You can review it at: https://acs-www.shapeblue.com/docs/pr84

Build Log Output:

No log errors found to report.

Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
@rohityadavcloud rohityadavcloud added the feature New feature or request label Dec 29, 2019
@PaulAngus
Copy link
Member

requesting docbuild

@ACSGitBot
Copy link

Your request had been received, i'll go and build the documentation and check the output log for errors.

This shouldn't take long.

@ACSGitBot
Copy link

Build finished. You can review it at: https://acs-www.shapeblue.com/docs/pr84

Build Log Output:

No log errors found to report.

@PaulAngus
Copy link
Member

requesting docbuild

@ACSGitBot
Copy link

Your request had been received, i'll go and build the documentation and check the output log for errors.

This shouldn't take long.

@ACSGitBot
Copy link

Build finished. You can review it at: https://acs-www.shapeblue.com/docs/pr84

Build Log Output:

No log errors found to report.

Copy link
Contributor Author

@shwstppr shwstppr left a comment

Choose a reason for hiding this comment

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

Some minor suggestions @PaulAngus

###############


To import a VM which has data disks attached, a map of the disk ID and corresponding disk offering ID must be passed via the *datadiskofferinglist* parameter.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@PaulAngus We also have diskofferingid parameter for specifying offering for root disk so do we need to mention that here.

Copy link
Member

Choose a reason for hiding this comment

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

oh boy, what happens in code? the diskoffering is a messy subject, a user doesn't choose a disk offering for the root, it's linked to the serviceoffering ID

Copy link
Contributor Author

Choose a reason for hiding this comment

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

While importing it uses disk offeering passedc with param diskofferingid. It is a required parameter for importUnmanagedInstance API

Copy link
Member

Choose a reason for hiding this comment

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

hmm. if you consider creating a VM from a template, you don't pass a disk offering, it's implicit from the service offering...

Paul Angus and others added 3 commits January 15, 2020 14:10
Co-Authored-By: Abhishek Kumar <abhishek.mrt22@gmail.com>
Co-Authored-By: Abhishek Kumar <abhishek.mrt22@gmail.com>
Co-Authored-By: Abhishek Kumar <abhishek.mrt22@gmail.com>
@PaulAngus PaulAngus changed the title [WIP DO NOT MERGE] VM ingestion VM ingestion and B&R Jan 30, 2020
@shwstppr
Copy link
Contributor Author

requesting docbuild

@ACSGitBot
Copy link

Your request had been received, i'll go and build the documentation and check the output log for errors.

This shouldn't take long.

@ACSGitBot
Copy link

Build finished. You can review it at: https://acs-www.shapeblue.com/docs/WIP-PROOFING/pr84

Build Log Output:

No log errors found to report.

@andrijapanicsb
Copy link
Contributor

@shwstppr Can you please update this one:

  • there is conflict
  • it mentioned Backup and Recovery, but there is actually a separate B&R PR [WIP] Backup & Recovery Doc Guide #85
  • can you sync with Rohit and see what is what and to have this one (VM ingestion) merged.

@PaulAngus
Copy link
Member

if you merge B&R, the commits will match.
B&R was used as the basis for this to avoid conflicts

@andrijapanicsb
Copy link
Contributor

andrijapanicsb commented May 13, 2020

@PaulAngus are you saying that this one is reviewed and LGTM?

@PaulAngus
Copy link
Member

@shwstppr it doesn't look like you've updated this PR after review by Steve Roles ..?

@PaulAngus PaulAngus changed the title VM ingestion and B&R VM ingestion May 13, 2020
@shwstppr
Copy link
Contributor Author

@PaulAngus I don't think Steve ever reviewed this one. There was no draft word doc for it. After initial updates it was mostly updated by you.

@shwstppr
Copy link
Contributor Author

@andrijapanic will resolve the conflict tomorrow morning

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr
Copy link
Contributor Author

@andrijapanicsb I've fixed conflicts. cc @PaulAngus

@andrijapanicsb
Copy link
Contributor

requesting docbuild

@ACSGitBot
Copy link

Your request had been received, i'll go and build the documentation and check the output log for errors.

This shouldn't take long.

@ACSGitBot
Copy link

Build finished. You can review it at: https://acs-www.shapeblue.com/docs/WIP-PROOFING/pr84

Build Log Output:

No log errors found to report.

@andrijapanicsb
Copy link
Contributor

@shwstppr @PaulAngus - can you advise if this is good to go or not, we need it asap for 4.14.
thx

@andrijapanicsb
Copy link
Contributor

pinging again for LGTM if possible.

@andrijapanicsb
Copy link
Contributor

Merging as it is, looks LGTM in general, but we can always improve it if needed.
Required for 4.14 to go out.

@andrijapanicsb andrijapanicsb merged commit 95b7382 into apache:master May 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants