search

aws / aws-network-policy-agent

Apache License 2.0
45 stars 29 forks source link

Conntrack enhancements #151

Closed jayanthvn closed 10 months ago

jayanthvn commented 10 months ago

Issue #, if available:

Description of changes: previously we had env CONNTRACK_CACHE_CLEANUP_PERIOD in seconds which would determine the conntrack cache cleanup interval and moving it to be a flag instead.

Default - 300

{"level":"info","ts":"2023-11-30T22:25:10.975Z","logger":"controllers.policyEndpoints","caller":"runtime/proc.go:267","msg":"ConntrackTTL","cleanupPeriod":300}

Set to 100

{"level":"info","ts":"2023-11-30T22:26:31.128Z","logger":"controllers.policyEndpoints","caller":"runtime/proc.go:267","msg":"ConntrackTTL","cleanupPeriod":100}
{"level":"info","ts":"2023-11-30T22:14:21.616Z","logger":"ebpf-client","caller":"wait/backoff.go:227","msg":"Done cleanup of conntrack map"}
{"level":"info","ts":"2023-11-30T22:19:21.641Z","logger":"ebpf-client","caller":"wait/backoff.go:227","msg":"Done cleanup of conntrack map"}
{"level":"info","ts":"2023-11-30T22:24:21.666Z","logger":"ebpf-client","caller":"wait/backoff.go:227","msg":"Done cleanup of conntrack map"}
{"level":"info","ts":"2023-11-30T22:34:51.286Z","logger":"ebpf-client","caller":"wait/backoff.go:227","msg":"Done cleanup of conntrack map"}
{"level":"info","ts":"2023-11-30T22:36:31.299Z","logger":"ebpf-client","caller":"wait/backoff.go:227","msg":"Done cleanup of conntrack map"}
{"level":"info","ts":"2023-11-30T22:38:11.314Z","logger":"ebpf-client","caller":"wait/backoff.go:227","msg":"Done cleanup of conntrack map"}

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

achevuru commented 10 months ago

I think it's better to convert this in to an arg as well, similar to our existing config flags