Re: [pve-devel] Bug 1458 - PVE 5 live migration downtime degraded to several seconds (compared to PVE 4)

2017-08-09 Thread Alexandre DERUMIER
Just tested the patches on my old xeon, works fine, around 100ms of pause max. But, it don't work will migration insecure (I think we should established the mtunnel also for insecure migration, to pass resume command) - Mail original - De: "aderumier" À:

Re: [pve-devel] Bug 1458 - PVE 5 live migration downtime degraded to several seconds (compared to PVE 4)

2017-08-09 Thread Fabian Grünbichler
On Tue, Aug 08, 2017 at 07:02:21PM +0200, Alexandre DERUMIER wrote: > Hi, > > I'm back from holiday, > > I'll test last mtunnel patches tomorrow . Thanks ! the relevant ones in qemu-server are already applied and packages are available in pve-no-subscription :) results in the forum indicate an

Re: [pve-devel] Bug 1458 - PVE 5 live migration downtime degraded to several seconds (compared to PVE 4)

2017-08-09 Thread Alexandre DERUMIER
>>I will see how we can speed upthe one with local storage a bit more as well, >>probably next week. I'm not sure how we can do faster pending write flush on block-job-complete. Maybe implementing COLO in the future (not sure it all parts are already implemented in qemu) could help to have

[pve-devel] [PATCH manager 5/5] add summary to the ceph warning window

2017-08-09 Thread Dominik Csapak
and format it better Signed-off-by: Dominik Csapak --- www/manager6/ceph/Status.js | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/www/manager6/ceph/Status.js b/www/manager6/ceph/Status.js index 132c5e0a..b0527337 100644 ---

[pve-devel] [PATCH manager 4/5] make the ceph warning window modal

2017-08-09 Thread Dominik Csapak
Signed-off-by: Dominik Csapak --- www/manager6/ceph/Status.js | 1 + 1 file changed, 1 insertion(+) diff --git a/www/manager6/ceph/Status.js b/www/manager6/ceph/Status.js index b5130986..132c5e0a 100644 --- a/www/manager6/ceph/Status.js +++ b/www/manager6/ceph/Status.js @@

[pve-devel] [PATCH manager 3/5] fix ceph gui icons

2017-08-09 Thread Dominik Csapak
this adds a new css class for fontawesome which we need under certain circumstances (it is copied from fontawesome minus a font-size:inherited) and it changes the icons in the osd tree from x-fa to fa (which is the correct one there) Signed-off-by: Dominik Csapak ---

[pve-devel] [PATCH manager 2/5] ceph: add application parameter to createpool

2017-08-09 Thread Dominik Csapak
and use 'rbd' by default, since we cannot create a cephfs or rgw from the gui currently Signed-off-by: Dominik Csapak --- PVE/API2/Ceph.pm | 13 + 1 file changed, 13 insertions(+) diff --git a/PVE/API2/Ceph.pm b/PVE/API2/Ceph.pm index 7cf07b70..c4d6ffcb 100644

[pve-devel] [PATCH manager] build: ensure REPOID has a length of 8

2017-08-09 Thread Fabian Grünbichler
because unlike the git-rev-parse man page says, the default length is actually calculated based on the number of packed objects in the clone of the repository. we don't want this to depend on the build environment and how the repository was cloned there, so fix a length of 8 for now.

[pve-devel] [PATCH manager 1/5] ceph: make size 3/min 2 default with createpool

2017-08-09 Thread Dominik Csapak
in the gui this is already the default, so make it also the default in the backend (also 2/1 is really bad as a default) Signed-off-by: Dominik Csapak --- PVE/API2/Ceph.pm | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/PVE/API2/Ceph.pm

[pve-devel] applied: [PATCH manager] build: ensure REPOID has a length of 8

2017-08-09 Thread Wolfgang Bumiller
applied... On Wed, Aug 09, 2017 at 11:42:32AM +0200, Fabian Grünbichler wrote: > because unlike the git-rev-parse man page says, the default > length is actually calculated based on the number of packed > objects in the clone of the repository. > > we don't want this to depend on the build

[pve-devel] applied: [PATCH docs] added missing ipv4/ip_forward in routed network section

2017-08-09 Thread Wolfgang Bumiller
applied On Wed, Aug 02, 2017 at 10:43:30AM +0200, Alwin Antreich wrote: > Signed-off-by: Alwin Antreich > --- > pve-network.adoc | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/pve-network.adoc b/pve-network.adoc > index 315ba63..62cad85 100644 > ---

[pve-devel] [RFC PATCH 1/2] do not use PVE::API2 in spiceproxy.pm

2017-08-09 Thread Dominik Csapak
we do not need it there and withouth this we save ~30MB memory for this daemon and its workers Signed-off-by: Dominik Csapak --- PVE/Service/spiceproxy.pm | 1 - 1 file changed, 1 deletion(-) diff --git a/PVE/Service/spiceproxy.pm b/PVE/Service/spiceproxy.pm index

[pve-devel] [RFC PATCH 2/2] add ver get parameter for js/css files to improve cache behaviour

2017-08-09 Thread Dominik Csapak
we do not set an Expire header for our js files, and certain browsers do not even make the request then (on which we would answer with a correct 304 not modified) so to force the browser to load a new version of the gui when we change something, we add the package version as a get parameter when

[pve-devel] [RFC PATCH 0/2] daemon and cache changes

2017-08-09 Thread Dominik Csapak
i post this as rfc because i am not completely sure if i overlooked some implications of those code changes the first patch removes a use statement from the spiceproxy daemon, which saves about ~30M memory. i tested it and it worked, but maybe someone else can test this again? the second patch

Re: [pve-devel] [RFC PATCH 1/2] do not use PVE::API2 in spiceproxy.pm

2017-08-09 Thread Dietmar Maurer
Looks quite strange to me, because PVE::HTTPServer calls: PVE::API2->find_handler so we should use PVE::API2 in PVE::HTTPServer? Not sure why it works without?? > On August 9, 2017 at 2:08 PM Dominik Csapak wrote: > > > we do not need it there and withouth this we