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 subhamkrai committed Jan 5, 2022
1 parent 210a8a4 commit edad3d8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/operator/ceph/object/rgw.go
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,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 started", c.store.Name, deployment.Name)

Expand Down

0 comments on commit edad3d8

Please sign in to comment.