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

mgr: adding CEPH_ARGS to the mgr pod so radosgw-admin can use it #13256

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

rkachach
Copy link
Contributor

@rkachach rkachach commented Nov 22, 2023

Ceph dashboard uses radosgw-admin for certain tasks that aren't accessible via the rgw REST API. Due to the absence of a valid ceph.conf file at /etc/ceph/ceph.conf within the mgr pod, radosgw-admin fails to operate, resulting in 500 errors across various 'Object Gateway' views on the dashboard. This change adds CEPH_ARGS environment variable to the mgr pod enabling its propagation and utilization by the dashboard/radosgw-admin for executing rgw commands.

closes: #13255

Testing:

Create a new rook cluster and make sure Dashboard views work correcly: rgw items information is listed without errors. Make sure envs are available on the mgr pod and the radosgw-admin works without issues:

[root@rook-ceph-mgr-a-8449d46459-9ntft ceph]# env | grep -i ceph_args
CEPH_ARGS=-m [v2:10.105.33.195:3300,v1:10.105.33.195:6789] -k /etc/ceph/keyring-store/keyring

[root@rook-ceph-mgr-a-8449d46459-9ntft ceph]# radosgw-admin -n mgr.a realm list
{
    "default_info": "f03de77f-9f7a-4acd-92d9-9615d78ccd11",
    "realms": [
        "my-store"
    ]
}

Which issue is resolved by this Pull Request:
Resolves #

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
  • Reviewed the developer guide on Submitting a Pull Request
  • Pending release notes updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

@rkachach rkachach force-pushed the fix_issue_radosgw_admin branch 2 times, most recently from 31a94f0 to b4e8fa8 Compare November 22, 2023 17:45
@rkachach rkachach changed the title mgr: adding CEPH_ARGS to the mgr pod radosgw-admin can access use it mgr: adding CEPH_ARGS to the mgr pod so radosgw-admin can access use it Nov 22, 2023
@rkachach rkachach changed the title mgr: adding CEPH_ARGS to the mgr pod so radosgw-admin can access use it mgr: adding CEPH_ARGS to the mgr pod so radosgw-admin can use it Nov 22, 2023
@rkachach rkachach marked this pull request as ready for review November 22, 2023 18:22
ceph dashboard uses radosgw-admin for certain tasks that
aren't accessible via the rgw REST API. Due to the absence of
a valid ceph.conf file at /etc/ceph/ceph.conf within the mgr pod,
radosgw-admin fails to operate, resulting in 500 errors across
various 'Object Gateway' views on the dashboard. This change
adds CEPH_ARGS environment variable to the mgr pod enabling
its propagation and utilization by the dashboard/radosgw-admin
for executing rgw commands.

closes: rook#13255

Signed-off-by: Redouane Kachach <rkachach@redhat.com>
@travisn travisn merged commit 49c4163 into rook:master Nov 27, 2023
51 checks passed
mergify bot added a commit that referenced this pull request Nov 27, 2023
mgr: adding CEPH_ARGS to the mgr pod so radosgw-admin can use it (backport #13256)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

radosgw-admin fails when run from ceph-mgr pod as it cannot connect to ceph monitors
5 participants