commit 1dc856c26c6e9f6bd09dc9a999e7c9518ea60487
author Adithya Chakilam <achakilam@nvidia.com> Tue Aug 19 08:29:43 2025 -0500
committer Adithya Chakilam <achakilam@nvidia.com> Tue Aug 19 13:53:32 2025 -0700
tree 67a60667a975aff3376e2596976e1e0092af8418
parent 89b64c301c26ef2121fc265a38701f6534dc1c3c

Fix task quota namespacing

`.git` extension only appears when user specifies it, so make the regex
independent of it. Also, use the Namekey#parse function to remove that
extension and trailing slashes. Some gerrit installations might also
have a leading slash and hence makes it optional.

Change-Id: I772e50046dbdd276e13438749767434d7cb3f430

src/main/java/com/googlesource/gerrit/plugins/quota/TaskQuotas.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/quota/TaskQuotas.java b/src/main/java/com/googlesource/gerrit/plugins/quota/TaskQuotas.java
index 40aaa36..0119c42 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/quota/TaskQuotas.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/quota/TaskQuotas.java
@@ -39,7 +39,7 @@
   private final QuotaFinder quotaFinder;
   private final Map<Integer, List<TaskQuota>> quotasByTask = new ConcurrentHashMap<>();
   private final Map<QuotaSection, List<TaskQuota>> quotasByNamespace = new HashMap<>();
-  private final Pattern PROJECT_PATTERN = Pattern.compile("\\s+(.*\\.git)\\s+(\\S+)$");
+  private final Pattern PROJECT_PATTERN = Pattern.compile("\\s+/?(.*)\\s+(\\(\\S+\\))$");
   private final Config quotaConfig;
   private final TaskQuota.BuildInfo baseBuildInfo;
 
@@ -123,6 +123,6 @@
   private Optional<Project.NameKey> estimateProject(WorkQueue.Task<?> task) {
     Matcher matcher = PROJECT_PATTERN.matcher(task.toString());
 
-    return matcher.find() ? Optional.of(Project.nameKey(matcher.group(1))) : Optional.empty();
+    return matcher.find() ? Optional.of(Project.NameKey.parse(matcher.group(1))) : Optional.empty();
   }
 }