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

Blocker: HTML test report generated by ocs-ci is reaching gigantic file sizes #5567

Closed
haubenr opened this issue Mar 9, 2022 · 4 comments · Fixed by #5572
Closed

Blocker: HTML test report generated by ocs-ci is reaching gigantic file sizes #5567

haubenr opened this issue Mar 9, 2022 · 4 comments · Fixed by #5572
Assignees
Labels
High Priority High priority issues team/ecosystem Ecosystem team related issues/PRs

Comments

@haubenr
Copy link

haubenr commented Mar 9, 2022

Today I noticed that the the HTML reports generated by ocs-ci for e.g. test tier runs are all of a sudden massive in size, e.g.:

IBM Z / LinuxONE:
tier4a: > 450MB

IBM Power Systems:
tier4a: > 580MB
tier1: > 660MB
tier2: > 980MB

These large HTML reports can no longer be handled properly as both Firefox and Chrome are unable to load / render them.

This issue was introduced into ocs-ci only recently, it's present in branches master as well as stable.

@clacroix12 clacroix12 added the High Priority High priority issues label Mar 9, 2022
@clacroix12
Copy link
Contributor

I'm not sure when this change was introduced but I think this is due to the entire log being captured in the html report for passed test cases. If you view http://magna002.ceph.redhat.com/ocsci-jenkins/openshift-clusters/j-154vuf1cs36-t4a/j-154vuf1cs36-t4a_20220308T163009/logs/deployment_report_1646756339.html you can see the log being displayed below the test case status.

@petr-balogh
Copy link
Member

I think it might be related to the issue I brought up in email here:
https://lists.corp.redhat.com/archives/ocs-qe/2022-March/000258.html

We need to identify the root cause of this ASAP.

1 similar comment
@petr-balogh
Copy link
Member

I think it might be related to the issue I brought up in email here:
https://lists.corp.redhat.com/archives/ocs-qe/2022-March/000258.html

We need to identify the root cause of this ASAP.

petr-balogh added a commit to petr-balogh/ocs-ci that referenced this issue Mar 10, 2022
Fixes: red-hat-storage#5567

In release notes here:
https://pytest-html.readthedocs.io/en/latest/changelog.html#id13
Respect --capture=no, --show-capture=no, and -s pytest flags

Which might be related to the pytest update and change:
red-hat-storage@5fd10d5

Signed-off-by: Petr Balogh <pbalogh@redhat.com>
@petr-balogh
Copy link
Member

Might be related to this issue:
pytest-dev/pytest-html#171

And with recent update of pytest and change of one parameter here:
5fd10d5

I hope that upgrade of pytest-html will fix the issue for us.

petr-balogh added a commit to petr-balogh/ocs-ci that referenced this issue Mar 10, 2022
Fixes: red-hat-storage#5567

In release notes here:
https://pytest-html.readthedocs.io/en/latest/changelog.html#id13
Respect --capture=no, --show-capture=no, and -s pytest flags

Which might be related to the pytest update and change:
red-hat-storage@5fd10d5

Signed-off-by: Petr Balogh <pbalogh@redhat.com>
@petr-balogh petr-balogh added the team/ecosystem Ecosystem team related issues/PRs label Mar 11, 2022
@petr-balogh petr-balogh self-assigned this Mar 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
High Priority High priority issues team/ecosystem Ecosystem team related issues/PRs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants