[pve-devel] [PATCH container] fix #3367: skip bind mounts when converting to template

[Fabian Ebner]

Restoring a backup of a template with a bind mount point failed, because the
volume_has_feature check would fail.

Fix this by skipping bind mounts on template creation. There is no check against
adding a bind mount with 'pct set' *after* converting to a template, so it
shouldn't be an issue if we also allow conversion if bind mounts are present *at*
conversion time.

Signed-off-by: Fabian Ebner <f.ebner at proxmox.com>
---
 src/PVE/LXC.pm | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/PVE/LXC.pm b/src/PVE/LXC.pm
index 6395d12..5a9f76d 100644
--- a/src/PVE/LXC.pm
+++ b/src/PVE/LXC.pm
@@ -1186,6 +1186,8 @@ sub template_create {
     PVE::LXC::Config->foreach_volume($conf, sub {
 	my ($ms, $mountpoint) = @_;
 
+	return if $mountpoint->{type} eq 'bind';
+
 	my $volid = $mountpoint->{volume};
 
 	die "Template feature is not available for '$volid'\n"
@@ -1195,6 +1197,8 @@ sub template_create {
     PVE::LXC::Config->foreach_volume($conf, sub {
 	my ($ms, $mountpoint) = @_;
 
+	return if $mountpoint->{type} eq 'bind';
+
 	my $volid = $mountpoint->{volume};
 
 	PVE::Storage::activate_volumes($storecfg, [$volid]);
-- 
2.20.1