-
Notifications
You must be signed in to change notification settings - Fork 341
SRE-3591 ci: pip install to not showing progress #17522
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
base: master
Are you sure you want to change the base?
Conversation
|
Errors are Unable to load ticket data |
The pip install command produces a lot of unnecessary lines in the logs showing the progress of download. This makes analysis of the logs very difficult as you have to scroll many screens to skip the installation command. Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
e31c808 to
f97866f
Compare
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com> Cancel-prev-build: false
JohnMalmberg
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be better to fix the /etc/pip.conf file that is being setup for those stages.
ci/rpm/test_daos_node.sh
Outdated
| source venv/bin/activate | ||
| pip install --upgrade pip | ||
| pip install -r $FTEST/requirements-ftest.txt | ||
| pip install --quiet --upgrade pip |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The bug is actually that the /etc/pip.conf in the docker container or a target node is missing these two lines or something similar as is being setup by scripts in other places.
[global]
progress_bar off
no_color true
This quits the logs for CI, but leaves it to the developer to decide if they want to disable the default behavior when doing manual testing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The following solution used everywhere:
cat <<EOF > venv/pip.conf
[global]
progress_bar = off
no_color = true
quiet = 1
EOF
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com> Cancel-prev-build: false Priority: 2
ci/unit/test_main_node.sh
Outdated
| touch venv/pip.conf | ||
|
|
||
| cat <<EOF > venv/pip.conf | ||
| [global] | ||
| progress_bar = off | ||
| no_color = true | ||
| EOF | ||
|
|
||
| pip install --upgrade pip | ||
|
|
||
| pip config set global.progress_bar off | ||
| pip config set global.no_color true | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need to create the config and use pip config set?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 252a0d5
|
Test stage NLT on EL 8.8 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-17522/6/testReport/ |
|
Test stage NLT on EL 8.8 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-17522/5/testReport/ |
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com> Cancel-prev-build: false Priority: 2
|
Test stage NLT on EL 8.8 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-17522/7/testReport/ |
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com> Cancel-prev-build: false Priority: 2 Allow-unstable-test: true Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
|
Test stage Functional Hardware Medium Verbs Provider MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-17522/6/execution/node/1398/log |
|
Test stage NLT on EL 8.8 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-17522/9/testReport/ |
The
pip installcommand produces a lot of unnecessary lines in the logs showing the progress of download.This makes analysis of the logs very difficult as you have to scroll many screens to skip the installation command.
Steps for the author:
After all prior steps are complete: