Re: [pve-devel] pvedaemon hanging because of qga retry

2018-05-17 Thread Dietmar Maurer
If you simply skip commands like 'guest-fsfreeze-thaw' your VM will get totally unusable (frozen). So I am not sure what you want to suggest? A correct fix would be to implement an async command queue inside qemu... > On May 18, 2018 at 7:13 AM Alexandre DERUMIER wrote: >

Re: [pve-devel] pvedaemon hanging because of qga retry

2018-05-17 Thread Dietmar Maurer
> we use $elapsed >= $timeout. > > Isn't this timeout for command execution time and not connect time ? > > I'm seeing at the end: > $self->{mux}->set_timeout($fh, $timeout); > > seem to be the command execution time in the muxer > I guess both should be shorter than $timeout?

Re: [pve-devel] pvedaemon hanging because of qga retry

2018-05-17 Thread Alexandre DERUMIER
Seem to be introduced a long time ago in 2012 https://git.proxmox.com/?p=qemu-server.git;a=blobdiff;f=PVE/QMPClient.pm;h=9829986ae77e82d340974e4d4128741ef85b4a0e;hp=d026f4d4c3012203d96660a311b1890e84e6aa18;hb=6d04217600f2145ee80d5d62231b8ade34f2e5ff;hpb=037a97463447b06ebf79a7f1d40c596d9955acee

Re: [pve-devel] pvedaemon hanging because of qga retry

2018-05-17 Thread Alexandre DERUMIER
in qmpclient : open_connection for (;;) { $count++; $fh = IO::Socket::UNIX->new(Peer => $sname, Blocking => 0, Timeout => 1); last if $fh; if ($! != EINTR && $! != EAGAIN) { die "unable to connect to VM $vmid $sotype socket - $!\n"; }

[pve-devel] pvedaemon hanging because of qga retry

2018-05-17 Thread Alexandre DERUMIER
Hi, I had a strange behaviour today, with a vm running + qga enabled, but qga service down in the vm after theses attempts, May 17 21:54:01 kvm14 pvedaemon[20088]: VM 745 qmp command failed - VM 745 qmp command 'guest-fsfreeze-thaw' failed - unable to connect to VM 745 qga socket - timeout

[pve-devel] Little Bug in GUI for iSCSI Storage

2018-05-17 Thread Bastian Sebode
Hey Guys, when I disable an iSCSI Storage it gets disabled in /etc/pve/storage.cfg by adding a "disabled"-line. The GUI shows it correct, but you can't reactivate it because when you edit it, it thinks it is enabled. When you enable it by removing the line in storage.cfg everything is fine

[pve-devel] [PATCH firewall] introduce ebtables_enable option to cluster config

2018-05-17 Thread Stoiko Ivanov
minimally fixes #1764, by introducing ebtables_enable as option in cluster.fw Signed-off-by: Stoiko Ivanov --- Note: A better option would be to just not overwrite any output of ebtables-save, not containing pve-specific interface names or PVE, however this patch should at

Re: [pve-devel] [PATCH v2 storage] Cephfs storage plugin

2018-05-17 Thread Fabian Grünbichler
some general remarks from a quick run-through: - backup does not seem to work, probably requires qemu-server/pve-container changes? [1] - parts of this could be merged with RBDPlugin code, either by extending it, or by refactoring to use a common helper module - ceph-fuse and the kernel have a

[pve-devel] [PATCH V2 ifupdown2 0/2] ifupdown2 package

2018-05-17 Thread Alexandre Derumier
Changelog v2: - use submodule for ifupdown2 src - split proxmox/extra patches - add description in 0004-add-dummy-mtu-bridgevlanport-modules.patch - add a note in this cover letter about systemd-networkd and ipv6 madness Hi, Theses last months, I'm working on vxlan implementation. (I'll send

[pve-devel] [PATCH V2 ifupdown2 1/2] add debian dir

2018-05-17 Thread Alexandre Derumier
--- debian/changelog | 174 + debian/compat | 1 + debian/control | 31 debian/copyright | 28

[pve-devel] [PATCH V2 ifupdown2 2/2] add ifupdown2 submodule

2018-05-17 Thread Alexandre Derumier
--- .gitmodules | 3 +++ ifupdown2 | 1 + 2 files changed, 4 insertions(+) create mode 100644 .gitmodules create mode 16 ifupdown2 diff --git a/.gitmodules b/.gitmodules new file mode 100644 index 000..a9d2457 --- /dev/null +++ b/.gitmodules @@ -0,0 +1,3 @@ +[submodule "ifupdown2"] +

[pve-devel] [PATCH v2 storage] Cephfs storage plugin

2018-05-17 Thread Alwin Antreich
- ability to mount through kernel and fuse client - allow mount options - get MONs from ceph config if not in storage.cfg - allow the use of ceph config with fuse client Signed-off-by: Alwin Antreich --- PVE/API2/Storage/Config.pm | 2 +- PVE/Storage.pm

[pve-devel] applied: [PATCH manager] ui: vm: allow to add socket backed serial devices

2018-05-17 Thread Wolfgang Bumiller
applied On Wed, May 16, 2018 at 03:30:59PM +0200, Thomas Lamprecht wrote: > We show and can remove serial devices but couldn't add new ones > through the WebUI. > Add a simple component to allow adding serial ports backed by a > socket, which can be especially useful now with xterm.js > >

Re: [pve-devel] [PATCH 1/1] initial ifupdown2 package

2018-05-17 Thread Wolfgang Bumiller
On Wed, May 16, 2018 at 12:01:40PM +0200, Alexandre Derumier wrote: > --- > Makefile | 42 + > debian/changelog | 174 > + > debian/compat | 1 + >

[pve-devel] [PATCH v2 storage] Cephfs storage plugin

2018-05-17 Thread Alwin Antreich
- ability to mount through kernel and fuse client - allow mount options - get MONs from ceph config if not in storage.cfg - allow the use of ceph config with fuse client Signed-off-by: Alwin Antreich --- PVE/API2/Storage/Config.pm | 2 +-

[pve-devel] [PATCH v2 manager] Cephfs storage wizard

2018-05-17 Thread Alwin Antreich
Add internal and external storage wizard for cephfs Signed-off-by: Alwin Antreich --- www/manager6/Makefile | 1 + www/manager6/Utils.js | 10 ++ www/manager6/storage/CephFSEdit.js | 71 ++ 3 files

[pve-devel] [RFC v2 storage/manager 0/2] Cephfs storage plugin

2018-05-17 Thread Alwin Antreich
This patch series is an update and adds the Cephfs to our list of storages. You can mount the storage through the kernel or fuse client. The plugin for now allows all content formats, but this needs further testing. Config and keyfile locations are the same as in the RBD plugin. Example entry: