inspec / train

Transport Interface to unify communication over SSH, WinRM, and friends.
Apache License 2.0
119 stars 87 forks source link

Prevent Cisco devices being detected as Arista #736

Closed trickyearlobe closed 1 year ago

trickyearlobe commented 2 years ago

Signed-off-by: Richard Nixon richard.nixon@btinternet

Description

When running inspec detect -t ssh://nexus001 against a virtual Nexus 9300 (NXOS) the platform was being mis-detected as Arista EOS with no version number.

After applying this bugfix, virtual Nexus and virtual Arista devices are correctly detected.

Type of change

Checklist:

chef-expeditor[bot] commented 2 years ago

Hello trickyearlobe! Thanks for the pull request!

Here is what will happen next:

  1. Your PR will be reviewed by the maintainers.
  2. Possible Outcomes a. If everything looks good, one of them will approve it, and your PR will be merged. b. The maintainer may request follow-on work (e.g. code fix, linting, etc). We would encourage you to address this work in 2-3 business days to keep the conversation going and to get your contribution in sooner. c. Cases exist where a PR is neither aligned to Chef InSpec's product roadmap, or something the team can own or maintain long-term. In these cases, the maintainer will provide justification and close out the PR.

Thank you for contributing!