Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1210)

Issue 6849126: firewaller: skip flaky test

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 5 months ago by fwereade
Modified:
11 years, 5 months ago
Reviewers:
mp+137258
Visibility:
Public.

Description

firewaller: skip flaky test Note that the test itself is bascially fine; it exposes problems with the firewaller initialization, which need to be addressed, but which would respond well to expert attention. https://code.launchpad.net/~fwereade/juju-core/firewaller-skip-flaky-test/+merge/137258 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : firewaller: skip flaky test #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
M worker/firewaller/firewaller_test.go View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 4
fwereade
Please take a look.
11 years, 5 months ago (2012-11-30 16:51:45 UTC) #1
TheMue
LGTM as discussed on irc.
11 years, 5 months ago (2012-11-30 16:57:14 UTC) #2
niemeyer
LGTM
11 years, 5 months ago (2012-11-30 17:00:46 UTC) #3
fwereade
11 years, 5 months ago (2012-11-30 17:15:52 UTC) #4
*** Submitted:

firewaller: skip flaky test

Note that the test itself is bascially fine; it exposes problems with the
firewaller initialization, which need to be addressed, but which would
respond well to expert attention.

R=TheMue, niemeyer
CC=
https://codereview.appspot.com/6849126
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b