Update methods NewGroup, DeleteGroup and GetGroup of OFSwitch

Signed-off-by: Hongliang Liu <lhongliang@vmware.com>

Author: hongliangl

.golangci.yml

@@ -12,13 +12,10 @@ linters-settings:
 linters:
   disable-all: true
   enable: # see https://golangci-lint.run/usage/linters/
-    - deadcode
+    - unused
     - staticcheck
     - govet
     - gofmt
     - goimports
     - gosec
     - misspell
-
-run:
-  deadline: 5m

Makefile

@@ -20,7 +20,7 @@ test: docker-test-integration
 # code linting
 .golangci-bin:
 	@echo "===> Installing Golangci-lint <==="
-	@curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $@ v1.41.1
+	@curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $@ v1.50.1
 
 .PHONY: golangci
 golangci: .golangci-bin

VERSION

@@ -1 +1 @@
-v0.6.7
+v0.6.9

ofctrl/dial.go

@@ -1,3 +1,4 @@
+//go:build linux || darwin
 // +build linux darwin
 
 package ofctrl

ofctrl/fgraph.go

@@ -1,4 +1,4 @@
-/***
+/*
 Copyright 2014 Cisco Systems Inc. All rights reserved.
 
 Licensed under the Apache License, Version 2.0 (the "License");

ofctrl/fgraphFlood.go

@@ -1,4 +1,4 @@
-/***
+/*
 Copyright 2014 Cisco Systems Inc. All rights reserved.
 
 Licensed under the Apache License, Version 2.0 (the "License");

ofctrl/fgraphFlow.go

@@ -1,4 +1,4 @@
-/***
+/*
 Copyright 2014 Cisco Systems Inc. All rights reserved.
 
 Licensed under the Apache License, Version 2.0 (the "License");
@@ -1906,10 +1906,10 @@ func (self *Flow) ConnTrack(commit bool, force bool, tableID *uint8, zoneID *uin
 
 // Special Actions to to the flow to set conjunctions
 // Note:
-//   1) nclause should be in [2, 64].
-//   2) clause value should be less than or equals to ncluase, and its value should be started from 1.
-//      actual clause in libopenflow messages is started from 0, here would decrement 1 to keep the display
-//      value is consistent with expected configuration
+//  1. nclause should be in [2, 64].
+//  2. clause value should be less than or equals to ncluase, and its value should be started from 1.
+//     actual clause in libopenflow messages is started from 0, here would decrement 1 to keep the display
+//     value is consistent with expected configuration
 func (self *Flow) AddConjunction(conjID uint32, clause uint8, nClause uint8) error {
 	conjunction, err := NewNXConjunctionAction(conjID, clause, nClause)
 	if err != nil {

ofctrl/fgraphOutput.go

@@ -1,4 +1,4 @@
-/***
+/*
 Copyright 2014 Cisco Systems Inc. All rights reserved.
 
 Licensed under the Apache License, Version 2.0 (the "License");

ofctrl/fgraphSwitch.go

@@ -1,4 +1,4 @@
-/***
+/*
 Copyright 2014 Cisco Systems Inc. All rights reserved.
 
 Licensed under the Apache License, Version 2.0 (the "License");
@@ -18,6 +18,7 @@ package ofctrl
 
 import (
 	"errors"
+	"fmt"
 
 	"antrea.io/libOpenflow/openflow15"
 )
@@ -96,31 +97,39 @@ func (self *OFSwitch) DefaultTable() *Table {
 	return self.tableDb[0]
 }
 
-// Create a new group. return an error if it already exists
-func (self *OFSwitch) NewGroup(groupId uint32, groupType GroupType) (*Group, error) {
-	// check if the group already exists
-	if self.groupDb[groupId] != nil {
-		return nil, errors.New("group already exists")
+// NewGroup creates a new group; when using cache, returns an error if the group ID already exists, otherwise returns the
+// group object; when not using cache, returns the new group object， and the caller should ensure the groupID is not
+// duplicated.
+func (self *OFSwitch) NewGroup(groupID uint32, groupType GroupType, useCache bool) (*Group, error) {
+	// Check if the group already exists.
+	if useCache {
+		if self.groupDb[groupID] != nil {
+			return nil, errors.New("group already exists")
+		}
 	}
 
-	// Create a new group
-	group := newGroup(groupId, groupType, self)
-	// Save it in the DB
-	self.groupDb[groupId] = group
+	// Create a new group.
+	group := newGroup(groupID, groupType, self)
+	if useCache {
+		// Save it in cache.
+		self.groupDb[groupID] = group
+	}
 
 	return group, nil
 }
 
-// Delete a group.
-// Return an error if there are flows refer pointing at it
+// DeleteGroup deletes a group in cache.
 func (self *OFSwitch) DeleteGroup(groupId uint32) error {
 	delete(self.groupDb, groupId)
 	return nil
 }
 
-// GetGroup Returns a group
-func (self *OFSwitch) GetGroup(groupId uint32) *Group {
-	return self.groupDb[groupId]
+// GetGroup returns a group if it is cached.
+func (self *OFSwitch) GetGroup(groupId uint32) (*Group, error) {
+	if _, exists := self.groupDb[groupId]; !exists {
+		return nil, fmt.Errorf("group %d does not exist in cache", groupId)
+	}
+	return self.groupDb[groupId], nil
 }
 
 // Create a new meter. return an error if it already exists

ofctrl/fgraphTable.go

@@ -1,4 +1,4 @@
-/***
+/*
 Copyright 2014 Cisco Systems Inc. All rights reserved.
 
 Licensed under the Apache License, Version 2.0 (the "License");