Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

tests: TestRktListCreatedStarted: fix timing issue #2366

Merged
merged 1 commit into from
Apr 6, 2016
Merged

tests: TestRktListCreatedStarted: fix timing issue #2366

merged 1 commit into from
Apr 6, 2016

Conversation

@alban
Copy link
Member

@alban alban commented Apr 1, 2016

Also fix the error messages

Fix #2220

/cc @krnowak

@alban alban added this to the v1.4.0 milestone Apr 1, 2016
@alban alban force-pushed the alban/TestRktListCreatedStarted branch from 624a100 to 49de29b Compare April 1, 2016 11:56
@alban
Copy link
Member Author

alban commented Apr 1, 2016

It now passes on Debian:

=== RUN   TestRktListCreatedStarted
--- PASS: TestRktListCreatedStarted (11.55s)
    rkt_tests.go:113: Running command: /var/tmp/rkt/builds/build-rkt-coreos-/build-rkt-1.2.1+git/tmp/functional/rkt --dir=/tmp/datadir-391073246 --local-config=/tmp/localdir-878887845 --system-config=/tmp/systemdir-727110336 --insecure-options=image prepare /var/tmp/rkt/builds/build-rkt-coreos-/build-rkt-1.2.1+git/tmp/functional/test-tmp/rkt-list-creation-start-time-test.aci
    rkt_tests.go:113: Running command: /var/tmp/rkt/builds/build-rkt-coreos-/build-rkt-1.2.1+git/tmp/functional/rkt --dir=/tmp/datadir-391073246 --local-config=/tmp/localdir-878887845 --system-config=/tmp/systemdir-727110336 run-prepared 12bdd27e-2445-45d4-8bed-52a3dbe38565
    rkt_tests.go:113: Running command: /var/tmp/rkt/builds/build-rkt-coreos-/build-rkt-1.2.1+git/tmp/functional/rkt --dir=/tmp/datadir-391073246 --local-config=/tmp/localdir-878887845 --system-config=/tmp/systemdir-727110336 list --full

@alban alban force-pushed the alban/TestRktListCreatedStarted branch from 49de29b to 6bd9f7c Compare April 6, 2016 10:01
@alban
Copy link
Member Author

alban commented Apr 6, 2016

@iaguis @s-urbaniak PTAL

@iaguis
Copy link
Member

iaguis commented Apr 6, 2016

This just makes everything slowly enough that the test works on slow machines. LGTM

@iaguis iaguis merged commit fc4829f into rkt:master Apr 6, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

area/testing reviewed/lgtm

Projects

None yet

Milestone

v1.4.0

Development

Successfully merging this pull request may close these issues.

TestRktListCreatedStarted fails on slow machines

2 participants

@alban @iaguis