OTA-1927: Update cluster-updates-critical queries to match console - #3060
Conversation
Align the cluster-updates-critical eval queries with the prompts used by the console. conv_001 was 37,384 chars, over the 32,000-char LLMRequest.query cap, so /v1/query returned HTTP 422 and the eval logged an ERROR that tripped test_cluster_updates. Fixes the periodic CI job: https://prow.ci.openshift.org/job-history/gs/test-platform-results/logs/periodic-ci-openshift-lightspeed-service-main-4.21-ols-cluster-updates-periodics Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Signed-off-by: Fabricio Aguiar <fabricio.aguiar@gmail.com> rh-pre-commit.version: 2.3.2 rh-pre-commit.check-secrets: ENABLED
|
@fao89: This pull request references OTA-1927 which is a valid jira issue. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. 💤 Files selected but had no reviewable changes (1)
⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
/cc @JoaoFula @sriroopar |
|
/lgtm |
|
@sriroopar could you please approve? |
sriroopar
left a comment
There was a problem hiding this comment.
PR fixes the query length overflow for cluster updates periodic CI job to pass.
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sriroopar The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@fao89: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
Align the cluster-updates-critical eval queries with the prompts used by the console. conv_001 was 37,384 chars, over the 32,000-char LLMRequest.query cap, so /v1/query returned HTTP 422 and the eval logged an ERROR that tripped test_cluster_updates.
Fixes the periodic CI job:
https://prow.ci.openshift.org/job-history/gs/test-platform-results/logs/periodic-ci-openshift-lightspeed-service-main-4.21-ols-cluster-updates-periodics
Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com
Signed-off-by: Fabricio Aguiar fabricio.aguiar@gmail.com
rh-pre-commit.version: 2.3.2
rh-pre-commit.check-secrets: ENABLED
Description
Type of change
Related Tickets & Documents
Checklist before requesting a review
Testing