[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