Skip to content

Update the AI model to GPT-5.4 in AI log agent - #4883

Open
lubaihua33 wants to merge 1 commit into
mainfrom
baihua/gpt
Open

lubaihua33 wants to merge 1 commit into
mainfrom
baihua/gpt

Conversation

@lubaihua33

@lubaihua33 lubaihua33 commented Nov 14, 2026 •

Copy link
Copy Markdown
Collaborator

Update the AI model to GPT-5.4 in AI log agent. The accuracy is 93% using GPT5.4

Copilot AI balanced review requested due to automatic review settings November 14, 2026 05:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The runbook still documents GPT-4 prerequisites and old deployment defaults, so users who omit optional deployment settings may encounter analysis failures because the requested gpt-5.4 deployment is unavailable.

1 open issue
What changed in this PR

Updates LISA’s AI log-analysis defaults to use the Azure deployment named gpt-5.4 for both general and software analysis.

Changes:

  • Updates both deployment defaults in the log-agent notifier configuration.
  • Aligns the analyze() function defaults with the notifier while preserving explicit overrides.

Suggested validation (not run): Enable the notifier and exercise a controlled failed-test notification with defaults and explicit overrides. A passing smoke test alone will not exercise AI analysis.

Key Test Cases:
smoke_test

Impacted LISA Features:
None — changes are limited to AI analysis and notifier defaults.

Tested Azure Marketplace Images:

  • canonical 0001-com-ubuntu-server-jammy 22_04-lts-gen2 latest
File Description
lisa/​notifiers/​log_agent.py Sets both notifier deployment defaults to gpt-5.4.
lisa/​ai/​log_agent.py Sets both analyze() deployment defaults to gpt-5.4.

Review effort: Balanced

# AI model deployment configuration
general_deployment_name: str = field(
default="gpt-4o",
default="gpt-5.4",

This branch has not been deployed

No deployments
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