Bug 22945 - Wiederherstellen von Snapshots mit / im Namen schlägt fehl
Wiederherstellen von Snapshots mit / im Namen schlägt fehl
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Virtualization - UVMM
UCS 2.4
Other Linux
: P5 normal (vote)
: ---
Assigned To: Bugzilla Mailingliste
:
Depends on:
Blocks: 26681
  Show dependency treegraph
 
Reported: 2011-06-30 16:36 CEST by Moritz Muehlenhoff
Modified: 2023-06-28 10:46 CEST (History)
4 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Moritz Muehlenhoff univentionstaff 2011-06-30 16:36:35 CEST
Ich hatte einen Snapshot "Experimenteller Build mit Xorg 1.10 / Intel 2.15" erzeugt. Das schlug fehl, da der / im Dateinamen als Unterverzeichnis interpretiert wird.

Der / sollte escapt oder zumindest in der Eingabemaske nicht erlaubt werden.
Comment 1 Philipp Hahn univentionstaff 2011-08-03 11:18:33 CEST
Ggf. fixed upstream in libvirt-0.9.0:
 qemu: Escape snapshot name passed to {save,load,del}vm (Jiri Denemark)
Comment 2 Philipp Hahn univentionstaff 2011-10-19 12:51:43 CEST
Das ist intern gerade nochmal aufgetreten: qemu hat den Sicherungspunkt noch in der Qcow2-Datei angelegt, aber beim erzeugen der xml-Datei durch libvirt ist das ganze dann gescheitert.

So ein Snapshot lässt sich notfalls noch retten:

$ vm="arequate_ucs_3.0-0-DCM.55"
$ img="/var/lib/libvirt/images/arequate_ucs_30MS1.qcow2"
$ ~/src/VIRT/qcow2.py -r -2 -1 "$img"
# 0000000000000000: + snapshots_offset=0x000000026d610000
$ offset=$((0x26d610000))
$ orig=$(mktemp) mod=$(mktemp)
$ dd bs=1 skip=$offset count=4096 if="$img" of="$orig"
$ cp "$orig" "$mod"
$ vim -b "$mod"
# s|ucs_3.0-0 w/o samba4|ucs_3.0-0 w_o samba4|
$ dd bs=1 conv=notrunc seek=$offset of="$img" if="$mod"
$ qemu-img check "$img"
$ qemu-img info "$img"
$ ~/src/VIRT/qcow2-snap-scan.sh "$img" "$vm"
$ pkill -SIGHUP libvirtd
$ virsh snapshot-list "$vm"
Comment 3 Arvid Requate univentionstaff 2011-10-19 14:07:57 CEST
Die Anpassung per Comment 2 hat leider nicht geholfen. Meiner Meinung nach sollte das UMC Modul einfach den Buchstaben "/" beim Anlegen von Snapshots verbieten. So wie es ist, lässt man den Anwender in das klassische Backup-Problem rennen, wo einem erst bei Restore auffällt, dass sich das Backup nicht wiederherstellen lässt.
Comment 4 Alexander Kläser univentionstaff 2011-11-08 19:47:32 CET
(In reply to comment #3)
> Die Anpassung per Comment 2 hat leider nicht geholfen. Meiner Meinung nach
> sollte das UMC Modul einfach den Buchstaben "/" beim Anlegen von Snapshots
> verbieten. So wie es ist, lässt man den Anwender in das klassische
> Backup-Problem rennen, wo einem erst bei Restore auffällt, dass sich das Backup
> nicht wiederherstellen lässt.

In 3.0 wurde dies so angepasst, dass keine "/" sowie Punkte am Namensanfang angegeben werden können.