Re: [pve-devel] ZFS encryption

2018-04-04 Thread Fabian Grünbichler
On Tue, Apr 03, 2018 at 08:45:59PM +0200, Andreas Steinel wrote: > Hi everyone, > > are you (Proxmox staff) actively testing encrypted ZFS or are you > waiting for the upstream "activation"? if you are talking about upstream's native encryption, then AFAIK none of us are testing that (yet). it's

Re: [pve-devel] [RFC kernel 2/2] debian/scripts: add import-upstream-tag

2018-04-04 Thread Thomas Lamprecht
Am 04/03/2018 um 01:30 PM schrieb Fabian Grünbichler: $ import-upstream-tag path/to/kernel/submodule path/to/kernel/patches tag [rebase] fetches 'tag' from default remote, optionally imports, rebases and exports patchqueue, checks out 'tag' and commits the resulting changes. Signed-off-by: F

Re: [pve-devel] [RFC manager 4/5] ui: add cluster join window POC

2018-04-04 Thread Dominik Csapak
On 04/03/2018 02:50 PM, Dominik Csapak wrote: On 04/03/2018 02:43 PM, Thomas Lamprecht wrote: Am 04/03/2018 um 01:52 PM schrieb Dominik Csapak: On 04/03/2018 10:49 AM, Thomas Lamprecht wrote: Am 04/03/2018 um 10:16 AM schrieb Dominik Csapak: even with the autoflush patch, i could not get this

Re: [pve-devel] [PATCH librados2-perl] Split method pve_rados_connect

2018-04-04 Thread Alwin Antreich
On Tue, Apr 03, 2018 at 02:13:18PM +0200, Dietmar Maurer wrote: > comments inline > > > On March 30, 2018 at 12:25 PM Alwin Antreich wrote: > > > > > > To be able to connect through librados2 without a config file, the > > method pve_rados_connect is split up into pve_rados_connect and > > pve_rad

Re: [pve-devel] [PATCH librados2-perl] Split method pve_rados_connect

2018-04-04 Thread Alwin Antreich
On Tue, Apr 03, 2018 at 10:25:53AM +0200, Thomas Lamprecht wrote: > > Am 03/30/2018 um 12:25 PM schrieb Alwin Antreich: > > To be able to connect through librados2 without a config file, the > > method pve_rados_connect is split up into pve_rados_connect and > > pve_rados_conf_read_file. > > > > Si

Re: [pve-devel] [RFC manager 0/1] introduce pvesrd, replacing systemd pvesr.{timer, service}

2018-04-04 Thread Dietmar Maurer
My plan was to use pvestatd for that. We need to improve that daemon to run a forked worker for each storage, so I think it would be ease to add this functionality. Would save some memory ... > On April 3, 2018 at 4:02 PM Thomas Lamprecht wrote: > > > This is a POC of replacing the systemd ti

Re: [pve-devel] [RFC kernel 0/2] pve-kernel helper scripts for patch-queue management

2018-04-04 Thread Thomas Lamprecht
Am 04/03/2018 um 01:30 PM schrieb Fabian Grünbichler: this patch series introduces helper scripts for - importing the exported patchqueue into a patchqueue branch inside the submodule - exporting the (updated) patchqueue from the patchqueue branch inside the submodule - importing a new upst

Re: [pve-devel] [RFC kernel 2/2] debian/scripts: add import-upstream-tag

2018-04-04 Thread Fabian Grünbichler
On Wed, Apr 04, 2018 at 11:31:44AM +0200, Thomas Lamprecht wrote: > Am 04/03/2018 um 01:30 PM schrieb Fabian Grünbichler: > > $ import-upstream-tag path/to/kernel/submodule path/to/kernel/patches tag > > [rebase] > > > > fetches 'tag' from default remote, optionally imports, rebases and exports >

[pve-devel] [PATCH kernel 4.13 2/2] bump version to 4.13-46

2018-04-04 Thread Fabian Grünbichler
Signed-off-by: Fabian Grünbichler --- Makefile | 2 +- debian/changelog | 6 ++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 02f07ba..560d3a4 100644 --- a/Makefile +++ b/Makefile @@ -4,7 +4,7 @@ RELEASE=5.1 KERNEL_VER=4.13.16 KREL=1 -PKGR

[pve-devel] [PATCH kernel 4.13 1/2] update SPL/ZFS to 0.7.7

2018-04-04 Thread Fabian Grünbichler
and manually set the executable build on this new helper script Signed-off-by: Fabian Grünbichler --- Makefile | 2 ++ submodules/zfsonlinux | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 553a30f..02f07ba 100644 --- a/Makefile +++ b/M

[pve-devel] [PATCH kernel 4.13 0/2] update ZFS/SPL to 0.7.7

2018-04-04 Thread Fabian Grünbichler
Fabian Grünbichler (2): update SPL/ZFS to 0.7.7 bump version to 4.13-46 Makefile | 4 +++- debian/changelog | 6 ++ submodules/zfsonlinux | 2 +- 3 files changed, 10 insertions(+), 2 deletions(-) -- 2.14.2 ___ pve-devel mai

[pve-devel] [PATCH kernel-meta] bump version to 5.1-3

2018-04-04 Thread Fabian Grünbichler
bump ABI to 4.15.15-1-pve Signed-off-by: Fabian Grünbichler --- Makefile | 4 ++-- debian/changelog | 6 ++ 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index cea6489..6fe46c1 100644 --- a/Makefile +++ b/Makefile @@ -1,8 +1,8 @@ RELEASE=5.1 -PKG

[pve-devel] [PATCH kernel 4.15 0/5] update to 4.15.0-14.15, ZFS 0.7.7

2018-04-04 Thread Fabian Grünbichler
pve-kernel-meta / pve-kernel-4.15: Fabian Grünbichler (1): bump version to 5.1-3 Makefile | 4 ++-- debian/changelog | 6 ++ 2 files changed, 8 insertions(+), 2 deletions(-) pve-kernel / pve-kernel-4.15: Fabian Grünbichler (5): update sources to Ubuntu-4.15.0-14.15 rebase pat

[pve-devel] [PATCH kernel 4.15 3/5] revert broken kernel Makefile modification

2018-04-04 Thread Fabian Grünbichler
see [1] for upstream report 1: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1758856/comments/1 Signed-off-by: Fabian Grünbichler --- Note: this will likely need to be rebased on the next upstream release, and a similar changed will be required for the next 4.4 and 4.13 upstream imports as

[pve-devel] [PATCH kernel 4.15 4/5] update SPL/ZFS to 0.7.7

2018-04-04 Thread Fabian Grünbichler
and manually set the executable build on this new helper script Signed-off-by: Fabian Grünbichler --- Makefile | 2 ++ submodules/zfsonlinux | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/Makefile b/Makefile index aaf55f2..58c6094 100644 --- a/Makefile +++ b/M

[pve-devel] [PATCH kernel 4.15 5/5] bump version to 4.15-5, bump ABI to 4.15.15-1-pve

2018-04-04 Thread Fabian Grünbichler
Signed-off-by: Fabian Grünbichler --- Makefile | 4 ++-- debian/changelog | 10 ++ 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index 58c6094..3ddd72f 100644 --- a/Makefile +++ b/Makefile @@ -1,10 +1,10 @@ RELEASE=5.1 # also update pve-k

[pve-devel] [PATCH kernel 4.15 2/5] rebase patches on top of Ubuntu-4.15.0-14.15

2018-04-04 Thread Fabian Grünbichler
and drop those applied upstream Signed-off-by: Fabian Grünbichler --- ...overrides-for-missing-ACS-capabilities-4..patch | 6 +- ...-not-wait-for-lock_page-in-shmem_unused_h.patch | 103 - ...ot-wait-for-lock_page-in-deferred_split_s.patch | 46 - 3 files changed,

[pve-devel] [PATCH kernel 4.15 1/5] update sources to Ubuntu-4.15.0-14.15

2018-04-04 Thread Fabian Grünbichler
Signed-off-by: Fabian Grünbichler --- submodules/ubuntu-bionic | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/submodules/ubuntu-bionic b/submodules/ubuntu-bionic index 6dc5db9..e4a338d 16 --- a/submodules/ubuntu-bionic +++ b/submodules/ubuntu-bionic @@ -1 +1 @@ -Subprojec

[pve-devel] [PATCH manager v2 3/5] dc/Cluster: allow to get join information

2018-04-04 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- changes v1 -> v2: * no need for join info window to be resizable (Dominik) www/manager6/dc/Cluster.js | 20 +++ www/manager6/dc/ClusterEdit.js | 77 ++ 2 files changed, 97 insertions(+) diff --git a/www/man

[pve-devel] [PATCH manager v2 2/5] dc/Cluster: allow cluster create over WebUI

2018-04-04 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- changes v1 -> v2: * add 'allowBlank: false' to cluster name field, it's required by the API (Dominik) www/manager6/Makefile | 1 + www/manager6/dc/Cluster.js | 23 +++ www/manager6/dc/ClusterEdit.js | 32 +++

[pve-devel] [PATCH manager v2 0/5] cluster create/join UI

2018-04-04 Thread Thomas Lamprecht
Second iteration of the WebUI Cluster Create, Join series[1]. Changes between v1 and v2 are described in each respective patch. Look at [1] for the v1 cover letter. Dominik comments, and a bit more, should be addressed now. While 4/5 is not my ideal looking UI, it's simple and works, we can enhan

[pve-devel] [PATCH manager v2 1/5] dc: add simple cluster panel

2018-04-04 Thread Thomas Lamprecht
Show configured cluster nodes with their addresses, votes, IDs. Also show cluster name, config_version, and node count. Prepares for creating and joining a cluster over the WebUI. Signed-off-by: Thomas Lamprecht --- changes v1 -> v2: * re-used an already defined gettext for "no cluster configur

[pve-devel] [PATCH manager v2 5/5] ui: silence auth failures during cluster join

2018-04-04 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- no changes www/manager6/Workspace.js | 2 +- www/manager6/dc/ClusterEdit.js | 12 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/www/manager6/Workspace.js b/www/manager6/Workspace.js index c7b88761..f75356c5 100644 --- a/www

[pve-devel] [PATCH manager v2 4/5] ui: add cluster join window POC

2018-04-04 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- no changes www/manager6/dc/Cluster.js | 21 + www/manager6/dc/ClusterEdit.js | 190 + 2 files changed, 211 insertions(+) diff --git a/www/manager6/dc/Cluster.js b/www/manager6/dc/Cluster.js index 8e955f76..b04

[pve-devel] applied: [PATCH kernel 4.13 0/2] update ZFS/SPL to 0.7.7

2018-04-04 Thread Thomas Lamprecht
Am 04/04/2018 um 02:20 PM schrieb Fabian Grünbichler: Fabian Grünbichler (2): update SPL/ZFS to 0.7.7 bump version to 4.13-46 Makefile | 4 +++- debian/changelog | 6 ++ submodules/zfsonlinux | 2 +- 3 files changed, 10 insertions(+), 2 deletions(-) applied. Wh

Re: [pve-devel] ZFS encryption

2018-04-04 Thread Andreas Steinel
Hi Fabian, On Wed, Apr 4, 2018 at 9:45 AM, Fabian Grünbichler wrote: > On Tue, Apr 03, 2018 at 08:45:59PM +0200, Andreas Steinel wrote: >> Hi everyone, >> >> are you (Proxmox staff) actively testing encrypted ZFS or are you >> waiting for the upstream "activation"? > > if you are talking about up

Re: [pve-devel] ZFS encryption

2018-04-04 Thread Fabian Grünbichler
On Wed, Apr 04, 2018 at 03:47:57PM +0200, Andreas Steinel wrote: > Hi Fabian, > > On Wed, Apr 4, 2018 at 9:45 AM, Fabian Grünbichler > wrote: > > On Tue, Apr 03, 2018 at 08:45:59PM +0200, Andreas Steinel wrote: > >> Hi everyone, > >> > >> are you (Proxmox staff) actively testing encrypted ZFS or

Re: [pve-devel] ZFS encryption

2018-04-04 Thread Andreas Steinel
On Wed, Apr 4, 2018 at 4:02 PM, Fabian Grünbichler wrote: >> > do you have specific use cases in mind? >> You mean besides fulfilling GDPR? > > I meant more from a technical point of view ;) the reasons for wanting > to encrypt stuff are rather obvious, although there are a lot. Best would be if

[pve-devel] [PATCH v2 librados2-perl 2/3] Split method pve_rados_connect

2018-04-04 Thread Alwin Antreich
To be able to connect through librados2 without a config file, the method pve_rados_connect is split up into pve_rados_connect and pve_rados_conf_read_file. Signed-off-by: Alwin Antreich --- changes from v1 -> v2: - die if the supplied ceph config in %params does not exist - removed the FIXME,

[pve-devel] [PATCH v2 librados2-perl 1/3] white space cleanup

2018-04-04 Thread Alwin Antreich
Signed-off-by: Alwin Antreich --- note: seems that, this one is need for the following patches to apply PVE/RADOS.pm | 12 ++-- RADOS.xs | 32 2 files changed, 22 insertions(+), 22 deletions(-) diff --git a/PVE/RADOS.pm b/PVE/RADOS.pm index aa6a102..

[pve-devel] [PATCH v2 librados2-perl 3/3] allow to specify the userid with rados_create

2018-04-04 Thread Alwin Antreich
This allows to connect to a cluster with a different user, besides admin Signed-off-by: Alwin Antreich --- PVE/RADOS.pm | 4 +++- RADOS.xs | 13 ++--- 2 files changed, 13 insertions(+), 4 deletions(-) diff --git a/PVE/RADOS.pm b/PVE/RADOS.pm index 2ed92b7..d53f655 100644 --- a/PVE/

Re: [pve-devel] [PATCH v2 librados2-perl 1/3] white space cleanup

2018-04-04 Thread Thomas Lamprecht
Still doesn't applies I recreated your patch with s/\s+$//g and compared, all the empty "stay lines" (i.e., lines from context which are not touched by the patch) miss their initial space (in patches the first character in a line is special: '-' means remove, '+' means add line, ' ' (spac