applied
Also, I just noticed, you duplicated the subject line into the message
body. Note that git-am already includes the subject line, so now the
commits contain the first line twice. (Not a problem, just something to
look out for in future contributions.)
On Wed, Sep 13, 2017 at 12:30:35PM +0200, Matthias Urban wrote:
> VM.Snapshot.Rollback privilege added
>
> Signed-off-by: Matthias Urban <matthias.ur...@pure-systems.com>
> ---
> src/PVE/API2/LXC/Snapshot.pm | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/PVE/API2/LXC/Snapshot.pm b/src/PVE/API2/LXC/Snapshot.pm
> index 6ba6fb3..edccbf4 100644
> --- a/src/PVE/API2/LXC/Snapshot.pm
> +++ b/src/PVE/API2/LXC/Snapshot.pm
> @@ -225,7 +225,7 @@ __PACKAGE__->register_method({
> proxyto => 'node',
> description => "Rollback LXC state to specified snapshot.",
> permissions => {
> - check => ['perm', '/vms/{vmid}', [ 'VM.Snapshot' ]],
> + check => ['perm', '/vms/{vmid}', [ 'VM.Snapshot',
> 'VM.Snapshot.Rollback' ], any => 1],
> },
> parameters => {
> additionalProperties => 0,
> @@ -328,7 +328,7 @@ __PACKAGE__->register_method({
> proxyto => 'node',
> description => "Get snapshot configuration",
> permissions => {
> - check => ['perm', '/vms/{vmid}', [ 'VM.Snapshot' ]],
> + check => ['perm', '/vms/{vmid}', [ 'VM.Snapshot',
> 'VM.Snapshot.Rollback' ], any => 1],
> },
> parameters => {
> additionalProperties => 0,
> --
> 2.14.1
_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
