Skip to content

Commit 734c13f

Browse files
roaks3anoopkverma-google
authored andcommitted
Modify labeler to consider issues with more than 3 services impacted as cross-provider (GoogleCloudPlatform#12863)
1 parent 72f4880 commit 734c13f

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

tools/issue-labeler/cmd/compute_new_labels.go

+17
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,23 @@ func execComputeNewLabels() error {
4545
affectedResources := labeler.ExtractAffectedResources(issueBody)
4646
labels := labeler.ComputeLabels(affectedResources, regexpLabels)
4747

48+
// If there are more than 3 service labels, treat this as a cross-provider issue.
49+
// Note that labeler.ComputeLabels() currently only returns service labels, but
50+
// the logic here remains defensive in case that changes.
51+
var serviceLabels []string
52+
var nonServiceLabels []string
53+
for _, l := range labels {
54+
if strings.HasPrefix(l, "service/") {
55+
serviceLabels = append(serviceLabels, l)
56+
} else {
57+
nonServiceLabels = append(nonServiceLabels, l)
58+
}
59+
}
60+
if len(serviceLabels) > 3 {
61+
serviceLabels = []string{"service/terraform"}
62+
}
63+
labels = append(nonServiceLabels, serviceLabels...)
64+
4865
if len(labels) > 0 {
4966
labels = append(labels, "forward/review")
5067
sort.Strings(labels)

0 commit comments

Comments
 (0)