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

supporting redfish #12819

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pettershao-ragilenetworks
Copy link
Contributor

@pettershao-ragilenetworks pettershao-ragilenetworks commented Nov 24, 2022

Why I did it

Redfish support communication between SONiC and BMC, It is a simple & safe & modern & high scalability & datasheet readable industry standard specification.

How I did it

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205

Description for the changelog

Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU.

Link to config_db schema for YANG module changes

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

@pettershao-ragilenetworks
Copy link
Contributor Author

/easycla

@Blueve
Copy link
Contributor

Blueve commented Nov 29, 2022

Do we have any design specification regarding this feature? Why is it not containerized?
How the redfish server enabled and configured? Is there a feature flag or build parameter to disable it?

@lguohan
Copy link
Collaborator

lguohan commented Dec 17, 2022

adding redfish support is a new feature, please go through standard sonic community design review process.

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.

3 participants