[pve-devel] [PATCH docs] qm/pct: Add explanation for --purge

[pve-devel] [PATCH docs] qm/pct: Add explanation for --purge

[Dominic Jäger]

This was apparently not clear enough.
For example, see #3184.

A bit long, but I think this is a better place for additional
information about ACL and firewall than the GUI tooltip or the
API parameter description.

Signed-off-by: Dominic Jäger <d.jaeger@proxmox.com>
---
 pct.adoc | 10 ++++++++++
 qm.adoc  |  8 ++++++++
 2 files changed, 18 insertions(+)

diff --git a/pct.adoc b/pct.adoc
index 1f0bcef..ddc16e2 100644
--- a/pct.adoc
+++ b/pct.adoc
@@ -791,6 +791,16 @@ Reduce the memory of the container to 512MB
 # pct set 100 -memory 512
 ----
 
+Destroying a container always removes it from Access Control Lists and it always
+removes the firewall configuration of the container. You have to activate
+'--purge', if you want to additionally remove the container from replication jobs,
+backup jobs and HA resource configurations.
+
+----
+# pct destroy 100 --purge
+----
+
+
 
 Obtaining Debugging Logs
 ~~~~~~~~~~~~~~~~~~~~~~~~
diff --git a/qm.adoc b/qm.adoc
index 5e53e4a..9c54903 100644
--- a/qm.adoc
+++ b/qm.adoc
@@ -1326,6 +1326,14 @@ Same as above, but only wait for 40 seconds.
 
  qm shutdown 300 && qm wait 300 -timeout 40
 
+Destroying a VM always removes it from Access Control Lists and it always
+removes the firewall configuration of the VM. You have to activate
+'--purge', if you want to additionally remove the VM from replication jobs,
+backup jobs and HA resource configurations.
+
+ qm destroy 300 --purge
+
+
 
 [[qm_configuration]]
 Configuration
-- 
2.20.1

[pve-devel] [PATCH manager] safe destroy: Add HA to purge tooltip

[Dominic Jäger]

This makes the enumeration complete.

Signed-off-by: Dominic Jäger <d.jaeger@proxmox.com>
---
 www/manager6/window/SafeDestroy.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/www/manager6/window/SafeDestroy.js b/www/manager6/window/SafeDestroy.js
index cc32f6e0..4506fad3 100644
--- a/www/manager6/window/SafeDestroy.js
+++ b/www/manager6/window/SafeDestroy.js
@@ -134,7 +134,7 @@ Ext.define('PVE.window.SafeDestroy', {
 		    checked: false,
 		    autoEl: {
 			tag: 'div',
-			'data-qtip': gettext('Remove from replication and backup jobs')
+			'data-qtip': gettext('Remove from replication & backup jobs and HA resource configuration.')
 		    }
 		}
 	    ]
-- 
2.20.1

[pve-devel] applied-series: Re: [PATCH docs] qm/pct: Add explanation for --purge

[Thomas Lamprecht]

On 10/12/2020 09:15, Dominic Jäger wrote:
> This was apparently not clear enough.
> For example, see #3184.
> 
> A bit long, but I think this is a better place for additional
> information about ACL and firewall than the GUI tooltip or the
> API parameter description.
> 
> Signed-off-by: Dominic Jäger <d.jaeger@proxmox.com>
> ---
>  pct.adoc | 10 ++++++++++
>  qm.adoc  |  8 ++++++++
>  2 files changed, 18 insertions(+)
> 
>

applied both patches, thanks!