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

Docker: Update 0.6.1 (includes API changes) #32

Merged
merged 17 commits into from
Apr 17, 2014

Conversation

cevich
Copy link
Member

@cevich cevich commented Apr 4, 2014

Signed-off-by: Chris Evich cevich@redhat.com

@cevich cevich changed the title Docker: Update 0.6.1 includes dockertest API changes Docker: Update 0.6.1 (includes API changes) Apr 4, 2014
@cevich cevich added the tracking label Apr 4, 2014
@cevich cevich added this to the Dockertest API Version 0.6.1 milestone Apr 4, 2014
@cevich cevich self-assigned this Apr 4, 2014
@cevich
Copy link
Member Author

cevich commented Apr 4, 2014

Please leave this open for me, I'll merge it in after all the other 0.6.1 stuff goes in.

*  Neither of these subclasses is a true external-interface
   extension and will lead to combersome interface in the
   future.  Since it's used by only a single test, just
   move it there.

*  Found docker 0.9.1 changes formatting of 'ps -a' output
   for STATUS.  Updated containers module to account for this,
   added some debugging.

*  Found docker 0.9.1 changes content of STATUS column causing
   ps test to incorrectly fail.  Updated checks to account for
   both 'old way' and 'new way'

*  Fixed up unittests

*  Added TODO's to fix this missing functionality in more
   general way later.

*  Double-checked there are no references in docs

Signed-off-by: Chris Evich <cevich@redhat.com>
Jiří Župka and others added 4 commits April 16, 2014 11:10
run interactive & disconnect, attach to stdin/stdout/stderr.

Signed-off-by: Jiří Župka <jzupka@redhat.com>
Signed-off-by: Jiří Župka <jzupka@redhat.com>
autotest#64

Signed-off-by: Chris Evich <cevich@redhat.com>
Signed-off-by: Ouyang Guohua <gouyang@redhat.com>
Signed-off-by: Chris Evich <cevich@redhat.com>
Jiří Župka and others added 8 commits April 17, 2014 13:11
Subtest:
  Make cleanup function more verbose.

Signed-off-by: Jiří Župka <jzupka@redhat.com>
*  As of docker-io 0.91 upper-case image names are rejected
   for some reason.  Several tests each implement their own
   image-name generation which results in mixed-case names.

*  Add a get_unique_name() method to images module, with attribute
   to force lower-case names (defaults to True).

*  Update all existing tests that frequently cause failures on
   image naming.

*  Left one of the "tag" sub-sbutests failing so this problem is
   overlooked.  If lower-only is a legit requirement, then
   the adjust ``gen_lower_only`` config option.

autotest#54

Signed-off-by: Chris Evich <cevich@redhat.com>

squash! 246963aed930268138342084a6231a3dfbfd2a8d

Signed-off-by: Chris Evich <cevich@redhat.com>
…to integration

Signed-off-by: Chris Evich <cevich@redhat.com>

Conflicts:
	config_defaults/subtests/docker_cli/start.ini
	subtests/docker_cli/start/start.py
Signed-off-by: Chris Evich <cevich@redhat.com>
…into integration

Conflicts:
	dockertest/containers.py
	subtests/docker_cli/psa/psa.py
SIGUSR isn't handled uniformly by docker and this test isn't
about signals.  Switch to the 'ole reliable SIGUSR1 and
leave the signal testing to more appropriate tests.

Signed-off-by: Chris Evich <cevich@redhat.com>
The else was missaligned to another branch causing fals-positive results

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>
Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>

Conflicts:
	index.rst
Signed-off-by: Chris Evich <cevich@redhat.com>
… integration

Conflicts:
	dockertest/containers.py
cevich added a commit that referenced this pull request Apr 17, 2014
Docker: Update 0.6.1 (includes API changes)
@cevich cevich merged commit ef43046 into autotest:master Apr 17, 2014
@cevich cevich deleted the docker_0.6.1 branch April 21, 2014 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants