Skip to content

Conversation

@rodrigovalin
Copy link
Contributor

@rodrigovalin rodrigovalin commented Mar 10, 2021

Summary

The readiness probe understands the replication state of this MongoDB replication member. It will only set the Ready state when the Replication state is PRIMARY or SECONDARY.

All Submissions:

  • Have you opened an Issue before filing this PR?
  • Have you signed our CLA?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

Copy link
Contributor

@chatton chatton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Can we also increase the readiness probe version in release.json (I guess to 1.0.2)

There is no automated process yet, but we want to push the new image here https://quay.io/repository/mongodb/mongodb-kubernetes-readinessprobe

The dockerfile is Dockerfile.readiness

@rodrigovalin rodrigovalin force-pushed the CLOUDP-65923_replication_state_aware_readinessprobe branch from b89c314 to 0d40d6e Compare March 30, 2021 09:33
@rodrigovalin rodrigovalin requested a review from chatton March 30, 2021 10:33
Copy link
Contributor

@chatton chatton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@rodrigovalin rodrigovalin force-pushed the CLOUDP-65923_replication_state_aware_readinessprobe branch from ce6dbe5 to 3749c26 Compare March 31, 2021 13:19
@rodrigovalin rodrigovalin force-pushed the CLOUDP-65923_replication_state_aware_readinessprobe branch from 3749c26 to d50e6bc Compare March 31, 2021 13:21
@rodrigovalin rodrigovalin merged commit a8c7508 into master Mar 31, 2021
@rodrigovalin rodrigovalin deleted the CLOUDP-65923_replication_state_aware_readinessprobe branch March 31, 2021 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants