fix: Don't panic on multiple $patch: delete strategic merge patches in a single patch file (#5859)

* chore: add test for multiple $patch: delete patches not panicking

* fix: don't panic on multiple deletion SM patches
This commit is contained in:
Timur Demin
2025-03-28 18:04:40 +05:00
committed by GitHub
parent 2e80cebf21
commit dd08aec23e
2 changed files with 70 additions and 0 deletions

View File

@@ -181,6 +181,10 @@ func (m *resWrangler) GetMatchingResourcesByAnyId(
matches IdMatcher) []*resource.Resource {
var result []*resource.Resource
for _, r := range m.rList {
if r.RNode.IsNilOrEmpty() {
continue
}
for _, id := range append(r.PrevIds(), r.CurId()) {
if matches(id) {
result = append(result, r)