[pve-devel] [PATCH http-server 4/4] bsys: add Utils.pm to install target

2019-02-22 Thread Stoiko Ivanov
Signed-off-by: Stoiko Ivanov --- Makefile | 1 + PVE/APIServer/AnyEvent.pm | 0 2 files changed, 1 insertion(+) mode change 100755 => 100644 PVE/APIServer/AnyEvent.pm diff --git a/Makefile b/Makefile index 5c5b088..e23ee97 100644 --- a/Makefile +++ b/Makefile @@ -57,6 +57,7 @@

[pve-devel] [PATCH manager 2/4] remove default values from pveproxy ssl config

2019-02-22 Thread Stoiko Ivanov
they are in PVE::APIServer::AnyEvent Signed-off-by: Stoiko Ivanov --- PVE/Service/pveproxy.pm | 14 +++--- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/PVE/Service/pveproxy.pm b/PVE/Service/pveproxy.pm index ee74db4c..e63c90ea 100755 --- a/PVE/Service/pveproxy.pm +++

[pve-devel] [PATCH manager 4/4] remove read_proxy_conf from PVE::API2Tools

2019-02-22 Thread Stoiko Ivanov
Signed-off-by: Stoiko Ivanov --- PVE/API2Tools.pm | 46 -- 1 file changed, 46 deletions(-) diff --git a/PVE/API2Tools.pm b/PVE/API2Tools.pm index b2e3a6bd..2d1bdf25 100644 --- a/PVE/API2Tools.pm +++ b/PVE/API2Tools.pm @@ -211,52 +211,6 @@ sub

[pve-devel] [PATCH manager 3/4] pveproxy, spiceproxy: new read_proxy_conf location

2019-02-22 Thread Stoiko Ivanov
Signed-off-by: Stoiko Ivanov --- PVE/Service/pveproxy.pm | 4 ++-- PVE/Service/spiceproxy.pm | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/PVE/Service/pveproxy.pm b/PVE/Service/pveproxy.pm index e63c90ea..467748b1 100755 --- a/PVE/Service/pveproxy.pm +++

[pve-devel] [PATCH 0/8] move ssl-defaults and proxy-configuration from pve-manager to pve-http-server

2019-02-22 Thread Stoiko Ivanov
Moving the parsing of '/etc/default/pveproxy' from PVE::API2Tools (in pve-manager) to PVE::APIServer::Utils (pve-http-server) (and making the configfile variable) enables us to use the same logic in pmgproxy. Additionally the default-values for SSL-related settings are put into

[pve-devel] [PATCH http-server 2/4] Move read_proxy_conf from PVE::API2Tools

2019-02-22 Thread Stoiko Ivanov
into new module PVE::APIServer::Utils. It now takes the proxy_name (e.g. pveproxy, pmgproxy) as variable to be used for the configfile location (/etc/default/$proxy_name) This serves as preparation to make pmgproxy configurable in the same way as pveproxy. Signed-off-by: Stoiko Ivanov ---

[pve-devel] [PATCH manager 1/4] add dependency on pve-http-server (>= 2.0.12)

2019-02-22 Thread Stoiko Ivanov
needed for the move of read_proxy_conf from PVE::API2Tools to PVE::APIServer::Utils Signed-off-by: Stoiko Ivanov --- debian/control | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/debian/control b/debian/control index 3618fa4a..819060f1 100644 --- a/debian/control +++

[pve-devel] [PATCH http-server 1/4] add ssl fallback values to AnyEvent->new

2019-02-22 Thread Stoiko Ivanov
This allows for sharing the values between pveproxy and pmgproxy Signed-off-by: Stoiko Ivanov --- PVE/APIServer/AnyEvent.pm | 18 ++ 1 file changed, 18 insertions(+) diff --git a/PVE/APIServer/AnyEvent.pm b/PVE/APIServer/AnyEvent.pm index 468edb4..b2330a5 100755 ---

[pve-devel] [PATCH http-server 3/4] Add short comment about users of read_proxy_config

2019-02-22 Thread Stoiko Ivanov
Signed-off-by: Stoiko Ivanov --- PVE/APIServer/Utils.pm | 2 ++ 1 file changed, 2 insertions(+) diff --git a/PVE/APIServer/Utils.pm b/PVE/APIServer/Utils.pm index 36b9c1f..e843e5f 100644 --- a/PVE/APIServer/Utils.pm +++ b/PVE/APIServer/Utils.pm @@ -5,6 +5,8 @@ use warnings; use Net::IP; +#

Re: [pve-devel] [PATCH common] fix #2111: regex match email address without tld

2019-02-22 Thread Oguz Bektas
do not apply this (just noticed after some internal discussion with thomas and dominik that it could possibly cause problems), i'll take a better look on monday. On Fri, Feb 22, 2019 at 03:10:13PM +0100, Oguz Bektas wrote: > for matching root@localhost or similar. > > expression taken from

[pve-devel] [PATCH common] fix #2111: regex match email address without tld

2019-02-22 Thread Oguz Bektas
for matching root@localhost or similar. expression taken from emailregex.com for html5. Signed-off-by: Oguz Bektas --- src/PVE/JSONSchema.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm index 36fa994..b5f9621 100644 ---

[pve-devel] [PATCH v3 cluster] allow to add and delete qdevice to cluster

2019-02-22 Thread Oguz Bektas
v3 changes: * die if standalone mode * remove redundant quorum checks, as we already check if all nodes are online * remove 'ERROR' from print statements * move the cluster member even/odd count check near the start Signed-off-by: Oguz Bektas --- data/PVE/CLI/pvecm.pm | 250

Re: [pve-devel] applied: [PATCH 0/7] Add zsh command completion generation

2019-02-22 Thread Thomas Lamprecht
On 2/22/19 12:57 PM, Fabian Grünbichler wrote: > On Fri, Feb 22, 2019 at 11:53:49AM +0100, Thomas Lamprecht wrote: >> On 2/21/19 2:24 PM, Christian Ebner wrote: >>> Adds the zsh command completion generation for most of the pve CLI tools. >>> >>> [...] >> >> applied series, many thanks! > >

Re: [pve-devel] applied: [PATCH 0/7] Add zsh command completion generation

2019-02-22 Thread Fabian Grünbichler
On Fri, Feb 22, 2019 at 11:53:49AM +0100, Thomas Lamprecht wrote: > On 2/21/19 2:24 PM, Christian Ebner wrote: > > Adds the zsh command completion generation for most of the pve CLI tools. > > > > [...] > > applied series, many thanks! doesn't this also require a versioned build-depends on

[pve-devel] applied: [PATCH 0/7] Add zsh command completion generation

2019-02-22 Thread Thomas Lamprecht
On 2/21/19 2:24 PM, Christian Ebner wrote: > Adds the zsh command completion generation for most of the pve CLI tools. > > pve-firewall: > Christian Ebner (1): > 1891 Add zsh command completion for pve-firewall > > src/Makefile | 8 +++- > 1 file changed, 7 insertions(+), 1 deletion(-) >

Re: [pve-devel] [RFC v2 cluster] allow to add and delete qdevice to cluster

2019-02-22 Thread Thomas Lamprecht
On 2/21/19 3:10 PM, Oguz Bektas wrote: > Allows adding and deleting qdevice through pvecm. > > Requirements: > * All hosts need corosync-qdevice installed. > * Box serving as QDevice needs corosync-qnetd installed. > * Root SSH access from Proxmox host to QDevice > > Original email with patch

[pve-devel] [PATCH docs v2 1/1] add ivshmem documentation

2019-02-22 Thread Dominik Csapak
Signed-off-by: Dominik Csapak --- qm.adoc | 19 +++ 1 file changed, 19 insertions(+) diff --git a/qm.adoc b/qm.adoc index 30e473f..77fec98 100644 --- a/qm.adoc +++ b/qm.adoc @@ -703,6 +703,25 @@ you need to set the client resolution in the OVMF menu(which you can reach with a

[pve-devel] [PATCH qemu-server/docs v2] add ivshmem device and docs

2019-02-22 Thread Dominik Csapak
adds ivshmem device to qemu config, and add the documentation for an explanation and use case changes since v1: * include it in local resources to prevent live migration * delete it always on stop and document it in the description * add documentation Dominik Csapak (1): add ivshmem device to

[pve-devel] [PATCH qemu-server v2 1/1] add ivshmem device to config

2019-02-22 Thread Dominik Csapak
with such a shared memory device, a vm can share data with other vms or with the host via memory one of the use cases is looking-glass[1] with pci-passthrough, which copies the guest fb to the host and you get a high-speed, low-latency display client for the vm on vm stop we delete the file

Re: [pve-devel] proxmox wiki update: removing No-Subscription debian repo

2019-02-22 Thread Tim Marx
We will rework the installation chapter and I will consider your input, thanks again for the contribution! > Herman van Rink hat am 22. Februar 2019 um 11:04 > geschrieben: > > > On 2/20/19 10:05 AM, Fabian Grünbichler wrote: > > On Thu, Feb 14, 2019 at 10:30:54AM +0100, Herman van Rink

Re: [pve-devel] proxmox wiki update: removing No-Subscription debian repo

2019-02-22 Thread Herman van Rink
On 2/20/19 10:05 AM, Fabian Grünbichler wrote: > On Thu, Feb 14, 2019 at 10:30:54AM +0100, Herman van Rink wrote: >> Hi, >> >> Is this the best place to suggest changes to the pve wiki? It seems that >> account creation is disabled there these days. >> >> The installation steps[1] have a section

[pve-devel] applied: [PATCH qemu-server] fix #2101: ipv6 ending in ':' not parsed as a string

2019-02-22 Thread Thomas Lamprecht
On 2/21/19 4:18 PM, David Limbeck wrote: > Space or newline after ':' is recognized as a mapping and as a result an > ipv6 ending in ':' is not parsed as a string. The solution is to quote > the address. For consistency all other addresses (including mac) are > quoted. > > Signed-off-by: David