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

test/kvs-cmd: Fix race in tests #937

Merged
merged 1 commit into from Jan 6, 2017

Conversation

Projects
None yet
4 participants
@chu11
Copy link
Contributor

chu11 commented Jan 6, 2017

Fix race with kvs watch background processes in which output
from a prior test's watch_out output is seen by the current
test.

Fixes #936

test/kvs-cmd: Fix race in tests
Fix race with kvs watch background processes in which output
from a prior test's watch_out output is seen by the current
test.

Fixes #936

@garlick garlick added the review label Jan 6, 2017

@codecov-io

This comment has been minimized.

Copy link

codecov-io commented Jan 6, 2017

Current coverage is 76.06% (diff: 100%)

Merging #937 into master will decrease coverage by 0.03%

@@             master       #937   diff @@
==========================================
  Files           149        149          
  Lines         25951      25951          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits          19748      19739     -9   
- Misses         6203       6212     +9   
  Partials          0          0          

Powered by Codecov. Last update dbd8883...7aad0a8

@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 6, 2017

Coverage Status

Coverage decreased (-0.03%) to 76.347% when pulling 7aad0a8 on chu11:issue936 into dbd8883 on flux-framework:master.

@garlick

This comment has been minimized.

Copy link
Member

garlick commented Jan 6, 2017

Thanks! Merging.

@garlick garlick merged commit 3cf4c55 into flux-framework:master Jan 6, 2017

4 checks passed

codecov/patch Coverage not affected when comparing dbd8883...7aad0a8
Details
codecov/project Absolute coverage decreased by -0.03% but relative coverage increased by +23.90% compared to dbd8883
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage decreased (-0.03%) to 76.347%
Details

@garlick garlick removed the review label Jan 6, 2017

@grondo grondo referenced this pull request Mar 28, 2017

Closed

0.7.0 Release Notes #1019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.