[GH-ISSUE #53] inverted commas in example #50

New issue

Closed

opened 2026-02-26 17:44:10 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 17:44:10 +03:00

Owner

Originally created by @romanshein on GitHub (Sep 19, 2021).
Original GitHub issue: https://github.com/Corsinvest/cv4pve-autosnap/issues/53

Generated following the example:
cv4pve-autosnap --host=192.168.1.230 --username=root@pam --password=XXXXXX --vmid=500 snap --label='daily' --keep=2 at Windows machine.

The result is as follows:
ACTION Snap
VMs: 500
Label: 'daily'
Keep: 2
State: False
Timeout: 30000
Timestamp format: yyMMddHHmmss
----- VM 500 Qemu -----
Create snapshot: auto'daily'210919215747
snapname : invalid format - invalid configuration ID 'auto'daily'210919215747'

The removal of ' (inverted commas) solved the problem. I suggest to edit the sample script by removing the inverted commas.
The inverted commas are redundant. The commas are useful for the phrases containing spaces, yet it is unlikely that Proxmox would be ok with the snapshot names with spaces anyway. Even if it does, it is still not a good idea to put spaces into the snapshot name, so it should be discouraged.

Originally created by @romanshein on GitHub (Sep 19, 2021). Original GitHub issue: https://github.com/Corsinvest/cv4pve-autosnap/issues/53 Generated following the example: cv4pve-autosnap --host=192.168.1.230 --username=root@pam --password=XXXXXX --vmid=500 snap --label='daily' --keep=2 at Windows machine. The result is as follows: ACTION Snap VMs: 500 Label: 'daily' Keep: 2 State: False Timeout: 30000 Timestamp format: yyMMddHHmmss ----- VM 500 Qemu ----- Create snapshot: auto'daily'210919215747 snapname : invalid format - invalid configuration ID 'auto'daily'210919215747' The removal of ' (inverted commas) solved the problem. I suggest to edit the sample script by removing the inverted commas. The inverted commas are redundant. The commas are useful for the phrases containing spaces, yet it is unlikely that Proxmox would be ok with the snapshot names with spaces anyway. Even if it does, it is still not a good idea to put spaces into the snapshot name, so it should be discouraged.