You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "KeDeng (Code Review)" <ge...@cloudera.org> on 2023/04/25 09:29:57 UTC

[kudu-CR] KUDU-3413 [multi-tenancy] update server key for multi-tenancy

Hello Tidy Bot, Alexey Serbin, Yuqi Du, Ashwani Raina, Yingchun Lai, Attila Bukor, Yifan Zhang, Kudu Jenkins, Wang Xixu, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/19622

to look at the new patch set (#6).

Change subject: KUDU-3413 [multi-tenancy] update server key for multi-tenancy
......................................................................

KUDU-3413 [multi-tenancy] update server key for multi-tenancy

If we want to introduce the multi-tenant feature on the
basis of data at rest encryption, we have to solve the
issue of version upgrade between the two features. Such
as do some refactor to create a new default tenant.

In this submission, I replaced the server key with the
tenant key, which belongs to the default tenant. I want
to use this patch as the beginning of the multi-tenant
feature development. However, the operation of custom
tenant is not supported at present. At the same time,
I mainly solved the upgrade problem by assigning the
existing server key to the default tenant and clearing
the existed server key. And I also added a unit test
for this upgrade process to ensure the upgrade logic is
effective.

Change-Id: I9e450d73940eb1dbaac6f905a46d6ccd084f15cf
---
M src/kudu/consensus/raft_consensus_quorum-test.cc
M src/kudu/fs/default_key_provider-test.cc
M src/kudu/fs/default_key_provider.h
M src/kudu/fs/dir_manager.h
M src/kudu/fs/fs.proto
M src/kudu/fs/fs_manager-test.cc
M src/kudu/fs/fs_manager.cc
M src/kudu/fs/fs_manager.h
M src/kudu/fs/key_provider.h
M src/kudu/fs/ranger_kms_key_provider.cc
M src/kudu/fs/ranger_kms_key_provider.h
M src/kudu/integration-tests/raft_consensus-itest.cc
M src/kudu/mini-cluster/external_mini_cluster.cc
M src/kudu/mini-cluster/external_mini_cluster.h
M src/kudu/mini-cluster/internal_mini_cluster.cc
M src/kudu/mini-cluster/internal_mini_cluster.h
M src/kudu/ranger-kms/ranger_kms_client.cc
M src/kudu/ranger-kms/ranger_kms_client.h
M src/kudu/server/server_base.cc
M src/kudu/server/server_base_options.cc
M src/kudu/server/server_base_options.h
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_common.cc
M src/kudu/tools/tool_action_common.h
M src/kudu/tools/tool_action_fs.cc
M src/kudu/tools/tool_action_pbc.cc
M src/kudu/tools/tool_action_wal.cc
M src/kudu/tserver/tablet_copy_client-test.cc
M src/kudu/tserver/tablet_server-test-base.cc
M src/kudu/util/env-test.cc
M src/kudu/util/env.h
M src/kudu/util/env_posix.cc
M src/kudu/util/test_util.cc
M src/kudu/util/test_util.h
34 files changed, 771 insertions(+), 235 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/22/19622/6
-- 
To view, visit http://gerrit.cloudera.org:8080/19622
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9e450d73940eb1dbaac6f905a46d6ccd084f15cf
Gerrit-Change-Number: 19622
Gerrit-PatchSet: 6
Gerrit-Owner: KeDeng <kd...@gmail.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ar...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <ab...@apache.org>
Gerrit-Reviewer: KeDeng <kd...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Wang Xixu <14...@qq.com>
Gerrit-Reviewer: Yifan Zhang <ch...@163.com>
Gerrit-Reviewer: Yingchun Lai <la...@apache.org>
Gerrit-Reviewer: Yuqi Du <sh...@gmail.com>