Open
Conversation
- Add secondary providers to internal DAOS gRPC/dRPC messages. - Add secondary providers and URIs to member database. - Parse multiple providers and their corresponding interfaces/ports from the yaml config file. Signed-off-by: Kris Jacque <kristin.jacque@intel.com>
This patch allows the agent to support secondary providers if any are returned by the server in the GetAttachInfo call. The client requires no knowledge of primary and secondary providers. The agent decides which provider to return based on its config file. - Add support for secondary providers to GetAttachInfo messages. - Add optional "provider" parameter for the agent configuration. If specified the agent will search secondary providers for the requested provider, and return the appropriate network info to the client as if it was the primary provider. Signed-off-by: Kris Jacque <kristin.jacque@intel.com>
* DAOS-9928 srv: xstreams for secondary cart contexts This patch adjusted xstream starting and core assignment code to support xstreams for secondary Cart contexts. It also have two minor fixes to current code: - Oversubscribe doesn't work in NUMA mode, fallthrough to legacy mode when oversubscribe is specified; - 'Core offset' should be ignored when NUMA mode is used; Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Features: control
Use a comma to separate the multiple provider list instead of a space. This matches up with what CART is expecting. Signed-off-by: Kris Jacque <kristin.jacque@intel.com>
The new parameter in the server config, secondary_provider_endpoints, defines the number of secondary CART contexts to be created by each engine for each secondary fabric provider. The primary provider is not affected. Signed-off-by: Kris Jacque <kristin.jacque@intel.com>
- Check all the fabric interfaces in the server. Test-tag: pr daily Signed-off-by: Kris Jacque <kristin.jacque@intel.com>
* DAOS-9623 agent: Let client specify desired fabric iface If the client has an OFI_INTERFACE/OFI_DOMAIN specified, the agent should select the GetAttachInfo payload for the provider that matches their choice. - Include OFI_INTERFACE/OFI_DOMAIN in client GetAttachInfo request. - If the requested interface/domain doesn't exist in the system, the primary provider is used. - The client requested interface and domain will be used even if not detected or they don't seem to support the configured provider. The mismatch will be logged as an error to make errors with the env variables easier to debug. Features: control Signed-off-by: Kris Jacque <kristin.jacque@intel.com>
Features: control
Features: control
Features: control
Required-githooks: true
Cart level changes for multiprovider support, phase1. No bulk support yet. Signed-off-by: Alexander A Oganezov <alexander.a.oganezov@intel.com>
Required-githooks: true
Required-githooks: true
Required-githooks: true
…erge Signed-off-by: Niu Yawei <yawei.niu@hpe.com>
|
Errors are component not formatted correctly,Ticket number prefix incorrect,PR title is malformatted. See https://daosio.atlassian.net/wiki/spaces/DC/pages/11133911069/Commit+Comments,Unable to load ticket data |
Collaborator
|
Test stage Unit Test completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-17709/1/testReport/ |
tanabarr
approved these changes
Mar 16, 2026
Contributor
tanabarr
left a comment
tanabarr
left a comment
There was a problem hiding this comment.
Go changes LGTM. It looks like some of the Copyright and compiled protobuf changes may not be strictly necessary but this isn't a big deal.
kjacque
requested changes
Mar 16, 2026
Contributor
kjacque
left a comment
kjacque
left a comment
There was a problem hiding this comment.
Not ready to merge into master yet
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Steps for the author:
After all prior steps are complete: