Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [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 device |
| 18 | |
| 19 | import ( |
| 20 | "context" |
| 21 | "errors" |
| 22 | "fmt" |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 23 | "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" |
khenaidoo | 0db4c81 | 2020-05-27 15:27:30 -0400 | [diff] [blame] | 32 | "strconv" |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 33 | ) |
| 34 | |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 35 | // listLogicalDeviceFlows returns logical device flows |
| 36 | func (agent *LogicalAgent) listLogicalDeviceFlows() map[uint64]*ofp.OfpFlowStats { |
Kent Hagerman | fa9d6d4 | 2020-05-25 11:49:40 -0400 | [diff] [blame] | 37 | flowIDs := agent.flowLoader.ListIDs() |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 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 Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 48 | //updateFlowTable updates the flow table of that logical device |
| 49 | func (agent *LogicalAgent) updateFlowTable(ctx context.Context, flow *ofp.OfpFlowMod) error { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 50 | logger.Debug(ctx, "UpdateFlowTable") |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 51 | if flow == nil { |
| 52 | return nil |
| 53 | } |
| 54 | |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 55 | 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 |
| 72 | func (agent *LogicalAgent) flowAdd(ctx context.Context, mod *ofp.OfpFlowMod) error { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 73 | logger.Debugw(ctx, "flowAdd", log.Fields{"flow": mod}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 74 | if mod == nil { |
| 75 | return nil |
| 76 | } |
| 77 | flow, err := fu.FlowStatsEntryFromFlowModMessage(mod) |
| 78 | if err != nil { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 79 | logger.Errorw(ctx, "flowAdd-failed", log.Fields{"flowMod": mod, "err": err}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 80 | return err |
| 81 | } |
| 82 | var updated bool |
| 83 | var changed bool |
| 84 | if changed, updated, err = agent.decomposeAndAdd(ctx, flow, mod); err != nil { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 85 | logger.Errorw(ctx, "flow-decompose-and-add-failed ", log.Fields{"flowMod": mod, "err": err}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 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 | |
| 97 | func (agent *LogicalAgent) decomposeAndAdd(ctx context.Context, flow *ofp.OfpFlowStats, mod *ofp.OfpFlowMod) (bool, bool, error) { |
| 98 | changed := false |
| 99 | updated := false |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 100 | var flowToReplace *ofp.OfpFlowStats |
| 101 | |
| 102 | //if flow is not found in the map, create a new entry, otherwise get the existing one. |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 103 | flowHandle, created, err := agent.flowLoader.LockOrCreate(ctx, flow) |
| 104 | if err != nil { |
| 105 | return changed, updated, err |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 106 | } |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 107 | defer flowHandle.Unlock() |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 108 | |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 109 | flows := make([]*ofp.OfpFlowStats, 0) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 110 | checkOverlap := (mod.Flags & uint32(ofp.OfpFlowModFlags_OFPFF_CHECK_OVERLAP)) != 0 |
| 111 | if checkOverlap { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 112 | // TODO: this currently does nothing |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 113 | if overlapped := fu.FindOverlappingFlows(flows, mod); len(overlapped) != 0 { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 114 | // TODO: should this error be notified other than being logged? |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 115 | logger.Warnw(ctx, "overlapped-flows", log.Fields{"logicaldeviceId": agent.logicalDeviceID}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 116 | } else { |
| 117 | // Add flow |
| 118 | changed = true |
| 119 | } |
| 120 | } else { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 121 | if !created { |
| 122 | flowToReplace = flowHandle.GetReadOnly() |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 123 | 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 | } |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 135 | logger.Debugw(ctx, "flowAdd-changed", log.Fields{"changed": changed, "updated": updated}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 136 | if changed { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 137 | updatedFlows := map[uint64]*ofp.OfpFlowStats{flow.Id: flow} |
| 138 | |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 139 | flowMeterConfig, err := agent.GetMeterConfig(ctx, updatedFlows) |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 140 | if err != nil { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 141 | logger.Error(ctx, "Meter-referred-in-flow-not-present") |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 142 | return changed, updated, err |
| 143 | } |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 144 | |
Kent Hagerman | fa9d6d4 | 2020-05-25 11:49:40 -0400 | [diff] [blame] | 145 | groupIDs := agent.groupLoader.ListIDs() |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 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 Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 155 | if err != nil { |
| 156 | return changed, updated, err |
| 157 | } |
| 158 | |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 159 | logger.Debugw(ctx, "rules", log.Fields{"rules": deviceRules.String()}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 160 | // Update store and cache |
| 161 | if updated { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 162 | if err := flowHandle.Update(ctx, flow); err != nil { |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 163 | return changed, updated, err |
| 164 | } |
| 165 | } |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 166 | respChannels := agent.addFlowsAndGroupsToDevices(ctx, deviceRules, toMetadata(flowMeterConfig)) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 167 | // Create the go routines to wait |
| 168 | go func() { |
| 169 | // Wait for completion |
| 170 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChannels...); res != nil { |
Matteo Scandolo | 45e514a | 2020-08-05 15:27:10 -0700 | [diff] [blame] | 171 | logger.Infow(ctx, "failed-to-add-flows-will-attempt-deletion", log.Fields{ |
| 172 | "errors": res, |
| 173 | "logical-device-id": agent.logicalDeviceID, |
| 174 | "flows": flows, |
| 175 | "groups": groups, |
| 176 | }) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 177 | // Revert added flows |
Rohan Agrawal | cf12f20 | 2020-08-03 04:42:01 +0000 | [diff] [blame] | 178 | if err := agent.revertAddedFlows(log.WithSpanFromContext(context.Background(), ctx), mod, flow, flowToReplace, deviceRules, toMetadata(flowMeterConfig)); err != nil { |
Matteo Scandolo | 45e514a | 2020-08-05 15:27:10 -0700 | [diff] [blame] | 179 | logger.Errorw(ctx, "failure-to-delete-flows-after-failed-addition", log.Fields{ |
| 180 | "error": err, |
| 181 | "logical-device-id": agent.logicalDeviceID, |
| 182 | "flows": flows, |
| 183 | "groups": groups, |
| 184 | }) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 185 | } |
| 186 | } |
| 187 | }() |
| 188 | } |
| 189 | return changed, updated, nil |
| 190 | } |
| 191 | |
| 192 | // revertAddedFlows reverts flows after the flowAdd request has failed. All flows corresponding to that flowAdd request |
| 193 | // will be reverted, both from the logical devices and the devices. |
| 194 | func (agent *LogicalAgent) revertAddedFlows(ctx context.Context, mod *ofp.OfpFlowMod, addedFlow *ofp.OfpFlowStats, replacedFlow *ofp.OfpFlowStats, deviceRules *fu.DeviceRules, metadata *voltha.FlowMetadata) error { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 195 | logger.Debugw(ctx, "revertFlowAdd", log.Fields{"added-flow": addedFlow, "replaced-flow": replacedFlow, "device-rules": deviceRules, "metadata": metadata}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 196 | |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 197 | flowHandle, have := agent.flowLoader.Lock(addedFlow.Id) |
| 198 | if !have { |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 199 | // Not found - do nothing |
Girish Kumar | 3e8ee21 | 2020-08-19 17:50:11 +0000 | [diff] [blame] | 200 | logger.Debugw(ctx, "flow-not-found", log.Fields{"added-flow": addedFlow}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 201 | return nil |
| 202 | } |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 203 | defer flowHandle.Unlock() |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 204 | |
| 205 | if replacedFlow != nil { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 206 | if err := flowHandle.Update(ctx, replacedFlow); err != nil { |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 207 | return err |
| 208 | } |
| 209 | } else { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 210 | if err := flowHandle.Delete(ctx); err != nil { |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 211 | return err |
| 212 | } |
| 213 | } |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 214 | |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 215 | // Revert meters |
| 216 | if changedMeterStats := agent.updateFlowCountOfMeterStats(ctx, mod, addedFlow, true); !changedMeterStats { |
| 217 | return fmt.Errorf("Unable-to-revert-meterstats-for-flow-%s", strconv.FormatUint(addedFlow.Id, 10)) |
| 218 | } |
| 219 | |
| 220 | // Update the devices |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 221 | respChnls := agent.deleteFlowsAndGroupsFromDevices(ctx, deviceRules, metadata, mod) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 222 | |
| 223 | // Wait for the responses |
| 224 | go func() { |
| 225 | // Since this action is taken following an add failure, we may also receive a failure for the revert |
| 226 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 227 | logger.Warnw(ctx, "failure-reverting-added-flows", log.Fields{ |
Matteo Scandolo | 367162b | 2020-06-22 15:07:33 -0700 | [diff] [blame] | 228 | "logical-device-id": agent.logicalDeviceID, |
| 229 | "flow-cookie": mod.Cookie, |
| 230 | "errors": res, |
| 231 | }) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 232 | } |
| 233 | }() |
| 234 | |
| 235 | return nil |
| 236 | } |
| 237 | |
| 238 | //flowDelete deletes a flow from the flow table of that logical device |
| 239 | func (agent *LogicalAgent) flowDelete(ctx context.Context, mod *ofp.OfpFlowMod) error { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 240 | logger.Debug(ctx, "flowDelete") |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 241 | if mod == nil { |
| 242 | return nil |
| 243 | } |
| 244 | |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 245 | //build a list of what to delete |
| 246 | toDelete := make(map[uint64]*ofp.OfpFlowStats) |
| 247 | |
| 248 | // add perfectly matching entry if exists |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 249 | fs, err := fu.FlowStatsEntryFromFlowModMessage(mod) |
| 250 | if err != nil { |
| 251 | return err |
| 252 | } |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 253 | if handle, have := agent.flowLoader.Lock(fs.Id); have { |
| 254 | toDelete[fs.Id] = handle.GetReadOnly() |
| 255 | handle.Unlock() |
| 256 | } |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 257 | |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 258 | // search through all the flows |
Kent Hagerman | fa9d6d4 | 2020-05-25 11:49:40 -0400 | [diff] [blame] | 259 | for flowID := range agent.flowLoader.ListIDs() { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 260 | if flowHandle, have := agent.flowLoader.Lock(flowID); have { |
| 261 | if flow := flowHandle.GetReadOnly(); fu.FlowMatchesMod(flow, mod) { |
| 262 | toDelete[flow.Id] = flow |
| 263 | } |
| 264 | flowHandle.Unlock() |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 265 | } |
| 266 | } |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 267 | |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 268 | //Delete the matched flows |
| 269 | if len(toDelete) > 0 { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 270 | logger.Debugw(ctx, "flowDelete", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "toDelete": len(toDelete)}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 271 | |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 272 | for _, flow := range toDelete { |
| 273 | if flowHandle, have := agent.flowLoader.Lock(flow.Id); have { |
| 274 | // TODO: Flow should only be updated if meter is updated, and meter should only be updated if flow is updated |
| 275 | // currently an error while performing the second operation will leave an inconsistent state in kv. |
| 276 | // This should be a single atomic operation down to the kv. |
| 277 | if changedMeter := agent.updateFlowCountOfMeterStats(ctx, mod, flowHandle.GetReadOnly(), false); !changedMeter { |
| 278 | flowHandle.Unlock() |
| 279 | return fmt.Errorf("cannot-delete-flow-%d. Meter-update-failed", flow.Id) |
| 280 | } |
| 281 | // Update store and cache |
| 282 | if err := flowHandle.Delete(ctx); err != nil { |
| 283 | flowHandle.Unlock() |
| 284 | return fmt.Errorf("cannot-delete-flows-%d. Delete-from-store-failed", flow.Id) |
| 285 | } |
| 286 | flowHandle.Unlock() |
| 287 | // TODO: since this is executed in a loop without also updating meter stats, and error part way through this |
| 288 | // operation will leave inconsistent state in the meter stats & flows on the devices. |
| 289 | // This & related meter updates should be a single atomic operation down to the kv. |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 290 | } |
| 291 | } |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 292 | |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 293 | metersConfig, err := agent.GetMeterConfig(ctx, toDelete) |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 294 | if err != nil { // This should never happen |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 295 | logger.Error(ctx, "Meter-referred-in-flows-not-present") |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 296 | return err |
| 297 | } |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 298 | |
| 299 | groups := make(map[uint32]*ofp.OfpGroupEntry) |
Kent Hagerman | fa9d6d4 | 2020-05-25 11:49:40 -0400 | [diff] [blame] | 300 | for groupID := range agent.groupLoader.ListIDs() { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 301 | if groupHandle, have := agent.groupLoader.Lock(groupID); have { |
| 302 | groups[groupID] = groupHandle.GetReadOnly() |
| 303 | groupHandle.Unlock() |
| 304 | } |
| 305 | } |
| 306 | |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 307 | var respChnls []coreutils.Response |
| 308 | var partialRoute bool |
| 309 | var deviceRules *fu.DeviceRules |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 310 | deviceRules, err = agent.flowDecomposer.DecomposeRules(ctx, agent, toDelete, groups) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 311 | if err != nil { |
| 312 | // A no route error means no route exists between the ports specified in the flow. This can happen when the |
| 313 | // child device is deleted and a request to delete flows from the parent device is received |
| 314 | if !errors.Is(err, route.ErrNoRoute) { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 315 | logger.Errorw(ctx, "unexpected-error-received", log.Fields{"flows-to-delete": toDelete, "error": err}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 316 | return err |
| 317 | } |
| 318 | partialRoute = true |
| 319 | } |
| 320 | |
| 321 | // Update the devices |
| 322 | if partialRoute { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 323 | respChnls = agent.deleteFlowsFromParentDevice(ctx, toDelete, toMetadata(metersConfig), mod) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 324 | } else { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 325 | respChnls = agent.deleteFlowsAndGroupsFromDevices(ctx, deviceRules, toMetadata(metersConfig), mod) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 326 | } |
| 327 | |
| 328 | // Wait for the responses |
| 329 | go func() { |
| 330 | // Wait for completion |
| 331 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 332 | logger.Errorw(ctx, "failure-updating-device-flows", log.Fields{"logicalDeviceId": agent.logicalDeviceID, "errors": res}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 333 | // TODO: Revert the flow deletion |
| 334 | } |
| 335 | }() |
| 336 | } |
| 337 | //TODO: send announcement on delete |
| 338 | return nil |
| 339 | } |
| 340 | |
| 341 | //flowDeleteStrict deletes a flow from the flow table of that logical device |
| 342 | func (agent *LogicalAgent) flowDeleteStrict(ctx context.Context, mod *ofp.OfpFlowMod) error { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 343 | logger.Debugw(ctx, "flowDeleteStrict", log.Fields{"mod": mod}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 344 | if mod == nil { |
| 345 | return nil |
| 346 | } |
| 347 | |
| 348 | flow, err := fu.FlowStatsEntryFromFlowModMessage(mod) |
| 349 | if err != nil { |
| 350 | return err |
| 351 | } |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 352 | logger.Debugw(ctx, "flow-id-in-flow-delete-strict", log.Fields{"flowID": flow.Id}) |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 353 | flowHandle, have := agent.flowLoader.Lock(flow.Id) |
| 354 | if !have { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 355 | logger.Debugw(ctx, "Skipping-flow-delete-strict-request. No-flow-found", log.Fields{"flowMod": mod}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 356 | return nil |
| 357 | } |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 358 | defer flowHandle.Unlock() |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 359 | |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 360 | groups := make(map[uint32]*ofp.OfpGroupEntry) |
Kent Hagerman | fa9d6d4 | 2020-05-25 11:49:40 -0400 | [diff] [blame] | 361 | for groupID := range agent.groupLoader.ListIDs() { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 362 | if groupHandle, have := agent.groupLoader.Lock(groupID); have { |
| 363 | groups[groupID] = groupHandle.GetReadOnly() |
| 364 | groupHandle.Unlock() |
| 365 | } |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 366 | } |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 367 | |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 368 | if changedMeter := agent.updateFlowCountOfMeterStats(ctx, mod, flow, false); !changedMeter { |
| 369 | return fmt.Errorf("Cannot delete flow - %s. Meter update failed", flow) |
| 370 | } |
| 371 | |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 372 | flowsToDelete := map[uint64]*ofp.OfpFlowStats{flow.Id: flowHandle.GetReadOnly()} |
| 373 | |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 374 | flowMetadata, err := agent.GetMeterConfig(ctx, flowsToDelete) |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 375 | if err != nil { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 376 | logger.Error(ctx, "meter-referred-in-flows-not-present") |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 377 | return err |
| 378 | } |
| 379 | var respChnls []coreutils.Response |
| 380 | var partialRoute bool |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 381 | deviceRules, err := agent.flowDecomposer.DecomposeRules(ctx, agent, flowsToDelete, groups) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 382 | if err != nil { |
| 383 | // A no route error means no route exists between the ports specified in the flow. This can happen when the |
| 384 | // child device is deleted and a request to delete flows from the parent device is received |
| 385 | if !errors.Is(err, route.ErrNoRoute) { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 386 | logger.Errorw(ctx, "unexpected-error-received", log.Fields{"flows-to-delete": flowsToDelete, "error": err}) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 387 | return err |
| 388 | } |
| 389 | partialRoute = true |
| 390 | } |
| 391 | |
| 392 | // Update the model |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 393 | if err := flowHandle.Delete(ctx); err != nil { |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 394 | return err |
| 395 | } |
| 396 | // Update the devices |
| 397 | if partialRoute { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 398 | respChnls = agent.deleteFlowsFromParentDevice(ctx, flowsToDelete, toMetadata(flowMetadata), mod) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 399 | } else { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 400 | respChnls = agent.deleteFlowsAndGroupsFromDevices(ctx, deviceRules, toMetadata(flowMetadata), mod) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 401 | } |
| 402 | |
| 403 | // Wait for completion |
| 404 | go func() { |
| 405 | if res := coreutils.WaitForNilOrErrorResponses(agent.defaultTimeout, respChnls...); res != nil { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 406 | logger.Warnw(ctx, "failure-deleting-device-flows", log.Fields{ |
Matteo Scandolo | 367162b | 2020-06-22 15:07:33 -0700 | [diff] [blame] | 407 | "flow-cookie": mod.Cookie, |
| 408 | "logical-device-id": agent.logicalDeviceID, |
| 409 | "errors": res, |
| 410 | }) |
Kent Hagerman | 3136fbd | 2020-05-14 10:30:45 -0400 | [diff] [blame] | 411 | //TODO: Revert flow changes |
| 412 | } |
| 413 | }() |
| 414 | |
| 415 | return nil |
| 416 | } |
| 417 | |
| 418 | //flowModify modifies a flow from the flow table of that logical device |
| 419 | func (agent *LogicalAgent) flowModify(mod *ofp.OfpFlowMod) error { |
| 420 | return errors.New("flowModify not implemented") |
| 421 | } |
| 422 | |
| 423 | //flowModifyStrict deletes a flow from the flow table of that logical device |
| 424 | func (agent *LogicalAgent) flowModifyStrict(mod *ofp.OfpFlowMod) error { |
| 425 | return errors.New("flowModifyStrict not implemented") |
| 426 | } |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 427 | |
| 428 | // TODO: Remove this helper, just pass the map through to functions directly |
| 429 | func toMetadata(meters map[uint32]*ofp.OfpMeterConfig) *voltha.FlowMetadata { |
| 430 | ctr, ret := 0, make([]*ofp.OfpMeterConfig, len(meters)) |
| 431 | for _, meter := range meters { |
| 432 | ret[ctr] = meter |
| 433 | ctr++ |
| 434 | } |
| 435 | return &voltha.FlowMetadata{Meters: ret} |
| 436 | } |
| 437 | |
| 438 | func (agent *LogicalAgent) deleteFlowsHavingMeter(ctx context.Context, meterID uint32) error { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 439 | logger.Infow(ctx, "Delete-flows-matching-meter", log.Fields{"meter": meterID}) |
Kent Hagerman | fa9d6d4 | 2020-05-25 11:49:40 -0400 | [diff] [blame] | 440 | for flowID := range agent.flowLoader.ListIDs() { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 441 | if flowHandle, have := agent.flowLoader.Lock(flowID); have { |
| 442 | if flowMeterID := fu.GetMeterIdFromFlow(flowHandle.GetReadOnly()); flowMeterID != 0 && flowMeterID == meterID { |
| 443 | if err := flowHandle.Delete(ctx); err != nil { |
| 444 | //TODO: Think on carrying on and deleting the remaining flows, instead of returning. |
| 445 | //Anyways this returns an error to controller which possibly results with a re-deletion. |
| 446 | //Then how can we handle the new deletion request(Same for group deletion)? |
| 447 | return err |
| 448 | } |
| 449 | } |
| 450 | flowHandle.Unlock() |
| 451 | } |
| 452 | } |
| 453 | return nil |
| 454 | } |
| 455 | |
| 456 | func (agent *LogicalAgent) deleteFlowsHavingGroup(ctx context.Context, groupID uint32) (map[uint64]*ofp.OfpFlowStats, error) { |
Rohan Agrawal | 31f2180 | 2020-06-12 05:38:46 +0000 | [diff] [blame] | 457 | logger.Infow(ctx, "Delete-flows-matching-group", log.Fields{"groupID": groupID}) |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 458 | flowsRemoved := make(map[uint64]*ofp.OfpFlowStats) |
Kent Hagerman | fa9d6d4 | 2020-05-25 11:49:40 -0400 | [diff] [blame] | 459 | for flowID := range agent.flowLoader.ListIDs() { |
Kent Hagerman | 433a31a | 2020-05-20 19:04:48 -0400 | [diff] [blame] | 460 | if flowHandle, have := agent.flowLoader.Lock(flowID); have { |
| 461 | if flow := flowHandle.GetReadOnly(); fu.FlowHasOutGroup(flow, groupID) { |
| 462 | if err := flowHandle.Delete(ctx); err != nil { |
| 463 | return nil, err |
| 464 | } |
| 465 | flowsRemoved[flowID] = flow |
| 466 | } |
| 467 | flowHandle.Unlock() |
| 468 | } |
| 469 | } |
| 470 | return flowsRemoved, nil |
| 471 | } |