Skip to content

Conversation

@rjdenney
Copy link
Contributor

@rjdenney rjdenney commented Jul 25, 2023

Reason for Change:

Updating CNI version in dropgz to CNI v1.5.9 that has a fix for preventing the CNS to panic when no NCs are found on an NNC.

Issue Fixed:

Requirements:

Notes:

camrynl
camrynl previously approved these changes Jul 25, 2023
@vipul-21 vipul-21 added the cni Related to CNI. label Jul 25, 2023
@rjdenney rjdenney changed the title fix: updating CNI to v1.5.7 for dropgz fix: updating CNI to v1.5.8 for dropgz Aug 2, 2023
@rjdenney rjdenney closed this Aug 2, 2023
@rjdenney rjdenney reopened this Aug 2, 2023
@rjdenney rjdenney enabled auto-merge (squash) August 4, 2023 19:06
ramiro-gamarra
ramiro-gamarra previously approved these changes Aug 4, 2023
paulyufan2
paulyufan2 previously approved these changes Aug 4, 2023
Copy link
Contributor

@paulyufan2 paulyufan2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@rjdenney rjdenney dismissed stale reviews from paulyufan2 and ramiro-gamarra via 8b8dc0f August 7, 2023 21:09
@rjdenney rjdenney requested a review from a team as a code owner August 7, 2023 21:09
@rjdenney rjdenney disabled auto-merge August 7, 2023 21:14
@rjdenney rjdenney changed the title fix: updating CNI to v1.5.8 for dropgz fix: updating CNI to v1.5.9 for dropgz Aug 10, 2023
@rjdenney rjdenney enabled auto-merge (squash) August 10, 2023 13:38
@rjdenney rjdenney merged commit 34b6c9e into master Aug 10, 2023
@rjdenney rjdenney deleted the updateCNI branch August 10, 2023 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cni Related to CNI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants