-
Notifications
You must be signed in to change notification settings - Fork 103
[jira/grooming] add sprint filter and support no time range to grooming command #88
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
base: main
Are you sure you want to change the base?
Conversation
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bryan-cox, jiajliu The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
| - **`/jira:create` `<type> [project-key] <summary> [--component <name>] [--version <version>] [--parent <key>]`** - Create Jira issues (story, epic, feature, task, bug) with proper formatting | ||
| - **`/jira:generate-test-plan` `[JIRA issue key] [GitHub PR URLs]`** - Generate test steps for a JIRA issue | ||
| - **`/jira:grooming` `[project-filter] [time-period] [--component component-name] [--label label-name]`** - Analyze new bugs and cards added over a time period and generate grooming meeting agenda | ||
| - **`/jira:grooming` `[project-filter] [time-period] [--component component-name] [--label label-name] [--sprint sprint-name]`** - Analyze new bugs and cards added over a time period and generate grooming meeting agenda |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMHO, the sprint name also represents a time range, so I think we can merge it with the time-period field.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
E.g
/jira:grooming OCPSTRAT "OTA 277" --component "Control Plane" --label "performance"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to know that a string name could be provided in time-period field, but according to my local testing, they returned different results.
/jira:grooming OCPBUGS "" --component "Cluster Version Operator" --sprint "OTA 277"
It retrieves and returns the bugs(such as OCPBUGS-61872 and OCPBUGS-54151), which is consistent with jira’s Sprint "OTA 277".
...
- OCPBUGS-61872: https://issues.redhat.com/browse/OCPBUGS-61872
- OCPBUGS-54151: https://issues.redhat.com/browse/OCPBUGS-54151
- CI Logs: Available in issue OCPBUGS-61872 additional info section
● ---
The grooming agenda has been generated successfully for sprint OTA 277 with the Cluster Version Operator
component in the OCPBUGS project. The sprint shows excellent completion with both bugs resolved:
...
/jira:grooming OCPBUGS "OTA 277" --component "Cluster Version Operator"
It retrieves a time range (September 16 - October 6, 2025) according to provided string "OTA 277." Then, it will identify and return bugs(such as OCPBUGS-62416 and OCPBUGS-61872), that were created during this period but are not consistent with jira's Sprint "OTA 277".
...
Summary: During the OTA 277 sprint (Sept 16 - Oct 6, 2025), only 2 new bugs were created for the Cluster
Version Operator component, and both have already been verified and closed. This indicates excellent sprint
execution with 100% completion rate.
The agenda includes:
- Detailed analysis of both issues (OCPBUGS-61872 and OCPBUGS-62416)
...
From the QE side, what we need is the result returned from the first example. It can help us plan and participate in sprint work effectively.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IIUC, this would require changes to the implementation to support it.
What this PR does / why we need it:
The grooming command is helpful for QE grooming tasks. This pull request adds support for
sprintfilter, enabling list generation based on sprint names. Additionally, the time-period argument can now be left empty, allowing for an unlimited time range.Checklist: