-
Notifications
You must be signed in to change notification settings - Fork 297
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
DAOS-16352 control: Handle cases with static ifaces #14953
Conversation
Fabric interfaces defined statically in the daos_agent config file are fundamentally different from those detected via hardware scanning. They don't include information derived from the hardware such as their true device class or fabric provider(s). This patch adds some rigor to what is ignored regarding these manually-defined interfaces. - Ignore provider for statically-defined fabric interfaces, as we do not bother detecting it. They are assumed to be compatible with whatever provider the agent is using. - Silence confusing "no interfaces requested" error from WaitFabricReady by not calling it if there are no interfaces to check. - Remove some defunct logic related to detecting the provider in the multi-provider case. The agent may only use a single provider. Features: control Required-githooks: true Signed-off-by: Kris Jacque <kris.jacque@intel.com>
Ticket title is 'daos_agent crash with static fabric config' |
Test stage Functional Hardware Large completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-14953/1/execution/node/1594/log |
Test failure is a known issue in master: https://daosio.atlassian.net/issues/DAOS-16334 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for what I understand.
Fabric interfaces defined statically in the daos_agent config file are fundamentally different from those detected via hardware scanning. They don't include information derived from the hardware such as their true device class or fabric provider(s). This patch adds some rigor to what is ignored regarding these manually-defined interfaces. - Ignore provider for statically-defined fabric interfaces, as we do not bother detecting it. They are assumed to be compatible with whatever provider the agent is using. - Silence confusing "no interfaces requested" error from WaitFabricReady by not calling it if there are no interfaces to check. - Remove some defunct logic related to detecting the provider in the multi-provider case. The agent may only use a single provider. Features: control Required-githooks: true Signed-off-by: Kris Jacque <kris.jacque@intel.com>
Fabric interfaces defined statically in the daos_agent config file are fundamentally different from those detected via hardware scanning. They don't include information derived from the hardware such as their true device class or fabric provider(s). This patch adds some rigor to what is ignored regarding these manually-defined interfaces. - Ignore provider for statically-defined fabric interfaces, as we do not bother detecting it. They are assumed to be compatible with whatever provider the agent is using. - Silence confusing "no interfaces requested" error from WaitFabricReady by not calling it if there are no interfaces to check. - Remove some defunct logic related to detecting the provider in the multi-provider case. The agent may only use a single provider. Signed-off-by: Kris Jacque <kris.jacque@intel.com>
Fabric interfaces defined statically in the daos_agent config file are fundamentally different from those detected via hardware scanning. They don't include information derived from the hardware such as their true device class or fabric provider(s).
This patch adds some rigor to what is ignored regarding these manually-defined interfaces.
Features: control
Required-githooks: true
Before requesting gatekeeper:
Features:
(orTest-tag*
) commit pragma was used or there is a reason documented that there are no appropriate tags for this PR.Gatekeeper: