Passed
Pull Request — master (#137)
by Carlos
02:45
created

build.main.Main._send_flow_mods_from_request()   C

Complexity

Conditions 10

Size

Total Lines 37
Code Lines 26

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 25
CRAP Score 10

Importance

Changes 0
Metric Value
cc 10
eloc 26
nop 4
dl 0
loc 37
rs 5.9999
c 0
b 0
f 0
ccs 25
cts 25
cp 1
crap 10

How to fix   Complexity   

Complexity

Complex classes like build.main.Main._send_flow_mods_from_request() often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

1
"""kytos/flow_manager NApp installs, lists and deletes switch flows."""
2 1
from collections import OrderedDict
3 1
from copy import deepcopy
4
5 1
from flask import jsonify, request
6 1
from pyof.foundation.base import UBIntBase
7 1
from pyof.v0x01.asynchronous.error_msg import BadActionCode
8 1
from pyof.v0x01.common.phy_port import PortConfig
9 1
from werkzeug.exceptions import BadRequest, UnsupportedMediaType
10
11 1
from kytos.core import KytosEvent, KytosNApp, log, rest
12 1
from kytos.core.helpers import listen_to
13 1
from napps.kytos.flow_manager.match import match_flow
14 1
from napps.kytos.flow_manager.storehouse import StoreHouse
15 1
from napps.kytos.of_core.flow import FlowFactory
16
17 1
from .exceptions import InvalidCommandError
18 1
from .settings import CONSISTENCY_INTERVAL, FLOWS_DICT_MAX_SIZE
19
20
21 1
def cast_fields(flow_dict):
22
    """Make casting the match fields from UBInt() to native int ."""
23 1
    match = flow_dict['match']
24 1
    for field, value in match.items():
25 1
        if isinstance(value, UBIntBase):
26
            match[field] = int(value)
27 1
    flow_dict['match'] = match
28 1
    return flow_dict
29
30
31 1
class Main(KytosNApp):
32
    """Main class to be used by Kytos controller."""
33
34 1
    def setup(self):
35
        """Replace the 'init' method for the KytosApp subclass.
36
37
        The setup method is automatically called by the run method.
38
        Users shouldn't call this method directly.
39
        """
40 1
        log.debug("flow-manager starting")
41 1
        self._flow_mods_sent = OrderedDict()
42 1
        self._flow_mods_sent_max_size = FLOWS_DICT_MAX_SIZE
43
44
        # Storehouse client to save and restore flow data:
45 1
        self.storehouse = StoreHouse(self.controller)
46
47
        # Format of stored flow data:
48
        # {'flow_persistence': {'dpid_str': {'flow_list': [
49
        #                                     {'command': '<add|delete>',
50
        #                                      'flow': {flow_dict}}]}}}
51 1
        self.stored_flows = {}
52 1
        self.resent_flows = set()
53 1
        if CONSISTENCY_INTERVAL > 0:
54 1
            self.execute_as_loop(CONSISTENCY_INTERVAL)
55
56 1
    def execute(self):
57
        """Run once on NApp 'start' or in a loop.
58
59
        The execute method is called by the run method of KytosNApp class.
60
        Users shouldn't call this method directly.
61
        """
62
        self._load_flows()
63
64
        if CONSISTENCY_INTERVAL > 0:
65
            self.consistency_check()
66
67 1
    def shutdown(self):
68
        """Shutdown routine of the NApp."""
69
        log.debug("flow-manager stopping")
70
71 1
    @listen_to('kytos/of_core.handshake.completed')
72
    def resend_stored_flows(self, event):
73
        """Resend stored Flows."""
74
        # if consistency check is enabled, it should take care of this
75 1
        if CONSISTENCY_INTERVAL >= 0:
76
            return
77 1
        switch = event.content['switch']
78 1
        dpid = str(switch.dpid)
79
        # This can be a problem because this code is running a thread
80 1
        if dpid in self.resent_flows:
81
            log.debug(f'Flow already resent to the switch {dpid}')
82
            return
83 1
        if dpid in self.stored_flows:
84 1
            flow_list = self.stored_flows[dpid]['flow_list']
85 1
            for flow in flow_list:
86 1
                command = flow['command']
87 1
                flows_dict = {"flows": [flow['flow']]}
88 1
                self._install_flows(command, flows_dict, [switch])
89 1
            self.resent_flows.add(dpid)
90 1
            log.info(f'Flows resent to Switch {dpid}')
91
92 1
    def consistency_check(self):
93
        """Check the consistency of flows in each switch."""
94
        switches = self.controller.switches.values()
95
96
        for switch in switches:
97
            # Check if a dpid is a key in 'stored_flows' dictionary
98
            if switch.is_enabled():
99
                self.check_storehouse_consistency(switch)
100
101
                if switch.dpid in self.stored_flows:
102
                    self.check_switch_consistency(switch)
103
104 1
    def check_switch_consistency(self, switch):
105
        """Check consistency of installed flows for a specific switch."""
106 1
        dpid = switch.dpid
107
108
        # Flows stored in storehouse
109 1
        stored_flows = self.stored_flows[dpid]['flow_list']
110
111 1
        serializer = FlowFactory.get_class(switch)
112
113 1
        for stored_flow in stored_flows:
114 1
            command = stored_flow['command']
115 1
            stored_flow_obj = serializer.from_dict(stored_flow['flow'], switch)
116
117 1
            flow = {'flows': [stored_flow['flow']]}
118
119 1
            if stored_flow_obj not in switch.flows:
120 1
                if command == 'add':
121 1
                    log.info('A consistency problem was detected in '
122
                             f'switch {dpid}.')
123 1
                    self._install_flows(command, flow, [switch])
124 1
                    log.info(f'Flow forwarded to switch {dpid} to be '
125
                             'installed.')
126 1
            elif command == 'delete':
127 1
                log.info('A consistency problem was detected in '
128
                         f'switch {dpid}.')
129 1
                command = 'delete_strict'
130 1
                self._install_flows(command, flow, [switch])
131 1
                log.info(f'Flow forwarded to switch {dpid} to be deleted.')
132
133 1
    def check_storehouse_consistency(self, switch):
134
        """Check consistency of installed flows for a specific switch."""
135 1
        dpid = switch.dpid
136
137 1
        for installed_flow in switch.flows:
138 1
            if dpid not in self.stored_flows:
139
                log.info('A consistency problem was detected in '
140
                         f'switch {dpid}.')
141
                flow = {'flows': [installed_flow.as_dict()]}
142
                command = 'delete_strict'
143
                self._install_flows(command, flow, [switch])
144
                log.info(f'Flow forwarded to switch {dpid} to be deleted.')
145
            else:
146 1
                serializer = FlowFactory.get_class(switch)
147 1
                stored_flows = self.stored_flows[dpid]['flow_list']
148 1
                stored_flows_list = [serializer.from_dict(stored_flow['flow'],
149
                                                          switch)
150
                                     for stored_flow in stored_flows]
151
152 1
                if installed_flow not in stored_flows_list:
153 1
                    log.info('A consistency problem was detected in '
154
                             f'switch {dpid}.')
155 1
                    flow = {'flows': [installed_flow.as_dict()]}
156 1
                    command = 'delete_strict'
157 1
                    self._install_flows(command, flow, [switch])
158 1
                    log.info(f'Flow forwarded to switch {dpid} to be deleted.')
159
160
    # pylint: disable=attribute-defined-outside-init
161 1
    def _load_flows(self):
162
        """Load stored flows."""
163 1
        try:
164 1
            data = self.storehouse.get_data()['flow_persistence']
165 1
            if 'id' in data:
166
                del data['id']
167 1
            self.stored_flows = data
168
        except (KeyError, FileNotFoundError) as error:
169
            log.debug(f'There are no flows to load: {error}')
170
        else:
171 1
            log.info('Flows loaded.')
172
173 1
    def _store_changed_flows(self, command, flow, switch):
174
        """Store changed flows.
175
176
        Args:
177
            command: Flow command to be installed
178
            flow: Flows to be stored
179
            switch: Switch target
180
        """
181 1
        stored_flows_box = deepcopy(self.stored_flows)
182
        # if the flow has a destination dpid it can be stored.
183 1
        if not switch:
184
            log.info('The Flow cannot be stored, the destination switch '
185
                     f'have not been specified: {switch}')
186
            return
187 1
        installed_flow = {}
188 1
        flow_list = []
189 1
        installed_flow['command'] = command
190 1
        installed_flow['flow'] = flow
191 1
        deleted_flows = []
192
193 1
        serializer = FlowFactory.get_class(switch)
194 1
        installed_flow_obj = serializer.from_dict(flow, switch)
195
196 1
        if switch.id not in stored_flows_box:
197
            # Switch not stored, add to box.
198 1
            flow_list.append(installed_flow)
199 1
            stored_flows_box[switch.id] = {'flow_list': flow_list}
200
        else:
201 1
            stored_flows = stored_flows_box[switch.id].get('flow_list', [])
202
            # Check if flow already stored
203 1
            for stored_flow in stored_flows:
204 1
                stored_flow_obj = serializer.from_dict(stored_flow['flow'],
205
                                                       switch)
206
207 1
                version = switch.connection.protocol.version
208
209 1
                if installed_flow['command'] == 'delete':
210
                    # No strict match
211 1
                    if match_flow(flow, version, stored_flow['flow']):
212 1
                        deleted_flows.append(stored_flow)
213
214 1
                elif installed_flow_obj == stored_flow_obj:
215 1
                    if stored_flow['command'] == installed_flow['command']:
216
                        log.debug('Data already stored.')
217
                        return
218
                    # Flow with inconsistency in "command" fields : Remove the
219
                    # old instruction. This happens when there is a stored
220
                    # instruction to install the flow, but the new instruction
221
                    # is to remove it. In this case, the old instruction is
222
                    # removed and the new one is stored.
223 1
                    stored_flow['command'] = installed_flow.get('command')
224 1
                    deleted_flows.append(stored_flow)
225 1
                    break
226
227
            # if installed_flow['command'] != 'delete':
228 1
            stored_flows.append(installed_flow)
229 1
            for i in deleted_flows:
230 1
                stored_flows.remove(i)
231 1
            stored_flows_box[switch.id]['flow_list'] = stored_flows
232
233 1
        stored_flows_box['id'] = 'flow_persistence'
234 1
        self.storehouse.save_flow(stored_flows_box)
235 1
        del stored_flows_box['id']
236 1
        self.stored_flows = deepcopy(stored_flows_box)
237
238 1
    @rest('v2/flows')
239 1
    @rest('v2/flows/<dpid>')
240 1
    def list(self, dpid=None):
241
        """Retrieve all flows from a switch identified by dpid.
242
243
        If no dpid is specified, return all flows from all switches.
244
        """
245 1
        if dpid is None:
246 1
            switches = self.controller.switches.values()
247
        else:
248 1
            switches = [self.controller.get_switch_by_dpid(dpid)]
249
250 1
        switch_flows = {}
251
252 1
        for switch in switches:
253 1
            flows_dict = [cast_fields(flow.as_dict())
254
                          for flow in switch.flows]
255 1
            switch_flows[switch.dpid] = {'flows': flows_dict}
256
257 1
        return jsonify(switch_flows)
258
259 1
    @rest('v2/flows', methods=['POST'])
260 1
    @rest('v2/flows/<dpid>', methods=['POST'])
261 1
    def add(self, dpid=None):
262
        """Install new flows in the switch identified by dpid.
263
264
        If no dpid is specified, install flows in all switches.
265
        """
266 1
        return self._send_flow_mods_from_request(dpid, "add")
267
268 1
    @rest('v2/delete', methods=['POST'])
269 1
    @rest('v2/delete/<dpid>', methods=['POST'])
270 1
    @rest('v2/flows', methods=['DELETE'])
271 1
    @rest('v2/flows/<dpid>', methods=['DELETE'])
272 1
    def delete(self, dpid=None):
273
        """Delete existing flows in the switch identified by dpid.
274
275
        If no dpid is specified, delete flows from all switches.
276
        """
277 1
        return self._send_flow_mods_from_request(dpid, "delete")
278
279 1
    def _get_all_switches_enabled(self):
280
        """Get a list of all switches enabled."""
281 1
        switches = self.controller.switches.values()
282 1
        return [switch for switch in switches if switch.is_enabled()]
283
284 1
    def _send_flow_mods_from_request(self, dpid, command, flows_dict=None):
285
        """Install FlowsMods from request."""
286 1
        if flows_dict is None:
287 1
            flows_dict = request.get_json() or {}
288 1
            content_type = request.content_type
289
            # Get flow to check if the request is well-formed
290 1
            flows = flows_dict.get('flows', [])
291
292 1
            if content_type is None:
293 1
                result = 'The request body is empty'
294 1
                raise BadRequest(result)
295
296 1
            if content_type != 'application/json':
297 1
                result = ('The content type must be application/json '
298
                          f'(received {content_type}).')
299 1
                raise UnsupportedMediaType(result)
300
301 1
            if not any(flows_dict) or not any(flows):
302 1
                result = 'The request body is not well-formed.'
303 1
                raise BadRequest(result)
304
305 1
        if dpid:
306 1
            switch = self.controller.get_switch_by_dpid(dpid)
307 1
            if not switch:
308 1
                return jsonify({"response": 'dpid not found.'}), 404
309 1
            elif switch.is_enabled() is False:
310 1
                if command == "delete":
311 1
                    self._install_flows(command, flows_dict, [switch])
312
                else:
313 1
                    return jsonify({"response": 'switch is disabled.'}), 404
314
            else:
315 1
                self._install_flows(command, flows_dict, [switch])
316
        else:
317 1
            self._install_flows(command, flows_dict,
318
                                self._get_all_switches_enabled())
319
320 1
        return jsonify({"response": "FlowMod Messages Sent"})
321
322 1
    def _install_flows(self, command, flows_dict, switches=[]):
323
        """Execute all procedures to install flows in the switches.
324
325
        Args:
326
            command: Flow command to be installed
327
            flows_dict: Dictionary with flows to be installed in the switches.
328
            switches: A list of switches
329
        """
330 1
        for switch in switches:
331 1
            serializer = FlowFactory.get_class(switch)
332 1
            flows = flows_dict.get('flows', [])
333 1
            for flow_dict in flows:
334 1
                flow = serializer.from_dict(flow_dict, switch)
335 1
                if command == "delete":
336
                    flow_mod = flow.as_of_delete_flow_mod()
337 1
                elif command == "delete_strict":
338 1
                    flow_mod = flow.as_of_strict_delete_flow_mod()
339 1
                elif command == "add":
340 1
                    flow_mod = flow.as_of_add_flow_mod()
341
                else:
342
                    raise InvalidCommandError
343 1
                self._send_flow_mod(flow.switch, flow_mod)
344 1
                self._add_flow_mod_sent(flow_mod.header.xid, flow, command)
345
346 1
                self._send_napp_event(switch, flow, command)
347 1
                self._store_changed_flows(command, flow_dict, switch)
348
349 1
    def _add_flow_mod_sent(self, xid, flow, command):
350
        """Add the flow mod to the list of flow mods sent."""
351 1
        if len(self._flow_mods_sent) >= self._flow_mods_sent_max_size:
352
            self._flow_mods_sent.popitem(last=False)
353 1
        self._flow_mods_sent[xid] = (flow, command)
354
355 1
    def _send_flow_mod(self, switch, flow_mod):
356 1
        event_name = 'kytos/flow_manager.messages.out.ofpt_flow_mod'
357
358 1
        content = {'destination': switch.connection,
359
                   'message': flow_mod}
360
361 1
        event = KytosEvent(name=event_name, content=content)
362 1
        self.controller.buffers.msg_out.put(event)
363
364 1
    def _send_napp_event(self, switch, flow, command, **kwargs):
365
        """Send an Event to other apps informing about a FlowMod."""
366 1
        if command == 'add':
367 1
            name = 'kytos/flow_manager.flow.added'
368 1
        elif command in ('delete', 'delete_strict'):
369 1
            name = 'kytos/flow_manager.flow.removed'
370 1
        elif command == 'error':
371 1
            name = 'kytos/flow_manager.flow.error'
372
        else:
373
            raise InvalidCommandError
374 1
        content = {'datapath': switch,
375
                   'flow': flow}
376 1
        content.update(kwargs)
377 1
        event_app = KytosEvent(name, content)
378 1
        self.controller.buffers.app.put(event_app)
379
380 1
    @listen_to('.*.of_core.*.ofpt_error')
381
    def handle_errors(self, event):
382
        """Receive OpenFlow error and send a event.
383
384
        The event is sent only if the error is related to a request made
385
        by flow_manager.
386
        """
387 1
        message = event.content["message"]
388
389 1
        connection = event.source
390 1
        switch = connection.switch
391
392 1
        xid = message.header.xid.value
393 1
        error_type = message.error_type
394 1
        error_code = message.code
395 1
        error_data = message.data.pack()
396
397
        # Get the packet responsible for the error
398 1
        error_packet = connection.protocol.unpack(error_data)
399
400 1
        if message.code == BadActionCode.OFPBAC_BAD_OUT_PORT:
401
            actions = []
402
            if hasattr(error_packet, 'actions'):
403
                # Get actions from the flow mod (OF 1.0)
404
                actions = error_packet.actions
405
            else:
406
                # Get actions from the list of flow mod instructions (OF 1.3)
407
                for instruction in error_packet.instructions:
408
                    actions.extend(instruction.actions)
409
410
            for action in actions:
411
                iface = switch.get_interface_by_port_no(action.port)
412
413
                # Set interface to drop packets forwarded to it
414
                if iface:
415
                    iface.config = PortConfig.OFPPC_NO_FWD
416
417 1
        try:
418 1
            flow, error_command = self._flow_mods_sent[xid]
419
        except KeyError:
420
            pass
421
        else:
422 1
            self._send_napp_event(flow.switch, flow, 'error',
423
                                  error_command=error_command,
424
                                  error_type=error_type, error_code=error_code)
425