-
Notifications
You must be signed in to change notification settings - Fork 9
CLOUDP-314916: e2e OIDC multicluster tests #155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…-crd-propagation # Conflicts: # controllers/operator/authentication/authentication_mechanism.go
evergreen patch |
evergreen retry |
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.
Pull Request Overview
Adds end-to-end tests and fixtures to validate multi-cluster OIDC machine-to-machine authentication, updates sharded-cluster tests for multi-cluster support, and integrates new test tasks into Evergreen CI.
- New E2E test modules for multi-cluster OIDC user and group workflows
- Supporting YAML fixtures for multi-cluster OIDC scenarios
- Sharded-cluster test updates: multi-cluster logic, service name handling, and increased timeouts
- CI configuration (.evergreen.yml & .evergreen-tasks.yml) updated to run the new tests
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
docs/mongodbcommunity/users.md | Removed stray blank line |
docker/mongodb-kubernetes-tests/tests/multicluster/multi_cluster_oidc_m2m_user.py | Added multi-cluster OIDC M2M user tests |
docker/mongodb-kubernetes-tests/tests/multicluster/multi_cluster_oidc_m2m_group.py | Added multi-cluster OIDC M2M group tests |
docker/mongodb-kubernetes-tests/tests/multicluster/fixtures/oidc/oidc-user-multi.yaml | New fixture for OIDC M2M user |
docker/mongodb-kubernetes-tests/tests/multicluster/fixtures/oidc/mongodb-multi-m2m-user.yaml | New multi-cluster MongoDBMultiCluster fixture for user tests |
docker/mongodb-kubernetes-tests/tests/multicluster/fixtures/oidc/mongodb-multi-m2m-group.yaml | New multi-cluster MongoDBMultiCluster fixture for group tests |
docker/mongodb-kubernetes-tests/tests/authentication/sharded_cluster_oidc_m2m_user.py | Updated sharded-cluster OIDC user tests for multi-cluster adaptations |
docker/mongodb-kubernetes-tests/tests/authentication/sharded_cluster_oidc_m2m_group.py | Updated sharded-cluster OIDC group tests with multi-cluster logic |
.evergreen.yml | Added new E2E test tasks to CI task group |
.evergreen-tasks.yml | Registered new E2E test tasks |
Comments suppressed due to low confidence (1)
docker/mongodb-kubernetes-tests/tests/authentication/sharded_cluster_oidc_m2m_group.py:93
- The skip_if_multi_cluster decorator should be referenced without parentheses (i.e. @skip_if_multi_cluster) to avoid calling the function at import time.
@skip_if_multi_cluster()
docker/mongodb-kubernetes-tests/tests/multicluster/multi_cluster_oidc_m2m_user.py
Outdated
Show resolved
Hide resolved
docker/mongodb-kubernetes-tests/tests/multicluster/multi_cluster_oidc_m2m_user.py
Outdated
Show resolved
Hide resolved
docker/mongodb-kubernetes-tests/tests/multicluster/multi_cluster_oidc_m2m_group.py
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <[email protected]>
docker/mongodb-kubernetes-tests/tests/authentication/sharded_cluster_oidc_m2m_group.py
Show resolved
Hide resolved
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.
Very cool that you made hybrid tests, working in both single and multi cluster variants !
Summary
This pull request adds new (E2E) tests for multi-cluster OIDC M2M authentication scenarios:
Proof of Work
Passing e2e tests.
Checklist
Reminder (Please remove this when merging)