[pve-devel] [PATCH v6 manager 0/6] add pveceph install to gui

Tim Marx t.marx at proxmox.com
Wed Feb 27 15:01:14 CET 2019


v6:
Now a wizard is available which guides through the installation and
afterwards, if needed, it will offer to init ceph config.
Therefore I added the cmd paramater to the inline console as well,
resulting in an additional commit.

Some minor things still need some polish e.g. inline terminal connection
errors after successfull installation and info text after installation.

v5:
I added the install window to the remaining submenus.
This needed some changes regarding error handling to get the same
behavior for all subviews.

Only the manager repo got touched.

v4:
I addressed various inputs:
https://pve.proxmox.com/pipermail/pve-devel/2019-January/035387.html
https://pve.proxmox.com/pipermail/pve-devel/2019-January/035377.html
https://pve.proxmox.com/pipermail/pve-devel/2019-January/035378.html

v3:
I headed back to my original approach and declare the cmd parameter 
in each api endpoint separately. This results again in only 4 repos
touched instead of 5. This change is reasoned by Thomas input [0].
I decided against a module level variable because I think it improves
the readability.

All repos since v2 are untouched except the pve-manager.
Changed some commit messages slightly.

[0] https://pve.proxmox.com/pipermail/pve-devel/2019-January/035358.html

v2:
In general I changed the approach from passing a command directly to
passing a command keyword which gets matched in the api and not
directly executed. Only the following two out of now 5 (v1 were 4) repos 
got changed since v1.

v1:
With this patch series and the corresponding patches in the following repositories
its now possible to install ceph via the gui in the same way as someone would if using the
pveceph install command manually

Affected repositories:

*)pve-manager:
  * altered the ceph install check to check for the mon binary, 
    because the ceph binary is likely to be there already
  * added parameter to all 3 consoles (novnc,xterm,spice) to pass a custom command
  * in the gui the ceph status page checks if the not install error returns and 
    if so, shows a window which offers to install ceph

*)pve-xtermjs:
  * added new cmd paramater 
*)proxmox-widget-toolkit:
  * added new cmd paramater
*)novnc-pve:
  * added new cmd paramater


Tim Marx (6):
  unified the classification of the passed command for all shell apis
  deprecate old upgrade property
  add pveceph install to shell api
  added new parameter to pass a command to the newly opened terminal
  add cmd parameter & rework query string creation
  added basic ability to install ceph via gui

 PVE/API2/Nodes.pm                      |  98 ++++++++-----
 www/css/ext6-pve.css                   |   5 +
 www/manager6/Makefile                  |   3 +
 www/manager6/Utils.js                  |  52 ++++++-
 www/manager6/VNCConsole.js             |  32 +++-
 www/manager6/button/ConsoleButton.js   |  12 +-
 www/manager6/ceph/CephInstallWizard.js | 259 +++++++++++++++++++++++++++++++++
 www/manager6/ceph/Config.js            |  16 ++
 www/manager6/ceph/Crush.js             |  16 ++
 www/manager6/ceph/FS.js                |  42 +++++-
 www/manager6/ceph/Log.js               |  70 +++++++++
 www/manager6/ceph/Monitor.js           |  19 ++-
 www/manager6/ceph/OSD.js               |  19 ++-
 www/manager6/ceph/Pool.js              |  18 ++-
 www/manager6/ceph/Status.js            |  21 ++-
 www/manager6/node/Config.js            |   5 +-
 www/manager6/window/CephInstall.js     |  66 +++++++++
 17 files changed, 685 insertions(+), 68 deletions(-)
 create mode 100644 www/manager6/ceph/CephInstallWizard.js
 create mode 100644 www/manager6/ceph/Log.js
 create mode 100644 www/manager6/window/CephInstall.js

novnc:

Tim Marx (1):
  novnc: added new parameter to pass a command to the newly opened
    terminal

 .../patches/0012-pass-custom-command-to-vnc.patch  | 38 ++++++++++++++++++++++
 debian/patches/series                              |  1 +
 2 files changed, 39 insertions(+)
 create mode 100644 debian/patches/0012-pass-custom-command-to-vnc.patch

xterm:

Tim Marx (1):
  xterm: added new parameter to pass a command to the newly opened
    terminal

 src/www/main.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

widget:

Tim Marx (1):
  widget: added new parameter to pass a command to the newly opened
    terminal

 Utils.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)
-- 
2.11.0




More information about the pve-devel mailing list