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
Open
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 50 additions & 0 deletions issue-triaging.md
Original file line number Diff line number Diff line change
Expand Up @@ -71,3 +71,53 @@ triaged and is ready for work. If someone who is assigned to an issue is no long
| `Sponsor Needed` | A new component has been proposed, but implementation is not ready to begin. This can be because a sponsor has not yet been decided, or because some details on the component still need to be decided. |
| `Accepted Component` | A sponsor has elected to take on a component and implementation is ready to begin. |
| `Vendor Specific Component` | This should be applied to any component proposal where the functionality for the component is particular to a vendor. |

### State diagram

Here is a diagram outlining the potential issue states and how issues move through different stages:
VihasMakwana marked this conversation as resolved.
Show resolved Hide resolved

```mermaid
flowchart TD
subgraph graph1["**needs-triage**"]
n1(["Needs Triage"])
end
subgraph graph2["**waiting-for-codeowners**"]
n3["Waiting for Codeowners<br>to further validate the issue"]
VihasMakwana marked this conversation as resolved.
Show resolved Hide resolved
end
subgraph graph3["**waiting-for-author**"]
n4["Waiting for author to provide more details"]
end
subgraph graph4["**help-wanted**"]
n8["Mark as help-wanted"]
end
n1 --> n2["Has good repro steps <br>and/or description?"]
n2 -- Yes --> n3
n2 -- No/Need more details --> n4
n2 -- Invalid configuration/alternative available --> n5(["Provide a detailed comment on the issue and close it."])
n3 -- Invalid Issue --> n5
n3 -- Valid Issue --> n6["Codeowner has time<br>to fix it?"]
VihasMakwana marked this conversation as resolved.
Show resolved Hide resolved
n6 -- Yes --> n7["Valid issue and to be fixed <br>by a codeowner"]
n6 -- No --> n8
n7 --> n9(["Issue closed once fix is merged"])
VihasMakwana marked this conversation as resolved.
Show resolved Hide resolved
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.

n4 -- Once enough detailsare available --> n2
VihasMakwana marked this conversation as resolved.
Show resolved Hide resolved

n3@{ shape: rect}
n4@{ shape: rect}
n2@{ shape: diam}
n6@{ shape: diam}
n1:::Aqua
n3:::Ash
n4:::Ash
n8:::Ash
n2:::Ash
n2:::Peach
n5:::Rose
fatsheep9146 marked this conversation as resolved.
Show resolved Hide resolved
n6:::Peach
n7:::Ash
n9:::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