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

Fail to capture the failed testing details #64

Closed
gouyang opened this issue Apr 15, 2014 · 3 comments
Closed

Fail to capture the failed testing details #64

gouyang opened this issue Apr 15, 2014 · 3 comments

Comments

@gouyang
Copy link
Contributor

gouyang commented Apr 15, 2014

when a testing fail, it always gets:
DockerTestFail: Good: ['error_check_stdout', 'crash_check_stdout']; Not Good: ['error_check_stderr', 'crash_check_stderr']; Details: (error_check_stderr, No details); (crash_check_stderr, No details)

no details captured.

It looks like self.details in environment.py has no value assigned.

@cevich
Copy link
Member

cevich commented Apr 17, 2014

Could very well be, we'll check it out, thanks!

@cevich cevich added this to the Future Major/Minor Dockertest API revision milestone Apr 17, 2014
@cevich
Copy link
Member

cevich commented Apr 17, 2014

I am able to reproduce this, will investigate further...

[root@docker docker]# git reset --hard 0.5.5
HEAD is now at 833331b Docker: Release Update 0.5.5 (no API changes)
[root@docker docker]# ../../shared/version.py 
0.16.0-master-63-g35e93
[root@docker docker]# cat /etc/redhat-release 
Red Hat Enterprise Linux Server release 7.0 (Maipo)
[root@docker docker]# rpm -q docker-io
docker-io-0.9.1-3.collider.el7.x86_64
[root@docker docker]# git am /tmp/63.patch 
Applying: subtests..run_twice: Add run container twice test Signed-off-by: Ouyang Guohua <gouyang@redhat.com>
[root@docker docker]# cd ../..
[root@docker client]# reset; tools/make_clean; ./autotest-local run docker --verbose --args=docker_cli/run_twice
...
09:32:44 DEBUG| DockerTestFail: Good: ['crash_check_stdout', 'error_check_stdout', 'usage_check_stdout', 'crash_check_stderr', 'usage_check_stderr']; Not Good: ['error_check_stderr']; Details: (error_check_stderr, No details)
...

@cevich
Copy link
Member

cevich commented Apr 17, 2014

Okay, I got a small patch that fixes this and will go into 0.6.1.

@cevich cevich closed this as completed Apr 17, 2014
cevich added a commit to cevich/autotest-docker that referenced this issue Apr 17, 2014
autotest#64

Signed-off-by: Chris Evich <cevich@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants