Skip to content

rgw: user accounts implementation#35726

Closed
theanalyst wants to merge 73 commits intoceph:masterfrom
theanalyst:rgw/user-accounts
Closed

rgw: user accounts implementation#35726
theanalyst wants to merge 73 commits intoceph:masterfrom
theanalyst:rgw/user-accounts

Conversation

@theanalyst
Copy link
Member

@theanalyst theanalyst commented Jun 23, 2020

Draft PR, not ready for review yet

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug
Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@theanalyst theanalyst force-pushed the rgw/user-accounts branch 3 times, most recently from 59c54c9 to 916d704 Compare June 30, 2020 11:36
@theanalyst theanalyst force-pushed the rgw/user-accounts branch 3 times, most recently from e738a8e to a04aa67 Compare July 20, 2020 14:38
@theanalyst
initial account commit
ac36f6e 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: services: introduce an RGWAccount interface
189ce6f 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw_account: add get_info_by_tenant declaration
e7d8935 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: zone: add an account pool
fdb9846 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
svc: svc_account: add get_meta_key function
97882c2 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: svc_account: fix store info args
5cfafb3 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: svc: introduce svc_account_rados to do the metadata ops & sys objs
37366eb 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: account: compiling version of store_account_info
2de9357 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: svc_account: add be_handler & rename interface to Account_RADOS
ecf5190 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw_service: add account service
e41c9bc 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: svc_account_rados: mark functions as not implemented
7593736 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: rgw_account: init be_handler
8e7b68a 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw_account: mark destructor as default
8a5d754 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw zone: fix account pool name
9d63200 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: json enc: add account pool
3d59d2b 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: account pool: fix position of encode
f940486 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw_account: declaration for RGWAccountMetadataHandler
6a0cd40 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw_account: drop rgw_user fwd declarations
db12756 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw_account: implement json dump methods
a560b2d 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw_admin: skeleton for account create
120214c 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: service: add be_module for Account service
9bce314 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: admin: account user list implementation
6a6a6cc 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: account: add max users/roles setters
3b67f29 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw_account: use a const for default quota limits
8d67594 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: rest_account: introduce admin api for accounts
3076b4a 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: user: introduce link and unlink account admin interfaces
f4272ac 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: rest: admin: introduce link user account admin api
13d5af1 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: rest_account: use the new get_handler syntax
2429387 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: admin: formatter changes
d510bf2 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: account introduce AdminOp classes for reuseability
04b78ca 
Since both the cli & REST api use a similar interface introduce a common set of
functions that can be reused by both to call admin actions

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: admin: use account admin op interfaces for cli & rest
37d1500 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
qa: radosgw-admin: add account related test cases
3ec93fa 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: admin: add account cli options
bbb6043 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
qa: radosgw_admin_rest: add account rest test cases
8e15c42 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: user: fix account linking in multisite sync
6af5806 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
rgw: dencoder: AccountInfo generate test fixes
961686a 
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@stale
Copy link

stale bot commented Oct 31, 2020

This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days.
If you are a maintainer or core committer, please follow-up on this pull request to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

@stale stale bot added the stale label Oct 31, 2020
@theanalyst
Copy link
Member Author

theanalyst commented Nov 2, 2020

unstale me

@stale stale bot removed the stale label Nov 2, 2020
@github-actions
Copy link

github-actions bot commented Dec 16, 2020

This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved

@mattbenjamin
Copy link
Contributor

mattbenjamin commented Jan 11, 2021

hi @theanalyst

@stale
Copy link

stale bot commented Jul 21, 2021

This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days.
If you are a maintainer or core committer, please follow-up on this pull request to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

@stale stale bot added the stale label Jul 21, 2021
@cbodley
Copy link
Contributor

cbodley commented May 18, 2022

@alimaredia i'll start working through this rebase

@cbodley cbodley mentioned this pull request May 23, 2022
Closed
10 tasks
@cbodley
Copy link
Contributor

cbodley commented May 23, 2022

rebased in #46373

@cbodley cbodley closed this May 23, 2022
@cbodley cbodley mentioned this pull request Nov 3, 2023
Merged
52 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

needs-rebase needs-review rgw stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@theanalyst @mattbenjamin @cbodley @batrick