Skip to content

Conversation

@grom72
Copy link
Contributor

@grom72 grom72 commented Feb 6, 2026

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.

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Errors are Unable to load ticket data
https://daosio.atlassian.net/browse/SRE-3591

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>
@grom72 grom72 force-pushed the grom72/SRE-3591-pip-no-progress-bar branch from e31c808 to f97866f Compare February 6, 2026 19:03
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
@grom72 grom72 marked this pull request as ready for review February 8, 2026 14:44
@grom72 grom72 requested review from a team as code owners February 8, 2026 14:44
Copy link
Contributor

@JohnMalmberg JohnMalmberg left a 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.

source venv/bin/activate
pip install --upgrade pip
pip install -r $FTEST/requirements-ftest.txt
pip install --quiet --upgrade pip
Copy link
Contributor

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.

Copy link
Contributor Author

@grom72 grom72 Feb 10, 2026

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
Comment on lines 87 to 99
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

Copy link
Contributor

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 252a0d5

@daosbuild3
Copy link
Collaborator

@daosbuild3
Copy link
Collaborator

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

Cancel-prev-build: false
Priority: 2
@daosbuild3
Copy link
Collaborator

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>
@daosbuild3
Copy link
Collaborator

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

@daosbuild3
Copy link
Collaborator

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