Skip to content

Conversation

@muskaankapahi
Copy link

@muskaankapahi muskaankapahi commented Sep 15, 2023

add code to fix the issue for 1G SFP-GE-BX40 and BX10 (BiDi) on S9510-28DC-003/001: Shown as Unknown

The code for 1G-SFP-GE-BX40 and BX10 was not added as a part of the repo, so it was functioning but it wasn't being recorded in the onlpdump

@sonoble @jnealtowns can you please review my PR?

Signed-off-by: Muskaan Kapahi <[email protected]>
…-28DC-003/001: Shown as Unknown

Signed-off-by: Muskaan Kapahi <[email protected]>
@muskaankapahi
Copy link
Author

@sonoble @jnealtowns can you please have a look at my PR?

@muskaankapahi
Copy link
Author

muskaankapahi commented Sep 21, 2023

@jnealtowns @sonoble
I apologize for not clarifying this before, I am an Arrcus employee and have worked on a ticket to include the code for 1G-SFP-GE-BX40 and BX10 which was not added as a part of the repo. As a result it was functioning but it wasn't being recorded in the onlpdump
I have raised this PR from my personal github account

@sonoble
Copy link

sonoble commented Sep 21, 2023

For ONL we now use https://github.com/opennetlinux/bigcode if you can put the commit there I can merge it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants