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 Edgecore device types for 4630-54TE-O-AC-B, 5835-54X-O-AC-B, 7326-56X-O-AC-B #2276

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

Conversation

matofeder
Copy link

No description provided.

part_number: F0PZZ4654xxxx
u_height: 1.0
is_full_depth: true
comments: https://www.edge-core.com/_upload/images/2023-026-EPS201_AS4630-54TE_DS_R04_20230308.pdf
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hello, thank you for your contribution! In an attempt to have the most accurate and complete data we can, I have a few additional data points we would like you to add if possible.

  1. Please add the airflow data point if you have it or can find it. Often these are listed in the datasheet, but sometimes they are not
  2. Please add the weight and associated weight_unit data point if you have it or can find it. Often these are listed in the datasheet, but sometimes they are not

If you have any questions, please let us know :)

part_number: FP1ZZ5654xxxx
u_height: 1.0
is_full_depth: true
comments: https://www.edge-core.com/_upload/images/DCS201_AS5835-54X_DS_R06_20230717.pdf
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hello, thank you for your contribution! In an attempt to have the most accurate and complete data we can, I have a few additional data points we would like you to add if possible.

  1. Please add the airflow data point if you have it or can find it. Often these are listed in the datasheet, but sometimes they are not
  2. Please add the weight and associated weight_unit data point if you have it or can find it. Often these are listed in the datasheet, but sometimes they are not

If you have any questions, please let us know :)

part_number: FP4ZZ7656xxxx
u_height: 1.0
is_full_depth: true
comments: https://www.edge-core.com/_upload/images/DCS203_AS7326-56X_DS_R09_20230717.pdf
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hello, thank you for your contribution! In an attempt to have the most accurate and complete data we can, I have a few additional data points we would like you to add if possible.

  1. Please add the airflow data point if you have it or can find it. Often these are listed in the datasheet, but sometimes they are not
  2. Please add the weight and associated weight_unit data point if you have it or can find it. Often these are listed in the datasheet, but sometimes they are not

If you have any questions, please let us know :)

@harryajc harryajc added the status: revisions needed This issue requires additional information to be actionable label Oct 19, 2024
@danner26 danner26 marked this pull request as draft October 29, 2024 00:06
berendt added a commit to osism/netbox-manager that referenced this pull request Dec 4, 2024
@berendt
Copy link

berendt commented Dec 4, 2024

@matofeder Can you please add the missing information? Would be nice to have these 3 types in the device type library.

berendt added a commit to osism/netbox-manager that referenced this pull request Dec 4, 2024
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