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

Create AUTOCUT issues without "untriaged" label #498

Closed
AmiStrn opened this issue Sep 16, 2024 · 10 comments
Closed

Create AUTOCUT issues without "untriaged" label #498

AmiStrn opened this issue Sep 16, 2024 · 10 comments
Labels
enhancement New feature or request

Comments

@AmiStrn
Copy link

AmiStrn commented Sep 16, 2024

Is your feature request related to a problem? Please describe

The AUTOCUT tickets are generated with an untriaged label, this is redundant. There is nothing to triage here.

Describe the solution you'd like

the labeling code already recognizes there is a difference between Autocut and other issues, I suggest making so these tickets are created without the untriaged label. this will reduce noise in the triage meetings.

@param args.label <optional> - GitHub issue label to be attached along with 'untriaged'. Defaults to autocut.
@param args.daysToReOpen <optional> - Look for a closed Github issues older than `daysToReOpen`.
@param args.issueEdit <optional> - Updates the body of the issue, the default if not passed is to add a comment.
@param args.issueBodyFile <optional> - GitHub issue body from an `.md` file
*/
void call(Map args = [:]) {
label = args.label ?: 'autocut'

Describe alternatives you've considered

all other options are either manual or automating the manual work which is breakable and pointless.

Additional context

No response

@AmiStrn
Copy link
Author

AmiStrn commented Sep 16, 2024

@dblock fyi

AmiStrn added a commit to AmiStrn/opensearch-build-libraries that referenced this issue Sep 16, 2024
Signed-off-by: Amitai Stern <123amitai@gmail.com>
@AmiStrn
Copy link
Author

AmiStrn commented Sep 16, 2024

this was really small. im pretty sure the 2-liner i wrote fixes this

@prudhvigodithi
Copy link
Member

prudhvigodithi commented Sep 16, 2024

Thanks @AmiStrn In my opinion having an untriaged label is good, doing the same for Core repo Gradle Check Flaky Test Report issues (example opensearch-project/OpenSearch#15944). Having an untriaged label will help teams ensure that newly created autocut issues (the failures) are not ignored.

The expectation is during the triage meeting the autocut issue is assigned to an owner or triaged with a solution or the RCA for the created issue, having this will ensure the addressing of the autocut issue is not delayed till the release window.

Thanks

@peterzhuamazon
Copy link
Member

I also agree with @prudhvigodithi here as untriage label helps bring attention to maintainers to address newly fresh autocut issues.

@AmiStrn
Copy link
Author

AmiStrn commented Sep 18, 2024

We saw many AUTOCUT issues in the catch-all triage meeting (3 week old untriaged issues). I am not sure that the untriaged label is having the expected results with regard to the release window.

@AmiStrn
Copy link
Author

AmiStrn commented Sep 18, 2024

We check to see that the labels are correct, the issue is in the right repo, and described correctly. this is always the case for these issues, so from my point of view it is redundant. I see why you may want it for visibility in the other triage meetings.
@dblock anything else you wish to add here?

@peterzhuamazon
Copy link
Member

peterzhuamazon commented Sep 18, 2024

We saw many AUTOCUT issues in the catch-all triage meeting (3 week old untriaged issues). I am not sure that the untriaged label is having the expected results with regard to the release window.

I feel like that means the maintainers should check the AUTOCUTs on time and not letting them piled up in untriaged for a long time.

Happy to discuss more tho our team still believes this should help bring more attention to the AUTOCUT issues being solved as opposed to pure noice during the triage. 😄

Would love to see better way of solving this if possible. Thanks.

@AmiStrn
Copy link
Author

AmiStrn commented Sep 18, 2024

@dblock this kind of sounds like we should just ignore autocut in the catch all triage meetings. Wdyt?

@prudhvigodithi
Copy link
Member

prudhvigodithi commented Sep 18, 2024

In my opinion, autocut labelled issues should be treated like any other issues in the repository that need attention. We should not exclude them, though the order of triaging priority for autocut issues can differ. If the autocut issues have a untriage label, it indicates that the component teams have not addressed them, potentially could be the release blockers.

We could introduce a new priority label to add to autocut issues. However, we have developed the habit of treating the untriage label as important, and this approach would need to be carried forward with the priority label as well.

Adding @getsaurabh02

@AmiStrn
Copy link
Author

AmiStrn commented Sep 30, 2024

Thanks @prudhvigodithi and @peterzhuamazon for the feedback.

So that you know, AUTOCUT issues that reach the catch-all meeting have the untriaged labels removed without assigning the issue.

I'm closing this issue for now since there is a desire to keep things as they are in this regard.

@AmiStrn AmiStrn closed this as completed Sep 30, 2024
@github-project-automation github-project-automation bot moved this from Backlog to ✅ Done in Engineering Effectiveness Board Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

4 participants