Skip to content

DRIVERS-3390 Add appName to OIDC test failpoints#1891

Merged
kevinAlbs merged 3 commits intomongodb:masterfrom
kevinAlbs:add-appName.DRIVERS-3197
Feb 6, 2026
Merged

DRIVERS-3390 Add appName to OIDC test failpoints#1891
kevinAlbs merged 3 commits intomongodb:masterfrom
kevinAlbs:add-appName.DRIVERS-3197

Conversation

@kevinAlbs
Copy link
Contributor

Summary

Add appName to isolate failpoints in OIDC tests.

Background & Motivation

This is intended to address test failures observed in C (example) and Rust (example), and maybe others drivers. Sometimes the GCP OIDC tests fail to trigger an expected failpoint.

Running repeatedly in the C driver reproduced the failure on a test configuring a failpoint. This patch build logs:

... "Command started" ... "configureFailPoint" ... \"failCommands\" : [ \"find\" ]
... "Command succeeded" ... "configureFailPoint" ...
... "Command started" ...  "find" ...
... "Command succeeded" ...  "find" ...

The "find" did not expectedly fail.

Diagnosing

Using mongosh to connect to an Atlas cluster created by the OIDC GCP scripts enabled testing a failpoint in the same environment:

db.adminCommand({"configureFailPoint": "failCommand", "mode": { "times": 2 }, "data": {"failCommands": [ "find" ], "errorCode": 391 }});
db.runCommand({find: "coll", filter: {}})

This usually throws the expected exception. However, if paused between configuring the failpoint and running the find command, the find sometimes surprisingly succeeds.

Printing server logs indicate the automation agent is connecting:

db.adminCommand({ getLog: "global" }).log.slice(-10).forEach((msg) => print(msg));
// Sometimes shows connection from "MongoDB Automation Agent".

So I expect another process may be triggering the failpoint.

Testing

With the appName added, a test of the C driver running on repeat 20 times passes:
Evergreen patch: https://spruce.mongodb.com/version/6983ff9b967f4f00073726da


Please complete the following before merging:

  • Is the relevant DRIVERS ticket in the PR title?
  • [ ] Update changelog. N/A. Test changes only.
  • Test changes in at least one language driver. Tested in C with 20x repeat in this patch
  • [ ] Test these changes against all server versions and topologies (including standalone, replica set, and sharded
    clusters).
    N/A. Tested OIDC tasks.

@kevinAlbs kevinAlbs requested a review from blink1073 February 5, 2026 15:36
@kevinAlbs kevinAlbs marked this pull request as ready for review February 5, 2026 15:36
@kevinAlbs kevinAlbs requested a review from a team as a code owner February 5, 2026 15:36
@blink1073
Copy link
Member

blink1073 commented Feb 5, 2026

Testing with pymongo in this patch, and PR.

Copy link
Member

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

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

LGTM!

@kevinAlbs kevinAlbs merged commit 4d11668 into mongodb:master Feb 6, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants