Skip to content

Commit 4839e52

Browse files
authored
PYTHON-5569: [Build Failure] Spec Resync job is failing silently (#2553)
1 parent e0767cf commit 4839e52

9 files changed

+1099
-82
lines changed

.evergreen/remove-unimplemented-tests.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ rm $PYMONGO/test/index_management/index-rawdata.json
4141
rm $PYMONGO/test/collection_management/modifyCollection-*.json
4242

4343
# PYTHON-5248 - Remove support for MongoDB 4.0
44-
rm $PYMONGO/test/**/pre-42-*.json
44+
find /$PYMONGO /test -type f -name 'pre-42-*.json' -delete
4545

4646
# PYTHON-3359 - Remove Database and Collection level timeout override
4747
rm $PYMONGO/test/csot/override-collection-timeoutMS.json

.evergreen/scripts/resync-all-specs.py

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,14 +30,18 @@ def resync_specs(directory: pathlib.Path, errored: dict[str, str]) -> None:
3030
print("Done syncing specs")
3131

3232

33-
def apply_patches():
33+
def apply_patches(errored):
3434
print("Beginning to apply patches")
3535
subprocess.run(["bash", "./.evergreen/remove-unimplemented-tests.sh"], check=True) # noqa: S603, S607
36-
subprocess.run(
37-
["git apply -R --allow-empty --whitespace=fix ./.evergreen/spec-patch/*"], # noqa: S607
38-
shell=True, # noqa: S602
39-
check=True,
40-
)
36+
try:
37+
subprocess.run(
38+
["git apply -R --allow-empty --whitespace=fix ./.evergreen/spec-patch/*"], # noqa: S607
39+
shell=True, # noqa: S602
40+
check=True,
41+
stderr=subprocess.PIPE,
42+
)
43+
except CalledProcessError as exc:
44+
errored["applying patches"] = exc.stderr
4145

4246

4347
def check_new_spec_directories(directory: pathlib.Path) -> list[str]:
@@ -85,7 +89,7 @@ def write_summary(errored: dict[str, str], new: list[str], filename: Optional[st
8589
pr_body += "\n -".join(succeeded)
8690
pr_body += "\n"
8791
if len(errored) > 0:
88-
pr_body += "\n\nThe following spec syncs encountered errors:\n -"
92+
pr_body += "\n\nThe following spec syncs encountered errors:"
8993
for k, v in errored.items():
9094
pr_body += f"\n -{k}\n```{v}\n```"
9195
pr_body += "\n"
@@ -106,7 +110,7 @@ def main(args: Namespace):
106110
directory = pathlib.Path("./test")
107111
errored: dict[str, str] = {}
108112
resync_specs(directory, errored)
109-
apply_patches()
113+
apply_patches(errored)
110114
new = check_new_spec_directories(directory)
111115
write_summary(errored, new, args.filename)
112116

.evergreen/scripts/resync-all-specs.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
#!/usr/bin/env bash
22
# Run spec syncing script and create PR
3+
set -eu
34

45
# SETUP
56
SRC_URL="https://github.com/mongodb/specifications.git"
Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
diff --git a/test/discovery_and_monitoring/unified/serverMonitoringMode.json b/test/discovery_and_monitoring/unified/serverMonitoringMode.json
2-
index 4b492f7d8..e44fad1bc 100644
2+
index e44fad1b..4b492f7d 100644
33
--- a/test/discovery_and_monitoring/unified/serverMonitoringMode.json
44
+++ b/test/discovery_and_monitoring/unified/serverMonitoringMode.json
5-
@@ -5,8 +5,7 @@
5+
@@ -5,7 +5,8 @@
66
{
77
"topologies": [
88
"single",
9-
+ "sharded"
10-
- "sharded",
11-
- "sharded-replicaset"
9+
- "sharded"
10+
+ "sharded",
11+
+ "sharded-replicaset"
1212
],
1313
"serverless": "forbid"
1414
}

0 commit comments

Comments
 (0)