AFAICT, the only existing callers using noerr=1 are in
__snapshot_delete_remove_drive, and in AbstractConfig's
foreach_volume_full, and they both should be fine with the change.
Signed-off-by: Fabian Ebner
---
Required for the next patch.
src/PVE/LXC/Config.pm | 4 +++-
1 file changed, 3 inser
For snapshot creation, the storage for the vmstate file is activated
via vdisk_alloc when the state file is created.
Do not activate the volumes themselves, as that has unnecessary side
effects (e.g. waiting for zvol device link for ZFS, mapping the volume
for RBD). If a storage can only do snapsh
Signed-off-by: Fabian Ebner
---
PVE/QemuConfig.pm | 19 +++
.../create/qemu-server/303.conf | 13 +++
.../delete/qemu-server/204.conf | 33 ++
.../rollback/qemu-server/303.conf | 34 +++
Signed-off-by: Fabian Ebner
---
Requires the previous patch, as otherwise, foreach_volume_full will
die when calling parse_volume with 'vmstate'.
src/PVE/LXC/Config.pm | 19 +++
.../snapshot-expected/create/lxc/204.conf | 10 ++
.../snapshot-expected/dele
to avoid a potential warning. parse_volume is called with noerr=1, so this might
be undef instead of a hash.
Signed-off-by: Fabian Ebner
---
src/PVE/LXC/Config.pm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/PVE/LXC/Config.pm b/src/PVE/LXC/Config.pm
index 8557e4c..f832
Signed-off-by: Fabian Ebner
---
src/PVE/AbstractConfig.pm | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/PVE/AbstractConfig.pm b/src/PVE/AbstractConfig.pm
index 3348d8a..b3f70f2 100644
--- a/src/PVE/AbstractConfig.pm
+++ b/src/PVE/AbstractConfig.pm
@@ -829,8 +829,6 @@ sub snapshot_creat
to make it work when the storage is just not active yet, and have
early errors when the storage cannot be activated. Also prohibits
snapshot operations when an involved storage is disabled.
Both qemu-server and pve-container build-depend upon pve-guest-common
for the added tests.
pve-guest-comm
changes to v1:
* also renamed iso/vztmpl REs
* new naming schema: $_EXT_RE_<# of capture groups>
for example: $BACKUP_EXT_RE_2
pve-storage:
Lorenz Stechauner (2):
storage: rename REs for iso and vztmpl extensions
storage/plugin: factoring out regex for backup extension re
PVE/API2/Stora
these changes make it more clear, how many capture groups each
RE inclues.
Signed-off-by: Lorenz Stechauner
---
PVE/API2/Storage/Status.pm | 8
PVE/Storage.pm | 11 +++
PVE/Storage/Plugin.pm | 8
3 files changed, 15 insertions(+), 12 deletions(-)
dif
Signed-off-by: Lorenz Stechauner
---
PVE/Storage.pm| 12 +++-
PVE/Storage/Plugin.pm | 4 ++--
2 files changed, 9 insertions(+), 7 deletions(-)
diff --git a/PVE/Storage.pm b/PVE/Storage.pm
index b5c2460..9bc799d 100755
--- a/PVE/Storage.pm
+++ b/PVE/Storage.pm
@@ -108,6 +108,8 @@
Signed-off-by: Lorenz Stechauner
---
PVE/APLInfo.pm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/PVE/APLInfo.pm b/PVE/APLInfo.pm
index 5cee1af8..1eff7107 100644
--- a/PVE/APLInfo.pm
+++ b/PVE/APLInfo.pm
@@ -84,7 +84,7 @@ sub read_aplinfo_from_fh {
my $template;
by not dying when the dataset is already unmounted. Can be triggered
for a container by doing two rollbacks in a row.
Signed-off-by: Fabian Ebner
---
PVE/Storage/ZFSPoolPlugin.pm | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/PVE/Storage/ZFSPoolPlugin.pm b/PVE/Storage/ZF
was missed in commit 92e7ce43e1d06b4065cd8237cb8b5977224dc3d0
Signed-off-by: Oguz Bektas
---
www/manager6/Utils.js | 1 +
1 file changed, 1 insertion(+)
diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js
index a22eaaa7..4041c010 100644
--- a/www/manager6/Utils.js
+++ b/www/manager6/Util
On 8/3/21 9:34 AM, Fabian Ebner wrote:
Am 19.07.21 um 16:52 schrieb Aaron Lauterer:
The goal of this is to expand the move-disk API endpoint to make it
possible to move a disk to another VM. Previously this was only possible
with manual intervertion either by renaming the VM disk or by manuall
quote from extjs docs (AbstractChart.js):
If you do have a chart inside a scrollable view, even if it has no
interactions, you have to set its touchAction config to the following:
touchAction: {
panX: true,
panY: true
}
Otherwise, if a touch action started on a chart,
a swipe wil
first, '43' is a typo, it should say '113' (if it really is like
legacy shorewall [0]). this tcp port corresponds to the ident or
authentication service protocol.
second, nowdays this reject is not included in shorewall anymore.
furthermore it would make no sense to reject specifically this
one po
applied, thanks :)
___
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
@bootorder only contains entries for non-legacy bootorder entries,
but the default one contains all cdroms anyway, and if the user
explicitely disabled cdroms, it is ok to not add them back
for the new cdrom drive.
Signed-off-by: Dominik Csapak
---
view with -w to see the actual change
PVE/API2/
thanks!
On August 5, 2021 1:53 pm, Dominik Csapak wrote:
> @bootorder only contains entries for non-legacy bootorder entries,
> but the default one contains all cdroms anyway, and if the user
> explicitely disabled cdroms, it is ok to not add them back
> for the new cdrom drive.
>
> Signed-off-by
On 8/3/21 10:14 AM, Fabian Ebner wrote:
Am 19.07.21 um 16:52 schrieb Aaron Lauterer:
The goal of this is to expand the move-volume API endpoint to make it
possible to move a container volume / mountpoint to another container.
Currently it works for regular mountpoints though it would be nice
- display zones list in global permissions management pathselector
- remove vmbrX bridges from bridgeselector if user have permissions on vnets
Alexandre Derumier (2):
permpathstore: add sdn zones
api2 : network: anybridge: don't display bridges if user have access
to vnets.
PVE/API2/Net
Signed-off-by: Alexandre Derumier
---
PVE/Network/SDN.pm | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/PVE/Network/SDN.pm b/PVE/Network/SDN.pm
index d3399ce..b95dd5b 100644
--- a/PVE/Network/SDN.pm
+++ b/PVE/Network/SDN.pm
@@ -193,7 +193,7 @@ sub get_local_vnets {
my
Signed-off-by: Alexandre Derumier
---
www/manager6/data/PermPathStore.js | 3 +++
1 file changed, 3 insertions(+)
diff --git a/www/manager6/data/PermPathStore.js
b/www/manager6/data/PermPathStore.js
index 1dc276b6..cf702c03 100644
--- a/www/manager6/data/PermPathStore.js
+++ b/www/manager6/data
This remove vmbr* from bridgeselector if user have access to vnets.
(as currently, we don't have have permission management on vmbr$)
Signed-off-by: Alexandre Derumier
---
PVE/API2/Network.pm | 19 ---
1 file changed, 12 insertions(+), 7 deletions(-)
diff --git a/PVE/API2/Networ
This remove vmbr* from bridgeselector if user have access to vnets.
(as currently, we don't have have permission management on vmbr$)
Signed-off-by: Alexandre Derumier
---
PVE/API2/Network.pm | 19 ---
1 file changed, 12 insertions(+), 7 deletions(-)
diff --git a/PVE/API2/Networ
Signed-off-by: Alexandre Derumier
---
src/PVE/AccessControl.pm | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/PVE/AccessControl.pm b/src/PVE/AccessControl.pm
index 3d8d01c..a9d97e8 100644
--- a/src/PVE/AccessControl.pm
+++ b/src/PVE/AccessControl.pm
@@ -944,6 +944,7 @@ sub check_path {
26 matches
Mail list logo
