The backup_container() method is called after forking, so changes to
$self will not be visible in later method calls done by the parent.

Signed-off-by: Fiona Ebner <f.eb...@proxmox.com>
---
 src/PVE/BackupProvider/Plugin/Base.pm | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/PVE/BackupProvider/Plugin/Base.pm 
b/src/PVE/BackupProvider/Plugin/Base.pm
index 424feb7..4a35330 100644
--- a/src/PVE/BackupProvider/Plugin/Base.pm
+++ b/src/PVE/BackupProvider/Plugin/Base.pm
@@ -595,11 +595,16 @@ that is made available for the mechanism returned by 
C<backup_get_mechanism>.
 Returns when done backing up. Ideally, the method should log the progress 
during
 backup.
 
-Note that this function is executed as the ID-mapped root user of the 
container,
+Note that this method is executed as the ID-mapped root user of the container,
 so a potentially unprivileged user. The ID is passed along as part of C<$info>.
 Use the C<backup_container_prepare()> method for preparation. For example, to
 make credentials available to the potentially unprivileged user.
 
+Note that changes to C<$self> made during this method will not be visible in
+later method calls. This is because the method is executed in a separate
+execution context after forking. Use the C<backup_container_prepare()> method
+if you need persistent changes to C<$self>.
+
 Parameters:
 
 =over
-- 
2.39.5



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to