Skip to content

Conversation

@getspaghett
Copy link
Contributor

Created definitions for Intel H2312XXLR2 blade system.
Specifications taken from Intel.com

@getspaghett getspaghett marked this pull request as ready for review March 6, 2023 16:59
@getspaghett getspaghett closed this Mar 6, 2023
@getspaghett getspaghett reopened this Mar 6, 2023
@getspaghett
Copy link
Contributor Author

Sorry, i think i have changed this from a draft to a pull request too soon. :(

@getspaghett getspaghett marked this pull request as draft March 6, 2023 17:46
@getspaghett
Copy link
Contributor Author

found out how to change it back to draft

@getspaghett getspaghett changed the title Intel h2312xxlr2 Added Intel H2312XXLR2 blade system Mar 6, 2023
@getspaghett
Copy link
Contributor Author

i am sorry but i do not have enough git skills yet to remove my .gitignore from this branch ...

added new line at end of file
@getspaghett
Copy link
Contributor Author

ok i think i figured it out, the gitignore is no longer in this branch ... sorry for the messy commits

@getspaghett getspaghett marked this pull request as ready for review March 8, 2023 09:16
@danner26
Copy link
Member

danner26 commented Mar 8, 2023

Its great seeing you move through the issues and overcome them :) I will work on reviewing this soon. Thanks in advance for your contribution!

@danner26 danner26 added the status: revisions needed This issue requires additional information to be actionable label Mar 8, 2023
@danner26
Copy link
Member

danner26 commented Mar 9, 2023

Do the PSU modules already exist? If so then just the one last correction, otherwise please add the applicable PSU modules to this PR :)

@getspaghett getspaghett requested a review from danner26 March 9, 2023 15:40
@danner26
Copy link
Member

danner26 commented Mar 9, 2023

Hello, unfortunately today we had to make a change to the schema in which weight is defined by. The upstream NetBox API requires weight to be passed to it in the following format:

weight: 12.01
weight_unit: lb

Due to this change, I have updated your branch against the current master. This might require you to update your PR, if you had weight currently defined, so that your PR is in-line with the new weight schema. If you have any issues, please let me know or refer to this PR for more information.

@danner26
Copy link
Member

danner26 commented Mar 9, 2023

My bad, I forgot to tell you to put '' around the 1 and 2. I fixed it for you though

danner26 added 2 commits March 9, 2023 11:36
Fix power-ports reference
fix power-ports definition
@danner26 danner26 added status: accepted and removed status: revisions needed This issue requires additional information to be actionable labels Mar 9, 2023
@danner26 danner26 merged commit 38bd217 into netbox-community:master Mar 9, 2023
@getspaghett getspaghett deleted the Intel-H2312XXLR2 branch March 9, 2023 16:44
@getspaghett
Copy link
Contributor Author

Thank you for the fixes :)

etherwrangler pushed a commit to etherwrangler/devicetype-library that referenced this pull request Jul 16, 2023
* ignore my custom venv dir

* Created definition for Intel H2312XXLR2

* changed power-ports type to iec-60320-c22

* Delete .gitignore

deleted my changed .gitignore

* restored original .gitignore

* Update .gitignore

added new line at end of file

* changed power ports type to iec-60320-c14

* added weight, airflow and changed power-ports to module bays

* removed comment

* created module-types for intel 1600w and 2130w redundant power supplies

* changed weight to fit new schema and changed position on module-bays

* fixed PSU module name to reflect position in device

* Update H2312XXLR2.yaml

Fixed position reference

* Update FXX1600PCRPS.yaml

Fix power-ports reference

* Update FXX2130PCRPS.yaml

fix power-ports definition

---------

Co-authored-by: Daniel W. Anner <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants