KusionStack / KusionStack/kuperator

Bug: When using PodToDelete along with decreasing replicas, the priority should consider PodToDelete list

Open
#269 1 comment 0 reactions 1 assignee View on GitHub

@ColdsteelRail is already working on this.

Since Sep 6, 2024.

kind/bug
Dominant language
Go
Stars
100
Forks
12
PR merge metrics
No merged PRs in 30d

Description

Minimal reproduce step

Scale in 2 pods, along with more than 2, let's say 3 podToDelete.

What did you expect to see?

2 Pods from podToDelete list should be delete with scale-in label, and 1 Pod should be recreated.

What did you see instead

The deletion priority may not consider PodToDelete list.
https://github.com/KusionStack/kuperator/blob/v0.5.3/pkg/controllers/collaset/synccontrol/sync_control.go#L403

What is your KusionStack components and its version?

0.5.0

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.