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

[477 idl part2] sharable flag, SR.create and SR.attach API change #56

Merged
merged 6 commits into from
Feb 8, 2018
Merged

[477 idl part2] sharable flag, SR.create and SR.attach API change #56

merged 6 commits into from
Feb 8, 2018

Conversation

edwintorok
Copy link
Contributor

@edwintorok edwintorok commented Feb 6, 2018

This is part of the SMAPIv3 interface change from feature/REQ477/master,
has to be merged together with these PRs:
xapi-project/sm-cli#26
xapi-project/xapi-storage#70
xapi-project/xcp-idl#202
xapi-project/xen-api#3433

The sharable flag is explained in the xcp-idl commit: xapi-project/xcp-idl@c8bfaf9
The activated on exception/redirection is explained here (SMAPIv3 can require snapshots to be taken on the host that has the VM running, instead of the master): xapi-project/xen-api@620acd2

SMAPIv3: SR.create and SR.attach need an uuid parameter too, this is a breaking API change, the xapi PR updates XAPI accordingly.

jonludlam and others added 6 commits February 6, 2018 10:21
Plumb through uuid

Signed-off-by: Jon Ludlam <jonathan.ludlam@citrix.com>
Signed-off-by: Edwin Török <edvin.torok@citrix.com>
For this to work the GFS2 SR will need to implement Volume.set, and retrieving
the 'keys' field.

Signed-off-by: Edwin Török <edvin.torok@citrix.com>
Signed-off-by: Edwin Török <edvin.torok@citrix.com>
Signed-off-by: Edwin Török <edvin.torok@citrix.com>
Signed-off-by: Edwin Török <edvin.torok@citrix.com>
Copy link
Contributor

@gaborigloi gaborigloi left a comment

Choose a reason for hiding this comment

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

I'm not too familiar with this repo but the changes have already been reviewed.

@edwintorok edwintorok merged commit fadc079 into xapi-project:master Feb 8, 2018
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.

None yet

3 participants