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

octopus: mgr/dashboard: Splitting tenant$user when creating rgw user #40297

Merged
merged 1 commit into from Apr 12, 2021

Conversation

nizamial09
Copy link
Member

Fixes: https://tracker.ceph.com/issues/47378
Signed-off-by: Nizamudeen A <nia@redhat.com>
(cherry picked from commit 7f4387d)

 Conflicts:
	src/pybind/mgr/dashboard/frontend/src/app/ceph/rgw/rgw-user-details/rgw-user-details.component.html
    -  Accepted the current changes and pasted the new change
	src/pybind/mgr/dashboard/frontend/src/app/ceph/rgw/rgw-user-form/rgw-user-form.component.ts
    -  Accepted incoming change and changed const modalRef =
       this.ModalService.show(RgwUserSubuserModalComponent); to const modalRef = this.bsModalService.show(RgwUserSubuserModalComponent);
    -  Made some modification in getUID() function to adapt with octopus
	src/pybind/mgr/dashboard/frontend/src/app/ceph/rgw/rgw-user-list/rgw-user-list.component.ts
    -  Accepted the current change and pasted the new change. Changed
       the $localize to this.i18n to match the octopus way
@nizamial09 nizamial09 added this to the octopus milestone Mar 22, 2021
@nizamial09 nizamial09 requested a review from a team as a code owner March 22, 2021 05:34
@nizamial09 nizamial09 added this to In progress in Dashboard via automation Mar 22, 2021
@avanthakkar avanthakkar added the skip-teuthology For PRs whose changes do not have an effect on QA runs/changes are not being tested in Teuthology label Mar 22, 2021
Dashboard automation moved this from In progress to Reviewer approved Mar 22, 2021
@epuertat epuertat moved this from Reviewer approved to Ready-to-merge in Dashboard Mar 23, 2021
Dashboard automation moved this from Ready-to-merge to Reviewer approved Mar 31, 2021
@nizamial09 nizamial09 moved this from Reviewer approved to Ready-to-merge in Dashboard Apr 1, 2021
@epuertat epuertat merged commit 2a1d2c6 into ceph:octopus Apr 12, 2021
Dashboard automation moved this from Ready-to-merge to Done Apr 12, 2021
@epuertat epuertat deleted the split-tenant-octopus branch April 12, 2021 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dashboard needs-review skip-teuthology For PRs whose changes do not have an effect on QA runs/changes are not being tested in Teuthology
Projects
Archived in project
Dashboard
  
Done
4 participants