X-Git-Url: https://gerrit.o-ran-sc.org/r/gitweb?a=blobdiff_plain;f=policy-agent%2Fsrc%2Fmain%2Fjava%2Forg%2Foransc%2Fpolicyagent%2Fcontrollers%2FServiceController.java;h=012d40a935540be9936b0d62ecce7feffab9dea5;hb=f26d17f375a8ab4d521549543d4fcc36bdc98865;hp=cc1d711bb2ecde80107900f4732d7cc6677973f4;hpb=d1b7bb42fe20c8c0131959f3e96668d8e6669d04;p=nonrtric.git

diff --git a/policy-agent/src/main/java/org/oransc/policyagent/controllers/ServiceController.java b/policy-agent/src/main/java/org/oransc/policyagent/controllers/ServiceController.java
index cc1d711b..012d40a9 100644
--- a/policy-agent/src/main/java/org/oransc/policyagent/controllers/ServiceController.java
+++ b/policy-agent/src/main/java/org/oransc/policyagent/controllers/ServiceController.java
@@ -41,6 +41,7 @@ import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
+import org.springframework.web.bind.annotation.PostMapping;
 import org.springframework.web.bind.annotation.PutMapping;
 import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RequestParam;
@@ -63,15 +64,16 @@ public class ServiceController {
     }
 
     @GetMapping("/services")
-    @ApiOperation(value = "Returns service information", response = ServiceStatus.class)
-    @ApiResponses(value = {@ApiResponse(code = 200, message = "OK")})
+    @ApiOperation(value = "Returns service information")
+    @ApiResponses(
+        value = {@ApiResponse(code = 200, message = "OK", response = ServiceStatus.class, responseContainer = "List")})
     public ResponseEntity<String> getServices( //
         @RequestParam(name = "name", required = false) String name) {
 
         Collection<ServiceStatus> servicesStatus = new Vector<>();
         synchronized (this.services) {
             for (Service s : this.services.getAll()) {
-                if (name == null || name.equals(s.name())) {
+                if (name == null || name.equals(s.getName())) {
                     servicesStatus.add(toServiceStatus(s));
                 }
             }
@@ -82,25 +84,24 @@ public class ServiceController {
     }
 
     private ServiceStatus toServiceStatus(Service s) {
-        return ImmutableServiceStatus.builder() //
-            .name(s.name()) //
-            .keepAliveInterval(s.getKeepAliveInterval().toSeconds()) //
-            .timeSincePing(s.timeSinceLastPing().toSeconds()) //
-            .build();
+        return new ServiceStatus(s.getName(), s.getKeepAliveInterval().toSeconds(), s.timeSinceLastPing().toSeconds());
     }
 
+    @ApiOperation(value = "Register a service")
+    @ApiResponses(value = {@ApiResponse(code = 200, message = "OK", response = String.class)})
     @PutMapping("/service")
     public ResponseEntity<String> putService( //
-        @RequestBody String jsonBody) {
+        @RequestBody ServiceRegistrationInfo registrationInfo) {
         try {
-            ServiceRegistrationInfo s = gson.fromJson(jsonBody, ImmutableServiceRegistrationInfo.class);
-            this.services.put(toService(s));
+            this.services.put(toService(registrationInfo));
             return new ResponseEntity<String>("OK", HttpStatus.OK);
         } catch (Exception e) {
             return new ResponseEntity<String>(e.getMessage(), HttpStatus.NO_CONTENT);
         }
     }
 
+    @ApiOperation(value = "Delete a service")
+    @ApiResponses(value = {@ApiResponse(code = 200, message = "OK")})
     @DeleteMapping("/services")
     public ResponseEntity<String> deleteService( //
         @RequestParam(name = "name", required = true) String name) {
@@ -115,17 +116,32 @@ public class ServiceController {
         }
     }
 
+    @ApiOperation(value = "Keep the poilicies alive for a service")
+    @ApiResponses(
+        value = {@ApiResponse(code = 200, message = "Policies timeout supervision refreshed"),
+            @ApiResponse(code = 404, message = "The service is not found, needs re-registration")})
+    @PostMapping("/services/keepalive")
+    public ResponseEntity<String> keepAliveService( //
+        @RequestParam(name = "name", required = true) String name) {
+        try {
+            services.getService(name).ping();
+            return new ResponseEntity<String>("OK", HttpStatus.OK);
+        } catch (Exception e) {
+            return new ResponseEntity<String>(e.getMessage(), HttpStatus.NOT_FOUND);
+        }
+    }
+
     private Service removeService(String name) throws ServiceException {
         synchronized (this.services) {
             Service service = this.services.getService(name);
-            this.services.remove(service.name());
+            this.services.remove(service.getName());
             return service;
         }
     }
 
     private void removePolicies(Service service) {
         synchronized (this.policies) {
-            Vector<Policy> policyList = new Vector<>(this.policies.getForService(service.name()));
+            Vector<Policy> policyList = new Vector<>(this.policies.getForService(service.getName()));
             for (Policy policy : policyList) {
                 this.policies.remove(policy);
             }
@@ -133,7 +149,7 @@ public class ServiceController {
     }
 
     private Service toService(ServiceRegistrationInfo s) {
-        return new Service(s.name(), Duration.ofSeconds(s.keepAliveInterval()), s.callbackUrl());
+        return new Service(s.name, Duration.ofSeconds(s.keepAliveIntervalSeconds), s.callbackUrl);
     }
 
 }