Skip to content

Commit

Permalink
Merge pull request #2 from ssttehrani/main
Browse files Browse the repository at this point in the history
docs: add comments to improve insight
  • Loading branch information
ssttehrani authored Dec 3, 2023
2 parents 29a71b9 + 50e961a commit 855b4ae
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
4 changes: 2 additions & 2 deletions deploy/charts/svc-lb-to-cilium-netpolicy-operator/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@ type: application
# This is the chart version. This version number should be incremented each time you make changes
# to the chart and its templates, including the app version.
# Versions are expected to follow Semantic Versioning (https://semver.org/)
version: 1.0.3
version: 1.0.4
# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application. Versions are not expected to
# follow Semantic Versioning. They should reflect the version the application is using.
# It is recommended to use it with quotes.
appVersion: "1.1.0"
appVersion: "v1.1.1"
6 changes: 6 additions & 0 deletions internal/controller/service/setup.go
Original file line number Diff line number Diff line change
Expand Up @@ -263,6 +263,9 @@ func (re *ReconcilerExtended) networkPolicyEventHandler(ctx context.Context, obj
} else {
// Add exclusion reason for the namespace indicating the source of the exclusion
cacheEntry[noNetworkPolicyExistence] = empty{}

logger.Info(consts.NamespaceExcludedNoNetworkPolicyMatchInfo +
consts.NamespaceExclusionReasonAddedInfo)
}

case updateEvent:
Expand Down Expand Up @@ -424,6 +427,9 @@ func (re *ReconcilerExtended) ciliumNetworkPolicyEventHandler(ctx context.Contex
} else {
// Add exclusion reason for the namespace indicating the source of the exclusion
cacheEntry[noUnmanagedCiliumNetworkPolicyExistence] = empty{}

logger.Info(consts.NamespaceExcludedNoCiliumNetworkPolicyMatchInfo +
consts.NamespaceExclusionReasonAddedInfo)
}
}

Expand Down

0 comments on commit 855b4ae

Please sign in to comment.