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

PWX-4572: added UT to watch multiple updates #31

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

PWX-4572: added UT to watch multiple updates #31

wants to merge 18 commits into from

Conversation

sdeoras
Copy link
Contributor

@sdeoras sdeoras commented Jun 21, 2018

added UT to watch multiple events in kvdb and updated Sirupsen/logrun to sirupsen/logrus
Signed-off-by: Saurabh Deoras [email protected]

@sdeoras sdeoras requested a review from sangleganesh June 21, 2018 22:56
@sangleganesh
Copy link
Contributor

it looks good, the idea is to create a test that can hit the bug in consul implementation. once we know thats happening, we can review/merge the PR.

sdeoras and others added 18 commits July 23, 2018 10:06
Signed-off-by: Saurabh Deoras <[email protected]>
Signed-off-by: Saurabh Deoras <[email protected]>
Signed-off-by: Saurabh Deoras <[email protected]>
Signed-off-by: Saurabh Deoras <[email protected]>
Signed-off-by: Saurabh Deoras <[email protected]>
Signed-off-by: Saurabh Deoras <[email protected]>
Signed-off-by: Saurabh Deoras <[email protected]>
Signed-off-by: Saurabh Deoras <[email protected]>
Signed-off-by: Saurabh Deoras <[email protected]>
Signed-off-by: Saurabh Deoras <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants