blob: b06b0f78e2287ebac2d264a627b1855db7a87eb2 [file] [log] [blame]
Kent Hagerman3136fbd2020-05-14 10:30:45 -04001/*
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
17package device
18
19import (
20 "context"
21 "errors"
22 "fmt"
Kent Hagerman3136fbd2020-05-14 10:30:45 -040023 "github.com/gogo/protobuf/proto"
24 "github.com/opencord/voltha-go/rw_core/route"
25 coreutils "github.com/opencord/voltha-go/rw_core/utils"
26 fu "github.com/opencord/voltha-lib-go/v3/pkg/flows"
27 "github.com/opencord/voltha-lib-go/v3/pkg/log"
28 ofp "github.com/opencord/voltha-protos/v3/go/openflow_13"
29 "github.com/opencord/voltha-protos/v3/go/voltha"
30 "google.golang.org/grpc/codes"
31 "google.golang.org/grpc/status"
khenaidoo0db4c812020-05-27 15:27:30 -040032 "strconv"
Kent Hagerman3136fbd2020-05-14 10:30:45 -040033)
34
Kent Hagerman433a31a2020-05-20 19:04:48 -040035// listLogicalDeviceFlows returns logical device flows
36func (agent *LogicalAgent) listLogicalDeviceFlows() map[uint64]*ofp.OfpFlowStats {
37 flowIDs := agent.flowLoader.List()
38 flows := make(map[uint64]*ofp.OfpFlowStats, len(flowIDs))
39 for flowID := range flowIDs {
40 if flowHandle, have := agent.flowLoader.Lock(flowID); have {
41 flows[flowID] = flowHandle.GetReadOnly()
42 flowHandle.Unlock()
43 }
44 }
45 return flows
46}
47
Kent Hagerman3136fbd2020-05-14 10:30:45 -040048//updateFlowTable updates the flow table of that logical device
49func (agent *LogicalAgent) updateFlowTable(ctx context.Context, flow *ofp.OfpFlowMod) error {
50 logger.Debug("UpdateFlowTable")
51 if flow == nil {
52 return nil
53 }
54
Kent Hagerman3136fbd2020-05-14 10:30:45 -040055 switch flow.GetCommand() {
56 case ofp.OfpFlowModCommand_OFPFC_ADD:
57 return agent.flowAdd(ctx, flow)
58 case ofp.OfpFlowModCommand_OFPFC_DELETE:
59 return agent.flowDelete(ctx, flow)
60 case ofp.OfpFlowModCommand_OFPFC_DELETE_STRICT:
61 return agent.flowDeleteStrict(ctx, flow)
62 case ofp.OfpFlowModCommand_OFPFC_MODIFY:
63 return agent.flowModify(flow)
64 case ofp.OfpFlowModCommand_OFPFC_MODIFY_STRICT:
65 return agent.flowModifyStrict(flow)
66 }
67 return status.Errorf(codes.Internal,
68 "unhandled-command: lDeviceId:%s, command:%s", agent.logicalDeviceID, flow.GetCommand())
69}
70
71//flowAdd adds a flow to the flow table of that logical device
72func (agent *LogicalAgent) flowAdd(ctx context.Context, mod *ofp.OfpFlowMod) error {
73 logger.Debugw("flowAdd", log.Fields{"flow": mod})
74 if mod == nil {
75 return nil
76 }
77 flow, err := fu.FlowStatsEntryFromFlowModMessage(mod)
78 if err != nil {
79 logger.Errorw("flowAdd-failed", log.Fields{"flowMod": mod, "err": err})
80 return err
81 }
82 var updated bool
83 var changed bool
84 if changed, updated, err = agent.decomposeAndAdd(ctx, flow, mod); err != nil {
85 logger.Errorw("flow-decompose-and-add-failed ", log.Fields{"flowMod": mod, "err": err})
86 return err
87 }
88 if changed && !updated {
89 if dbupdated := agent.updateFlowCountOfMeterStats(ctx, mod, flow, false); !dbupdated {
90 return fmt.Errorf("couldnt-updated-flow-stats-%s", strconv.FormatUint(flow.Id, 10))
91 }
92 }
93 return nil
94
95}
96
97func (agent *LogicalAgent) decomposeAndAdd(ctx context.Context, flow *ofp.OfpFlowStats, mod *ofp.OfpFlowMod) (bool, bool, error) {
98 changed := false
99 updated := false
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400100 var flowToReplace *ofp.OfpFlowStats
101
102 //if flow is not found in the map, create a new entry, otherwise get the existing one.
Kent Hagerman433a31a2020-05-20 19:04:48 -0400103 flowHandle, created, err := agent.flowLoader.LockOrCreate(ctx, flow)
104 if err != nil {
105 return changed, updated, err
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400106 }
Kent Hagerman433a31a2020-05-20 19:04:48 -0400107 defer flowHandle.Unlock()
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400108
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400109 flows := make([]*ofp.OfpFlowStats, 0)
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400110 checkOverlap := (mod.Flags & uint32(ofp.OfpFlowModFlags_OFPFF_CHECK_OVERLAP)) != 0
111 if checkOverlap {
Kent Hagerman433a31a2020-05-20 19:04:48 -0400112 // TODO: this currently does nothing
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400113 if overlapped := fu.FindOverlappingFlows(flows, mod); len(overlapped) != 0 {
Kent Hagerman433a31a2020-05-20 19:04:48 -0400114 // TODO: should this error be notified other than being logged?
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400115 logger.Warnw("overlapped-flows", log.Fields{"logicaldeviceId": agent.logicalDeviceID})
116 } else {
117 // Add flow
118 changed = true
119 }
120 } else {
Kent Hagerman433a31a2020-05-20 19:04:48 -0400121 if !created {
122 flowToReplace = flowHandle.GetReadOnly()
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400123 if (mod.Flags & uint32(ofp.OfpFlowModFlags_OFPFF_RESET_COUNTS)) != 0 {
124 flow.ByteCount = flowToReplace.ByteCount
125 flow.PacketCount = flowToReplace.PacketCount
126 }
127 if !proto.Equal(flowToReplace, flow) {
128 changed = true
129 updated = true
130 }
131 } else {
132 changed = true
133 }
134 }
135 logger.Debugw("flowAdd-changed", log.Fields{"changed": changed, "updated": updated})
136 if changed {
Kent Hagerman433a31a2020-05-20 19:04:48 -0400137 updatedFlows := map[uint64]*ofp.OfpFlowStats{flow.Id: flow}
138
139 flowMeterConfig, err := agent.GetMeterConfig(updatedFlows)
140 if err != nil {
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400141 logger.Error("Meter-referred-in-flow-not-present")
142 return changed, updated, err
143 }
Kent Hagerman433a31a2020-05-20 19:04:48 -0400144
145 groupIDs := agent.groupLoader.List()
146 groups := make(map[uint32]*ofp.OfpGroupEntry, len(groupIDs))
147 for groupID := range groupIDs {
148 if groupHandle, have := agent.groupLoader.Lock(groupID); have {
149 groups[groupID] = groupHandle.GetReadOnly()
150 groupHandle.Unlock()
151 }
152 }
153
154 deviceRules, err := agent.flowDecomposer.DecomposeRules(ctx, agent, updatedFlows, groups)
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400155 if err != nil {
156 return changed, updated, err
157 }
158
159 logger.Debugw("rules", log.Fields{"rules": deviceRules.String()})
160 // Update store and cache
161 if updated {
Kent Hagerman433a31a2020-05-20 19:04:48 -0400162 if err := flowHandle.Update(ctx, flow); err != nil {
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400163 return changed, updated, err
164 }
165 }
Kent Hagerman433a31a2020-05-20 19:04:48 -0400166 respChannels := agent.addFlowsAndGroupsToDevices(deviceRules, toMetadata(flowMeterConfig))
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400167 // Create the go routines to wait
168 go func() {
169 // Wait for completion
170 if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChannels...); res != nil {
171 logger.Infow("failed-to-add-flows-will-attempt-deletion", log.Fields{"errors": res, "logical-device-id": agent.logicalDeviceID})
172 // Revert added flows
Kent Hagerman433a31a2020-05-20 19:04:48 -0400173 if err := agent.revertAddedFlows(context.Background(), mod, flow, flowToReplace, deviceRules, toMetadata(flowMeterConfig)); err != nil {
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400174 logger.Errorw("failure-to-delete-flows-after-failed-addition", log.Fields{"logical-device-id": agent.logicalDeviceID, "error": err})
175 }
176 }
177 }()
178 }
179 return changed, updated, nil
180}
181
182// revertAddedFlows reverts flows after the flowAdd request has failed. All flows corresponding to that flowAdd request
183// will be reverted, both from the logical devices and the devices.
184func (agent *LogicalAgent) revertAddedFlows(ctx context.Context, mod *ofp.OfpFlowMod, addedFlow *ofp.OfpFlowStats, replacedFlow *ofp.OfpFlowStats, deviceRules *fu.DeviceRules, metadata *voltha.FlowMetadata) error {
185 logger.Debugw("revertFlowAdd", log.Fields{"added-flow": addedFlow, "replaced-flow": replacedFlow, "device-rules": deviceRules, "metadata": metadata})
186
Kent Hagerman433a31a2020-05-20 19:04:48 -0400187 flowHandle, have := agent.flowLoader.Lock(addedFlow.Id)
188 if !have {
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400189 // Not found - do nothing
190 log.Debugw("flow-not-found", log.Fields{"added-flow": addedFlow})
191 return nil
192 }
Kent Hagerman433a31a2020-05-20 19:04:48 -0400193 defer flowHandle.Unlock()
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400194
195 if replacedFlow != nil {
Kent Hagerman433a31a2020-05-20 19:04:48 -0400196 if err := flowHandle.Update(ctx, replacedFlow); err != nil {
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400197 return err
198 }
199 } else {
Kent Hagerman433a31a2020-05-20 19:04:48 -0400200 if err := flowHandle.Delete(ctx); err != nil {
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400201 return err
202 }
203 }
Kent Hagerman433a31a2020-05-20 19:04:48 -0400204
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400205 // Revert meters
206 if changedMeterStats := agent.updateFlowCountOfMeterStats(ctx, mod, addedFlow, true); !changedMeterStats {
207 return fmt.Errorf("Unable-to-revert-meterstats-for-flow-%s", strconv.FormatUint(addedFlow.Id, 10))
208 }
209
210 // Update the devices
khenaidoo0db4c812020-05-27 15:27:30 -0400211 respChnls := agent.deleteFlowsAndGroupsFromDevices(deviceRules, metadata)
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400212
213 // Wait for the responses
214 go func() {
215 // Since this action is taken following an add failure, we may also receive a failure for the revert
216 if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil {
217 logger.Warnw("failure-reverting-added-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res})
218 }
219 }()
220
221 return nil
222}
223
224//flowDelete deletes a flow from the flow table of that logical device
225func (agent *LogicalAgent) flowDelete(ctx context.Context, mod *ofp.OfpFlowMod) error {
226 logger.Debug("flowDelete")
227 if mod == nil {
228 return nil
229 }
230
Kent Hagerman433a31a2020-05-20 19:04:48 -0400231 //build a list of what to delete
232 toDelete := make(map[uint64]*ofp.OfpFlowStats)
233
234 // add perfectly matching entry if exists
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400235 fs, err := fu.FlowStatsEntryFromFlowModMessage(mod)
236 if err != nil {
237 return err
238 }
Kent Hagerman433a31a2020-05-20 19:04:48 -0400239 if handle, have := agent.flowLoader.Lock(fs.Id); have {
240 toDelete[fs.Id] = handle.GetReadOnly()
241 handle.Unlock()
242 }
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400243
Kent Hagerman433a31a2020-05-20 19:04:48 -0400244 // search through all the flows
245 for flowID := range agent.flowLoader.List() {
246 if flowHandle, have := agent.flowLoader.Lock(flowID); have {
247 if flow := flowHandle.GetReadOnly(); fu.FlowMatchesMod(flow, mod) {
248 toDelete[flow.Id] = flow
249 }
250 flowHandle.Unlock()
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400251 }
252 }
Kent Hagerman433a31a2020-05-20 19:04:48 -0400253
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400254 //Delete the matched flows
255 if len(toDelete) > 0 {
256 logger.Debugw("flowDelete", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "toDelete": len(toDelete)})
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400257
Kent Hagerman433a31a2020-05-20 19:04:48 -0400258 for _, flow := range toDelete {
259 if flowHandle, have := agent.flowLoader.Lock(flow.Id); have {
260 // TODO: Flow should only be updated if meter is updated, and meter should only be updated if flow is updated
261 // currently an error while performing the second operation will leave an inconsistent state in kv.
262 // This should be a single atomic operation down to the kv.
263 if changedMeter := agent.updateFlowCountOfMeterStats(ctx, mod, flowHandle.GetReadOnly(), false); !changedMeter {
264 flowHandle.Unlock()
265 return fmt.Errorf("cannot-delete-flow-%d. Meter-update-failed", flow.Id)
266 }
267 // Update store and cache
268 if err := flowHandle.Delete(ctx); err != nil {
269 flowHandle.Unlock()
270 return fmt.Errorf("cannot-delete-flows-%d. Delete-from-store-failed", flow.Id)
271 }
272 flowHandle.Unlock()
273 // TODO: since this is executed in a loop without also updating meter stats, and error part way through this
274 // operation will leave inconsistent state in the meter stats & flows on the devices.
275 // This & related meter updates should be a single atomic operation down to the kv.
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400276 }
277 }
Kent Hagerman433a31a2020-05-20 19:04:48 -0400278
279 metersConfig, err := agent.GetMeterConfig(toDelete)
280 if err != nil { // This should never happen
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400281 logger.Error("Meter-referred-in-flows-not-present")
282 return err
283 }
Kent Hagerman433a31a2020-05-20 19:04:48 -0400284
285 groups := make(map[uint32]*ofp.OfpGroupEntry)
286 for groupID := range agent.groupLoader.List() {
287 if groupHandle, have := agent.groupLoader.Lock(groupID); have {
288 groups[groupID] = groupHandle.GetReadOnly()
289 groupHandle.Unlock()
290 }
291 }
292
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400293 var respChnls []coreutils.Response
294 var partialRoute bool
295 var deviceRules *fu.DeviceRules
Kent Hagerman433a31a2020-05-20 19:04:48 -0400296 deviceRules, err = agent.flowDecomposer.DecomposeRules(ctx, agent, toDelete, groups)
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400297 if err != nil {
298 // A no route error means no route exists between the ports specified in the flow. This can happen when the
299 // child device is deleted and a request to delete flows from the parent device is received
300 if !errors.Is(err, route.ErrNoRoute) {
301 logger.Errorw("unexpected-error-received", log.Fields{"flows-to-delete": toDelete, "error": err})
302 return err
303 }
304 partialRoute = true
305 }
306
307 // Update the devices
308 if partialRoute {
Kent Hagerman433a31a2020-05-20 19:04:48 -0400309 respChnls = agent.deleteFlowsFromParentDevice(toDelete, toMetadata(metersConfig))
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400310 } else {
Kent Hagerman433a31a2020-05-20 19:04:48 -0400311 respChnls = agent.deleteFlowsAndGroupsFromDevices(deviceRules, toMetadata(metersConfig))
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400312 }
313
314 // Wait for the responses
315 go func() {
316 // Wait for completion
317 if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil {
318 logger.Errorw("failure-updating-device-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res})
319 // TODO: Revert the flow deletion
320 }
321 }()
322 }
323 //TODO: send announcement on delete
324 return nil
325}
326
327//flowDeleteStrict deletes a flow from the flow table of that logical device
328func (agent *LogicalAgent) flowDeleteStrict(ctx context.Context, mod *ofp.OfpFlowMod) error {
329 logger.Debugw("flowDeleteStrict", log.Fields{"mod": mod})
330 if mod == nil {
331 return nil
332 }
333
334 flow, err := fu.FlowStatsEntryFromFlowModMessage(mod)
335 if err != nil {
336 return err
337 }
338 logger.Debugw("flow-id-in-flow-delete-strict", log.Fields{"flowID": flow.Id})
Kent Hagerman433a31a2020-05-20 19:04:48 -0400339 flowHandle, have := agent.flowLoader.Lock(flow.Id)
340 if !have {
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400341 logger.Debugw("Skipping-flow-delete-strict-request. No-flow-found", log.Fields{"flowMod": mod})
342 return nil
343 }
Kent Hagerman433a31a2020-05-20 19:04:48 -0400344 defer flowHandle.Unlock()
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400345
Kent Hagerman433a31a2020-05-20 19:04:48 -0400346 groups := make(map[uint32]*ofp.OfpGroupEntry)
347 for groupID := range agent.groupLoader.List() {
348 if groupHandle, have := agent.groupLoader.Lock(groupID); have {
349 groups[groupID] = groupHandle.GetReadOnly()
350 groupHandle.Unlock()
351 }
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400352 }
Kent Hagerman433a31a2020-05-20 19:04:48 -0400353
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400354 if changedMeter := agent.updateFlowCountOfMeterStats(ctx, mod, flow, false); !changedMeter {
355 return fmt.Errorf("Cannot delete flow - %s. Meter update failed", flow)
356 }
357
Kent Hagerman433a31a2020-05-20 19:04:48 -0400358 flowsToDelete := map[uint64]*ofp.OfpFlowStats{flow.Id: flowHandle.GetReadOnly()}
359
360 flowMetadata, err := agent.GetMeterConfig(flowsToDelete)
361 if err != nil {
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400362 logger.Error("meter-referred-in-flows-not-present")
363 return err
364 }
365 var respChnls []coreutils.Response
366 var partialRoute bool
Kent Hagerman433a31a2020-05-20 19:04:48 -0400367 deviceRules, err := agent.flowDecomposer.DecomposeRules(ctx, agent, flowsToDelete, groups)
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400368 if err != nil {
369 // A no route error means no route exists between the ports specified in the flow. This can happen when the
370 // child device is deleted and a request to delete flows from the parent device is received
371 if !errors.Is(err, route.ErrNoRoute) {
372 logger.Errorw("unexpected-error-received", log.Fields{"flows-to-delete": flowsToDelete, "error": err})
373 return err
374 }
375 partialRoute = true
376 }
377
378 // Update the model
Kent Hagerman433a31a2020-05-20 19:04:48 -0400379 if err := flowHandle.Delete(ctx); err != nil {
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400380 return err
381 }
382 // Update the devices
383 if partialRoute {
Kent Hagerman433a31a2020-05-20 19:04:48 -0400384 respChnls = agent.deleteFlowsFromParentDevice(flowsToDelete, toMetadata(flowMetadata))
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400385 } else {
Kent Hagerman433a31a2020-05-20 19:04:48 -0400386 respChnls = agent.deleteFlowsAndGroupsFromDevices(deviceRules, toMetadata(flowMetadata))
Kent Hagerman3136fbd2020-05-14 10:30:45 -0400387 }
388
389 // Wait for completion
390 go func() {
391 if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil {
392 logger.Warnw("failure-deleting-device-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res})
393 //TODO: Revert flow changes
394 }
395 }()
396
397 return nil
398}
399
400//flowModify modifies a flow from the flow table of that logical device
401func (agent *LogicalAgent) flowModify(mod *ofp.OfpFlowMod) error {
402 return errors.New("flowModify not implemented")
403}
404
405//flowModifyStrict deletes a flow from the flow table of that logical device
406func (agent *LogicalAgent) flowModifyStrict(mod *ofp.OfpFlowMod) error {
407 return errors.New("flowModifyStrict not implemented")
408}
Kent Hagerman433a31a2020-05-20 19:04:48 -0400409
410// TODO: Remove this helper, just pass the map through to functions directly
411func toMetadata(meters map[uint32]*ofp.OfpMeterConfig) *voltha.FlowMetadata {
412 ctr, ret := 0, make([]*ofp.OfpMeterConfig, len(meters))
413 for _, meter := range meters {
414 ret[ctr] = meter
415 ctr++
416 }
417 return &voltha.FlowMetadata{Meters: ret}
418}
419
420func (agent *LogicalAgent) deleteFlowsHavingMeter(ctx context.Context, meterID uint32) error {
421 logger.Infow("Delete-flows-matching-meter", log.Fields{"meter": meterID})
422 for flowID := range agent.flowLoader.List() {
423 if flowHandle, have := agent.flowLoader.Lock(flowID); have {
424 if flowMeterID := fu.GetMeterIdFromFlow(flowHandle.GetReadOnly()); flowMeterID != 0 && flowMeterID == meterID {
425 if err := flowHandle.Delete(ctx); err != nil {
426 //TODO: Think on carrying on and deleting the remaining flows, instead of returning.
427 //Anyways this returns an error to controller which possibly results with a re-deletion.
428 //Then how can we handle the new deletion request(Same for group deletion)?
429 return err
430 }
431 }
432 flowHandle.Unlock()
433 }
434 }
435 return nil
436}
437
438func (agent *LogicalAgent) deleteFlowsHavingGroup(ctx context.Context, groupID uint32) (map[uint64]*ofp.OfpFlowStats, error) {
439 logger.Infow("Delete-flows-matching-group", log.Fields{"groupID": groupID})
440 flowsRemoved := make(map[uint64]*ofp.OfpFlowStats)
441 for flowID := range agent.flowLoader.List() {
442 if flowHandle, have := agent.flowLoader.Lock(flowID); have {
443 if flow := flowHandle.GetReadOnly(); fu.FlowHasOutGroup(flow, groupID) {
444 if err := flowHandle.Delete(ctx); err != nil {
445 return nil, err
446 }
447 flowsRemoved[flowID] = flow
448 }
449 flowHandle.Unlock()
450 }
451 }
452 return flowsRemoved, nil
453}