sync: support reloading sync config when the config file changes

[peusebiu]

Signed-off-by: Petu Eusebiu peusebiu@cisco.com

What type of PR is this?

Which issue does this PR fix:

What does this PR do / Why do we need it:

If an issue # is not available please add repro steps and logs from IPAMD/CNI showing the issue:

Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:

Does this change require updates to the CNI daemonset config files to work?:

Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[codecov]

Codecov Report

Merging #406 (32ea614) into main (7e8cc3c) will increase coverage by 0.14%.
The diff coverage is 89.31%.

@@            Coverage Diff             @@
##             main     #406      +/-   ##
==========================================
+ Coverage   83.80%   83.94%   +0.14%     
==========================================
  Files          49       50       +1     
  Lines       10210    10290      +80     
==========================================
+ Hits         8556     8638      +82     
+ Misses       1294     1293       -1     
+ Partials      360      359       -1     

Impacted Files	Coverage Δ
pkg/extensions/extensions.go	80.82% <50.00%> (ø)
pkg/cli/config_reloader.go	78.04% <78.04%> (ø)
pkg/cli/root.go	92.95% <89.74%> (+4.18%)	⬆️
pkg/api/controller.go	91.82% <100.00%> (+0.80%)	⬆️
pkg/extensions/sync/sync.go	90.96% <100.00%> (+0.20%)	⬆️
pkg/extensions/sync/utils.go	79.64% <100.00%> (ø)

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

[rchincha]

lgtm