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

[Github] Add LLVM Premerge Checks to the watchlist #120230

Merged
merged 2 commits into from
Dec 18, 2024

Conversation

Keenuts
Copy link
Contributor

@Keenuts Keenuts commented Dec 17, 2024

LLVM Premerge Checks is running on the new GCP cluster. Tracking its metrics will allow us to determine the stability of the presubmit and make sure the new infra is working as intended.

LLVM Premerge Checks is running on the new GCP cluster. Tracking its
metrics will allow us to determine the stability of the presubmit and
make sure the new infra is working as intended.

Signed-off-by: Nathan Gauër <[email protected]>
Copy link

github-actions bot commented Dec 17, 2024

✅ With the latest revision this PR passed the Python code formatter.

Signed-off-by: Nathan Gauër <[email protected]>
Copy link
Contributor

@boomanaiden154 boomanaiden154 left a comment

Choose a reason for hiding this comment

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

Tested manually since we don't have any automated testing:

[JobMetrics(job_name='Check code formatting', queue_time=6, run_time=172, status=1, created_at_ns=1734474477000000000, workflow_id=12382528649), JobMetrics(job_name='LLVM Premerge Checks', queue_time=667, run_time=1013, status=1, created_at_ns=1734472175000000000, workflow_id=12382027516)]
Fake uploaded 2 metrics
[JobMetrics(job_name='Check code formatting', queue_time=6, run_time=172, status=1, created_at_ns=1734474605000000000, workflow_id=12382556504), JobMetrics(job_name='Check code formatting', queue_time=5, run_time=228, status=1, created_at_ns=1734474504000000000, workflow_id=12382535076)]
Fake uploaded 2 metrics
No metrics found to upload.
No metrics found to upload.
[JobMetrics(job_name='Check code formatting', queue_time=6, run_time=178, status=1, created_at_ns=1734474704000000000, workflow_id=12382576218)]
Fake uploaded 1 metrics
[JobMetrics(job_name='Check code formatting', queue_time=6, run_time=176, status=1, created_at_ns=1734474735000000000, workflow_id=12382582860)]
Fake uploaded 1 metrics
[JobMetrics(job_name='Check code formatting', queue_time=7, run_time=170, status=0, created_at_ns=1734474813000000000, workflow_id=12382598716), JobMetrics(job_name='Check code formatting', queue_time=6, run_time=181, status=1, created_at_ns=1734474789000000000, workflow_id=12382593396)]
Fake uploaded 2 metrics
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
[JobMetrics(job_name='Check code formatting', queue_time=5, run_time=186, status=1, created_at_ns=1734475015000000000, workflow_id=12382643443)]
Fake uploaded 1 metrics
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
No metrics found to upload.
[JobMetrics(job_name='Check code formatting', queue_time=7, run_time=184, status=1, created_at_ns=1734475208000000000, workflow_id=12382681715)]
Fake uploaded 1 metrics
[JobMetrics(job_name='LLVM Premerge Checks', queue_time=5, run_time=1801, status=0, created_at_ns=1734473648000000000, workflow_id=12382347323)]
Fake uploaded 1 metrics

Seems like everything is working, so LGTM. We'll have to modify the dashboard to add everything. To get this deployed we might also need to delete the pod to force it to pull the new container version. Theoretically we should always pull the container when recreating the pod, but I'm not sure if it's explicitly setup that way.

@Keenuts Keenuts merged commit 3bcfa1a into llvm:main Dec 18, 2024
8 checks passed
@Keenuts Keenuts deleted the add-presubmit-wf-track branch December 18, 2024 08:59
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.

2 participants