Skip to content

Search: support multiple mongots#817

Draft
lsierant wants to merge 23 commits intosearch/sharded-clusterfrom
search/multiple-mongot
Draft

Search: support multiple mongots#817
lsierant wants to merge 23 commits intosearch/sharded-clusterfrom
search/multiple-mongot

Conversation

@lsierant
Copy link
Contributor

@lsierant lsierant commented Feb 20, 2026 •
edited
Loading

Based on PR #816

Chain of upstream PRs as of 2026-03-03

Summary

Proof of Work

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?

@lsierant Graphite App
Copy link
Contributor Author

lsierant commented Feb 20, 2026 •
edited by Julien-Ben
Loading

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@lsierant lsierant mentioned this pull request Feb 20, 2026
Open
@lsierant lsierant force-pushed the search/sharded-cluster branch from a917d32 to 915b024 Compare February 20, 2026 10:52
@lsierant lsierant force-pushed the search/multiple-mongot branch 2 times, most recently from 28b7edd to ceaced6 Compare February 20, 2026 10:58
This was referenced Feb 20, 2026
Draft
Closed
@lsierant lsierant force-pushed the search/sharded-cluster branch from f449329 to 29a42da Compare February 20, 2026 12:59
@lsierant lsierant force-pushed the search/multiple-mongot branch 2 times, most recently from 4ae9bfc to 2cedafd Compare February 20, 2026 14:15
@lsierant lsierant mentioned this pull request Feb 20, 2026
Closed
@lsierant lsierant changed the base branch from search/sharded-cluster to graphite-base/817 February 20, 2026 15:47
@lsierant lsierant force-pushed the search/multiple-mongot branch from 2cedafd to f89d675 Compare February 21, 2026 09:02
@lsierant lsierant changed the base branch from graphite-base/817 to search/sharded-cluster February 21, 2026 09:03
@lsierant lsierant force-pushed the search/multiple-mongot branch from f89d675 to f599344 Compare February 21, 2026 09:13
@lsierant lsierant mentioned this pull request Feb 23, 2026
Merged
@lsierant lsierant changed the base branch from search/sharded-cluster to graphite-base/817 February 23, 2026 12:25
@lsierant lsierant force-pushed the search/multiple-mongot branch from f599344 to b823e40 Compare February 23, 2026 20:47
@lsierant lsierant changed the base branch from graphite-base/817 to search/sharded-cluster February 23, 2026 20:47
@lsierant lsierant mentioned this pull request Feb 23, 2026
Merged
@viveksinghggits viveksinghggits changed the base branch from search/sharded-cluster to graphite-base/817 February 25, 2026 18:44
This was referenced Feb 25, 2026
Merged
Merged
@lsierant lsierant force-pushed the search/multiple-mongot branch from 77d6c13 to b292b51 Compare February 26, 2026 12:23
@lsierant lsierant changed the base branch from graphite-base/817 to search/sharded-cluster February 26, 2026 12:23
@Julien-Ben Julien-Ben mentioned this pull request Feb 26, 2026
Merged
3 tasks
lsierant added a commit that referenced this pull request Mar 3, 2026
@lsierant
Revert cluster index from replicaset and fix e2e tests (#853)
21e7c0a 
<!-- start git-machete generated -->

# Based on PR #817

## Chain of upstream PRs as of 2026-03-03

* PR #806:
  `master` ← `search/base`

  * PR #816:
    `search/base` ← `search/sharded-cluster`

    * PR #817:
      `search/sharded-cluster` ← `search/multiple-mongot`

      * **PR #853 (THIS ONE)**:
`search/multiple-mongot` ← `search/lsierant/revert-rs-cluster-index`

<!-- end git-machete generated -->
@lsierant lsierant force-pushed the search/sharded-cluster branch from 4c82ef8 to f36f5bb Compare March 3, 2026 21:26
lsierant added a commit that referenced this pull request Mar 3, 2026
@lsierant
Revert cluster index from replicaset and fix e2e tests (#853)
70b4fe1 
<!-- start git-machete generated -->

# Based on PR #817

## Chain of upstream PRs as of 2026-03-03

* PR #806:
  `master` ← `search/base`

  * PR #816:
    `search/base` ← `search/sharded-cluster`

    * PR #817:
      `search/sharded-cluster` ← `search/multiple-mongot`

      * **PR #853 (THIS ONE)**:
`search/multiple-mongot` ← `search/lsierant/revert-rs-cluster-index`

<!-- end git-machete generated -->
@lsierant lsierant force-pushed the search/multiple-mongot branch from 21e7c0a to 70b4fe1 Compare March 3, 2026 21:31
viveksinghggits
viveksinghggits reviewed Mar 6, 2026
View reviewed changes
...ongodb-kubernetes-tests/tests/search/search_sharded_enterprise_external_mongod_managed_lb.py
Copy link
Collaborator

@viveksinghggits viveksinghggits Mar 6, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

@Julien-Ben Some of the things from the test file can be improved now that we have extracted some of the code.

@lsierant lsierant force-pushed the search/sharded-cluster branch from f36f5bb to f36ca1e Compare March 6, 2026 13:30
lsierant and others added 23 commits March 6, 2026 14:30
@lsierant
Search: support multiple mongots
5e91b01 
# Conflicts:
#	controllers/operator/mongodbshardedcluster_controller.go
#	controllers/searchcontroller/enterprise_search_source_test.go
@lsierant
Sharded multiple mongot: review fixes (#827)
a76c57a 
Fixes to search with multiple-mongot
# Conflicts:
#	controllers/searchcontroller/mongodbsearch_reconcile_helper.go
@Julien-Ben @lsierant
E2E tests
abf5385
@Julien-Ben @lsierant
Updates to existing controllers
14a119d
@Julien-Ben @lsierant
New controller
c953203
@Julien-Ben @lsierant
Types
7d2ece8
@Julien-Ben @lsierant
Add deployments permission
a08ce46
@Julien-Ben @lsierant
Fix existing tests
d5ce403
@Julien-Ben @lsierant
Fix method call
d936763
@Julien-Ben @lsierant
Lint
a808254
@Julien-Ben @lsierant
Lint
5689e77
@lsierant
Aligned naming conventions for all resources
0e4ac0b
@lsierant
Fixing e2e tests for naming alignment
039be37
@lsierant
Fix e2e search test naming alignment issues
51371f7 
- Fix try_load() by removing the FIXME early-return that was causing
  get_ops_manager() to return unloaded resources (empty status), resulting
  in empty baseUrl in OpsManager connection ConfigMaps on om80 variant
- Fix endpoint template in search-sharded-external-lb.yaml
- Fix username mismatch in enterprise_tls sample_movies_helper
- Fix CRD field name sharded→shardedCluster in sharded_enterprise_external_mongod
- Fix CA configmap and mongodbResourceRef overrides in sharded_enterprise_managed_lb
- Fix username mismatches in sharded_external_mongod_single_mongot
@lsierant
Rebase and deployment permission fix
621fe24
@lsierant
Added new variant for large search tests
7b511c6
@lsierant
Split tests to large variants
33281c5
@lsierant
Fix context files
a01265b
@lsierant
Revert cluster index from replicaset and fix e2e tests (#853)
a35bc83 
<!-- start git-machete generated -->

# Based on PR #817

## Chain of upstream PRs as of 2026-03-03

* PR #806:
  `master` ← `search/base`

  * PR #816:
    `search/base` ← `search/sharded-cluster`

    * PR #817:
      `search/sharded-cluster` ← `search/multiple-mongot`

      * **PR #853 (THIS ONE)**:
`search/multiple-mongot` ← `search/lsierant/revert-rs-cluster-index`

<!-- end git-machete generated -->
@viveksinghggits @lsierant
Deduplicate the code between the search test files (#846)
c9a714f
@viveksinghggits @lsierant
Move envoy related utils to a separate file (#861)
81f9f50
@viveksinghggits @lsierant
Create helper to move MongoDB related utils, to make them usable acro…
74a903e 
…ss tests (#863)

# Summary

<!-- Enter your PR summary here. Try to emphasize on WHY this change is
needed, followed by what's being done in the PR. -->

## 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 added changelog file?
    - use `skip-changelog` label if not needed
- refer to [Changelog files and Release
Notes](https://github.com/mongodb/mongodb-kubernetes/blob/master/CONTRIBUTING.md#changelog-files-and-release-notes)
section in CONTRIBUTING.md for more details
@Julien-Ben @lsierant
Managed Proxy Improvements (#849)
32a6973 
# Summary

Improve managed Envoy proxy: replace custom config types with go-control-plane, support external mongod sources, and add CRD configurability

Replace hand-rolled Envoy config structs with the official envoyproxy/go-control-plane protobuf types for type-safe JSON config generation 

Refactor the Envoy controller to share database source resolution with the main search controller via getSearchSource, enabling managed LB support for external mongod sharded sources. Add CRD fields for Envoy image and resource requirements overrides, proper security contexts, and a new e2e test for sharded enterprise with external mongod managed LB.
@lsierant lsierant force-pushed the search/multiple-mongot branch from 5ba63f4 to 32a6973 Compare March 6, 2026 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@viveksinghggits viveksinghggits viveksinghggits left review comments

@Julien-Ben Julien-Ben Awaiting requested review from Julien-Ben

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lsierant @viveksinghggits @Julien-Ben