Skip to content

Add N77-SUP2E #1124

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

Closed

Conversation

ascension2020
Copy link
Contributor

No description provided.

@ascension2020 ascension2020 marked this pull request as ready for review February 9, 2023 20:42
@danner26
Copy link
Member

Please add the datasheet as a comment, I tried to myself but you seem to have the "Allow edits from maintainers" disabled

@danner26 danner26 added the status: revisions needed This issue requires additional information to be actionable label Feb 10, 2023
@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, you will be required to update your branch/PR from the new master branch. 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.

model: N77-SUP2E
part_number: N77-SUP2E
interfaces:
- name: mgmt0
Copy link
Member

Choose a reason for hiding this comment

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

This will cause a problem when inserting it into a chassis where there is two mgmt0 interfaces. I believe it actually throws an exception.

Choose a reason for hiding this comment

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

I added the following to my documents to address this issue.
interfaces:

  • name: Sup{module} mgmt0
    type: 1000base-t
    mgmt_only: true
    console-ports:
  • name: Sup{module} Con0
    type: rj-45

Copy link
Member

Choose a reason for hiding this comment

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

@scoffman77 The change needs to be made on the module, not the upstream

@github-actions
Copy link
Contributor

github-actions bot commented May 2, 2023

This PR has been automatically marked as stale because it has not had recent activity. It will be closed automatically if no further progress is made.

@github-actions github-actions bot added the stale label May 2, 2023
@github-actions
Copy link
Contributor

github-actions bot commented May 9, 2023

This PR has been automatically closed due to lack of activity.

@github-actions github-actions bot closed this May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale status: revisions needed This issue requires additional information to be actionable
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants