Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make issue labeler apply service/terraform label if more than 3 services are impacted #20818

Open
melinath opened this issue Jan 6, 2025 · 4 comments · May be fixed by GoogleCloudPlatform/magic-modules#12863

Comments

@melinath
Copy link
Collaborator

melinath commented Jan 6, 2025

What kind of contribution is this issue about?

Other (specify in details)

Details

Currently we can end up applying a large number of labels to cross-product issues. We should instead label them with service/terraform.

Note: this may also have impact on how we label PRs (though that may have similar logic.) There may be an opportunity to consolidate the implementations.

References

#20729

@BBBmau BBBmau added the size/s label Jan 13, 2025
@BBBmau BBBmau added this to the Goals milestone Jan 13, 2025
@roaks3
Copy link
Collaborator

roaks3 commented Jan 24, 2025

I think there are valid issues that involve multiple service teams where we would want to forward to each team, but that situation gets less likely as the number of services increases. Perhaps we just set a threshold (like 4 or more) where we consider involvement of that many services means it is actually a provider-level issue. Note though that this solution would not cleanly allow for overriding this assumption; we would probably need to split the issue in those cases.

@melinath
Copy link
Collaborator Author

TBH I've been wondering if we should "force"/require splits of any issues with more than one label... it muddies the water in terms of responsibility.

@melinath
Copy link
Collaborator Author

Although in the context of this ticket, that would mean:

  • more than three -> service/terraform
  • 2 or 3 -> split

@roaks3
Copy link
Collaborator

roaks3 commented Jan 24, 2025

Yea, and:

  • more than three that actually should be forwarded to those service teams -> split

Also, I somehow missed the title on the first read, so my previous comment probably seems a bit weird, but also I'm glad I came to the same solution anyway!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants