Skip to content

create S5732-H24S6Q.yaml #2970

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

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Conversation

trnki
Copy link
Contributor

@trnki trnki commented Apr 1, 2025

added new huawei device model

trnki added 4 commits April 1, 2025 10:36
device model for Huawei switch S5732-H24S6Q
airflow modified
modified part number
type: usb-a
- name: console0
type: rj-45
power-ports:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hi Thank you for contribution,
Looking at the spec sheet these are hot swappable PSUs therefore they should be modelled as module-bays within the device template with the required PSUs added as modules

@harryajc harryajc added the status: revisions needed This issue requires additional information to be actionable label Apr 3, 2025
@danner26 danner26 requested a review from Copilot April 6, 2025 00:28
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

device-types/Huawei/S5732-H24S6Q.yaml:86

  • [nitpick] The interface name 'MEth0/0/1' is inconsistent with the naming conventions used for other interfaces (e.g. GigabitEthernet, XGigabitEthernet, 40GE0/0/1) and may be a typo. Consider verifying the intended name and potentially renaming it to match the standard conventions.
- name: MEth0/0/1

@danner26 danner26 marked this pull request as draft April 6, 2025 00:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: revisions needed This issue requires additional information to be actionable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants