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

DAOS-13151 client: cache and reuse the attach info for the default system #14318

Merged
merged 2 commits into from
May 6, 2024

Conversation

mjmac
Copy link
Contributor

@mjmac mjmac commented May 6, 2024

Contains the following backports:

mchaarawi added 2 commits May 6, 2024 16:16
…stem (#14172)

improve the daos_init() and pool_connect() process to reuse the attach info
instead of doing agent drpc upcalls multiple times.

Required-githooks: true

Change-Id: I404ca66ba986f9e0284738875347da5e057a534a
Signed-off-by: Mohamad Chaarawi <[email protected]>
Copy link

github-actions bot commented May 6, 2024

Bug-tracker data:
Ticket title is 'libdaos should call get_attach_info() only once per normal startup'
Status is 'Resolved'
Labels: 'scrubbed,triaged'
https://daosio.atlassian.net/browse/DAOS-13151

@mjmac mjmac changed the title mjmac/DAOS 13151 2.4 DAOS-13151 client: cache and reuse the attach info for the default system May 6, 2024
@mjmac mjmac requested review from jolivier23 and techbasset May 6, 2024 16:27
@mjmac
Copy link
Contributor Author

mjmac commented May 6, 2024

@techbasset: FYI, I'd like to get this merged before we do the weekly update.

Copy link
Collaborator

@daosbuild1 daosbuild1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. No errors found by checkpatch.

@mjmac mjmac merged commit 6ab0729 into google/2.4 May 6, 2024
34 of 36 checks passed
@mjmac mjmac deleted the mjmac/DAOS-13151-2.4 branch May 6, 2024 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants