New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #18306: Create healthcheck API #3249
Fixes #18306: Create healthcheck API #3249
Conversation
...udder-core/src/main/scala/com/normation/rudder/services/healthcheck/HealthcheckService.scala
Outdated
Show resolved
Hide resolved
847ef8f
to
2f7851c
Compare
Commit modified |
...udder-core/src/main/scala/com/normation/rudder/services/healthcheck/HealthcheckService.scala
Outdated
Show resolved
Hide resolved
...udder-core/src/main/scala/com/normation/rudder/services/healthcheck/HealthcheckService.scala
Outdated
Show resolved
Hide resolved
...udder-core/src/main/scala/com/normation/rudder/services/healthcheck/HealthcheckService.scala
Outdated
Show resolved
Hide resolved
...udder-core/src/main/scala/com/normation/rudder/services/healthcheck/HealthcheckService.scala
Outdated
Show resolved
Hide resolved
...ources/rudder/rudder-rest/src/main/scala/com/normation/rudder/rest/lift/HealthcheckApi.scala
Outdated
Show resolved
Hide resolved
...udder-core/src/main/scala/com/normation/rudder/services/healthcheck/HealthcheckService.scala
Outdated
Show resolved
Hide resolved
2f7851c
to
10b76e4
Compare
Commit modified |
10b76e4
to
556625d
Compare
Commit modified |
...udder-core/src/main/scala/com/normation/rudder/services/healthcheck/HealthcheckService.scala
Outdated
Show resolved
Hide resolved
...udder-core/src/main/scala/com/normation/rudder/services/healthcheck/HealthcheckService.scala
Show resolved
Hide resolved
...udder-core/src/main/scala/com/normation/rudder/services/healthcheck/HealthcheckService.scala
Outdated
Show resolved
Hide resolved
...sources/rudder/rudder-rest/src/main/scala/com/normation/rudder/rest/RestDataSerializer.scala
Outdated
Show resolved
Hide resolved
556625d
to
31b40ec
Compare
Commit modified |
31b40ec
to
e21c080
Compare
Commit modified |
...ources/rudder/rudder-rest/src/main/scala/com/normation/rudder/rest/EndpointsDefinition.scala
Outdated
Show resolved
Hide resolved
e21c080
to
d7de582
Compare
Commit modified |
availableCores <- IOResult.effect(getRuntime.availableProcessors) | ||
} yield { | ||
availableCores match { | ||
case 0 => Critical("No Core available") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it should really be case i if i <= 0 =>
because 0 core is an error (it can't be otherwise), and in case of error, you can't, make any assumption about what will be returned beside what the type contraints (an error is often a negative number in C-like semantic).
d7de582
to
b8823dc
Compare
Commit modified |
https://issues.rudder.io/issues/18306