Skip to content

AlertManager - fix deployment template issue - #106

Merged
Yuting Jiang (yukirora) merged 1 commit into
devfrom
yutji/alert-deploy
Nov 6, 2025
Merged

AlertManager - fix deployment template issue#106
Yuting Jiang (yukirora) merged 1 commit into
devfrom
yutji/alert-deploy

Conversation

@yukirora

@yukirora Yuting Jiang (yukirora) commented Oct 24, 2025

Copy link
Copy Markdown
Contributor

Major Revision

  • fix deployment template parse error when cluster-utilization, abnormal detector and related setting not configured

Copilot AI review requested due to automatic review settings October 24, 2025 07:56

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.

Pull Request Overview

This PR fixes a deployment template parsing error that occurs when optional configuration sections (lucia, abnormal-detector) are not present in the cluster configuration. The changes add conditional checks to prevent template rendering failures when these optional settings are missing.

Key changes:

  • Added conditional guards around LUCIA-related environment variables
  • Added conditional guard around abnormal-detector configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@yukirora
Yuting Jiang (yukirora) merged commit 8365ac4 into dev Nov 6, 2025
3 checks passed
@yukirora Yuting Jiang (yukirora) mentioned this pull request Nov 11, 2025
25 tasks
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.

5 participants