Skip to content

Conversation

sknat
Copy link
Collaborator

@sknat sknat commented Oct 7, 2025

This patch fixes the behavior of CNI server state
reload on error. It addresses the two following issues.

  • If the state file is corrupted and parsing errors, we should return, not proceed as the parsed cniServerState might contain partial data.
  • When VPP re-programmation completes, we overwrite the state file so that we remove pods that yielded errors or for which the linux ns was removed.

@sknat
Copy link
Collaborator Author

sknat commented Oct 7, 2025

This is a proposal to superseed #811

This patch fixes the behavior of CNI server state
reload on error. It addresses the two following issues.
- If the state file is corrupted and parsing errors,
we should return, not proceed as the parsed cniServerState
might contain partial data.
- When VPP re-programmation completes, we overwrite the
state file so that we remove pods that yielded errors or
for which the linux ns was removed.

Signed-off-by: Nathan Skrzypczak <[email protected]>
@sknat sknat force-pushed the nsk-fix-load-cni-state-onerror branch from 4749e36 to 52a42af Compare October 7, 2025 08:55
@sknat sknat changed the title Fix CNI server state behavior on reload. Fix CNI server state error handling Oct 7, 2025
@sknat sknat self-assigned this Oct 7, 2025
@sknat sknat merged commit d56a21e into master Oct 15, 2025
5 checks passed
@sknat sknat deleted the nsk-fix-load-cni-state-onerror branch October 15, 2025 08:20
@sknat sknat mentioned this pull request Oct 15, 2025
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.

2 participants