Commit 0f58ef3
CLOUDP-314916: OIDC e2e test single cluster (#55)
# Summary
### Test File Summary
- **replica_set_oidc_m2m_user.py**
- Tests OIDC machine-to-machine (M2M) user authentication for a replica
set.
- Verifies user creation, role updates, access restrictions, and
negative authentication scenarios.
- **replica_set_oidc_m2m_group.py**
- Tests OIDC group-based (Workload Identity Federation) authentication
in a replica set.
- Covers creation, OIDC provider/role updates, and removal of OIDC
configs and roles.
- **replica_set_oidc_workforce.py**
- Tests OIDC workforce (human identity) authentication in a replica set.
- Validates user creation and correct automation config for multiple
OIDC providers.
- **sharded_cluster_oidc_m2m_group.py**
- Tests OIDC group-based authentication in a sharded cluster.
- Verifies creation, connectivity, provider/role updates, and automation
config state.
- **sharded_cluster_oidc_m2m_user.py**
- Tests OIDC machine-to-machine user authentication in a sharded
cluster, including user creation, role assignment, and
access restrictions.
- Verifies correct OIDC provider configuration, user propagation, and
negative authentication scenarios.
### Additional PR Changes
- **OIDC Callback Integration:**
Introduced a custom OIDC callback handler utilizing AWS Cognito for
token acquisition, allowing automated OIDC authentication in E2E tests.
- **Automation Config Tester Improvements:**
Enhanced assertion helpers to validate OIDC-specific state in the Ops
Manager automation config, including provider counts, configuration
details, and user propagation.
- **New OIDC Fixture Files:**
Added YAML resource definitions for OIDC-enabled replica sets, sharded
clusters, and MongoDB users, supporting a wide range of authentication
and authorization test cases.
- **Core Controller and Logic Adjustments:**
Minor changes in Go controller code to ensure robust handling of OIDC
provider configs and roles, and to support expanded test coverage.
### AWS Setup:
The project uses AWS Cognito in the mongodb-mms-testing AWS account to
facilitate OIDC authentication testing. This setup includes:
- User Pool: A user pool in Cognito manages the identities.
- Users: We use the user credentials to do authentication.
- App Client: An app client is configured for machine-to-machine (M2M)
authentication.
- Groups: Cognito groups are used to manage users from the user pool for
GroupMembership access.
Environment variables and secrets required for these tests (like client
IDs, URLs, and user IDs, as seen in the Python code) are stored in
Evergreen and fetched from there during test execution.
[Link to the session](http://go/k8s-oidc-session) where I explained the
AWS setup for OIDC
---
## Proof of Work
Added tests are passing.
## Checklist
- [x] 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: Maciej Karaś <[email protected]>
Co-authored-by: Lucian Tosa <[email protected]>
Co-authored-by: Lucian Tosa <[email protected]>1 parent f4465ab commit 0f58ef3
File tree
25 files changed
+1037
-4
lines changed- controllers
- om
- operator/authentication
- docker/mongodb-kubernetes-tests
- kubetester
- tests
- authentication
- fixtures/oidc
- webhooks/fixtures
- scripts
- dev/contexts
- evergreen
- deployments/test-app/templates
- e2e
25 files changed
+1037
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
4 | 11 | | |
5 | 12 | | |
6 | 13 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1240 | 1240 | | |
1241 | 1241 | | |
1242 | 1242 | | |
| 1243 | + | |
| 1244 | + | |
| 1245 | + | |
| 1246 | + | |
| 1247 | + | |
| 1248 | + | |
| 1249 | + | |
| 1250 | + | |
| 1251 | + | |
| 1252 | + | |
| 1253 | + | |
| 1254 | + | |
| 1255 | + | |
| 1256 | + | |
| 1257 | + | |
| 1258 | + | |
| 1259 | + | |
| 1260 | + | |
| 1261 | + | |
| 1262 | + | |
| 1263 | + | |
| 1264 | + | |
| 1265 | + | |
| 1266 | + | |
| 1267 | + | |
| 1268 | + | |
1243 | 1269 | | |
1244 | 1270 | | |
1245 | 1271 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
759 | 759 | | |
760 | 760 | | |
761 | 761 | | |
| 762 | + | |
| 763 | + | |
| 764 | + | |
| 765 | + | |
| 766 | + | |
| 767 | + | |
762 | 768 | | |
763 | 769 | | |
764 | 770 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
639 | 639 | | |
640 | 640 | | |
641 | 641 | | |
642 | | - | |
643 | | - | |
| 642 | + | |
| 643 | + | |
644 | 644 | | |
645 | 645 | | |
646 | | - | |
| 646 | + | |
| 647 | + | |
| 648 | + | |
| 649 | + | |
| 650 | + | |
| 651 | + | |
| 652 | + | |
| 653 | + | |
| 654 | + | |
| 655 | + | |
| 656 | + | |
| 657 | + | |
647 | 658 | | |
648 | 659 | | |
649 | 660 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
264 | 264 | | |
265 | 265 | | |
266 | 266 | | |
| 267 | + | |
267 | 268 | | |
268 | 269 | | |
269 | 270 | | |
| |||
Lines changed: 12 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
97 | 97 | | |
98 | 98 | | |
99 | 99 | | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
100 | 112 | | |
101 | 113 | | |
102 | 114 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
404 | 404 | | |
405 | 405 | | |
406 | 406 | | |
| 407 | + | |
| 408 | + | |
| 409 | + | |
| 410 | + | |
| 411 | + | |
| 412 | + | |
| 413 | + | |
| 414 | + | |
| 415 | + | |
| 416 | + | |
| 417 | + | |
| 418 | + | |
| 419 | + | |
| 420 | + | |
| 421 | + | |
| 422 | + | |
| 423 | + | |
| 424 | + | |
| 425 | + | |
| 426 | + | |
| 427 | + | |
| 428 | + | |
| 429 | + | |
| 430 | + | |
| 431 | + | |
| 432 | + | |
| 433 | + | |
| 434 | + | |
| 435 | + | |
| 436 | + | |
| 437 | + | |
| 438 | + | |
| 439 | + | |
| 440 | + | |
| 441 | + | |
407 | 442 | | |
408 | 443 | | |
409 | 444 | | |
| |||
Lines changed: 56 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
| 4 | + | |
4 | 5 | | |
5 | 6 | | |
6 | 7 | | |
| |||
11 | 12 | | |
12 | 13 | | |
13 | 14 | | |
| 15 | + | |
| 16 | + | |
14 | 17 | | |
15 | 18 | | |
16 | 19 | | |
| |||
61 | 64 | | |
62 | 65 | | |
63 | 66 | | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
64 | 79 | | |
65 | 80 | | |
66 | 81 | | |
| |||
277 | 292 | | |
278 | 293 | | |
279 | 294 | | |
| 295 | + | |
| 296 | + | |
| 297 | + | |
| 298 | + | |
| 299 | + | |
| 300 | + | |
| 301 | + | |
| 302 | + | |
| 303 | + | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
| 322 | + | |
| 323 | + | |
| 324 | + | |
| 325 | + | |
| 326 | + | |
| 327 | + | |
| 328 | + | |
| 329 | + | |
| 330 | + | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
280 | 336 | | |
281 | 337 | | |
282 | 338 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
Lines changed: 13 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
0 commit comments