Skip to content

Conversation

@filipcirtog
Copy link
Contributor

@filipcirtog filipcirtog commented Nov 24, 2025

Summary

Resource Management and Configuration Updates:

  • Updated the get_mdbmc helper to set the MongoDB version, configure the resource with the correct cluster spec, and update the associated ConfigMap to use the new external OM base URL, ensuring that MongoDBMulti resources in member clusters use the external address for OM.
  • Adjusted the get_sharded helper to explicitly set the custom MongoDB version.

Test Logic and Assertion Improvements:

  • Added cross cluster support so OM is actually reachable, before we were not actually accessing om and thus weren't able to download the agent
  • Refined test cases to assert the running phase of all relevant resource types (replica sets, multi-cluster, sharded, and standalone) instead of using one that didn't work

Proof of Work

passing https://spruce.mongodb.com/task/mongodb_kubernetes_e2e_operator_race_ubi_with_telemetry_e2e_om_reconcile_race_with_telemetry_patch_c6c02208534df8bed84316b5062db92c8f4e1087_69306528ba7d830007e2495c_25_12_03_16_28_27/logs?execution=0

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 added changelog file?

@github-actions
Copy link

github-actions bot commented Nov 24, 2025

⚠️ (this preview might not be accurate if the PR is not rebased on current master branch)

MCK 1.6.1 Release Notes

Bug Fixes

  • Backed up the agent password in a secret for SCRAM authentication to prevent unnecessary password rotations.

@filipcirtog filipcirtog changed the title Reconcile races test fix Misuse of get_all_rs in E2E Test Nov 24, 2025
@nammn nammn changed the title Misuse of get_all_rs in E2E Test CLOUDP-361352 - fix get_all_rs and related methods in multi_cluster_reconcile_races E2E Test Dec 3, 2025
@nammn nammn added the skip-changelog Use this label in Pull Request to not require new changelog entry file label Dec 3, 2025
@nammn nammn marked this pull request as ready for review December 3, 2025 18:37
@nammn nammn requested a review from a team as a code owner December 3, 2025 18:37
Comment on lines +46 to +47
# Enable external connectivity so member clusters can reach OM
resource["spec"]["externalConnectivity"] = {"type": "LoadBalancer"}
Copy link
Contributor

Choose a reason for hiding this comment

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

This test has Istio enabled, it shouldn't require this.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Why not - do I missunderstand something? This test requires the multi cluster components to be able to access OM, which the latest commit adds to fix this test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Use this label in Pull Request to not require new changelog entry file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants