Skip to content

Commit

Permalink
Merge pull request #1286 from s1061123/fix-type-scope-comment
Browse files Browse the repository at this point in the history
Not exposed APIReadyCheckFunc to outside of package
  • Loading branch information
dougbtv committed Jun 6, 2024
2 parents 9f5c023 + d23856b commit 4757be1
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions pkg/server/api/shim.go
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,8 @@ type ShimNetConf struct {
LogToStderr bool `json:"logToStderr,omitempty"`
}

// Define a type for API readiness check functions
type APIReadyCheckFunc func(string) error
// readyCheckFunc defines a type for API readiness check functions
type readyCheckFunc func(string) error

// CmdAdd implements the CNI spec ADD command handler
func CmdAdd(args *skel.CmdArgs) error {
Expand Down Expand Up @@ -74,7 +74,7 @@ func CmdDel(args *skel.CmdArgs) error {
return nil
}

func postRequest(args *skel.CmdArgs, readinessCheck APIReadyCheckFunc) (*Response, string, error) {
func postRequest(args *skel.CmdArgs, readinessCheck readyCheckFunc) (*Response, string, error) {
multusShimConfig, err := shimConfig(args.StdinData)
if err != nil {
return nil, "", fmt.Errorf("invalid CNI configuration passed to multus-shim: %w", err)
Expand Down

0 comments on commit 4757be1

Please sign in to comment.