Commit Graph

4 Commits

Author SHA1 Message Date
yxxhero 92205e827a fix: prevent duplicate isReleaseInstalled checks in DetectReleasesToBeDeletedForSync
When a release with 'installed: false' appears multiple times in the release list
(due to duplicate entries or being included via needs), the isReleaseInstalled
function was being called multiple times for the same release, resulting in
duplicate 'Listing releases' log messages.

This fix adds a deduplication check using a 'checked' map to ensure each release
is only checked once, preventing duplicate helm list calls and log messages.

This fixes the diff-args integration test where 'Listing releases matching
^uninstalled$' appeared twice in stderr output.

Signed-off-by: yxxhero <aiopsclub@163.com>
2026-03-18 08:25:32 +08:00
yxxhero 20928620a8 fix tests
Signed-off-by: yxxhero <aiopsclub@163.com>
2026-03-18 08:25:32 +08:00
yxxhero 48254a990b fix tests
Signed-off-by: yxxhero <aiopsclub@163.com>
2026-03-18 08:25:32 +08:00
Tim Ramlot 824e5a8b92
Use logger for helm output (#1585)
* use logger for helm output

Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>

* update integration test output

Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>

* make logging output configurable

Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>

* also compare stderr in integration tests

Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>

---------

Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>
2024-07-04 13:53:31 +09:00