Fixes #133

[Enkidu93]

Implemented same logic in ClearML as in S3.

---

This change is 

[codecov-commenter]

Codecov Report

Attention: 8 lines in your changes are missing coverage. Please review.

Files	Coverage Δ
...NetCore/Configuration/IMachineBuilderExtensions.cs	0.00% <0.00%> (ø)
....Machine.AspNetCore/Services/ClearMLHealthCheck.cs	0.00% <0.00%> (ø)

📢 Thoughts on this report? Let us know!

[ddaspit]

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @Enkidu93 and @johnml1135)

---

src/SIL.Machine.AspNetCore/Services/ClearMLHealthCheck.cs line 35 at r1 (raw file):

                    $"No ClearML agents are available for configured queue \"{_options.CurrentValue.Queue}\""
                );
            _numConsecutiveFailures = 0;

You should be locking all access to _numConsecutiveFailures.

[Enkidu93]

Reviewable status: 1 of 3 files reviewed, 1 unresolved discussion (waiting on @ddaspit and @johnml1135)

---

src/SIL.Machine.AspNetCore/Services/ClearMLHealthCheck.cs line 35 at r1 (raw file):

Previously, ddaspit (Damien Daspit) wrote…

You should be locking all access to _numConsecutiveFailures.

Done. I noticed that S3 was missing a lock as well in the same place, so I changed that as well.

[ddaspit]

Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @johnml1135)

[johnml1135]

Reviewed 1 of 2 files at r1, 2 of 2 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Enkidu93)