)]}'
{"networking_generic_switch/devices/netmiko_devices/dell.py":[{"author":{"_account_id":29100,"name":"Jason Anderson","email":"jasonanderson@uchicago.edu","username":"jasonanderson"},"change_message_id":"c7898deff048a011b17e038af7128f7135e385fc","unresolved":false,"context_lines":[{"line_number":19,"context_line":"        ADD_NETWORK \u003d ("},{"line_number":20,"context_line":"            \u0027interface vlan {segmentation_id}\u0027,"},{"line_number":21,"context_line":"            \u0027name {network_id}\u0027,"},{"line_number":22,"context_line":"            \u0027exit\u0027,"},{"line_number":23,"context_line":"        )"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"        DELETE_NETWORK \u003d ("}],"source_content_type":"text/x-python","patch_set":2,"id":"5fc1f717_96747200","line":22,"updated":"2019-03-19 05:14:16.000000000","message":"I know I am asking very long after this was merged, but do you remember the reason for adding the \u0027exit\u0027 commands here? I was pulling in upstream patches for a fork of this plugin we use in production and this causes problems like the following:\n\n  GenericSwitchNetmikoConnectError: Netmiko connection error: {...}, error: Socket is closed\n\nRemoving the \u0027exit\u0027 commands at the end of each operation fixes this. I noticed that the other device files don\u0027t have this.","commit_id":"d8d58a81565116a8fae796d55ab5ee6fe90d83c7"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"559c202c6889ddcc266c3a515cc0df4d933458e3","unresolved":false,"context_lines":[{"line_number":19,"context_line":"        ADD_NETWORK \u003d ("},{"line_number":20,"context_line":"            \u0027interface vlan {segmentation_id}\u0027,"},{"line_number":21,"context_line":"            \u0027name {network_id}\u0027,"},{"line_number":22,"context_line":"            \u0027exit\u0027,"},{"line_number":23,"context_line":"        )"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"        DELETE_NETWORK \u003d ("}],"source_content_type":"text/x-python","patch_set":2,"id":"5fc1f717_5c77090a","line":22,"in_reply_to":"5fc1f717_96747200","updated":"2019-03-19 08:33:56.000000000","message":"Hi Jason, the reason for adding exit is so that we can jump up to the top level of the configuration, in order to add subsequent commands. For networks with trunk ports we need to create the network and plug one or more ports in.","commit_id":"d8d58a81565116a8fae796d55ab5ee6fe90d83c7"}]}
