You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2018/12/14 18:08:27 UTC

[kudu-CR] [security] enhance warning on conflicting OpenSSL initialization

Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12089


Change subject: [security] enhance warning on conflicting OpenSSL initialization
......................................................................

[security] enhance warning on conflicting OpenSSL initialization

This patch enhances the warning message on detected conflicting
initialization of the OpenSSL library, making it more actionable.

This patch does not contain any functional changes.

Change-Id: I72e989dce82586ba43d2cb4c00e2ef6948c2bd2a
---
M src/kudu/security/openssl_util.cc
1 file changed, 5 insertions(+), 2 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/89/12089/1
-- 
To view, visit http://gerrit.cloudera.org:8080/12089
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I72e989dce82586ba43d2cb4c00e2ef6948c2bd2a
Gerrit-Change-Number: 12089
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>

[kudu-CR] [security] enhance warning on conflicting OpenSSL initialization

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12089 )

Change subject: [security] enhance warning on conflicting OpenSSL initialization
......................................................................


Patch Set 1: Verified+1

Unrelated flake in DeleteTableITest.TestAutoTombstoneAfterCrashDuringTabletCopy


-- 
To view, visit http://gerrit.cloudera.org:8080/12089
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I72e989dce82586ba43d2cb4c00e2ef6948c2bd2a
Gerrit-Change-Number: 12089
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Comment-Date: Fri, 14 Dec 2018 18:45:20 +0000
Gerrit-HasComments: No

[kudu-CR] [security] enhance warning on conflicting OpenSSL initialization

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12089 )

Change subject: [security] enhance warning on conflicting OpenSSL initialization
......................................................................

[security] enhance warning on conflicting OpenSSL initialization

This patch enhances the warning message on detected conflicting
initialization of the OpenSSL library, making it more actionable.

This patch does not contain any functional changes.

Change-Id: I72e989dce82586ba43d2cb4c00e2ef6948c2bd2a
Reviewed-on: http://gerrit.cloudera.org:8080/12089
Tested-by: Alexey Serbin <as...@cloudera.com>
Reviewed-by: Adar Dembo <ad...@cloudera.com>
---
M src/kudu/security/openssl_util.cc
1 file changed, 5 insertions(+), 2 deletions(-)

Approvals:
  Alexey Serbin: Verified
  Adar Dembo: Looks good to me, approved

-- 
To view, visit http://gerrit.cloudera.org:8080/12089
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I72e989dce82586ba43d2cb4c00e2ef6948c2bd2a
Gerrit-Change-Number: 12089
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>

[kudu-CR] [security] enhance warning on conflicting OpenSSL initialization

Posted by "Adar Dembo (Code Review)" <ge...@cloudera.org>.
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12089 )

Change subject: [security] enhance warning on conflicting OpenSSL initialization
......................................................................


Patch Set 1: Code-Review+2


-- 
To view, visit http://gerrit.cloudera.org:8080/12089
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I72e989dce82586ba43d2cb4c00e2ef6948c2bd2a
Gerrit-Change-Number: 12089
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Comment-Date: Fri, 14 Dec 2018 19:56:28 +0000
Gerrit-HasComments: No

[kudu-CR] [security] enhance warning on conflicting OpenSSL initialization

Posted by "Alexey Serbin (Code Review)" <ge...@cloudera.org>.
Alexey Serbin has removed Kudu Jenkins from this change.  ( http://gerrit.cloudera.org:8080/12089 )

Change subject: [security] enhance warning on conflicting OpenSSL initialization
......................................................................


Removed reviewer Kudu Jenkins with the following votes:

* Verified-1 by Kudu Jenkins (120)
-- 
To view, visit http://gerrit.cloudera.org:8080/12089
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: deleteReviewer
Gerrit-Change-Id: I72e989dce82586ba43d2cb4c00e2ef6948c2bd2a
Gerrit-Change-Number: 12089
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>