feat(approval-service): Consider nats connection in readiness probe #7723
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7723 +/- ##
=======================================
Coverage 60.00% 60.01%
=======================================
Files 558 558
Lines 31907 31907
Branches 1590 1590
=======================================
+ Hits 19147 19149 +2
+ Misses 11462 11460 -2
Partials 1298 1298
Flags with carried forward coverage won't be shown. Click here to find out more. |
b2dc8ff
to
6fa446c
Compare
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.
lgtm
approval-service/go.mod
Outdated
@@ -7,7 +7,7 @@ require ( | |||
github.com/google/uuid v1.3.0 | |||
github.com/kelseyhightower/envconfig v1.4.0 | |||
github.com/keptn/go-utils v0.15.1-0.20220512082458-c5e1b7519bec | |||
github.com/keptn/keptn/cp-connector v0.0.0-20220509104843-d040776782e2 | |||
github.com/keptn/keptn/cp-connector v0.0.0-20220512103206-c36faf0071dc |
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.
pls update dependency to the newest
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
7b410c0
to
99f07be
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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.
lgtm
Part of #7613
This PR modifies the readiness probe of the approval service to consider the current state of the control plane connector. The service will now be considered ready, only when it's connected to the controlplane. Until that is the case, previous pods of the approval-service will not be terminated. This will ensure that there is always at least one replica available that is ready to receive events.
Integration test run: https://github.com/keptn/keptn/actions/runs/2313756860