Skip to content

Write rerunner trigger events to the database#31931

Merged
ababushk merged 8 commits intoopenvinotoolkit:masterfrom
ababushk:rerunner_db_stats
Sep 9, 2025
Merged

Write rerunner trigger events to the database#31931
ababushk merged 8 commits intoopenvinotoolkit:masterfrom
ababushk:rerunner_db_stats

Conversation

@ababushk
Copy link
Copy Markdown
Contributor

@ababushk ababushk commented Sep 1, 2025

Details:

Create a database record every time the rerunner script triggers a workflow rerun.

@ababushk ababushk requested a review from a team as a code owner September 1, 2025 22:33
@github-actions github-actions bot added category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code labels Sep 1, 2025
Copy link
Copy Markdown
Contributor

@akashchi akashchi left a comment

Choose a reason for hiding this comment

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

I think env writing could be removed: https://github.com/openvinotoolkit/openvino/pull/31931/files#diff-4c2871984341ecac264e7272921614164e5d8eba25e5d21ca4e665eddd2eaeeeL69 since there will be no subsequent steps for statistics gathering.

Otherwise, LGTM.

@ababushk ababushk enabled auto-merge September 8, 2025 17:36
@ababushk ababushk added this pull request to the merge queue Sep 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 8, 2025
@ababushk ababushk added this pull request to the merge queue Sep 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 9, 2025
@ababushk ababushk enabled auto-merge September 9, 2025 16:08
@ababushk ababushk added this pull request to the merge queue Sep 9, 2025
Merged via the queue into openvinotoolkit:master with commit 1549a79 Sep 9, 2025
177 checks passed
@ababushk ababushk deleted the rerunner_db_stats branch September 9, 2025 21:50
github-merge-queue bot pushed a commit that referenced this pull request Jan 30, 2026
### Details:
My bad, these tests should have been added right in
#31931 . Because I
haven't done that, we've lost metrics for the past few days - metrics
recording has been broken by
#33798

This PR changes the script structure slightly to be able to test DB
integration. I tried to add a smoke test to prevent such mistakes in the
future.
Naseer-010 pushed a commit to Naseer-010/openvino that referenced this pull request Feb 18, 2026
### Details:
My bad, these tests should have been added right in
openvinotoolkit#31931 . Because I
haven't done that, we've lost metrics for the past few days - metrics
recording has been broken by
openvinotoolkit#33798

This PR changes the script structure slightly to be able to test DB
integration. I tried to add a smoke test to prevent such mistakes in the
future.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants