Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various CNI delegation corrections related to framework bump, and central IPAM support #123

Merged
merged 2 commits into from
Aug 8, 2019

Conversation

Levovar
Copy link
Collaborator

@Levovar Levovar commented Aug 8, 2019

A bug is corrected both in cnidel, and metacni package related to Free-ing IPs when an error happened during the execution of the same asynch thread.
The correction is to immediately refresh the network manifest after allocation to avoid working on an out-of-date local copy.
This bug is probably present from earlier, accidentally caught during testing.

CNI version support is now explicitly set to 0.3.1 to avoid any accidental mistranslation of CNI results.

Started correctly filling IPAM config for static CNI deletes when DANM was reserving their IP during setup.

DANM CNI log file can be found under /ar/log/danm.log from now on!

A bug is corrected both in cnidel, and metacni package related to Free-ing IPs when an error happened during the execution of the same asynch thread.
The correction is to immediately refresh the network manifest after allocation to avoid working on an out-of-date local copy.

CNI version support is now explicitly set to 0.3.1 to avoid any accidental mis-translation.

Started correctly filling IPAM config for static CNI deletes when DANM was reserving their IP during setup.
@Levovar Levovar force-pushed the central_ipam_fixes branch 2 times, most recently from 01cb1de to 37dd874 Compare August 8, 2019 19:35
Added Update counter checks to the cnidel / setup TCs.
These extra assertation now cover the cases related to the IP leaking bug.
@Levovar Levovar force-pushed the central_ipam_fixes branch from 37dd874 to ec5450f Compare August 8, 2019 21:14
@Levovar Levovar merged commit 5e15d4e into master Aug 8, 2019
@Levovar Levovar deleted the central_ipam_fixes branch August 8, 2019 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant