[HDDS-10181] Remove duplicate ACL protobuf definitions in interface-storage

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Won't Do
• Affects Version/s: 1.4.0
• Component/s: OM
• Labels: pull-request-available
• Assignee: Ivan Andika
• Reporter: Ivan Andika
• Created: 22/Jan/24 08:59
• Updated: 25/Jan/24 04:04
• Resolved: 25/Jan/24 04:03

Description

It's observed that the serialization (toProtobuf) and deserialization (fromProtobuf) of OmPrefixInfo uses OzoneAclStorageUtil, whereas OmKeyInfo, OmBucketInfo, OmVolumeArgs uses OzoneAclUtil. The logic for both classes looks the same.

There are also duplicate proto definitions for OzoneAclInfo in OmStorageProtocol.proto and OmClientProtocol.proto.

We need to remove the duplication to prevent any divergence.

Note: This seems the separation was done as part of HDDS-4242, under HDDS-4216 which are supposed to separate RPC and storage proto files for compatibility reasons by introducing interface-storage package. But it seems only PrefixInfo and UserVolumeInfo have been done for this. If these changes are intended, this can be closed and we can focus on separating the remaining protos instead.

Issue Links

• relates to: HDDS-4242 Copy PrefixInfo proto to new project hadoop-ozone/interface-storage
• links to: GitHub Pull Request #6068