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
Improve logging for test-cmd.sh #6391
Conversation
[test] |
👍 to this... finding the log in the end-to-end directory was awful |
@@ -153,7 +155,7 @@ os::util::sed "s/:7001$/:${ETCD_PEER_PORT}/g" ${SERVER_CONFIG_DIR}/master/master | |||
OPENSHIFT_ON_PANIC=crash openshift start master \ | |||
--config=${MASTER_CONFIG_DIR}/master-config.yaml \ | |||
--loglevel=4 \ |
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.
Shouldn't we bump this up if the added logging is all V(5)
?
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.
weird, I had that at one point, must have gotten dropped
[merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/7974/) (Image: devenv-rhel7_2991) |
Evaluated for origin merge up to 17e7c25 |
Evaluated for origin test up to 17e7c25 |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin/7974/) |
Merged by openshift-bot
No description provided.