-
Notifications
You must be signed in to change notification settings - Fork 10
Comparing changes
Open a pull request
base repository: mongodb/mongodb-kubernetes
base: 1.2.0
head repository: mongodb/mongodb-kubernetes
compare: master
- 6 commits
- 62 files changed
- 5 contributors
Commits on Jul 4, 2025
-
Revert change for trimming agents versions (#208)
# Summary Revert the change to only build last 3 OM versions with last 3 Operator versions. We want to build all possible OM versions with last 3 operator versions.
Configuration menu - View commit details
-
Copy full SHA for 6c9d6c9 - Browse repository at this point
Copy the full SHA 6c9d6c9View commit details -
agent release - add traces (#226)
# Summary We sometimes run into errors and having these traced makes it easier to know which ones have failed for later analysis. This pull request enhances the `queue_exception_handling` function in `pipeline.py` by integrating OpenTelemetry tracing and adding detailed metrics to improve observability of task queue error processing. ### Observability Enhancements: * Added OpenTelemetry tracing to `queue_exception_handling` * Introduced metrics to track task queue processing, including: - Total number of tasks (`mck.agent.queue.tasks_total`). - Count of tasks with exceptions (`mck.agent.queue.exceptions_count`). - Success rate of tasks (`mck.agent.queue.success_rate`). - Types of exceptions encountered (`mck.agent.queue.exception_types`). - Boolean flag indicating if exceptions were found (`mck.agent.queue.has_exceptions`) * Updated logging to provide more granular details about exceptions encountered in the task queue ## Proof of Work <!-- Enter your proof that it works here.--> ## Checklist - [ ] Have you linked a jira ticket and/or is the ticket in the title? - [ ] Have you checked whether your jira ticket required DOCSP changes? - [ ] Have you checked for release_note changes? ## Reminder (Please remove this when merging) - Please try to Approve or Reject Changes the PR, keep PRs in review as short as possible - Our Short Guide for PRs: [Link](https://docs.google.com/document/d/1T93KUtdvONq43vfTfUt8l92uo4e4SEEvFbIEKOxGr44/edit?tab=t.0) - Remember the following Communication Standards - use comment prefixes for clarity: * **blocking**: Must be addressed before approval. * **follow-up**: Can be addressed in a later PR or ticket. * **q**: Clarifying question. * **nit**: Non-blocking suggestions. * **note**: Side-note, non-actionable. Example: Praise * --> no prefix is considered a question --------- Co-authored-by: Anand <[email protected]> Co-authored-by: Anand Singh <[email protected]>
Configuration menu - View commit details
-
Copy full SHA for 4aca390 - Browse repository at this point
Copy the full SHA 4aca390View commit details
Commits on Jul 7, 2025
-
CLOUDP-312879: Add search reconciler test (#232)
# Summary This pull request adds unit tests for the `MongoDBSearchReconciler` in the `operator` package. These tests cover various reconciliation scenarios, including success cases, failure cases, and edge cases. Additionally, helper functions and mock setups are added. ### Unit Tests for `MongoDBSearchReconciler`: * **Basic Reconciliation Scenarios:** - Added tests for successful reconciliation (`TestMongoDBSearchReconcile_Success`) and scenarios where the `MongoDBSearch` resource or its source is missing (`TestMongoDBSearchReconcile_NotFound`, `TestMongoDBSearchReconcile_MissingSource`). * **Failure Cases:** - Implemented tests to validate failure conditions, such as unsupported MongoDB versions (`TestMongoDBSearchReconcile_InvalidVersion`), TLS-enabled configurations (`TestMongoDBSearchReconcile_TLSNotSupported`), and multiple `MongoDBSearch` resources referencing the same MongoDB instance (`TestMongoDBSearchReconcile_MultipleSearchResources`). ### Helper Functions and Mock Setup: * **Helper Functions:** - Added helper functions to create mock `MongoDBCommunity` and `MongoDBSearch` resources (`newMongoDBCommunity`, `newMongoDBSearch`) and build expected configurations (`buildExpectedMongotConfig`). ## Proof of Work Tests Pass ## Checklist - [x] Have you linked a jira ticket and/or is the ticket in the title? - [x] Have you checked whether your jira ticket required DOCSP changes? - [x] Have you checked for release_note changes?
Configuration menu - View commit details
-
Copy full SHA for 16e1d78 - Browse repository at this point
Copy the full SHA 16e1d78View commit details
Commits on Jul 8, 2025
-
Fix flakiness in the E2E test
e2e_multi_cluster_replica_set_scale_up
(#231) # Summary The E2E test `e2e_multi_cluster_replica_set_scale_up` has been flaky and @lucian-tosa suggested that we should fix it. It has been failing while waiting for statefulsets (STSs) to have correct number of in case multi cluster mongoDB deployment. And the problem was sometimes after the `MongoDBMultiCluster (mdbmc)` resource got into running phase (that would mean all the STSs are ready), some of the STSs got into not ready state. When we see that `mdbmc` resource is Running we try to make sure that STSs have correct number of replicas but because of above problem (STSs transitioning into not ready state from ready), STS didn't have the correct number of replicas and tests failed. The reason why STS was transitioning into not ready state from ready is, the pod that it was maintaining did the same, i.e., it transitioned from ready state to not ready state. After looking into it further we got to know that the pod is behaving like this because sometimes, it's readiness probe fails momentarily. And because of that the pod gets to ready and then transitioned to not ready (readiness probe failed) and then eventually becomes ready. This is documented in much more detail in the document [here](https://jira.mongodb.org/browse/CLOUDP-329231). The ideal fix of the problem would be to figure out why the readiness probe fails and then fix that. But this PR has the workaround that changes the test slightly to wait for STSs to get correct number of replicas. Jira ticket: https://jira.mongodb.org/browse/CLOUDP-329422 ## Proof of Work Ran the test `e2e_multi_cluster_replica_set_scale_up` manually locally to make sure that it's passing consistently. I am not able to reproduce the flakiness now. ## Checklist - [x] Have you linked a jira ticket and/or is the ticket in the title? - [x] Have you checked whether your jira ticket required DOCSP changes? - [x] Have you checked for release_note changes?
Configuration menu - View commit details
-
Copy full SHA for 5710105 - Browse repository at this point
Copy the full SHA 5710105View commit details
Commits on Jul 9, 2025
-
Configuration menu - View commit details
-
Copy full SHA for 9ed5f98 - Browse repository at this point
Copy the full SHA 9ed5f98View commit details -
Use operator name from env var for webhook setup (#222)
No need to hardcode operator name as we can determine it from the env var we already pass to the operator.
Configuration menu - View commit details
-
Copy full SHA for 2571d04 - Browse repository at this point
Copy the full SHA 2571d04View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff 1.2.0...master