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

ci: modifying stat count for L0_server_status #7820

Merged
merged 5 commits into from
Nov 21, 2024

Conversation

KrishnanPrash
Copy link
Contributor

What does the PR do?

This PR modifies the stat count in ModelMetadataTest::test_infer_stats_no_model() to account for potential changes made to qa_model_repository. Additionally, to assist in future debugging, a request is sent to the model repository index API to be able to catch any changes made to qa_model_repository by comparing job logs.

  • CI Pipeline ID: 20575134

@KrishnanPrash KrishnanPrash added the PR: ci Changes to our CI configuration files and scripts label Nov 20, 2024
@KrishnanPrash KrishnanPrash self-assigned this Nov 20, 2024
# Primarily meant to assist in debugging ModelMetadataTest::test_infer_stats_no_model
# Diff the output with a previous L0_server_status job to catch any changes to
# /data/inferenceserver/${REPO_VERSION}/qa_model_repository that were not accounted for.
curl -X POST http://localhost:8000/v2/repository/index
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice changes!

@KrishnanPrash KrishnanPrash merged commit b06d1b0 into main Nov 21, 2024
3 checks passed
@KrishnanPrash KrishnanPrash deleted the kprashanth-L0-server-status branch November 21, 2024 01:48
mc-nv pushed a commit that referenced this pull request Nov 21, 2024
mc-nv added a commit that referenced this pull request Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: ci Changes to our CI configuration files and scripts
Development

Successfully merging this pull request may close these issues.

3 participants