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

[RFE] Add Network attributes modification #385

Open
grafuls opened this issue Oct 19, 2023 · 1 comment
Open

[RFE] Add Network attributes modification #385

grafuls opened this issue Oct 19, 2023 · 1 comment
Assignees
Labels
BIOS enhancement New feature or request

Comments

@grafuls
Copy link
Collaborator

grafuls commented Oct 19, 2023

Is your feature request related to a problem? Please describe.
Since idrac9 (version 5.10) OEM network attributes are available for modification via the redfish API. This would allow us to set SRIOV attribute at a per NIC level. Develop this new functionality into BF.

What System / IPMI Platform?
idrac 9 (version 5.10)

Describe the Possible Solution
Add a new argument to badfish in the form of --get-nic-attribute / --set-nic-attribute in a similar fashion to the already existing --get-bios-attribute / --set-bios-attribute.

Additional Info
Reference: dell/iDRAC-Redfish-Scripting#50 (comment)

@dominikvagner
Copy link
Contributor

dominikvagner commented Dec 14, 2023

Copy of description from PR #392, which tries to solve this issue:

After extensive testing on my part and a joint investigation with @grafuls, this feature cannot be fully implemented in the current state of our lab, because of our iDRAC setup/versions.
Getting a list of attributes for specific NICs is working as it should along with getting information about a specific attribute. (The same way as we do it for BIOS attributes.)
Setting attributes though does not work, even if the somewhat incomplete feeling iDRAC documentation about this use case says it should for versions 5.x.x.x (testing was done against 5.10.50.0 as the newest version available to us).
Neither my attempt at solving this or the reference script from Dell works.
In our case the end-point accepting the setting of attributes shows no supported attributes to set.

The getting functionality was implemented and tested. The setting of attributes was made as a draft implementation supposed to completed in the future, when the environment will allow for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BIOS enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants