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 ThinkSystem SR635 #2856

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Add ThinkSystem SR635 #2856

wants to merge 1 commit into from

Conversation

psuet
Copy link
Contributor

@psuet psuet commented Feb 21, 2025

No description provided.

type: 1000base-t
mgmt_only: true
power-ports:
- name: PSU0
Copy link
Collaborator

Choose a reason for hiding this comment

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

As these are Modular PSUs they need to be modeled as module-bays and then add the psu's in the modules folder if they don't already exist.

slug: lenovo-thinksystem-sr635
part_number: 7Y98
u_height: 1
is_full_depth: true
Copy link
Collaborator

Choose a reason for hiding this comment

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

To make sure we have the most complete information, can you add the following parameters, airflow weight and weight_unit with the required data

part_number: 7Y98
u_height: 1
is_full_depth: true
subdevice_role: parent
Copy link
Collaborator

Choose a reason for hiding this comment

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

Subdevice_role: parent is not required as this has no 'device-bays'

@harryajc harryajc added the status: revisions needed This issue requires additional information to be actionable label Feb 21, 2025
@harryajc harryajc marked this pull request as draft February 22, 2025 12:36
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.

2 participants