From bff4a8edb557cc256f0f80d2efab7c9b0966bc97 Mon Sep 17 00:00:00 2001
From: fstagni <federico.stagni@cern.ch>
Date: Fri, 19 Jan 2024 11:07:25 +0100
Subject: [PATCH 1/2] fix: removed several fields that are not anymore in use

---
 .../DIRAC/JobMonitor/classes/JobMonitor.js    | 52 -------------------
 1 file changed, 52 deletions(-)

diff --git a/src/WebAppDIRAC/WebApp/static/DIRAC/JobMonitor/classes/JobMonitor.js b/src/WebAppDIRAC/WebApp/static/DIRAC/JobMonitor/classes/JobMonitor.js
index 042ba83d..be231f9f 100755
--- a/src/WebAppDIRAC/WebApp/static/DIRAC/JobMonitor/classes/JobMonitor.js
+++ b/src/WebAppDIRAC/WebApp/static/DIRAC/JobMonitor/classes/JobMonitor.js
@@ -79,13 +79,6 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
     return oReturn;
   },
   dataFields: [
-    {
-      name: "SystemPriority",
-      type: "float",
-    },
-    {
-      name: "ApplicationNumStatus",
-    },
     {
       name: "JobID",
       type: "int",
@@ -98,9 +91,6 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
     {
       name: "VerifiedFlag",
     },
-    {
-      name: "RetrievedFlag",
-    },
     {
       name: "Status",
     },
@@ -113,9 +103,6 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
       name: "RescheduleCounter",
       type: "int",
     },
-    {
-      name: "JobSplitType",
-    },
     {
       name: "MinorStatus",
     },
@@ -130,12 +117,6 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
     {
       name: "JobType",
     },
-    {
-      name: "MasterJobID",
-    },
-    {
-      name: "KilledFlag",
-    },
     {
       name: "RescheduleTime",
     },
@@ -151,12 +132,6 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
     {
       name: "JobName",
     },
-    {
-      name: "AccountedFlag",
-    },
-    {
-      name: "OSandboxReadyFlag",
-    },
     {
       name: "LastUpdateTime",
       type: "date",
@@ -173,21 +148,12 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
     {
       name: "OwnerGroup",
     },
-    {
-      name: "ISandboxReadyFlag",
-    },
     {
       name: "UserPriority",
     },
     {
       name: "Owner",
     },
-    {
-      name: "DeletedFlag",
-    },
-    {
-      name: "TaskQueueID",
-    },
     {
       name: "JobType",
     },
@@ -478,27 +444,9 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
           hidden: true,
         },
       },
-      AccountedFlag: {
-        dataIndex: "AccountedFlag",
-        properties: {
-          hidden: true,
-        },
-      },
-      OSandboxReadyFlag: {
-        dataIndex: "OSandboxReadyFlag",
-        properties: {
-          hidden: true,
-        },
-      },
       Owner: {
         dataIndex: "Owner",
       },
-      TaskQueueID: {
-        dataIndex: "TaskQueueID",
-        properties: {
-          hidden: true,
-        },
-      },
       OwnerGroup: {
         dataIndex: "OwnerGroup",
       },

From 024d0867b027865f80786ef4011ac99b611ce231 Mon Sep 17 00:00:00 2001
From: Federico Stagni <federico.stagni@cern.ch>
Date: Thu, 8 Feb 2024 17:28:36 +0100
Subject: [PATCH 2/2] fix: added VO selector

---
 src/WebAppDIRAC/WebApp/handler/JobMonitorHandler.py    |  5 ++++-
 .../static/DIRAC/JobMonitor/classes/JobMonitor.js      | 10 +++++++++-
 2 files changed, 13 insertions(+), 2 deletions(-)

diff --git a/src/WebAppDIRAC/WebApp/handler/JobMonitorHandler.py b/src/WebAppDIRAC/WebApp/handler/JobMonitorHandler.py
index f4b3ba7f..0aa78ec8 100644
--- a/src/WebAppDIRAC/WebApp/handler/JobMonitorHandler.py
+++ b/src/WebAppDIRAC/WebApp/handler/JobMonitorHandler.py
@@ -4,7 +4,6 @@
 from DIRAC import gConfig, gLogger
 from DIRAC.Core.Utilities import DictCache
 from DIRAC.Core.Utilities.Graphs.Palette import Palette
-from DIRAC.RequestManagementSystem.Client.Request import Request
 from DIRAC.RequestManagementSystem.Client.ReqClient import ReqClient
 from DIRAC.WorkloadManagementSystem.Client.JobMonitoringClient import JobMonitoringClient
 from DIRAC.WorkloadManagementSystem.Client.JobManagerClient import JobManagerClient
@@ -256,6 +255,10 @@ def _request(self):
         if ownerGroup:
             req["OwnerGroup"] = ownerGroup
 
+        vo = list(json.loads(self.get_argument("vo", "[]")))
+        if vo:
+            req["vo"] = vo
+
         if self.get_argument("startDate", ""):
             req["FromDate"] = self.get_argument("startDate")
             if self.get_argument("startTime", ""):
diff --git a/src/WebAppDIRAC/WebApp/static/DIRAC/JobMonitor/classes/JobMonitor.js b/src/WebAppDIRAC/WebApp/static/DIRAC/JobMonitor/classes/JobMonitor.js
index be231f9f..ab2ddf2b 100755
--- a/src/WebAppDIRAC/WebApp/static/DIRAC/JobMonitor/classes/JobMonitor.js
+++ b/src/WebAppDIRAC/WebApp/static/DIRAC/JobMonitor/classes/JobMonitor.js
@@ -145,6 +145,9 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
       type: "date",
       dateFormat: "Y-m-d H:i:s",
     },
+    {
+      name: "VO",
+    },
     {
       name: "OwnerGroup",
     },
@@ -224,6 +227,7 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
       appStatus: "Application Status",
       owner: "Owner",
       OwnerGroup: "OwnerGroup",
+      VO: "VO",
       jobGroup: "Job Group",
       jobType: "Job Type",
     };
@@ -249,6 +253,7 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
       ["status", "status"],
       ["types", "jobType"],
       ["OwnerGroup", "OwnerGroup"],
+      ["VO", "VO"],
     ];
 
     me.leftPanel = Ext.create("Ext.dirac.utils.DiracBaseSelector", {
@@ -380,6 +385,9 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
       Site: {
         dataIndex: "Site",
       },
+      VO: {
+        dataIndex: "VO",
+      },
       JobName: {
         dataIndex: "JobName",
         properties: {
@@ -675,7 +683,7 @@ Ext.define("DIRAC.JobMonitor.classes.JobMonitor", {
       mode: "local",
       store: new Ext.data.ArrayStore({
         fields: ["category"],
-        data: [["Status"], ["Site"], ["Minor Status"], ["Application Status"], ["Owner"], ["Owner Group"], ["Job Group"], ["Job Type"]],
+        data: [["Status"], ["Site"], ["Minor Status"], ["Application Status"], ["Owner"], ["Owner Group"], ["VO"], ["Job Group"], ["Job Type"]],
       }),
       triggerAction: "all",
       value: "Status",