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 api update API: daos_cont_open only supports uuid string. #521

Merged
merged 5 commits into from
Feb 22, 2022

Conversation

wiliamhuang
Copy link
Collaborator

Need to convert uuid_t to string then pass it to daos_cont_open().

Wang Shilong and others added 3 commits November 17, 2021 15:24
object format will be changed since 2.0, this patch
adds codes to be aware of new changes.

PR-repos-el7: daos@PR-6752
PR-repos-el8: daos@PR-6752
PR-repos-leap15: daos@PR-6752
Signed-off-by: Wang Shilong <shilong.wang@intel.com>
daos: fix to support new DAOS api
daos_cont_open only supports container uuid string.
uuid_t has to be converted to string.

Signed-off-by: Lei Huang <wiliam_huang@hotmail.com>
Copy link
Collaborator

@daltonbohning daltonbohning left a comment

Choose a reason for hiding this comment

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

@wiliamhuang The daos-stack/hpc fork diverged from hpc/mpifileutils for a temporary fix that is now in upstream. If creating a PR from daos-stack/hpc, you'll want to rebase on hpc/master first.

daos_cont_open only supports container uuid string.
uuid_t has to be converted to string.

Signed-off-by: Lei Huang <wiliam_huang@hotmail.com>
…pifileutils into daos_api_update_lei

Signed-off-by: Lei Huang <wiliam_huang@hotmail.com>
@daltonbohning daltonbohning added the daos DAOS-related issues label Feb 22, 2022
@daltonbohning
Copy link
Collaborator

@adammoody I'm going ahead and merging this one since the change is just one line, contained to DAOS only, and we'd like to update our internal tests accordingly.

@daltonbohning daltonbohning merged commit 0837021 into hpc:master Feb 22, 2022
@daltonbohning daltonbohning deleted the daos_api_update_lei branch February 22, 2022 21:00
@adammoody
Copy link
Member

No problem. Thanks @wiliamhuang and @daltonbohning

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
daos DAOS-related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants