Skip to content

Conversation

@varunraokadaparthi
Copy link
Contributor

@varunraokadaparthi varunraokadaparthi commented Feb 3, 2026

Krkn Aggregator:

Introduces a dedicated aggregator for KrknAI chaos testing results that reduces context for LLM analysis by parsing key summary files and excluding verbose cluster data.

Key features:

  • Parses all.csv and health_check_report.csv to extract scenario results
  • Extracts minimal config summary immune to key reordering from updateKrknConfig
  • Stores absolute paths for log artifacts enabling LLM tool access
  • Provides top N scenarios (by fitness), failed scenarios, and health check metrics

Files:

  • internal/aggregator/krknai.go: Aggregator implementation
  • internal/aggregator/krknai_test.go: Unit tests with 8 test cases

Krkn Engine:

Introduces a new analysis mode for KrknAI chaos test results alongside
the existing CI failure analysis. The engine uses LLM to analyze chaos
scenarios and provide cluster resilience recommendations.

Changes:

  • internal/analysisengine/krknai.go: KrknAI analysis engine that
    orchestrates aggregation, prompt rendering, LLM analysis, and result
    reporting for chaos test results
  • internal/llm/tools/tools.go: Updated tool registry to accept log
    artifacts directly, enabling read_file tool to access scenario files
  • internal/llm/tools/read_file_test.go: Updated tests to use
    []aggregator.LogEntry instead of *aggregator.AggregatedData
  • internal/prompts/templates/krknai.yaml: A dummy prompt template
    for chaos engineering analysis with JSON response schema

The KrknAI engine:

  • Collects results via KrknAIAggregator
  • Renders prompts with scenario summaries and artifact paths
  • Provides LLM with read_file tool for deep-dive analysis
  • Writes YAML summary to results directory
  • Supports Slack notifications

Co-authored-by: Cursor AIcursor@cursor.sh

Introduces a dedicated aggregator for KrknAI chaos testing results that
reduces context for LLM analysis by parsing key summary files and
excluding verbose cluster data.

Key features:
- Parses all.csv and health_check_report.csv to extract scenario results
- Extracts minimal config summary immune to key reordering from updateKrknConfig
- Stores absolute paths for log artifacts enabling LLM tool access
- Provides top N scenarios (by fitness), failed scenarios, and health
  check metrics

Files:
- internal/aggregator/krknai.go: Aggregator implementation
- internal/aggregator/krknai_test.go: Unit tests with 8 test cases

Co-authored-by: Cursor AI<cursor@cursor.sh>
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Feb 3, 2026
@openshift-ci-robot
Copy link

openshift-ci-robot commented Feb 3, 2026

@varunraokadaparthi: This pull request references SDCICD-1733 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Introduces a dedicated aggregator for KrknAI chaos testing results that reduces context for LLM analysis by parsing key summary files and excluding verbose cluster data.

Key features:

  • Parses all.csv and health_check_report.csv to extract scenario results
  • Extracts minimal config summary immune to key reordering from updateKrknConfig
  • Stores absolute paths for log artifacts enabling LLM tool access
  • Provides top N scenarios (by fitness), failed scenarios, and health check metrics

Files:

  • internal/aggregator/krknai.go: Aggregator implementation
  • internal/aggregator/krknai_test.go: Unit tests with 8 test cases

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.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 3, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: varunraokadaparthi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 3, 2026
Introduces a new analysis mode for KrknAI chaos test results alongside
the existing CI failure analysis. The engine uses LLM to analyze chaos
scenarios and provide cluster resilience recommendations.

Changes:
- internal/analysisengine/krknai.go: KrknAI analysis engine that
  orchestrates aggregation, prompt rendering, LLM analysis, and result
  reporting for chaos test results
- internal/llm/tools/tools.go: Updated tool registry to accept log
  artifacts directly, enabling read_file tool to access scenario files
- internal/llm/tools/read_file_test.go: Updated tests to use
  []aggregator.LogEntry instead of *aggregator.AggregatedData
- internal/prompts/templates/krknai.yaml: A dummy prompt template
  for chaos engineering analysis with JSON response schema

The KrknAI engine:
- Collects results via KrknAIAggregator
- Renders prompts with scenario summaries and artifact paths
- Provides LLM with read_file tool for deep-dive analysis
- Writes YAML summary to results directory
- Supports Slack notifications

Co-authored-by: Cursor AI<cursor@cursor.sh>
@openshift-ci-robot
Copy link

openshift-ci-robot commented Feb 4, 2026

@varunraokadaparthi: This pull request references SDCICD-1733 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Introduces a dedicated aggregator for KrknAI chaos testing results that reduces context for LLM analysis by parsing key summary files and excluding verbose cluster data.

Key features:

  • Parses all.csv and health_check_report.csv to extract scenario results
  • Extracts minimal config summary immune to key reordering from updateKrknConfig
  • Stores absolute paths for log artifacts enabling LLM tool access
  • Provides top N scenarios (by fitness), failed scenarios, and health check metrics

Files:

  • internal/aggregator/krknai.go: Aggregator implementation
  • internal/aggregator/krknai_test.go: Unit tests with 8 test cases

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.

@openshift-ci-robot
Copy link

openshift-ci-robot commented Feb 4, 2026

@varunraokadaparthi: This pull request references SDCICD-1733 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.22.0" version, but no target version was set.

Details

In response to this:

Krkn Aggregator:

Introduces a dedicated aggregator for KrknAI chaos testing results that reduces context for LLM analysis by parsing key summary files and excluding verbose cluster data.

Key features:

  • Parses all.csv and health_check_report.csv to extract scenario results
  • Extracts minimal config summary immune to key reordering from updateKrknConfig
  • Stores absolute paths for log artifacts enabling LLM tool access
  • Provides top N scenarios (by fitness), failed scenarios, and health check metrics

Files:

  • internal/aggregator/krknai.go: Aggregator implementation
  • internal/aggregator/krknai_test.go: Unit tests with 8 test cases

Krkn Engine:

Introduces a new analysis mode for KrknAI chaos test results alongside
the existing CI failure analysis. The engine uses LLM to analyze chaos
scenarios and provide cluster resilience recommendations.

Changes:

  • internal/analysisengine/krknai.go: KrknAI analysis engine that
    orchestrates aggregation, prompt rendering, LLM analysis, and result
    reporting for chaos test results
  • internal/llm/tools/tools.go: Updated tool registry to accept log
    artifacts directly, enabling read_file tool to access scenario files
  • internal/llm/tools/read_file_test.go: Updated tests to use
    []aggregator.LogEntry instead of *aggregator.AggregatedData
  • internal/prompts/templates/krknai.yaml: A dummy prompt template
    for chaos engineering analysis with JSON response schema

The KrknAI engine:

  • Collects results via KrknAIAggregator
  • Renders prompts with scenario summaries and artifact paths
  • Provides LLM with read_file tool for deep-dive analysis
  • Writes YAML summary to results directory
  • Supports Slack notifications

Co-authored-by: Cursor AIcursor@cursor.sh

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.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Feb 4, 2026

@varunraokadaparthi: all tests passed!

Full PR test history. Your PR dashboard.

Details

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 kubernetes-sigs/prow repository. I understand the commands that are listed here.

@varunraokadaparthi
Copy link
Contributor Author

/hold

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 4, 2026
@@ -0,0 +1,452 @@
package aggregator
Copy link
Contributor

Choose a reason for hiding this comment

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

Can the specific implementation go in krkai package ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ritmun You mean in internal/aggregator/krkn/krknai.go?

@varunraokadaparthi varunraokadaparthi changed the title SDCICD-1733: Add KrknAI chaos test results aggregator SDCICD-1733: [wip] Add KrknAI chaos test results aggregator Feb 4, 2026
@varunraokadaparthi varunraokadaparthi changed the title SDCICD-1733: [wip] Add KrknAI chaos test results aggregator SDCICD-1733: [WIP] Add KrknAI chaos test results aggregator Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants