[kudu-CR] security: simplify CertSigner interface

2017-02-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: security: simplify CertSigner interface .. Patch Set 6: Code-Review+2 Carrying Alexey's +2 -- To view, visit http://gerrit.cloudera.org:8080/5846 To

[kudu-CR] security: simplify CertSigner interface

2017-02-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: security: simplify CertSigner interface .. security: simplify CertSigner interface This makes CertSigner a "one-shot" type of instance, which no longer

[kudu-CR] security: simplify CertSigner interface

2017-02-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: security: simplify CertSigner interface .. Patch Set 6: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5846 To unsubscribe, visit

[kudu-CR] security: simplify CertSigner interface

2017-02-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: security: simplify CertSigner interface .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5846 To unsubscribe, visit

[kudu-CR] security: simplify CertSigner interface

2017-02-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: security: simplify CertSigner interface .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/5846/3/src/kudu/security/ca/cert_management-test.cc File

[kudu-CR] security: simplify CertSigner interface

2017-02-01 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: security: simplify CertSigner interface .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5846 To unsubscribe, visit

[kudu-CR] security: simplify CertSigner interface

2017-02-01 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: security: simplify CertSigner interface .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/5846/3/src/kudu/security/ca/cert_management-test.cc

[kudu-CR] security: simplify CertSigner interface

2017-01-31 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: security: simplify CertSigner interface .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5846/2/src/kudu/security/ca/cert_management-test.cc File

[kudu-CR] security: simplify CertSigner interface

2017-01-31 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Alexey Serbin, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5846 to look at the new patch set (#2). Change subject: security: simplify CertSigner interface ..

[kudu-CR] security: simplify CertSigner interface

2017-01-31 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Alexey Serbin, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5846 to review the following change. Change subject: security: simplify CertSigner interface .. security: