Skip to content

Conversation

@captainzmc
Copy link
Member

@captainzmc captainzmc commented Nov 19, 2020

What changes were proposed in this pull request?

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

What is the link to the Apache JIRA

There are few descriptions of Ranger in Ozone documents. I tried to install Ranger locally and verified the authentication strategy of Ozone. The authentication rules need to be added to the existing documentation.
The effects of the changes can be viewed here:
https://github.com/captainzmc/hadoop-ozone/blob/add-ranger-doc/hadoop-hdds/docs/content/security/SecurityWithRanger.md

How was this patch tested?

add doc no need tests

@captainzmc captainzmc changed the title HDDS-4485. [DOC]Modification of the operating introduction of ozone ranger. HDDS-4485. [DOC] add the authentication rules of the Ozone Ranger. Nov 19, 2020
@vivekratnavel
Copy link
Contributor

Overall LGTM. Added a couple of minor inline suggestions.

@captainzmc Thanks for working on this!

@captainzmc
Copy link
Member Author

Thanks for @vivekratnavel’s review.
Fixed review issues.

@vivekratnavel
Copy link
Contributor

Changes in English doc looks good to me. I will wait for someone else to review the zh version.

Copy link
Contributor

@cku328 cku328 left a comment

Choose a reason for hiding this comment

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

Changes in Chinese docs part, Overall LGTM.
I just have a minor suggestion.



Apache Ranger™ 是一个用于管理和监控 Hadoop 平台复杂数据权限的框架。版本大于 1.20 的 Apache Ranger 都可以用于管理 Ozone 集群
Apache Ranger™ 是一个用于管理和监控 Hadoop 平台复杂数据权限的框架。Apache Ranger 从2.0版本开始支持ozone鉴权。但由于在2.0中存在一些bug,因此我们更推荐使用Apache Ranger 2.1及以后版本
Copy link
Contributor

Choose a reason for hiding this comment

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

ozone -> Ozone
I think Ozone should start with a uppercase letter.

@captainzmc
Copy link
Member Author

Thanks for @cku328 ’s review. Fixed review issues.

@runzhiwang
Copy link
Contributor

@captainzmc Thanks the patch. @cku328 @vivekratnavel Thanks for review. I have merged the patch

@runzhiwang runzhiwang merged commit fc8b4ec into apache:master Dec 1, 2020
errose28 added a commit to errose28/ozone that referenced this pull request Jan 5, 2021
* master: (40 commits)
  HDDS-4473. Reduce number of sortDatanodes RPC calls (apache#1610)
  HDDS-4485. [DOC] add the authentication rules of the Ozone Ranger. (apache#1603)
  HDDS-4528. Upgrade slf4j to 1.7.30 (apache#1639)
  HDDS-4424. Update README with information how to report security issues (apache#1548)
  HDDS-4484. Use RaftServerImpl isLeader instead of periodic leader update logic in OM and isLeaderReady for read/write requests (apache#1638)
  HDDS-4429. Create unit test for SimpleContainerDownloader. (apache#1551)
  HDDS-4461. Reuse compiled binaries in acceptance test (apache#1588)
  HDDS-4511: Avoiding StaleNodeHandler to take effect in TestDeleteWithSlowFollower. (apache#1625)
  HDDS-4510. SCM can avoid creating RetriableDatanodeEventWatcher for deletion command ACK (apache#1626)
  HDDS-3363. Intermittent failure in testContainerImportExport (apache#1618)
  HDDS-4370. Datanode deletion service can avoid storing deleted blocks. (apache#1620)
  HDDS-4512. Remove unused netty3 transitive dependency (apache#1627)
  HDDS-4481. With HA OM can send deletion blocks to SCM multiple times. (apache#1608)
  HDDS-4487. SCM can avoid using RETRIABLE_DATANODE_COMMAND for datanode deletion commands. (apache#1621)
  HDDS-4471. GrpcOutputStream length can overflow (apache#1617)
  HDDS-4308. Fix issue with quota update (apache#1489)
  HDDS-4392. [DOC] Add Recon architecture to docs (apache#1602)
  HDDS-4501. Reload OM State fail should terminate OM for any exceptions. (apache#1622)
  HDDS-4492. CLI flag --quota should default to 'spaceQuota' to preserve backward compatibility. (apache#1609)
  HDDS-3689. Add various profiles to MiniOzoneChaosCluster to run different modes. (apache#1420)
  ...
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.

4 participants