fix(server): app initialization starts metrics after multiregion

[iainsproat]

Description & motivation

The knex & high frequency metrics were initialized before the multiregion module. These metrics query clients related to multi-region databases. When getting the clients, they will be initialized if they are not already. This initialization process starts the database migration for the multiregion database.

As metrics runs concurrently, the error would throw at a random time during the startup process and was unrelated to the log messages immediately prior which was confusing.

To avoid confusion the multiregion database migrations should remain the responsibility of the multiregion module.

The application may ultimately run into the same error, but it is then clearer as to what module has encountered the error and for what purpose.

Changes:

• Prometheus metrics are initialized after the modules have run.

To-do before merge:

Screenshots:

Validation of changes:

• validated by running locally in multi-region mode, and querying the /metrics endpoint.

Checklist:

• My pull request follows the guidelines in the Contributing guide?
• My pull request does not duplicate any other open Pull Requests for the same update/change?
• My commits are related to the pull request and do not amend unrelated code or documentation.
• My code follows a similar style to existing code.
• I have added appropriate tests.
• I have updated or added relevant documentation.

References