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

[202012][DellEMC] Z9332f PSU data is not updated in state DB #7543

Merged
merged 1 commit into from
May 6, 2021

Conversation

aravindmani-1
Copy link
Contributor

@aravindmani-1 aravindmani-1 commented May 6, 2021

Why I did it

  • PSU data is loaded into state DB.
    Following errors are seen in syslogs:
    "Failed to update PSU data - '<=' not supported between instances of 'float' and 'str'"

  • Issue is not seen in master image as the PSU API return type is different.

How I did it

  • Changed the return type in PSU API's.

How to verify it

  • Check state DB, syslogs.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

psu_UT.txt

A picture of a cute animal (not mandatory but encouraged)

@aravindmani-1
Copy link
Contributor Author

@jleveque can you please review this PR?.

@jleveque jleveque changed the title DellEMC: <202012 branch> Z9332f PSU data is not updated in state DB. [202012][DellEMC] Z9332f PSU data is not updated in state DB May 6, 2021
@jleveque jleveque merged commit 11aa05b into sonic-net:202012 May 6, 2021
@aravindmani-1 aravindmani-1 deleted the Fix_psu_issue branch May 7, 2021 03:01
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