From 53ce6092662680ed9618db4ec158ee4504c166a0 Mon Sep 17 00:00:00 2001
From: Nikolay Telepenin <ntelepenin@cloudlinux.com>
Date: Mon, 4 Sep 2023 15:49:30 +0100
Subject: [PATCH] Msg is mandatory field for the request

---
 pr_agent/git_providers/gerrit_provider.py | 2 +-
 pr_agent/servers/gerrit_server.py         | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/pr_agent/git_providers/gerrit_provider.py b/pr_agent/git_providers/gerrit_provider.py
index 480809b0..03faf2a1 100644
--- a/pr_agent/git_providers/gerrit_provider.py
+++ b/pr_agent/git_providers/gerrit_provider.py
@@ -350,7 +350,7 @@ class GerritProvider(GitProvider):
             msg.append(f'* {description}\n{full_path}')
 
         if msg:
-            add_comment(self.parsed_url, self.refspec, "---\n".join(msg))
+            add_comment(self.parsed_url, self.refspec, "\n".join(msg))
             return True
 
     def publish_comment(self, pr_comment: str, is_temporary: bool = False):
diff --git a/pr_agent/servers/gerrit_server.py b/pr_agent/servers/gerrit_server.py
index 9bb6f16b..04232ea9 100644
--- a/pr_agent/servers/gerrit_server.py
+++ b/pr_agent/servers/gerrit_server.py
@@ -30,7 +30,7 @@ class Action(str, Enum):
 class Item(BaseModel):
     refspec: str
     project: str
-    msg: str = ""
+    msg: str
 
 
 @router.post("/api/v1/gerrit/{action}")
@@ -46,7 +46,7 @@ async def handle_gerrit_request(action: Action, item: Item):
             )
     await PRAgent().handle_request(
         f"{item.project}:{item.refspec}",
-        f"/{action.value} {item.msg.strip()}"
+        f"/{item.msg.strip()}"
     )