Skip to content

pyocf: disable test debug logs by default - #711

Merged
robertbaldyga merged 1 commit into
Open-CAS:masterfrom
arutk:sdl
Oct 8, 2026
Merged

robertbaldyga merged 1 commit into
Open-CAS:masterfrom
arutk:sdl

Conversation

@arutk

@arutk arutk commented May 9, 2022

Copy link
Copy Markdown
Contributor

New commandline switch --debug-log and associated debug_log fixture
is added to enable debug logs on demand.

Signed-off-by: Adam Rutkowski adam.j.rutkowski@intel.com

@pep8speaks

pep8speaks commented May 9, 2022 •

Copy link
Copy Markdown

Hello @arutk! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 55:1: W293 blank line contains whitespace

Comment last updated at 2022-06-02 08:25:05 UTC

@codecov

codecov Bot commented May 9, 2022 •

Copy link
Copy Markdown

Codecov Report

Merging #711 (54013a9) into master (fcfdd26) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #711      +/-   ##
==========================================
- Coverage   69.04%   69.00%   -0.04%     
==========================================
  Files          90       90              
  Lines       10066    10105      +39     
==========================================
+ Hits         6950     6973      +23     
- Misses       3116     3132      +16     
Impacted Files Coverage Δ
src/mngt/ocf_mngt_cache.c 81.70% <0.00%> (-0.60%) ⬇️

jfckm
jfckm previously approved these changes May 31, 2022
rafalste
rafalste previously approved these changes Jun 2, 2022
@arutk
arutk dismissed stale reviews from rafalste and jfckm via 54013a9 June 2, 2022 08:25
kmajzero
kmajzero previously approved these changes Jun 21, 2022
Comment thread tests/functional/tests/conftest.py Outdated
New commandline switch --debug-log and associated debug_log fixture
is added to enable debug logs on demand.

Signed-off-by: Adam Rutkowski <adam.j.rutkowski@intel.com>
Signed-off-by: Robert Baldyga <robert.baldyga@unvertical.com>
@robertbaldyga
robertbaldyga merged commit 024149d into Open-CAS:master Oct 8, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants