Skip to content

Add explanations re:Storage Repositories to the Xen-API overview#7101

Open
shindere wants to merge 1 commit into
xapi-project:masterfrom
shindere:properly-introduce-storage-repositories
Open

Add explanations re:Storage Repositories to the Xen-API overview#7101
shindere wants to merge 1 commit into
xapi-project:masterfrom
shindere:properly-introduce-storage-repositories

Conversation

@shindere

Copy link
Copy Markdown
Contributor

The description of how to install a VM through the Xen API didmention storage repositories but only in passing. This PR proposes to fully introduce them before moving on to the installation of VMs.

Signed-off-by: Seb Hinderer <sebastien.hinderer@vates.tech>

@psafont psafont left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for these fixes!

- Next, we need to specify the UUID of the Storage Repository where the VM's
disks will be instantiated. We have to put this in the `sr` attribute in
- Next, we need to specify where the VM's disks will be stored by
default. We do so by putting `my_sr_ref` in the `sr` attribute in

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I haven't verified it. But from the code, it seems indeed it is UUID.
https://github.com/xapi-project/xen-api/blob/master/ocaml/xapi/xapi_templates.ml#L128

@gthvn1 gthvn1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, just need to check if it is a Ref or a UUID that you need to provide for the SR when modifying the other-config. It looks like it is a UUID.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants