MGMT-16157 Remove recover usage from DB transactions (#5795)

The downside of recover is that it may hide bugs like accessing nil
pointer and can create a situation where a function panic and instead of
returning an error it will return nil, in some cases it cause a hot loop
in our controllers, or just move a bug to an upper later.

To avoid those situations we decided to remove the recover function
usage in the transaction code and move to a new transaction style that
will make scope of the trnasaction and handling of errors clearer.

Fix unit tests

Remove CancelInstallCommitFailedEvent event, it doesn't help the user in any way