CMLDEV-12: Add posibility to switch serial console used by PyAts conn… #177
Conversation
virl2_client/models/cl_pyats.py
Outdated
| except KeyError: | ||
| raise PyatsDeviceNotFound(node_label) | ||
|
|
||
| node: Node = [node for node in self._lab.nodes()if node.label == node_label].pop() |
There was a problem hiding this comment.
| node: Node = [node for node in self._lab.nodes()if node.label == node_label].pop() | |
| node = self._lab.get_node_by_label(node_label) |
| node: Node = [node for node in self._lab.nodes()if node.label == node_label].pop() | ||
|
|
||
| # will raise API error inside if console does not exist for device | ||
| node.console_key(console_number) |
There was a problem hiding this comment.
you don't want to store and use the key? what's the reason of calling it then? I'd let it fail later
There was a problem hiding this comment.
yes, let it fail on connect if the console cannot get actually connected
virl2_client/models/cl_pyats.py
Outdated
| pattern = rf"({re.escape(node_label)})/\d+" | ||
| new_console_cfg = f"/{console_number}" | ||
|
|
||
| new_connect_command = re.sub(pattern, rf"\1{new_console_cfg}", old_connect_command) |
There was a problem hiding this comment.
wouldn't this work?
| new_connect_command = re.sub(pattern, rf"\1{new_console_cfg}", old_connect_command) | |
| new_connect_command = re.sub(pattern, rf"\1/{console_number}", old_connect_command) |
There was a problem hiding this comment.
I believe that the original console number is always 0, so a simpler replace command should work as well, e.g.,
pyats_device.connections["a"]["command"] = pyats_device.connections["a"]["command"][:-1] + console_number
tmikuska
force-pushed
the
CMLDEV-12-Add-possibility-to-enable-pyats-on-different-console
branch
from
ab65bf3 to
e00ddee
Compare
|
I rebased this from dev and changed the target branch to the same. |
| node: Node = [node for node in self._lab.nodes()if node.label == node_label].pop() | ||
|
|
||
| # will raise API error inside if console does not exist for device | ||
| node.console_key(console_number) |
There was a problem hiding this comment.
yes, let it fail on connect if the console cannot get actually connected
| new_console_cfg = f"/{console_number}" | ||
|
|
||
| new_connect_command = re.sub( | ||
| pattern, rf"\1{new_console_cfg}", old_connect_command |
There was a problem hiding this comment.
you should rather check that the console number is 0-3, and only change the last character. No need to use regex here
| """ | ||
| Switch to different serial console that is used to execute PyAts commands | ||
| should be executed after sync_testbed | ||
| and re-executed after every sync_testbed call. |
There was a problem hiding this comment.
speaking of - if you use console 0, then switch to console 1 using this method, will the next command perhaps reuse the connection it already has established with console 0? Maybe coordinate with the other ticket about closing an existing connection if the connection number does not match.
…ections