Skip to content

Conversation

@tanvipenumudy
Copy link
Contributor

What changes were proposed in this pull request?

This task is to add LEGACY to AllowedBucketLayouts in the CreateBucketHandler class.

CreateBucketHandler.java#L65

enum AllowedBucketLayouts {
    FILE_SYSTEM_OPTIMIZED("FILE_SYSTEM_OPTIMIZED"),
    OBJECT_STORE("OBJECT_STORE"),
    LEGACY("LEGACY")

While executing the ozone sh bucket create command and passing an INVALID bucket layout, we currently get the following error message:

Invalid value for option '--layout': expected one of [FILE_SYSTEM_OPTIMIZED, OBJECT_STORE, DEFAULT, ] (case-sensitive) but was 'INVALID'

After the changes, this would be modified to:

Invalid value for option '--layout': expected one of [FILE_SYSTEM_OPTIMIZED, OBJECT_STORE, LEGACY] (case-sensitive) but was 'INVALID'

Similarly, ozone sh bucket create --help gives the following description for "-l", "--layout":

-l, --layout=<allowedBucketLayout>
                           Allowed Bucket Layouts: FILE_SYSTEM_OPTIMIZED,
                           OBJECT_STORE,

After the changes, this would be modified to:

-l, --layout=<allowedBucketLayout>
                           Allowed Bucket Layouts: FILE_SYSTEM_OPTIMIZED,
                           OBJECT_STORE, LEGACY

What is the link to the Apache JIRA

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

How was this patch tested?

Added integration tests.

Copy link
Contributor

@rakeshadr rakeshadr left a comment

Choose a reason for hiding this comment

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

+1 LGTM, thanks @tanvipenumudy for the contribution.

Thanks a lot @JyotinderSingh for the offline discussions to understand the ${COMPLETION-CANDIDATES} types.

@JyotinderSingh
Copy link
Contributor

Thanks for the contribution @tanvipenumudy! The overall patch looks good to me, added a minor comment.

@JyotinderSingh
Copy link
Contributor

Looks like the CI is having some issues right now, will retrigger these tests in a little bit.

@JyotinderSingh
Copy link
Contributor

Thank you @tanvipenumudy for the patch, and @rakeshadr for the reviews and offline discussion. Merging this PR.

@JyotinderSingh JyotinderSingh merged commit c923a26 into apache:master Jun 2, 2022
errose28 added a commit to errose28/ozone that referenced this pull request Jun 7, 2022
* master: (87 commits)
  HDDS-6686. Do Leadship check before SASL token verification. (apache#3382)
  HDDS-4364: [FSO]List FileStatus : startKey can be a non-existed path (apache#3481)
  HDDS-6091. Add file checksum to OmKeyInfo (apache#3201)
  HDDS-6706. Exposing Volume Information Metrics to the DataNode UI (apache#3478)
  HDDS-6759: Add listblock API in MockDatanodeStorage (apache#3452)
  HDDS-5821 Container cache management for closing RockDB  (apache#3426)
  HDDS-6683. Refactor OM server bucket layout configuration usage (apache#3477)
  HDDS-6824. Revert changes made in proto.lock by HDDS-6768. (apache#3480)
  HDDS-6811. Bucket create message with layout type (apache#3479)
  HDDS-6810. Add a optional flag to trigger listStatus as part of listKeys for FSO buckets. (apache#3461)
  HDDS-6828. Revert RockDB version pending leak fixes (apache#3475)
  HDDS-6764: EC: DN ability to create RECOVERING containers for EC reconstruction. (apache#3458)
  HDDS-6795: EC: PipelineStateMap#addPipeline should not have precondition checks post db updates (apache#3453)
  HDDS-6823. Intermittent failure in TestOzoneECClient#testExcludeOnDNMixed (apache#3476)
  HDDS-6820. Bucket Layout Post-Finalization Validators for ACL Requests. (apache#3472)
  HDDS-6819. Add LEGACY to AllowedBucketLayouts in CreateBucketHandler (apache#3473)
  HDDS-4859. [FSO]ListKeys: seek all the files/dirs from startKey to keyPrefix (apache#3466)
  HDDS-6705 Add metrics for volume statistics including disk capacity, usage, Reserved (apache#3430)
  HDDS-6474. Add test to cover the FSO bucket list status with beyond batch boundary and cache. (apache#3379). Contributed by aswinshakil
  HDDS-6280. Support Container Balancer HA (apache#3423)
  ...
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.

3 participants