Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add cisco apic device #3428

Closed
wants to merge 4 commits into from
Closed

Conversation

yone2ks
Copy link
Contributor

@yone2ks yone2ks commented Apr 30, 2024

I added support for Cisco APIC. Tested on APIC version 6.0.
Since Cisco APIC is based on Linux, this new class inherits from the LinuxSSH class.
Within the class, the only implementation is a pass statement.

@yone2ks
Copy link
Contributor Author

yone2ks commented May 3, 2024

@ktbyers
It looks like the checks have failed. Is there anything I should do to fix this problem?
From what i tell, the failure doesn't seem to be related to the code changes I made.
I would appreciate any help you can give me in resolving the issue.

@ktbyers
Copy link
Owner

ktbyers commented May 16, 2024

@yone2ks Tests look like they are passing now...I still need to review the code, though.

@ktbyers
Copy link
Owner

ktbyers commented Jul 10, 2024

Superseded by #3461

@ktbyers ktbyers closed this Jul 10, 2024
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