From 2a78a83d079862ccb373744e351314eafa2ce38d Mon Sep 17 00:00:00 2001 From: travisn Date: Fri, 20 Oct 2023 14:57:26 -0600 Subject: [PATCH] Revert "core: use crash profile in crash daemon keyring" This reverts commit fab23d34072dac479a719ce362a448d02e856542. The mgr requires rw access for the cron job that collects the crashes. Signed-off-by: Travis Nielsen (cherry picked from commit a421143dddd31f8786d232a21ddda3de81f5b186) --- pkg/operator/ceph/cluster/nodedaemon/keyring.go | 4 ++-- pkg/operator/ceph/cluster/nodedaemon/keyring_test.go | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/pkg/operator/ceph/cluster/nodedaemon/keyring.go b/pkg/operator/ceph/cluster/nodedaemon/keyring.go index 0472ba3e7..af59694bf 100644 --- a/pkg/operator/ceph/cluster/nodedaemon/keyring.go +++ b/pkg/operator/ceph/cluster/nodedaemon/keyring.go @@ -34,7 +34,7 @@ const ( [client.crash] key = %s caps mon = "allow profile crash" - caps mgr = "allow profile crash" + caps mgr = "allow rw" ` ) @@ -59,7 +59,7 @@ func CreateCrashCollectorSecret(context *clusterd.Context, clusterInfo *client.C func cephCrashCollectorKeyringCaps() []string { return []string{ "mon", "allow profile crash", - "mgr", "allow profile crash", + "mgr", "allow rw", } } diff --git a/pkg/operator/ceph/cluster/nodedaemon/keyring_test.go b/pkg/operator/ceph/cluster/nodedaemon/keyring_test.go index 376c23b60..4dc9f244c 100644 --- a/pkg/operator/ceph/cluster/nodedaemon/keyring_test.go +++ b/pkg/operator/ceph/cluster/nodedaemon/keyring_test.go @@ -24,5 +24,5 @@ import ( func TestCephCrashCollectorKeyringCaps(t *testing.T) { caps := cephCrashCollectorKeyringCaps() - assert.Equal(t, caps, []string{"mon", "allow profile crash", "mgr", "allow profile crash"}) + assert.Equal(t, caps, []string{"mon", "allow profile crash", "mgr", "allow rw"}) }