This will allow to disable replication, when for instance we
add a disk on a non-replicatable storage.
The option is hidden in the wizard, because at that time no VM
replication has been set.
we used a static emptytext at creation, which is wrong after editing
now copied from qemu/Options.js (the name), but instead of deleting the
hostname on the backend on an empty field, we set it to CT<VMID>
(this is also the default in the wizard)
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
there were a number of problems in how we reselected vms when they
migrated and selected:
we relied on the data in the configpanel to be static, but since
pveselnode is a reference, it actually changes when the resourcestore
changes
also, handling the reselection as a response to the statusstore failure
is very inconsistent, because we know exactly when a vm moves node in
the tree
this patch removes the whole configpanel vm migration code,
and lets the tree handle it
for this, we have to check if we have to reselect the entry,
deselect it manually and after the treeupdate reselect it
also in the right click selection check we have to check if it is
already selected, else the next selection will get canceled
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
with using suspend/resumeEvents the reselection of vms when migrating,
was not working correctly, fixing this by resuming before selecting
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
The status '-' can confuse users, as it could indicate something is
missing, whereas OK indicates everything is alright (which is the status
we want to show)
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
so we always show one decimal place
(instead of all with seconds and none with minutes, hours, days)
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
to show the number in the url fragment instead of 'replication'
also change the itemid in the datacenter to 'replication' from 'replica'
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
instead of getting the repoid from .git/refs/heads/master
using HEAD as commit also means we get the correct commit hash for when
building the package (e.g. when building from stable-4)
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
API calls return dates in seconds where the JS Date() constructor
wants millisecond.
For the correct computation use render_timestamp_human_readable.
A similar fix was commited in 4092576b1b
for VMs but Containers missed the fix.
To have consistency with the CLI tools `pveceph destroyosd` command.
This is also more clear about the resulting action, as 'Remove' could
also just mean 'Remove from Config', for example.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Each command may not run longer than 10 seconds. This avoids
situations where no output gets generated because of a infinitely
hanging command (e.g. check for dead NFS storage)
Also ensure that commands are separated with a new line from each
other to improve readability.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
In a previous commit, the inputField of SafeDestroy.js was converted
from a numberfield to a textfield to allow ceph pools names to be
entered.
This means the type of 'value' changed from number to string and
the strong equal test === was failing against a numeric id.
To solve this, we now always stringify the resource id before comparing.
values.guest is only available when the field 'guest' is not a
displayfield (e.g. when in vm replication panel), so we have to
use the given vmid first and only when this is not available, fall back
to the value of the field
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
columns with fixed widths do not resize correctly in chrome when
resizing or having a different dpi scaling
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
this patch adds a replication grid, which shows the status/configuration
for the defined replication jobs, and allow them to be
added/edited/removed
in case the replication grid gets shown in the datacenter, we omit the
status fields, because we only have the configuration there
when opening the grid, we check if there are at least two nodes,
because currently we only allow replication in a cluster environment
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
this is a freely editable combobox, which shows five usable examples and
their syntax, allowing users to deduct how the syntax works without
reading documentation
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
we want the new behaviour of the proxmox button also in pve, but
we do not want to change to proxmox-widget-toolkit yet
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
we have translated 'Target Node' already and I add a translation for
german and italian for 'Parallel Jobs' in the next translation update
patches.
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
instead of having font-awesome copied into the repository,
depend on the debian package 'fonts-font-awesome', which provides all we
need
Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
this has the effect that a warning triangle instead of an "?" symbol
is displayed and that the default focus is on "No"
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
Reviewed-By: Dominik Csapak <d.csapak@proxmox.com>
