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

Set default log level to 0. #7

Merged
merged 1 commit into from
Dec 12, 2016
Merged

Conversation

dahorak
Copy link
Contributor

@dahorak dahorak commented Dec 7, 2016

This is required for printing summary of test results and other usefull information. This fixes incorrectly set value in commit d3b70f1.

[09:08:08,438] =======================================================================================
[09:08:08,438] Test-case[s] Summary: (3 found)
[09:08:08,439] ---------------------------------------------------------------------------------------
[09:08:08,439] PASS cluster import                          :  #tests:5     #fails:0     desc.:""
[09:08:08,439] PASS create volume                           :  #tests:4     #fails:0     desc.:""
[09:08:08,439] PASS delete volume                           :  #tests:4     #fails:0     desc.:""
[09:08:08,439] =======================================================================================
[09:08:08,439] List of known issues:
[09:08:08,440] ---------------------------------------------------------------------------------------
[09:08:08,440] =======================================================================================
[09:08:08,440] =======================================================================================
[09:08:08,440] Test-Cases Summary   #TOTAL: 3     #PASSED: 3     #FAILED: 0     #ERRORS: 0
[09:08:08,440] Test Summary : ERROR #TOTAl: 13    #PASSED: 13    #FAILED: 0     #ERRORS: 0   #WAIVES: 0
[09:08:08,441] Test name    : jenkins/workspace/3-dahorak-usm2-cluster-test/usmqe-tests
[09:08:08,441] Duration     : 0:00:13.591856
[09:08:08,441] Test on      : Red Hat Enterprise Linux Server release 7.2 (Maipo) x86_64
[09:08:08,441] =======================================================================================
[09:08:08,441] =======================================================================================
[09:08:08,442] Test finished: 09:08:08,438301

This is required for printing summary of test results and other usefull information.
@mbukatov
Copy link
Contributor

mbukatov commented Dec 7, 2016

I agree with this change as long as @ltrilety is ok with it as well.

Copy link
Contributor

@ltrilety ltrilety left a comment

Choose a reason for hiding this comment

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

I don't much agree with this change, however it could be done as temporary. I am afraid with the temporary part though.
We should sit on that and discuss which information should be there, which is important and what is not.

@mbukatov
Copy link
Contributor

Ok, I have created issue #8 so that we don't forget about it.

@mbukatov mbukatov merged commit f286741 into usmqe:master Dec 12, 2016
@dahorak dahorak deleted the default_log_level branch March 7, 2018 12:51
mbukatov pushed a commit that referenced this pull request Mar 22, 2019
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.

3 participants