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

Unifying the platform api for get_pcie_aer_stats with PcieBase #197

Merged
merged 2 commits into from
Jun 17, 2021

Conversation

sujinmkang
Copy link
Contributor

Unifying the platform api for get_pcie_aer_stats with PcieBase

Description

Motivation and Context

How Has This Been Tested?

Additional Information (Optional)

jleveque
jleveque previously approved these changes Jun 14, 2021
@jleveque
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sujinmkang
Copy link
Contributor Author

@jleveque Can you approve this PR now? The PR build passed now.

@sujinmkang sujinmkang merged commit 93641f3 into sonic-net:master Jun 17, 2021
andywongarista pushed a commit to andywongarista/sonic-platform-common that referenced this pull request Jun 18, 2021
…-net#197)

Unifying the platform api for get_pcie_aer_stats with PcieBase
qiluo-msft pushed a commit that referenced this pull request Jun 21, 2021
Unifying the platform api for get_pcie_aer_stats with PcieBase
qiluo-msft added a commit to qiluo-msft/sonic-platform-common that referenced this pull request Jul 19, 2021
qiluo-msft added a commit that referenced this pull request Jul 19, 2021
#197)" (#207)

This reverts commit 238d76b.
Due to build broken on sonic-platform-daemons
lguohan pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Jul 22, 2021
To include:
> e168f1d 2021-07-19 pettershao-ragilenetworks: [python coverage] fix result color bar (sonic-net/sonic-platform-common#202)
> 87c81de 2021-07-13 Prince George: Fix Xcvrd crash due to invalid key access in type_of_media_interface, host_electrical_interface, connector_dict (sonic-net/sonic-platform-common#206)
> 4533f82 2021-06-21 ngoc-do: Add a template function that returns list of asics on module (sonic-net/sonic-platform-common#185)
> 1e860c5 2021-06-18 Aravind Mani: Fix decode error when parsing EEPROM fields (sonic-net/sonic-platform-common#199)
> 93641f3 2021-06-17 Sujin Kang: Unifying the platform api for get_pcie_aer_stats with PcieBase (sonic-net/sonic-platform-common#197)
liat-grozovik added a commit that referenced this pull request Jul 26, 2021
sujinmkang pushed a commit that referenced this pull request Jul 26, 2021
qiluo-msft pushed a commit to sonic-net/sonic-buildimage that referenced this pull request Jul 27, 2021
…ns (#8262)

sonic-platform-daemons:
* c90bb29 [202012] Refactor Pcied and add unittest (sonic-net/sonic-platform-daemons#199)

sonic-platform-common:
* 9a59e19 Revert "Unifying the platform api for get_pcie_aer_stats with PcieBase (sonic-net/sonic-platform-common#197)" (sonic-net/sonic-platform-common#207)
* d35960b Revert "Revert "Unifying the platform api for get_pcie_aer_stats with PcieBase (sonic-net/sonic-platform-common#197)" (sonic-net/sonic-platform-common#207)" (sonic-net/sonic-platform-common#210)
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
To include:
> e168f1d 2021-07-19 pettershao-ragilenetworks: [python coverage] fix result color bar (sonic-net/sonic-platform-common#202)
> 87c81de 2021-07-13 Prince George: Fix Xcvrd crash due to invalid key access in type_of_media_interface, host_electrical_interface, connector_dict (sonic-net/sonic-platform-common#206)
> 4533f82 2021-06-21 ngoc-do: Add a template function that returns list of asics on module (sonic-net/sonic-platform-common#185)
> 1e860c5 2021-06-18 Aravind Mani: Fix decode error when parsing EEPROM fields (sonic-net/sonic-platform-common#199)
> 93641f3 2021-06-17 Sujin Kang: Unifying the platform api for get_pcie_aer_stats with PcieBase (sonic-net/sonic-platform-common#197)
judyjoseph pushed a commit that referenced this pull request Aug 20, 2021
Unifying the platform api for get_pcie_aer_stats with PcieBase
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-platform-common that referenced this pull request Oct 25, 2024
…for calling Y-Cable API's inside xcvrd (sonic-net#197)

Signed-off-by: vaibhav-dahiya [email protected]

Description
This PR integrates vendor specific class objects inside xcvrd for Y-Cable API's to be called.
Detailed designed document can be found sonic-net/SONiC#757

Motivation and Context
Basically xcvrd now has an interface for Y-Cable to interact with PMON docker and HOST
which can be uniform across all vendors. As part of this refactor, we will be moving towards a model where only xcvrd interacts with the cables/transceivers, and host-side processes will communicate with xcvrd rather than with the devices directly with Y-Cable.

How Has This Been Tested?
Ran the changes on Arista7050cx3 switch, making changes inside the container.

Signed-off-by: vaibhav-dahiya <[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.

4 participants