Skip to content

Customize the issue-labeler train workflow to specify a page size for pulls #11078

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

Merged

Conversation

jeffhandley
Copy link
Member

See Support a pullPageSize in the train.yml composite workflow (dotnet/issue-labeler#97).

The train workflow is not able to successfully download the pull requests for this repo to feed the training. Likely because of how many files are involved in many PRs in this repo, the queries for fetching pull requests get increasingly slow with each page of data, and 25 pulls per page is timing out. Local testing revealed that any page size over 1 is likely to time out quite a bit.

The reusable train.yml workflow from issue-labeler does not currently support an input parameter for the page sizes, let alone different page sizes for issues and pulls. To work around this, the logic from that reusable workflow is decomposed directly into this repo's workflow so the page size can be specified to the Downloader. The functionality of the train job will remain the same overall.

@jeffhandley jeffhandley requested review from RussKie and gewarren March 12, 2025 19:57
@jeffhandley jeffhandley self-assigned this Mar 12, 2025
@jeffhandley jeffhandley requested a review from a team as a code owner March 12, 2025 19:57
@ghost ghost added the area-Infrastructure label Mar 12, 2025
Copy link

Learn Build status updates of commit 2c83c4f:

✅ Validation status: passed

File Status Preview URL Details
.github/workflows/labeler-train.yml ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@gewarren gewarren merged commit 91e6999 into dotnet:main Mar 12, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants