Make persistRun transactional

Signed-off-by: Andrea Lamparelli <[email protected]>
Hyperfoil · Jan 16, 2025 · 4a0fa80 · 4a0fa80
1 parent ba3696c
commit 4a0fa80
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/horreum-api/src/main/java/io/hyperfoil/tools/horreum/api/services/RunService.java b/horreum-api/src/main/java/io/hyperfoil/tools/horreum/api/services/RunService.java
@@ -222,6 +222,7 @@ Response add(@QueryParam("test") String testNameOrId,
             @Parameter(name = "description", description = "Run description", example = "AWS runs"),
 
     })
+    @Produces(MediaType.TEXT_PLAIN)
     @APIResponses(value = {
             @APIResponse(responseCode = "202", description = "The request has been accepted for processing. Returns a list of created run IDs if available, "
                     + "or an empty list if processing is still ongoing. Label values and change detection processing " +
@@ -242,6 +243,7 @@ Response addRunFromData(@QueryParam("start") String start,
     @Path("data")
     @Consumes(MediaType.MULTIPART_FORM_DATA)
     @Operation(description = "Upload a new Run with metadata", hidden = true)
+    @Produces(MediaType.TEXT_PLAIN)
     @APIResponses(value = {
             @APIResponse(responseCode = "202", description = "The request has been accepted for processing. Returns a list of created run IDs if available, "
                     + "or an empty list if processing is still ongoing. Label values and change detection processing " +

diff --git a/horreum-backend/src/main/java/io/hyperfoil/tools/horreum/svc/RunServiceImpl.java b/horreum-backend/src/main/java/io/hyperfoil/tools/horreum/svc/RunServiceImpl.java
@@ -535,6 +535,7 @@ Response addRunFromData(String start, String stop, String test,
                 .build();
     }
 
+    @Transactional
     void persistRun(ServiceMediator.RunUpload runUpload) {
         runUpload.roles.add("horreum.system");
         roleManager.setRoles(String.join(",", runUpload.roles));