On 25.06.21 14:32, Fabian Ebner wrote:
> The '--targetstorage' parameter does not apply to shared storages.
>
> Example for a problem solved with the enabled check: Given a VM with
> images only on a shared storage 'storeA', not available on the target
> node (i.e. restricted by the nodes property
we will have multiple panels with the same widget. Instead
of raising an error in that case, simply ignore it, since
we normally only want to set the default initially, not when
users configured something else
Signed-off-by: Dominik Csapak
---
www/manager6/qemu/OSTypeEdit.js | 2 +-
1 file chang
the 'diskControllerMaxIDs' object in Utils does not describe the
'maximum ids', but the maximum *number* of ids, so the max is one less
correctly set that instead
the api rejected those values (e.g. ide4) already, so its only a ui change
Signed-off-by: Dominik Csapak
---
www/manager6/form/Cont
e.g. from scsi0 to scsi1 or from scsi0 to virtio0
Signed-off-by: Dominik Csapak
---
www/manager6/qemu/HDEdit.js | 10 ++
1 file changed, 10 insertions(+)
diff --git a/www/manager6/qemu/HDEdit.js b/www/manager6/qemu/HDEdit.js
index bbd4a2c6..2142c746 100644
--- a/www/manager6/qemu/HDEdit
we'll use them outside of the controllerSelector soon
Signed-off-by: Dominik Csapak
---
www/manager6/Utils.js | 46 +
www/manager6/form/ControllerSelector.js | 45 +---
2 files changed, 54 insertions(+), 37 deletions(-)
diff --git a/
more in line with our remaining code style
Signed-off-by: Dominik Csapak
---
www/manager6/qemu/HDEdit.js | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/www/manager6/qemu/HDEdit.js b/www/manager6/qemu/HDEdit.js
index ae7e3fde..bbd4a2c6 100644
--- a/www/manager6/qe
for the multi disk panel, we want it to be just a little wider, so
that all form fields are still readable
Signed-off-by: Dominik Csapak
---
www/manager6/window/Wizard.js | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/www/manager6/window/Wizard.js b/www/manager6/window/Wizar
Signed-off-by: Dominik Csapak
---
www/manager6/Utils.js | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/www/manager6/Utils.js b/www/manager6/Utils.js
index 8631a67c..71e5fc9a 100644
--- a/www/manager6/Utils.js
+++ b/www/manager6/Utils.js
@@ -13,7 +13,7 @@ Ext.define('PVE.Utils
those are helpers that will be useful when we want to change the
vmconfig for the ControllerSelector on the fly
Signed-off-by: Dominik Csapak
---
www/manager6/form/ControllerSelector.js | 16
1 file changed, 16 insertions(+)
diff --git a/www/manager6/form/ControllerSelector.js
we will sometimes directly give it a config, so the id can be selected
at the start
Signed-off-by: Dominik Csapak
---
www/manager6/form/ControllerSelector.js | 4
1 file changed, 4 insertions(+)
diff --git a/www/manager6/form/ControllerSelector.js
b/www/manager6/form/ControllerSelector.js
so that we can retrieve it in the MultiDiskPanel
Signed-off-by: Dominik Csapak
---
www/manager6/qemu/OSTypeEdit.js | 1 +
1 file changed, 1 insertion(+)
diff --git a/www/manager6/qemu/OSTypeEdit.js b/www/manager6/qemu/OSTypeEdit.js
index fad56e8a..d9a0988e 100644
--- a/www/manager6/qemu/OSTypeE
this series is intended to replace dominics and my previous attempts
at this [0][1][2]
splits the bandwidth options into their on tab on the disk panel and
introduces a 'MultiHDEdit' panel which creates/deletes the
HDEdit panels on demand.
The UX is modeled after Dominics first attempt, but a ver
for that we have to nest the now two tabs in a tabpanel into an inputpanel.
to prevent the options to be collected twice, we override the
'getValues' function of the 'sub-inputpanels' to return an empty object.
(we could make that an option for the inputpanel, but not necessary for
now)
also we ha
this adds a new panel where a user can add multiple disks.
Has a simple grid for displaying the already added disks and displays
a warning triangle if the disk is not valid.
This allows also to create a vm without any disk by removing all of them.
Signed-off-by: Dominik Csapak
---
www/manager6
"default local storage" is a bit confusing without the backticks,
add that to make it more clear that the 'local' storage will be used by
default for restore operations unless `--storage foo` is passed.
Signed-off-by: Oguz Bektas
---
pct.adoc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
From: Dominic Jäger
Like notes for nodes.
Signed-off-by: Dominic Jäger
Signed-off-by: Dominik Csapak
---
www/manager6/dc/Config.js | 6 +
www/manager6/panel/NotesView.js | 48 +
2 files changed, 31 insertions(+), 23 deletions(-)
diff --git a/www/man
From: Dominic Jäger
Similar to notes for nodes.
datacenter.cfg normally uses key-value pairs defined in the schema.
We bypass this to allow potentially long comments at the top.
Signed-off-by: Dominic Jäger
Signed-off-by: Dominik Csapak
---
data/PVE/DataCenterConfig.pm | 31 ++
resend of dominics previous series of this fix
changes from v1:
* removed the (imho) unecessary parsing of the nonexisting 'description'
field in the datacenter.cfg, we never had it before and now it only
exists as comment, no need to add it to the description (since it
cannot exist using onl
"default local storage" is confusing without the backticks. reword the
sentences to make it clearer about `storage` parameter usage.
Signed-off-by: Oguz Bektas
---
pct.adoc | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/pct.adoc b/pct.adoc
index c5ed243..9a9d4ff 100644
-
19 matches
Mail list logo
