Skip to content

fix(moduel): fix vm, vmop alerts#1079

Merged
fl64 merged 2 commits into
mainfrom
fix/module/fix-alerts
May 23, 2025
Merged

fix(moduel): fix vm, vmop alerts#1079
fl64 merged 2 commits into
mainfrom
fix/module/fix-alerts

Conversation

@fl64
Copy link
Copy Markdown
Member

@fl64 fl64 commented May 22, 2025

Description

fix vm, vmop alerts

Why do we need it, and what problem does it solve?

What is the expected result?

Checklist

  • The code is covered by unit tests.
  • e2e tests passed.
  • Documentation updated according to the changes.
  • Changes were tested in the Kubernetes cluster manually.

Changelog entries

section: module	
type: fix
summary: fix vm, vmop alerts
impact_level: low

@fl64 fl64 requested a review from Isteb4k as a code owner May 22, 2025 11:43
@fl64 fl64 added this to the v0.19.0 milestone May 22, 2025
@fl64 fl64 requested a review from hayer969 May 23, 2025 10:07
Comment thread monitoring/prometheus-rules/vm.state.yaml
Signed-off-by: Pavel Tishkov <pavel.tishkov@flant.com>
@fl64 fl64 force-pushed the fix/module/fix-alerts branch from 63ed86a to 55864a7 Compare May 23, 2025 12:42
@fl64 fl64 requested a review from hayer969 May 23, 2025 12:42
hayer969
hayer969 previously approved these changes May 23, 2025
> Requires live migration support.
#### Option B: Reboot the VM:
```bash
d8 v -n {{ $labels.namespace }} restart vm {{ $labels.name }}
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.

Without vm as well
Just the name

2. Choose one of the following options depending on your setup:
#### Option A: Evict the VM to another node (live migration):
```bash
d8 v -n {{ $labels.namespace }} evict vm {{ $labels.name }}
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.

d8 v -n {{ $labels.namespace }} restart {{ $labels.name }}
Restart accepts 1 arg (without vm, just the name)

Signed-off-by: Pavel Tishkov <pavel.tishkov@flant.com>
@fl64 fl64 merged commit bb6d27d into main May 23, 2025
21 of 22 checks passed
@fl64 fl64 deleted the fix/module/fix-alerts branch May 23, 2025 16:26
yachmenevas pushed a commit that referenced this pull request Oct 15, 2025
Signed-off-by: Pavel Tishkov <pavel.tishkov@flant.com>
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.

3 participants