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

[chore] diagram defining different states for issues #36975

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

VihasMakwana
Copy link
Contributor

@VihasMakwana VihasMakwana commented Dec 28, 2024

Description

See #36677

Closes #36677

flowchart TD
    n0(["New issue has been opened"]) --> n1
    n1(["Needs Triage"]) --> n2["Has good repro steps <br>and/or description?"]
  subgraph graph2["**waiting-for-codeowners**"]
        n3["Waiting for Codeowners<br>to further validate the issue"]
  end
  subgraph graph3["**waiting-for-author**"]
        n4["Waiting for author to provide more details"]
  end
  subgraph graph4["**help-wanted**"]
        n8["Waiting on community"]
  end
  subgraph graph5["**closed**"]
        n10(["Close the issue and provide details as needed"])
  end
    n2 -- Yes --> n3
    n2 -- No/Need more details --> n4
    n2 -- Invalid configuration/alternative available --> n10
    n3 -- Invalid Issue --> n10
    n3 -- Valid Issue -->  n6["Codeowner has time<br>to fix it?"]
    n6 -- Assign it to codeowner --> n7["Issue in being worked upon"]
    n6 -- No --> n8
    n7 -- Once PR is merged --> n10
    n8 -- When someone volunteers to provide a fix --> n11["Assign it to the person"]
    n12 -- Any activity? --> n8
    n8 -. Stale issue .-> n12["Issue in inactive"]
    n11 --> n7
    n12 -- Closed automatically after 120 days due to lack of activity --> n10
    n4 -- Once enough details are available --> n2

    n3@{ shape: rect}
    n4@{ shape: rect}
    n2@{ shape: diam}
    n6@{ shape: diam}

     n1:::Aqua
     n3:::Ash
     n4:::Ash
     n8:::Ash
     n2:::Ash
     n2:::Peach
     n6:::Peach
     n7:::Ash
     n10:::Rose
    classDef Rose stroke-width:1px, stroke-dasharray:none, stroke:#FF5978, fill:#FFDFE5, color:#8E2236
    classDef Aqua stroke-width:1px, stroke-dasharray:none, stroke:#46EDC8, fill:#DEFFF8, color:#378E7A
    classDef Peach stroke-width:1px, stroke-dasharray:none, stroke:#FBB35A, fill:#FFEFDB, color:#8F632D
    classDef Ash stroke-width:1px, stroke-dasharray:none, stroke:#999999, fill:#EEEEEE, color:#000000
Loading

issue-triaging.md Outdated Show resolved Hide resolved
@VihasMakwana VihasMakwana requested a review from atoulme December 31, 2024 13:42
issue-triaging.md Outdated Show resolved Hide resolved
issue-triaging.md Outdated Show resolved Hide resolved
@mx-psi mx-psi self-requested a review January 7, 2025 09:08
issue-triaging.md Outdated Show resolved Hide resolved
@fatsheep9146
Copy link
Contributor

image

Add a screenshot for current version in case other approvers have interest in this.

@fatsheep9146
Copy link
Contributor

ping @open-telemetry/collector-contrib-approvers for review

n6 -- Yes --> n7["Valid issue and to be fixed <br>by a codeowner"]
n6 -- No --> n8
n7 --> n9
n8 --> n9
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be some step between help-wanted and closed

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, I'll add some steps in between.

Copy link
Contributor

@jade-guiton-dd jade-guiton-dd Jan 29, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this actually resolved? It seems weird to me that "To be fixed by a codeowner" and "Mark as help wanted" immediately lead to closing the issue.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@songy23 @jade-guiton-dd It's now fixed. Please take look at issue description for updated flowchart image.

issue-triaging.md Outdated Show resolved Hide resolved
issue-triaging.md Outdated Show resolved Hide resolved
issue-triaging.md Show resolved Hide resolved
@VihasMakwana VihasMakwana requested a review from mx-psi February 3, 2025 13:52
n7 -- Once PR is merged --> n10
n8 -- When someone volunteers to provide a fix --> n11["Assign it to the person"]
n12 -- Any activity? --> n8
n8 -. Stale issue .-> n12["Issue in inactive"]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If issues of a certain component keep being ignored, becoming stale then closed, we should consider the component as unmaintained at some point. (We don't have to address it in this PR though)

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

Successfully merging this pull request may close these issues.

Define states an issue can be at
8 participants