Re: [pve-devel] XenServer to Proxmox in a single step

2017-11-27 Thread Gandalf Corvotempesta
Gilberto, did you try my script? Il 23 nov 2017 2:26 PM, "Gilberto Nunes" ha scritto: > Nice! Thanks > > --- > Gilberto Ferreira > > (47) 3025-5907 > (47) 99676-7530 > > Skype: gilberto.nunes36 > > > > > 2017-11-23 11:22 GMT-02:00 Gandalf Corvotempesta < > gandalf.corvotempe...@gmail.com>: > > >

[pve-devel] applied 1..3 [PATCH common 1/4] Add PVE::PTY helper class

2017-11-27 Thread Wolfgang Bumiller
Applied patches 1..3, waiting for feedback on 4. On Fri, Nov 24, 2017 at 11:24:22AM +0100, Wolfgang Bumiller wrote: > Signed-off-by: Wolfgang Bumiller > --- > This will be used primarily by the xtermjs console client. > I'm adding this to pve-common since it also presents the > opportunity to add

[pve-devel] [RFC cluster 2/6] node add: factor out code

2017-11-27 Thread Thomas Lamprecht
Factor out common code, which will be used by the new API endpoint to join a cluster and the old legacy SSH method which we will keep for a bit. Signed-off-by: Thomas Lamprecht --- data/PVE/CLI/pvecm.pm | 152 ++ data/PVE/Cluster.pm | 146 +++

[pve-devel] [RFC cluster 6/6] pvecm: add: use API by default

2017-11-27 Thread Thomas Lamprecht
Default to using the API for a add node procedure. But, allow the user to manually fall back to the legacy SSH method. Also fallback if the API detected an not up to date peer. This could be removed in a later release. Signed-off-by: Thomas Lamprecht --- data/PVE/CLI/pvecm.pm | 44 +

[pve-devel] [RFC cluster 4/6] api: add joint cluster endpoint

2017-11-27 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- data/PVE/API2/ClusterConfig.pm | 125 - debian/control.in | 2 + 2 files changed, 126 insertions(+), 1 deletion(-) diff --git a/data/PVE/API2/ClusterConfig.pm b/data/PVE/API2/ClusterConfig.pm index c38feb2

[pve-devel] [RFC cluster 5/6] api/join: check if peer supports addnode over API

2017-11-27 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- I tend to not check for the version, but simply do the POST request on "/cluster/config/join" and handle the "not implemented" error instead, seems to be straight forward and does not needs an timed version bump. data/PVE/API2/ClusterConfig.pm | 21 ++

[pve-devel] [RFC cluster 1/6] move addnode/delnode from CLI to cluster config API

2017-11-27 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- data/PVE/API2/ClusterConfig.pm | 210 ++ data/PVE/CLI/pvecm.pm | 223 + 2 files changed, 213 insertions(+), 220 deletions(-) diff --git a/data/PVE/API2/ClusterConfig.pm b/data

[pve-devel] [RFC cluster 0/6] Allow adding a node to a cluster over API

2017-11-27 Thread Thomas Lamprecht
Initially, I picked up Dietmars work on this posted last year: https://pve.proxmox.com/pipermail/pve-devel/2016-November/024222.html Adopting the then not-existing PVE API client library. Fabian then told me that the idea of a cluster CA was rejected as not ideal in off-list discussion, so I droppe

[pve-devel] [RFC cluster 3/6] return cluster config and authkey in addnode API call

2017-11-27 Thread Thomas Lamprecht
Signed-off-by: Thomas Lamprecht --- data/PVE/API2/ClusterConfig.pm | 22 -- 1 file changed, 20 insertions(+), 2 deletions(-) diff --git a/data/PVE/API2/ClusterConfig.pm b/data/PVE/API2/ClusterConfig.pm index fa01022..c38feb2 100644 --- a/data/PVE/API2/ClusterConfig.pm +++ b/d

[pve-devel] [PATCH manager] fix #1578: api/resource hastate: fallback to resource cfg state

2017-11-27 Thread Thomas Lamprecht
With ignored or still queued services we have no hastate for a service in the manager status available. As we use hastate in the web UI to determine if a service is configured for HA this could lead to confusion there. For example, the VM/CT 'Manage HA' window thinks tries to add the service again

[pve-devel] applied: [PATCH cluster] factor out reading a nodes ssl cert fingerprint

2017-11-27 Thread Fabian Grünbichler
and improved the Net::SSLeay error handling ___ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Re: [pve-devel] [PATCH manager v2 3/3] Add a proper confirmation message when we detach an used disk

2017-11-27 Thread Dominik Csapak
On 11/15/2017 11:54 AM, Emmanuel Kasper wrote: Signed-off-by: Emmanuel Kasper --- www/manager6/qemu/HardwareView.js | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/www/manager6/qemu/HardwareView.js b/www/manager6/qemu/HardwareView.js index 2cd0c940..e6bc023

Re: [pve-devel] [PATCH manager v2 2/3] Fix #1526: Use 'detach' instead of 'remove' when the disk is used

2017-11-27 Thread Dominik Csapak
comment inline: On 11/15/2017 11:54 AM, Emmanuel Kasper wrote: Signed-off-by: Emmanuel Kasper --- www/manager6/qemu/HardwareView.js | 28 +++- 1 file changed, 23 insertions(+), 5 deletions(-) diff --git a/www/manager6/qemu/HardwareView.js b/www/manager6/qemu/Hardwar

[pve-devel] applied: [PATCH cluster] setup_sshd_config: remove useless start_sshd parameter

2017-11-27 Thread Wolfgang Bumiller
applied On Thu, Nov 23, 2017 at 09:57:59AM +0100, Thomas Lamprecht wrote: > This controlled if we use reload-or-restart or try-reload-or-restart. > They differ in the following way: > > reload-or-restart - Reload one or more units if possible, otherwise > > start or restart > > > > try-reload-or-r

[pve-devel] applied: [PATCH cluster 1/2] pvecm: check early if the deletion cannot work

2017-11-27 Thread Wolfgang Bumiller
applied both patches On Thu, Nov 23, 2017 at 12:12:04PM +0100, Thomas Lamprecht wrote: > Signed-off-by: Thomas Lamprecht > --- > data/PVE/CLI/pvecm.pm | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/data/PVE/CLI/pvecm.pm b/data/PVE/CLI/pvecm.pm > index 9437368..ba

[pve-devel] applied: [PATCH container] add 'noescapechar' parameter to get_console_command

2017-11-27 Thread Wolfgang Bumiller
applied On Fri, Nov 24, 2017 at 03:43:32PM +0100, Dominik Csapak wrote: > this adds a '-e -1' to lxc-console, and disables the > escape character (which we do not want with the web console) > > Signed-off-by: Dominik Csapak > --- > src/PVE/API2/LXC.pm | 2 +- > src/PVE/LXC.pm | 13 +++