Skip to content

Commit d605e9f

Browse files
committed
filter default rules
1 parent ddbd123 commit d605e9f

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

pkg/resource/network_acl/hooks.go

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -88,10 +88,10 @@ func (rm *resourceManager) requiredFieldsMissingForCreateNetworkAcl(
8888

8989
func (rm *resourceManager) createEntries(
9090
ctx context.Context,
91-
r *resource,
91+
desired *resource,
9292
) error {
93-
if r.ko.Spec.Entries != nil {
94-
if err := rm.syncEntries(ctx, r, nil); err != nil {
93+
if desired.ko.Spec.Entries != nil {
94+
if err := rm.syncEntries(ctx, desired, nil); err != nil {
9595
return err
9696
}
9797
}
@@ -351,7 +351,6 @@ func (rm *resourceManager) syncEntries(
351351
toAdd = append(toAdd, desiredEntry)
352352
}
353353
}
354-
355354
if latest != nil {
356355
// Filter out AWS default rules from latest entries before comparison
357356
// to ensure consistent state management between desired and actual
@@ -382,6 +381,11 @@ func (rm *resourceManager) syncEntries(
382381
}
383382
}
384383

384+
// During create latest is nil, just add the entries when sync is called via createEntries
385+
if latest == nil {
386+
toAdd = append(toAdd, desired.ko.Spec.Entries...)
387+
}
388+
385389
for _, entry := range toAdd {
386390
if entry == nil {
387391
continue

0 commit comments

Comments
 (0)