Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(Ranger): Pull policies from the Ranger Service and update using resources policies #1388

Merged
merged 11 commits into from
Mar 17, 2023

Conversation

WHBANG
Copy link
Contributor

@WHBANG WHBANG commented Mar 13, 2023

#1054

This patch implements how to pull policies from the Ranger Service
and dump policies to remote storage.

  • Pull policies in JSON format from Ranger service and parse
    policies from JSON formated string.
  • Create the path to save policies in remote storage, and update
    using resources policies.
  • Dump policies to remote storage.
  • Sync policies to app envs.
  • Update the cached global/database resources policies.

@github-actions github-actions bot added the cpp label Mar 13, 2023
@WHBANG WHBANG changed the title feat(Ranger): Pull policies from the Ranger Service and update using resources policies. feat(Ranger): Pull policies from the Ranger Service and update using resources policies Mar 13, 2023
@github-actions github-actions bot added the build label Mar 13, 2023
@github-actions github-actions bot removed the build label Mar 14, 2023
acelyc111
acelyc111 previously approved these changes Mar 16, 2023
src/runtime/ranger/ranger_resource_policy_manager.cpp Outdated Show resolved Hide resolved
src/runtime/ranger/ranger_resource_policy_manager.cpp Outdated Show resolved Hide resolved
@acelyc111 acelyc111 merged commit 034cb9d into apache:master Mar 17, 2023
dsn::error_code update_policies_from_ranger_service();

// Pull policies in JSON format from Ranger service.
dsn::error_code pull_policies_from_ranger_service(std::string *ranger_policies) const;
Copy link
Contributor

Choose a reason for hiding this comment

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

Could pull_policies_from_ranger_service be defined in anonymous namespace in cpp file since it does not reference any members ?

@empiredan empiredan mentioned this pull request Aug 21, 2023
GehaFearless pushed a commit to GehaFearless/incubator-pegasus that referenced this pull request Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants