Skip to content

Add Wave Terminal console launcher for Cisco CML - #82

Open
hendapaim wants to merge 1 commit into
CiscoDevNet:masterfrom
hendapaim:feature/cml-breakout-wave-terminal
Open

Add Wave Terminal console launcher for Cisco CML#82
hendapaim wants to merge 1 commit into
CiscoDevNet:masterfrom
hendapaim:feature/cml-breakout-wave-terminal

Conversation

@hendapaim

Copy link
Copy Markdown

Summary

This pull request adds support for launching Cisco CML serial consoles directly in Wave Terminal on Linux.

Features

  • Automatic console discovery
  • One Wave block per console
  • Simple installation
  • Documentation
  • Example configuration

Motivation

This provides a modern alternative to the previous SecureCRT workflow for accessing CML breakout consoles.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file is generated and should not be committed.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is clearly AI-generated, but whatever prompt was used didn't prioritize user experience and getting started quickly. I suggest refactoring this to offer a quick start. More specific comments in line.


> **Repository:** `cml-breakout-wave-terminal`

Repository Base: https://github.com/hendapaim/cml-breakout-wave-terminal

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't needed, but perhaps a link to the Wave terminal would be useful.

2. `nc`
3. `ncat`

Install `netcat-openbsd` if none are already installed.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does one do this?

To temporarily use another console proxy:

```bash
CONSOLE_HOST=127.0.0.1 ./wave-cml2

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is meant by a console host? Is this another instance of the breakout tool? The breakout tool always binds locally. So why would one need another console host?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file should not have been committed.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file should not have been committed.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file should not have been committed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants