Skip to content

Commit

Permalink
libcni: set both GC valid attachment keys
Browse files Browse the repository at this point in the history
I made an (embarassing) error where SPEC.md and libcni disagreed on the
key for valid attachments for a GC operation. The spec has been
updated, but libcni should set both keys as a transition mechanism.

Signed-off-by: Casey Callendrello <c1@caseyc.net>
  • Loading branch information
squeed committed Jul 22, 2024
1 parent 6a68851 commit 692efbd
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions libcni/api.go
Original file line number Diff line number Diff line change
Expand Up @@ -817,6 +817,8 @@ func (c *CNIConfig) GCNetworkList(ctx context.Context, list *NetworkConfigList,
}
if args != nil {
inject["cni.dev/valid-attachments"] = args.ValidAttachments
// #1101: spec used incorrect variable name
inject["cni.dev/attachments"] = args.ValidAttachments
}

for _, plugin := range list.Plugins {
Expand Down

0 comments on commit 692efbd

Please sign in to comment.