-
Notifications
You must be signed in to change notification settings - Fork 106
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
go/tests/sentry: Add access control sanity checks #3062
Conversation
300777b
to
6a6432d
Compare
@@ -734,10 +733,6 @@ func (net *Network) startOasisNode( | |||
args = append(args, baseArgs...) | |||
args = append(args, extraArgs.vec...) | |||
|
|||
if !strings.HasPrefix(node.Name, "sentry-") && !strings.HasPrefix(node.Name, "ias-proxy") && len(net.byzantine) == 0 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So i'm not sure if the && len(net.byzantine) == 0
flag was needed before, since in the new code i also enable cert rotation in byzantine tests and tests don't fail
Codecov Report
@@ Coverage Diff @@
## master #3062 +/- ##
==========================================
+ Coverage 68.16% 68.18% +0.02%
==========================================
Files 372 372
Lines 36487 36487
==========================================
+ Hits 24870 24878 +8
- Misses 8398 8400 +2
+ Partials 3219 3209 -10
Continue to review full report at Codecov.
|
dd347fc
to
400740d
Compare
400740d
to
8ccfb84
Compare
No description provided.