Skip to content

Conversation

@smengcl
Copy link
Contributor

@smengcl smengcl commented Mar 4, 2022

What changes were proposed in this pull request?

Add a check in OM that the accessId must be in the form of tenantId$username in OMAssignUserToTenantRequest when assigning an accessId to a new tenant user.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-6366

How was this patch tested?

  • Amended TestOzoneTenantShell accordingly
  • Existing integration tests and acceptance tests shall pass.

…eManager

Change-Id: Iadf69883dc543593030e0c2cf5bc0aa495e16e17
smengcl added 2 commits March 6, 2022 08:10
Change-Id: I0fb63e822b5ec041ff2040ebaa66259c6809391f
Change-Id: I066ab0fe6cf565f45b08cd48df1324423254eca2
Copy link
Contributor

@errose28 errose28 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @smengcl LGTM.

smengcl added 2 commits March 7, 2022 15:20
Change-Id: Ieb261167fd62c5e2031fd7a8d224aa3fd7e0bc69
Conflicts:
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OMMultiTenantManagerImpl.java

Change-Id: I666ab9d4699e4cfdd42072cce13653c2a66cec8f
@smengcl
Copy link
Contributor Author

smengcl commented Mar 8, 2022

Thanks @errose28 for the review.

@smengcl smengcl merged commit 43deaa8 into apache:HDDS-4944 Mar 8, 2022
errose28 added a commit to errose28/ozone that referenced this pull request Mar 30, 2022
* HDDS-4944: (268 commits)
  HDDS-6366. [Multi-Tenant] Disallow specifying custom accessId in OzoneManager (apache#3166)
  HDDS-6275. [Multi-Tenant] Add feature documentation and CLI quick start guide (apache#3095)
  HDDS-6063. [Multi-Tenant] Use VOLUME_LOCK in read and write requests, and some minor refactoring (apache#3051)
  HDDS-6214. [Multi-Tenant] Fix KMS Encryption/Decryption (apache#3010)
  HDDS-6322. Fix Recon getting inccorrect sequenceNumber from OM (apache#3090)
  HDDS-5913. Avoid integer overflow when setting dfs.container.ratis.lo… (apache#2785)
  HDDS-6313. Remove replicas in ContainerStateMap when a container is deleted (apache#3086)
  HDDS-6186. Selective checks: skip integration check for unit test changes (apache#3061)
  HDDS-6310. Update json-smart to 2.4.7. (apache#3080)
  HDDS-6190. Cleanup unnecessary datanode id path checks. (apache#2993)
  HDDS-6304. Add enforcer to make sure ozone.version equals project.version (apache#3075)
  HDDS-6309. Update ozone-runner version to 20220212-1 (apache#3079)
  HDDS-6293. Allow using custom ozone-runner image (apache#3072)
  HDDS-4126. Freon key generator should support >2GB files. (apache#3054)
  HDDS-6088. Implement O3FS/OFS getFileChecksum() using file checksum helpers - addendum: fix checkstyle
  HDDS-6088. Implement O3FS/OFS getFileChecksum() using file checksum helpers. (apache#2935)
  HDDS-6084. [Multi-Tenant] Handle upgrades to version supporting S3 multi-tenancy (apache#3018)
  HDDS-6257. Wrong stack trace for S3 errors (apache#3066)
  HDDS-6278 Improve memory profile for listStatus API call. (apache#3053)
  HDDS-6285. ozonesecure-mr intermittently failing with timeout downloading packages (apache#3057)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants