Skip to content

DAOS-8655 tests: Fix mpi tmpdir_base assignment in threads.#7614

Merged
sylviachanoiyee merged 3 commits intomasterfrom
DAOS-8655_2
Dec 13, 2021
Merged

DAOS-8655 tests: Fix mpi tmpdir_base assignment in threads.#7614
sylviachanoiyee merged 3 commits intomasterfrom
DAOS-8655_2

Conversation

@phender
Copy link
Copy Markdown
Contributor

@phender phender commented Dec 9, 2021

Supplemental fix for previous server/metadata.py
test_metadata_server_restart test case where unique temporary base
directories for each orterun command where based on the previous unique
temporary directory that no longer existed when the next ior command was
issued.

Quick-functional: true
Test-tag: metadata_ior offline_extend_oclass
Test-repeat: 10

Signed-off-by: Phillip Henderson phillip.henderson@intel.com

Supplemental fix for previous server/metadata.py
test_metadata_server_restart test case where unique temporary base
directories for each orterun command where based on the previous unique
temporary directory that no longer existed when the next ior command was
issued.

Quick-functional: true
Test-tag: metadata_ior offline_extend_oclass
Test-repeat: 10

Signed-off-by: Phillip Henderson <phillip.henderson@intel.com>
@phender phender requested a review from kccain December 9, 2021 13:47
kccain
kccain previously approved these changes Dec 9, 2021
@phender phender requested a review from rpadma2 December 9, 2021 16:59
rpadma2
rpadma2 previously approved these changes Dec 9, 2021
Avoid using a previous tmp directory as the base for a new tmp directory
when assinging the orte_tmpdir_base value.

Quick-functional: true
Test-tag: metadata_ior offline_extend_oclass
Test-repeat: 10

Signed-off-by: Phillip Henderson <phillip.henderson@intel.com>
@phender phender dismissed stale reviews from rpadma2 and kccain via d9c1771 December 10, 2021 13:49
@phender
Copy link
Copy Markdown
Contributor Author

phender commented Dec 13, 2021

@phender phender requested review from kccain and rpadma2 December 13, 2021 13:53
Copy link
Copy Markdown
Contributor

@rpadma2 rpadma2 left a comment

Choose a reason for hiding this comment

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

LGTM.

@phender phender requested a review from a team December 13, 2021 15:36
@sylviachanoiyee sylviachanoiyee merged commit eb0f4f1 into master Dec 13, 2021
@sylviachanoiyee sylviachanoiyee deleted the DAOS-8655_2 branch December 13, 2021 15:42
phender added a commit that referenced this pull request Dec 13, 2021
Supplemental fix for previous server/metadata.py
test_metadata_server_restart test case where unique temporary base
directories for each orterun command where based on the previous unique
temporary directory that no longer existed when the next ior command was
issued.

Quick-functional: true
Test-tag: metadata_ior offline_extend_oclass
Test-repeat: 10

Signed-off-by: Phillip Henderson <phillip.henderson@intel.com>
johannlombardi pushed a commit that referenced this pull request Dec 14, 2021
…7655)

Supplemental fix for previous server/metadata.py
test_metadata_server_restart test case where unique temporary base
directories for each orterun command where based on the previous unique
temporary directory that no longer existed when the next ior command was
issued.

Signed-off-by: Phillip Henderson <phillip.henderson@intel.com>
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