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
To go deletednsseckeys #2369
To go deletednsseckeys #2369
Conversation
Refer to this link for build results (access rights to CI server needed): |
e6acdb2
to
b8e2e5a
Compare
Refer to this link for build results (access rights to CI server needed): |
b8e2e5a
to
37ef89f
Compare
Refer to this link for build results (access rights to CI server needed): |
37ef89f
to
45da8e7
Compare
Refer to this link for build results (access rights to CI server needed): |
b770644
to
f247c3a
Compare
Refer to this link for build results (access rights to CI server needed): |
03752ab
to
2476da0
Compare
Refer to this link for build results (access rights to CI server needed): |
Refer to this link for build results (access rights to CI server needed): |
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.
I'm concerned we may be leaking something here. Can you check on it?
api.HandleErr(w, r, http.StatusInternalServerError, nil, errors.New("getting riak cluster: "+err.Error())) | ||
return | ||
} | ||
if err := riakCluster.Start(); err != nil { |
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.
I was expecting to see this paired with a .Stop()
somewhere. Is it missing?
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.
Fixed.
2476da0
to
31eadad
Compare
Refer to this link for build results (access rights to CI server needed): |
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.
Yup, this is good to merge now.
31eadad
to
bb45571
Compare
Refer to this link for build results (access rights to CI server needed): |
bb45571
to
88b4e64
Compare
Refer to this link for build results (access rights to CI server needed): |
88b4e64
to
2db1d24
Compare
Refer to this link for build results (access rights to CI server needed): |
Branched from #2365 - recommend merging that first.