aneoconsulting / ArmoniK.Core

Task manager for ArmoniK (submission, scheduling, IO data, monitoring). Implements API located in https://github.com/aneoconsulting/ArmoniK.Api
https://aneoconsulting.github.io/ArmoniK.Core/
GNU Affero General Public License v3.0
10 stars 8 forks source link

fix: properly initialize rabbit health check #786

Closed aneojgurhem closed 2 days ago

aneojgurhem commented 2 days ago

Motivation

Properly initialize rabbit so that its health check does not return Unhealthy all the time preventing the container from starting when there is a health check set up.

Description

Variable isInitialized_ was not set to true, forcing the health check to be false.

This variable was not needed as it is enough to check if the model (represents the underlying connection) is not null. When the model is null, the plugin is not initialized yet. This pr replace the variable by the proper checks on the model.

Testing

Health checks added by #785 were used to validate that this development properly initialises the health check.

Impact

RabbitMQ plugin initialization now works, allowing the usage of the plugin.

Checklist