Use project-dawn-submit-access to govern CQ+2 permission.
This group includes all human committers (who can code review) as well as robot accounts like Rubber Stamper that need to be able to trigger CQ+2.
Bug: 356276600
Change-Id: I49041db2210271448ccf98b24c4d04b287d9d866
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/208160
Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Joey Scarr <jsca@google.com>
diff --git a/infra/config/global/generated/commit-queue.cfg b/infra/config/global/generated/commit-queue.cfg
index 6a9a98e..32c48bd 100644
--- a/infra/config/global/generated/commit-queue.cfg
+++ b/infra/config/global/generated/commit-queue.cfg
@@ -41,7 +41,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -728,7 +728,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -851,7 +851,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -977,7 +977,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -1103,7 +1103,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -1229,7 +1229,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -1355,7 +1355,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -1481,7 +1481,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -1607,7 +1607,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -1733,7 +1733,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -1859,7 +1859,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -1985,7 +1985,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -2114,7 +2114,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -2243,7 +2243,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -2372,7 +2372,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -2501,7 +2501,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -2633,7 +2633,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -2765,7 +2765,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -2897,7 +2897,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
@@ -3029,7 +3029,7 @@
}
verifiers {
gerrit_cq_ability {
- committer_list: "project-dawn-committers"
+ committer_list: "project-dawn-submit-access"
dry_run_access_list: "project-dawn-tryjob-access"
}
tryjob {
diff --git a/infra/config/global/main.star b/infra/config/global/main.star
index 42af106..57f1dc0 100755
--- a/infra/config/global/main.star
+++ b/infra/config/global/main.star
@@ -983,7 +983,7 @@
acls = [
acl.entry(
acl.CQ_COMMITTER,
- groups = "project-dawn-committers",
+ groups = "project-dawn-submit-access",
),
acl.entry(
acl.CQ_DRY_RUNNER,
