[pve-devel] [PATCH manager v2] ha: re-add node to service status
Dominik Csapak
d.csapak at proxmox.com
Thu Jan 5 14:02:19 CET 2017
Reviewed-by: Dominik Csapak <d.csapak at proxmox.com>
On 01/05/2017 12:23 PM, Thomas Lamprecht wrote:
> On the old HA status we saw where a service was located currently,
> this information was lost when we merged the resource and the status
> tab.
> Add this information again.
>
> Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
> ---
>
> changes since v1:
> * add 'node' also to the data model
>
> www/manager6/ha/Resources.js | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/www/manager6/ha/Resources.js b/www/manager6/ha/Resources.js
> index 8d35afc..9f716eb 100644
> --- a/www/manager6/ha/Resources.js
> +++ b/www/manager6/ha/Resources.js
> @@ -128,6 +128,12 @@ Ext.define('PVE.ha.ResourcesView', {
> dataIndex: 'state'
> },
> {
> + header: gettext('Node'),
> + width: 100,
> + sortable: true,
> + dataIndex: 'node'
> + },
> + {
> header: gettext('Request State'),
> width: 100,
> hidden: true,
> @@ -196,7 +202,7 @@ Ext.define('PVE.ha.ResourcesView', {
> extend: 'Ext.data.Model',
> fields: [
> 'sid', 'state', 'digest', 'errors', 'group', 'comment',
> - 'max_restart', 'max_relocate', 'type', 'status',
> + 'max_restart', 'max_relocate', 'type', 'status', 'node',
> 'crm_state', 'request_state'
> ],
> idProperty: 'sid'
>
More information about the pve-devel
mailing list