Skip to content

Conversation

pizu
Copy link

@pizu pizu commented Sep 7, 2025

Support of Cisco Next-Gen Switches Driver.

In my case for C1200-16T-2G - more can be added here. - any 1200 should be supported

https://www.cisco.com/c/en/us/products/collateral/switches/catalyst-1200-series-switches/nb-06-cat1200-ser-data-sheet-cte-en.html

This is related to #361

Support of Cisco Next-Gen Switches Driver.

In my case for C1200-16T-2G - more can be added here. - any 1200 should be supported

https://www.cisco.com/c/en/us/products/collateral/switches/catalyst-1200-series-switches/nb-06-cat1200-ser-data-sheet-cte-en.html
@pizu pizu changed the title Bew Cisco Next-Gen Switches Driver New Cisco Next-Gen Switches Driver Sep 7, 2025
Copy link
Contributor

@SCadilhac SCadilhac left a comment

Choose a reason for hiding this comment

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

Thanks for contributing!
Here are my comments.

@pizu
Copy link
Author

pizu commented Sep 9, 2025

will work on these changes and will get back to you. thanks :-)

@pizu
Copy link
Author

pizu commented Sep 12, 2025

Hi,

can you check?

Thanks.

@pizu pizu requested a review from SCadilhac September 12, 2025 13:12
Copy link
Contributor

@SCadilhac SCadilhac left a comment

Choose a reason for hiding this comment

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

Thanks for all the changes.
I still think we need a different name for this driver.

Copy link
Contributor

@SCadilhac SCadilhac left a comment

Choose a reason for hiding this comment

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

Was this driver successfully tested on real devices? Based on CLI object code, I'm not sure how this can work.

/* ========================= DRIVER META ========================= */

const Info = {
name: "CiscoNextGenSeries",
Copy link
Contributor

Choose a reason for hiding this comment

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

Cf. my other comment, we need a more specific name for the driver.

Copy link
Author

Choose a reason for hiding this comment

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

no problem, true all new switches are next generation hehe.

We can name it Cisco_Catalyst1200-1300.js - for me as long as the driver is supported the name doesn't matter :D

const Info = {
name: "CiscoNextGenSeries",
description: "Cisco Catalyst Next-Gen Series",
author: "DMica(Pizu)",
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add a space


function snapshot(cli, device, config) {
// Land in enable mode first.
try { cli.macro("enable"); } catch (e) {}
Copy link
Contributor

Choose a reason for hiding this comment

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

An error from cli.macro should be propagated (no catch) to snapshot task it fails there before attempting to issue commands in an unknown context.

username: {
// Matches "User Name:", "Username:", "login:"
prompt: /(User\s*Name|Username|login):\s*$/mi,
response: "%username%"
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this really work? Looks like IA-generated code :-) response is in pager context. There should be macros in each mode to transition to the target one.

Copy link
Author

Choose a reason for hiding this comment

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

Hi,

Yes had big parts generated with AI. We needed this driver in order to backup these new devices.

Thanks.

@pizu
Copy link
Author

pizu commented Sep 15, 2025

Was this driver successfully tested on real devices? Based on CLI object code, I'm not sure how this can work.

Yes, working.

so far no issues. Currently am backing up 6 devices with this driver.

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