On Tue, Jun 12, 2018 at 02:48:48PM +0200, Thomas Lamprecht wrote:
> On 6/11/18 11:31 AM, Alwin Antreich wrote:
> > - ability to mount through kernel and fuse client
> > - allow mount options
> > - get MONs from ceph config if not in storage.cfg
> > - allow the use of ceph config with fuse clien
On 06/12/2018 03:36 PM, Thomas Lamprecht wrote:
On 6/12/18 12:33 PM, Dominik Csapak wrote:
we have a param_mapping now, with which we can impement that
e.g. instead of using:
sub read_password {
# read password
}
we can do:
sub param_mapping {
my ($name) = @_;
my $password_ma
Please note that we group similar commands inside a single command class.
Our CLIHandler then nicely groups this commands inside help.
Signed-off-by: Dietmar Maurer
---
Makefile | 1 +
PVE/APIClient/Commands/GuestStatus.pm | 76 +++
P
hi, thanks for the patches
i looked briefly over them (and will dive deeper into the code in the
following days) and played a little around
here an initial review of what i saw/found
nitpicks:
you used the wrong git repository in the subject (container vs storage),
not really important, but
we do not complete them in bash copletion either and their just there
for backward compatibillity, so filter them out.
Signed-off-by: Thomas Lamprecht
---
src/PVE/CLIHandler.pm | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/PVE/CLIHandler.pm b/src/PVE/CLIHandler.pm
in
before:
$ pveum acl
ERROR: incomplete command 'pveum acl'
USAGE: pveum [ARGS] [OPTIONS]
pveum acl delete --roles [OPTIONS]
pveum acl modify --roles [OPTIONS]
pveum group add [OPTIONS]
pveum group delete
pveum group modify [OPTIONS]
pveum role add
we have a param_mapping now, with which we can impement that
e.g. instead of using:
sub read_password {
# read password
}
we can do:
sub param_mapping {
my ($name) = @_;
my $password_map = ['password', sub{
# read password
}, ' [$password_map],
'apicall2' => [$p
applied
___
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
---
PVE/APIClient/Commands/GuestStatus.pm | 44 +++
pveclient | 3 ++-
2 files changed, 46 insertions(+), 1 deletion(-)
diff --git a/PVE/APIClient/Commands/GuestStatus.pm
b/PVE/APIClient/Commands/GuestStatus.pm
index 7903a2b..50730db 1
applied
___
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
when a vm is suspended (e.g. autosuspend on windows)
we detect that it is not running, display the resume button,
but 'cont' does not wakeup the system from suspend
with this we can wake up suspended vms
Signed-off-by: Dominik Csapak
---
PVE/QemuServer.pm | 11 +--
1 file changed, 9 ins
---
Makefile | 18 ++---
PVE/{ => APIClient}/CLIHandler.pm | 29 +++--
PVE/APIClient/Commands/GuestStatus.pm | 8 +++---
PVE/APIClient/Commands/config.pm | 4 +--
PVE/APIClient/Commands/lxc.pm | 1 -
PVE/APIClient/Commands/re
see commit c7616abcb278e2b888febc082b557f60676fc376
---
PVE/Storage/CIFSPlugin.pm | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/PVE/Storage/CIFSPlugin.pm b/PVE/Storage/CIFSPlugin.pm
index ff24c8d..ffd16e3 100644
--- a/PVE/Storage/CIFSPlugin.pm
+++ b/PVE/Storage/CIFSPlugin.p
From: René Jochum
Signed-off-by: Dietmar Maurer
---
Changes in v2:
- use make instead of separate shell file
- do not copy Exception.pm
- install to correct path
- remove "use lib ..."
Makefile | 42 ++
PVE/{ => APIClient}/CLIHandler.pm | 27 +++
applied
___
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
Signed-off-by: Dietmar Maurer
---
Makefile | 19 +++
1 file changed, 11 insertions(+), 8 deletions(-)
diff --git a/Makefile b/Makefile
index 4c9572a..0e11af5 100644
--- a/Makefile
+++ b/Makefile
@@ -19,6 +19,16 @@ PVE_COMMON_FILES=\
SafeSyslog.pm
On 6/13/18 11:17 AM, Dominik Csapak wrote:
> when a vm is suspended (e.g. autosuspend on windows)
> we detect that it is not running, display the resume button,
> but 'cont' does not wakeup the system from suspend
>
> with this we can wake up suspended vms
>
> Signed-off-by: Dominik Csapak
> ---
Looks Good To Merge :)
On 2018-06-13 13:45, Dietmar Maurer wrote:
> Signed-off-by: Dietmar Maurer
> ---
> Makefile | 19 +++
> 1 file changed, 11 insertions(+), 8 deletions(-)
>
> diff --git a/Makefile b/Makefile
> index 4c9572a..0e11af5 100644
> --- a/Makefile
> +++ b/Makefile
---
PVE/APIClient/Commands/GuestStatus.pm | 18 ++
PVE/APIClient/Helpers.pm | 10 ++
2 files changed, 20 insertions(+), 8 deletions(-)
diff --git a/PVE/APIClient/Commands/GuestStatus.pm
b/PVE/APIClient/Commands/GuestStatus.pm
index 4a50164..7f90798 100644
---
This works but currently 'qemu' is hard coded in the
"merge_api_definition_properties" path which
has the behavior that we ALWAYS get qemu "properties" where we want LXC
properties often.
The only solution I can think of is to split those commands for Qemu/LXC
as in:
$ pveclient qemu start
comments inline:
> On June 13, 2018 at 2:17 PM René Jochum wrote:
>
>
> ---
> PVE/APIClient/Commands/GuestStatus.pm | 18 ++
> PVE/APIClient/Helpers.pm | 10 ++
> 2 files changed, 20 insertions(+), 8 deletions(-)
>
> diff --git a/PVE/APIClient/Commands/Gue
> This works but currently 'qemu' is hard coded in the
> "merge_api_definition_properties" path which
> has the behavior that we ALWAYS get qemu "properties" where we want LXC
> properties often.
>
> The only solution I can think of is to split those commands for Qemu/LXC
> as in:
>
> $ pv
Signed-off-by: Thomas Lamprecht
---
src/PVE/API2/LXC.pm | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/PVE/API2/LXC.pm b/src/PVE/API2/LXC.pm
index d85c1a2..9f94bf3 100644
--- a/src/PVE/API2/LXC.pm
+++ b/src/PVE/API2/LXC.pm
@@ -167,6 +167,12 @@ __PACKAGE__->register_method({
Signed-off-by: Thomas Lamprecht
---
www/manager6/lxc/CreateWizard.js | 9 +
1 file changed, 9 insertions(+)
diff --git a/www/manager6/lxc/CreateWizard.js b/www/manager6/lxc/CreateWizard.js
index c39db169..a913ca0c 100644
--- a/www/manager6/lxc/CreateWizard.js
+++ b/www/manager6/lxc/Creat
On 6/13/18 1:21 PM, Wolfgang Link wrote:
> see commit c7616abcb278e2b888febc082b557f60676fc376
> ---
> PVE/Storage/CIFSPlugin.pm | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/PVE/Storage/CIFSPlugin.pm b/PVE/Storage/CIFSPlugin.pm
> index ff24c8d..ffd16e3 100644
> --- a
comments inline:
> On June 12, 2018 at 12:41 PM Stoiko Ivanov wrote:
>
>
> print_api_list takes an array of columns to print, the data, and the returns
> definitions of the API, and prints the data by using print_text_table.
>
> handle_cmd and handle_simple_cmd call the output sub with the ret
On 6/12/18 12:41 PM, Stoiko Ivanov wrote:
> These two function could serve as a generic output sub for various CLI
> utilities
> * print_text_table prints an array of objects in a tabular fashion,
>the formating is passed as an array containg hashes with titles, maximal
>lengths and defaul
On 6/12/18 12:41 PM, Stoiko Ivanov wrote:
> print_api_list takes an array of columns to print, the data, and the returns
> definitions of the API, and prints the data by using print_text_table.
>
> handle_cmd and handle_simple_cmd call the output sub with the returns
> definitons.
>
> Signed-off
On 6/12/18 12:41 PM, Stoiko Ivanov wrote:
> * The JSONSchema for most API calls share some properties, which this
> refactoring pulls out in a common_$type_properties hashref in order to
> minimize code duplication (All parameters, which are thus added to the API
> calls were optional)
> * Addit
Signed-off-by: Dietmar Maurer
---
src/PVE/CLIHandler.pm| 1 -
src/PVE/RESTHandler.pm | 2 +-
src/PVE/SectionConfig.pm | 1 -
3 files changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/PVE/CLIHandler.pm b/src/PVE/CLIHandler.pm
index 9bbc156..d871ba6 100644
--- a/src/PVE/CLIHandler.pm
30 matches
Mail list logo