Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2018-present Open Networking Foundation |
| 3 | |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | |
| 17 | package openflow |
| 18 | |
| 19 | import ( |
Rohan Agrawal | c32d993 | 2020-06-15 11:01:47 +0000 | [diff] [blame] | 20 | "context" |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 21 | "fmt" |
Kent Hagerman | 9b0ac0a | 2020-06-08 11:48:16 -0400 | [diff] [blame] | 22 | "math" |
| 23 | "testing" |
| 24 | |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 25 | "github.com/opencord/goloxi" |
| 26 | "github.com/opencord/goloxi/of13" |
| 27 | "github.com/opencord/ofagent-go/internal/pkg/holder" |
| 28 | "github.com/opencord/ofagent-go/internal/pkg/mock" |
Maninder | 12b909f | 2020-10-23 14:23:36 +0530 | [diff] [blame] | 29 | "github.com/opencord/voltha-protos/v4/go/openflow_13" |
| 30 | "github.com/opencord/voltha-protos/v4/go/voltha" |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 31 | "github.com/stretchr/testify/assert" |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 32 | ) |
| 33 | |
| 34 | var msgSizeLimit = 64000 |
| 35 | |
| 36 | func createEapolFlow(id int) openflow_13.OfpFlowStats { |
| 37 | |
| 38 | portField := openflow_13.OfpOxmField{ |
| 39 | OxmClass: openflow_13.OfpOxmClass_OFPXMC_OPENFLOW_BASIC, |
| 40 | Field: &openflow_13.OfpOxmField_OfbField{ |
| 41 | OfbField: &openflow_13.OfpOxmOfbField{ |
| 42 | Type: openflow_13.OxmOfbFieldTypes_OFPXMT_OFB_IN_PORT, |
| 43 | Value: &openflow_13.OfpOxmOfbField_Port{Port: 16}, |
| 44 | }, |
| 45 | }, |
| 46 | } |
| 47 | |
| 48 | ethField := openflow_13.OfpOxmField{ |
| 49 | OxmClass: openflow_13.OfpOxmClass_OFPXMC_OPENFLOW_BASIC, |
| 50 | Field: &openflow_13.OfpOxmField_OfbField{ |
| 51 | OfbField: &openflow_13.OfpOxmOfbField{ |
| 52 | Type: openflow_13.OxmOfbFieldTypes_OFPXMT_OFB_ETH_TYPE, |
| 53 | Value: &openflow_13.OfpOxmOfbField_EthType{EthType: 2048}, |
| 54 | }, |
| 55 | }, |
| 56 | } |
| 57 | |
| 58 | vlanField := openflow_13.OfpOxmField{ |
| 59 | OxmClass: openflow_13.OfpOxmClass_OFPXMC_OPENFLOW_BASIC, |
| 60 | Field: &openflow_13.OfpOxmField_OfbField{ |
| 61 | OfbField: &openflow_13.OfpOxmOfbField{ |
| 62 | Type: openflow_13.OxmOfbFieldTypes_OFPXMT_OFB_VLAN_VID, |
| 63 | Value: &openflow_13.OfpOxmOfbField_VlanVid{VlanVid: 4096}, |
| 64 | }, |
| 65 | }, |
| 66 | } |
| 67 | |
| 68 | return openflow_13.OfpFlowStats{ |
| 69 | Id: uint64(id), |
| 70 | Priority: 10000, |
| 71 | Flags: 1, |
| 72 | Match: &openflow_13.OfpMatch{ |
| 73 | Type: openflow_13.OfpMatchType_OFPMT_OXM, |
| 74 | OxmFields: []*openflow_13.OfpOxmField{ |
| 75 | &portField, ðField, &vlanField, |
| 76 | }, |
| 77 | }, |
| 78 | Instructions: []*openflow_13.OfpInstruction{ |
| 79 | { |
| 80 | Type: uint32(openflow_13.OfpInstructionType_OFPIT_APPLY_ACTIONS), |
| 81 | Data: &openflow_13.OfpInstruction_Actions{ |
| 82 | Actions: &openflow_13.OfpInstructionActions{ |
| 83 | Actions: []*openflow_13.OfpAction{ |
| 84 | { |
| 85 | Type: openflow_13.OfpActionType_OFPAT_OUTPUT, |
| 86 | Action: &openflow_13.OfpAction_Output{ |
| 87 | Output: &openflow_13.OfpActionOutput{ |
| 88 | Port: 4294967293, |
| 89 | }, |
| 90 | }, |
| 91 | }, |
| 92 | }, |
| 93 | }, |
| 94 | }, |
| 95 | }, |
| 96 | { |
| 97 | Type: uint32(openflow_13.OfpInstructionType_OFPIT_WRITE_METADATA), |
| 98 | Data: &openflow_13.OfpInstruction_WriteMetadata{ |
| 99 | WriteMetadata: &openflow_13.OfpInstructionWriteMetadata{ |
| 100 | Metadata: 274877906944, |
| 101 | }, |
| 102 | }, |
| 103 | }, |
| 104 | { |
| 105 | Type: uint32(openflow_13.OfpInstructionType_OFPIT_METER), |
| 106 | Data: &openflow_13.OfpInstruction_Meter{ |
| 107 | Meter: &openflow_13.OfpInstructionMeter{ |
| 108 | MeterId: 2, |
| 109 | }, |
| 110 | }, |
| 111 | }, |
| 112 | }, |
| 113 | } |
| 114 | } |
| 115 | |
| 116 | func createRandomFlows(count int) []*openflow_13.OfpFlowStats { |
| 117 | |
| 118 | var flows []*openflow_13.OfpFlowStats |
| 119 | |
| 120 | n := 0 |
| 121 | for n < count { |
| 122 | |
| 123 | flow := createEapolFlow(n) |
| 124 | |
| 125 | flows = append(flows, &flow) |
| 126 | |
| 127 | n = n + 1 |
| 128 | } |
| 129 | return flows |
| 130 | } |
| 131 | |
| 132 | func createRandomPorts(count int) []*voltha.LogicalPort { |
| 133 | var ports []*voltha.LogicalPort |
| 134 | |
| 135 | n := 0 |
| 136 | for n < count { |
| 137 | |
| 138 | port := voltha.LogicalPort{ |
| 139 | Id: fmt.Sprintf("uni-%d", n), |
| 140 | OfpPort: &openflow_13.OfpPort{ |
| 141 | PortNo: uint32(n), |
| 142 | HwAddr: []uint32{8, 0, 0, 0, 0, uint32(n)}, |
| 143 | Name: fmt.Sprintf("BBSM-%d", n), |
| 144 | State: uint32(openflow_13.OfpPortState_OFPPS_LIVE), |
| 145 | Curr: 4128, |
| 146 | Advertised: 4128, |
| 147 | Peer: 4128, |
| 148 | CurrSpeed: 32, |
| 149 | MaxSpeed: 32, |
| 150 | Config: 1, |
| 151 | Supported: 1, |
| 152 | }, |
| 153 | } |
| 154 | |
| 155 | ports = append(ports, &port) |
| 156 | |
| 157 | n = n + 1 |
| 158 | } |
| 159 | return ports |
| 160 | } |
| 161 | |
| 162 | func newTestOFConnection(flowsCount int, portsCount int) *OFConnection { |
| 163 | |
| 164 | flows := openflow_13.Flows{ |
| 165 | Items: createRandomFlows(flowsCount), |
| 166 | } |
| 167 | |
| 168 | ports := voltha.LogicalPorts{ |
| 169 | Items: createRandomPorts(portsCount), |
| 170 | } |
| 171 | |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 172 | volthaClient := mock.MockVolthaClient{ |
| 173 | LogicalDeviceFlows: flows, |
| 174 | LogicalPorts: ports, |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 175 | } |
| 176 | |
| 177 | volthaClientHolder := &holder.VolthaServiceClientHolder{} |
| 178 | volthaClientHolder.Set(volthaClient) |
| 179 | |
| 180 | return &OFConnection{ |
| 181 | VolthaClient: volthaClientHolder, |
| 182 | flowsChunkSize: ofcFlowsChunkSize, |
| 183 | portsChunkSize: ofcPortsChunkSize, |
| 184 | portsDescChunkSize: ofcPortsDescChunkSize, |
| 185 | } |
| 186 | } |
| 187 | |
Andrea Campanella | f2cf13e | 2020-09-03 16:22:27 +0200 | [diff] [blame] | 188 | func TestHandleFlowStatsRequestExactlyDivisible(t *testing.T) { |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 189 | |
Andrea Campanella | f2cf13e | 2020-09-03 16:22:27 +0200 | [diff] [blame] | 190 | generatedFlowsCount := ofcFlowsChunkSize * 5 |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 191 | |
Andrea Campanella | f2cf13e | 2020-09-03 16:22:27 +0200 | [diff] [blame] | 192 | testCorrectChunkAndReplyMoreFlag(t, generatedFlowsCount) |
| 193 | } |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 194 | |
Andrea Campanella | f2cf13e | 2020-09-03 16:22:27 +0200 | [diff] [blame] | 195 | func TestHandleFlowStatsRequestNotExactlyDivisible(t *testing.T) { |
| 196 | |
| 197 | generatedFlowsCount := int(ofcFlowsChunkSize * 5.5) |
| 198 | |
| 199 | testCorrectChunkAndReplyMoreFlag(t, generatedFlowsCount) |
| 200 | } |
| 201 | |
| 202 | func testCorrectChunkAndReplyMoreFlag(t *testing.T, generatedFlowsCount int) { |
| 203 | ofc := newTestOFConnection(generatedFlowsCount, 0) |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 204 | request := of13.NewFlowStatsRequest() |
Rohan Agrawal | c32d993 | 2020-06-15 11:01:47 +0000 | [diff] [blame] | 205 | replies, err := ofc.handleFlowStatsRequest(context.Background(), request) |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 206 | assert.Equal(t, err, nil) |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 207 | // check that the correct number of messages is generated |
| 208 | assert.Equal(t, int(math.Ceil(float64(generatedFlowsCount)/ofcFlowsChunkSize)), len(replies)) |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 209 | n := 1 |
| 210 | entriesCount := 0 |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 211 | for _, r := range replies { |
| 212 | json, _ := r.Flags.MarshalJSON() |
| 213 | |
| 214 | // check that the ReplyMore flag is correctly set in the messages |
| 215 | if n == len(replies) { |
| 216 | assert.Equal(t, string(json), "{}") |
| 217 | } else { |
| 218 | assert.Equal(t, string(json), "{\"OFPSFReplyMore\": true}") |
| 219 | } |
| 220 | |
| 221 | // check the message size |
| 222 | enc := goloxi.NewEncoder() |
| 223 | if err := r.Serialize(enc); err != nil { |
| 224 | t.Fail() |
| 225 | } |
| 226 | bytes := enc.Bytes() |
| 227 | fmt.Println("FlowStats msg size: ", len(bytes)) |
| 228 | if len(bytes) > msgSizeLimit { |
| 229 | t.Fatal("Message size is bigger than 64KB") |
| 230 | } |
| 231 | |
| 232 | entriesCount = entriesCount + len(r.GetEntries()) |
| 233 | n++ |
| 234 | } |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 235 | // make sure all the generate item are included in the responses |
| 236 | assert.Equal(t, generatedFlowsCount, entriesCount) |
| 237 | } |
| 238 | |
| 239 | func TestHandlePortStatsRequest(t *testing.T) { |
| 240 | |
| 241 | generatedPortsCount := 2560 |
| 242 | |
| 243 | ofc := newTestOFConnection(0, generatedPortsCount) |
| 244 | |
| 245 | request := of13.NewPortStatsRequest() |
| 246 | |
| 247 | // request stats for all ports |
| 248 | request.PortNo = 0xffffffff |
| 249 | |
Girish Kumar | 01e0c63 | 2020-08-10 16:48:56 +0000 | [diff] [blame] | 250 | replies, err := ofc.handlePortStatsRequest(context.Background(), request) |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 251 | assert.Equal(t, err, nil) |
| 252 | |
| 253 | assert.Equal(t, int(math.Ceil(float64(generatedPortsCount)/ofcPortsChunkSize)), len(replies)) |
| 254 | |
| 255 | n := 1 |
| 256 | entriesCount := 0 |
| 257 | |
| 258 | for _, r := range replies { |
| 259 | json, _ := r.Flags.MarshalJSON() |
| 260 | |
| 261 | // check that the ReplyMore flag is correctly set in the messages |
| 262 | if n == len(replies) { |
| 263 | assert.Equal(t, string(json), "{}") |
| 264 | } else { |
| 265 | assert.Equal(t, string(json), "{\"OFPSFReplyMore\": true}") |
| 266 | } |
| 267 | |
| 268 | // check the message size |
| 269 | enc := goloxi.NewEncoder() |
| 270 | if err := r.Serialize(enc); err != nil { |
| 271 | t.Fail() |
| 272 | } |
| 273 | bytes := enc.Bytes() |
| 274 | fmt.Println("PortStats msg size: ", len(bytes)) |
| 275 | if len(bytes) > msgSizeLimit { |
| 276 | t.Fatal("Message size is bigger than 64KB") |
| 277 | } |
| 278 | |
| 279 | entriesCount = entriesCount + len(r.GetEntries()) |
| 280 | n++ |
| 281 | } |
| 282 | |
| 283 | // make sure all the generate item are included in the responses |
| 284 | assert.Equal(t, generatedPortsCount, entriesCount) |
| 285 | } |
| 286 | |
| 287 | func TestHandlePortDescStatsRequest(t *testing.T) { |
| 288 | |
| 289 | generatedPortsCount := 2560 |
| 290 | |
| 291 | ofc := newTestOFConnection(0, generatedPortsCount) |
| 292 | |
| 293 | request := of13.NewPortDescStatsRequest() |
| 294 | |
Girish Kumar | 01e0c63 | 2020-08-10 16:48:56 +0000 | [diff] [blame] | 295 | replies, err := ofc.handlePortDescStatsRequest(context.Background(), request) |
Matteo Scandolo | 256266d | 2020-06-01 13:44:07 -0700 | [diff] [blame] | 296 | assert.Equal(t, err, nil) |
| 297 | |
| 298 | // check that the correct number of messages is generated |
| 299 | assert.Equal(t, int(math.Ceil(float64(generatedPortsCount)/ofcPortsDescChunkSize)), len(replies)) |
| 300 | |
| 301 | n := 1 |
| 302 | entriesCount := 0 |
| 303 | |
| 304 | for _, r := range replies { |
| 305 | json, _ := r.Flags.MarshalJSON() |
| 306 | |
| 307 | // check that the ReplyMore flag is correctly set in the messages |
| 308 | if n == len(replies) { |
| 309 | assert.Equal(t, string(json), "{}") |
| 310 | } else { |
| 311 | assert.Equal(t, string(json), "{\"OFPSFReplyMore\": true}") |
| 312 | } |
| 313 | |
| 314 | // check the message size |
| 315 | enc := goloxi.NewEncoder() |
| 316 | if err := r.Serialize(enc); err != nil { |
| 317 | t.Fail() |
| 318 | } |
| 319 | bytes := enc.Bytes() |
| 320 | fmt.Println("PortDesc msg size: ", len(bytes)) |
| 321 | if len(bytes) > msgSizeLimit { |
| 322 | t.Fatal("Message size is bigger than 64KB") |
| 323 | } |
| 324 | |
| 325 | entriesCount = entriesCount + len(r.GetEntries()) |
| 326 | n++ |
| 327 | } |
| 328 | |
| 329 | // make sure all the generate item are included in the responses |
| 330 | assert.Equal(t, generatedPortsCount, entriesCount) |
| 331 | } |