pubsub/rabbitmq: return error if reconnect errors in Init

We should always return the error if we fail to connection during the
Init of a Component. This is because failing to do so has downstream
effects whereby a component is considered healthy when it is not.

Re-attempting an Init or otherwise whould be handled by the runtime,
rather than on a per-component basis.

Signed-off-by: joshvanl <me@joshvanl.dev>

pubsub/rabbitmq/rabbitmq.go

@@ -154,10 +154,10 @@ func (r *rabbitMQ) Init(_ context.Context, metadata pubsub.Metadata) error {
 
 	r.metadata = meta
 
-	r.reconnect(0)
-	// We do not return error on reconnect because it can cause problems if init() happens
-	// right at the restart window for service. So, we try it now but there is logic in the
-	// code to reconnect as many times as needed.
+	if err := r.reconnect(0); err != nil {
+		return err
+	}
+
 	return nil
 }