Skip to content

vmagent,vmanomaly: fixed VPA and HPA naming and orphans cleanup - #2519

Merged
AndrewChubatiuk merged 2 commits into
masterfrom
hpa-cleanup-fix
Aug 24, 2026
Merged

vmagent,vmanomaly: fixed VPA and HPA naming and orphans cleanup#2519
AndrewChubatiuk merged 2 commits into
masterfrom
hpa-cleanup-fix

Conversation

@AndrewChubatiuk

Copy link
Copy Markdown
Contributor

fixes #2518

@cubic-dev-ai cubic-dev-ai Bot 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.

All reported issues were addressed

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread internal/controller/operator/factory/vmagent/vmagent.go Outdated
Comment thread docs/CHANGELOG.md Outdated
@AndrewChubatiuk
AndrewChubatiuk force-pushed the hpa-cleanup-fix branch 6 times, most recently from 40b54d0 to e6e36d8 Compare August 20, 2026 12:15
if !cr.Spec.DaemonSetMode {
objsToRemove = append(objsToRemove, &appsv1.DaemonSet{ObjectMeta: objMeta})
}
// TODO: drop the cr.Name deletes below once enough time has passed since HPA/VPA objects

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Lets add a specific deadline - i.e. 0.80 version

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done for both vmagent and vmanomaly

@AndrewChubatiuk
AndrewChubatiuk enabled auto-merge (squash) August 24, 2026 15:55
@AndrewChubatiuk
AndrewChubatiuk merged commit 112087f into master Aug 24, 2026
9 checks passed
@AndrewChubatiuk
AndrewChubatiuk deleted the hpa-cleanup-fix branch August 24, 2026 16:03
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.

VMAgent: disabling HPA does not delete the HPA

2 participants