DAOS-14369 test: Rework pydaos and avocado install/setup process. #13565
Conversation
This file contains 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
|
Bug-tracker data: |
ashleypittman
changed the title
|
Test stage Unit Test on EL 8.8 completed with status UNSTABLE. https://build.hpdd.intel.com/job/daos-stack/job/daos//view/change-requests/job/PR-13565/9/testReport/ |
|
Test stage NLT on EL 8.8 completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-13565/9/execution/node/795/log |
|
Test stage Unit Test on EL 8.8 completed with status UNSTABLE. https://build.hpdd.intel.com/job/daos-stack/job/daos//view/change-requests/job/PR-13565/10/testReport/ |
|
Test stage NLT on EL 8.8 completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-13565/17/execution/node/795/log |
|
Test stage Functional on EL 8.8 completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-13565/19/execution/node/757/log |
|
Test stage Build RPM on Leap 15.5 completed with status UNSTABLE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-13565/20/execution/node/337/log |
|
Test stage Build DEB on Ubuntu 20.04 completed with status UNSTABLE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-13565/20/execution/node/310/log |
|
Test stage Functional on EL 8.8 completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-13565/22/execution/node/720/log |
ashleypittman
force-pushed
the
amd/avocado-version
branch
from
29208f5 to
1772bd1
Compare
|
Test stage Functional on EL 8.8 completed with status FAILURE. https://build.hpdd.intel.com/job/daos-stack/job/daos/job/PR-13565/30/display/redirect |
Update provisioning scripts to use pip bump the pip version of avocado. Update ansible to use the same version. Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Use venv for NLT. Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
Required-githooks: true Signed-off-by: Ashley Pittman <[email protected]>
|
@phender @daltonbohning @brianjmurrell can this be prioritised please? It's a big change and I'd like to get it in ASAP. |
daltonbohning
approved these changes
Apr 4, 2024
|
IMO we should move forward with this ASAP before it has conflicts yet again. And this work precedes being able to update to an even newer avocado version |
phender
approved these changes
Apr 4, 2024
|
Test stage Functional Hardware Medium Verbs Provider completed with status FAILURE. https://build.hpdd.intel.com//job/daos-stack/job/daos/view/change-requests/job/PR-13565/106/execution/node/1441/log |
ashleypittman
changed the title
daltonbohning
added a commit
that referenced
this pull request
Apr 16, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely. Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 16, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely. Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 16, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely. Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
18 tasks
daltonbohning
added a commit
that referenced
this pull request
Apr 16, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely. Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 16, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely. Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 23, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using "command -v daos" and support setting DAOS_TEST_PREFIX Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 23, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using "command -v daos" and support setting DAOS_TEST_PREFIX Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 23, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using "command -v daos" and support setting DAOS_TEST_PREFIX Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 23, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using "command -v daos" and support setting DAOS_TEST_PREFIX Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 23, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using "command -v daos" and support setting DAOS_TEST_PREFIX Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 25, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using "command -v daos" and support setting DAOS_TEST_PREFIX Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 25, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using shutil.which("daos") and support setting DAOS_TEST_PREFIX Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 26, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using shutil.which("daos") and support setting DAOS_TEST_PREFIX Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 26, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using shutil.which("daos") and support setting DAOS_TEST_PREFIX Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
daltonbohning
added a commit
that referenced
this pull request
Apr 26, 2024
Test-tag: pr Skip-unit-tests: true Skip-fault-injection-test: true PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using shutil.which("daos") and support setting DAOS_TEST_PREFIX Required-githooks: true Signed-off-by: Dalton Bohning <[email protected]>
phender
pushed a commit
that referenced
this pull request
Apr 29, 2024
PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using shutil.which("daos") and support setting DAOS_TEST_PREFIX Signed-off-by: Dalton Bohning <[email protected]>
mjmac
pushed a commit
that referenced
this pull request
Apr 30, 2024
PR #13565 accidentally broke how ftest determines the prefix from .build_vars.json because it is no longer installed. Eliminate the need for .build_vars.json in ftest entirely by using shutil.which("daos") and support setting DAOS_TEST_PREFIX Signed-off-by: Dalton Bohning <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Update pydaos install process. Include pydaos sources in daos-devel rpm to allow
installation on non-default python versions and virtual environments.
Use updated python version for NLT to improve reporting and verify pydaos install process.
Update dependencies for daos-client-tests rpm, add daos-devel to get pydaos sources
and remove some no longer required python libs.
Re-write entire python requirements.txt files whilst keeping install process the same.
requirements.txt is now split into build, utest and ftest files, the origional file pulls in all
three as well as cq for linting.
Update the build to only install build requirements, have utest install utest requirements and
ftest install ftest requirements.
Include avocado 82 in ftest requirements so this is used universally throughout CI.
Update code as required to no longer handle or patch older avocado versions and
add check that at least 82 is in use.
Signed-off-by: Ashley Pittman [email protected]