Skip to content

Commit

Permalink
rgw: raise errors when rgw daemon fails to be created
Browse files Browse the repository at this point in the history
The generation of the rgw deployment spec was swallowing errors
if any issues are raised such as the tls cert not being found
as expected in some configurations. We need to fail the reconcile
so the error will be logged and the admin can identify the issue.

Signed-off-by: Travis Nielsen <tnielsen@redhat.com>
(cherry picked from commit 9ecd0cb)
  • Loading branch information
travisn authored and mergify-bot committed Nov 10, 2021
1 parent 396ef3b commit 4a9491f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/operator/ceph/object/rgw.go
Expand Up @@ -142,7 +142,7 @@ func (c *clusterConfig) startRGWPods(realmName, zoneGroupName, zoneName string)
// Create deployment
deployment, err := c.createDeployment(rgwConfig)
if err != nil {
return nil
return errors.Wrap(err, "failed to create rgw deployment")
}
logger.Infof("object store %q deployment %q created", c.store.Name, deployment.Name)

Expand Down

0 comments on commit 4a9491f

Please sign in to comment.