Skip to content

Commit 62ff3fc

Browse files
committed
Fix CI test failures by removing authentication from workflows
- Root cause: Local dev removed auth in b2aac89, but CI workflows still had partial auth setup causing WaitTimeoutError in test_drop_capped_collection and test_drop_database tests - Delete user creation scripts from CI (users.adm.js, users.js) - Update util scripts to skip user creation in rsinit function - Remove adm:pass@ credentials from mongos connections in sh/run - Update e2etests.yml to use connection strings without credentials - Ensures consistency: both CI and local dev now run without authentication
1 parent b2aac89 commit 62ff3fc

File tree

8 files changed

+10
-55
lines changed

8 files changed

+10
-55
lines changed

.github/workflows/e2etests.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@ jobs:
4646
4747
- name: Run tests (pytest)
4848
run: |
49-
export TEST_SOURCE_URI=mongodb://adm:pass@rs00:30000
50-
export TEST_TARGET_URI=mongodb://adm:pass@rs10:30100
49+
export TEST_SOURCE_URI=mongodb://rs00:30000
50+
export TEST_TARGET_URI=mongodb://rs10:30100
5151
export TEST_PCSM_URL=http://127.0.0.1:2242
5252
export TEST_PCSM_BIN=./bin/pcsm_test
5353
@@ -96,8 +96,8 @@ jobs:
9696
9797
- name: Run tests (pytest)
9898
run: |
99-
export TEST_SOURCE_URI=mongodb://adm:pass@src-mongos:27017
100-
export TEST_TARGET_URI=mongodb://adm:pass@tgt-mongos:29017
99+
export TEST_SOURCE_URI=mongodb://src-mongos:27017
100+
export TEST_TARGET_URI=mongodb://tgt-mongos:29017
101101
export TEST_PCSM_URL=http://127.0.0.1:2242
102102
export TEST_PCSM_BIN=./bin/pcsm_test
103103

.github/workflows/rs/mongo/users.adm.js

Lines changed: 0 additions & 5 deletions
This file was deleted.

.github/workflows/rs/mongo/users.js

Lines changed: 0 additions & 11 deletions
This file was deleted.

.github/workflows/rs/util

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,5 @@ rsinit() {
5151
echo "uninitialized primary: $1"
5252
exit 1
5353
fi
54-
msh "$2" "/cfg/users.adm.js"
55-
msh "adm:pass@$2" "/cfg/users.js"
56-
msh "adm:pass@$2" "/cfg/scripts/deprioritize.js"
54+
msh "$2" "/cfg/scripts/deprioritize.js"
5755
}

.github/workflows/sh/mongo/users.adm.js

Lines changed: 0 additions & 5 deletions
This file was deleted.

.github/workflows/sh/mongo/users.js

Lines changed: 0 additions & 20 deletions
This file was deleted.

.github/workflows/sh/run

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,11 @@ mwait "src-rs00:30000" && rsinit "src/rs0" "src-rs00:30000"
1313
mwait "src-rs10:30100" && rsinit "src/rs1" "src-rs10:30100"
1414

1515
dcf up -d src-mongos && mwait "src-mongos:27017"
16-
msh "adm:pass@src-mongos:27017" --eval "
16+
msh "src-mongos:27017" --eval "
1717
sh.addShard('rs0/src-rs00:30000'); //
1818
sh.addShard('rs1/src-rs10:30100');
1919
"
20-
msh "adm:pass@src-mongos:27017" --eval "db.adminCommand('transitionFromDedicatedConfigServer');"
20+
msh "src-mongos:27017" --eval "db.adminCommand('transitionFromDedicatedConfigServer');"
2121

2222
dcf up -d tgt-cfg0 tgt-rs00 tgt-rs10
2323

@@ -26,8 +26,8 @@ mwait "tgt-rs00:40000" && rsinit "tgt/rs0" "tgt-rs00:40000"
2626
mwait "tgt-rs10:40100" && rsinit "tgt/rs1" "tgt-rs10:40100"
2727

2828
dcf up -d tgt-mongos && mwait "tgt-mongos:27017"
29-
msh "adm:pass@tgt-mongos:27017" --eval "
29+
msh "tgt-mongos:27017" --eval "
3030
sh.addShard('rs0/tgt-rs00:40000'); //
3131
sh.addShard('rs1/tgt-rs10:40100');
3232
"
33-
msh "adm:pass@tgt-mongos:27017" --eval "db.adminCommand('transitionFromDedicatedConfigServer');"
33+
msh "tgt-mongos:27017" --eval "db.adminCommand('transitionFromDedicatedConfigServer');"

.github/workflows/sh/util

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,5 @@ rsinit() {
5555
echo "uninitialized primary: $1"
5656
exit 1
5757
fi
58-
msh "$2" "/cfg/users.adm.js"
59-
msh "adm:pass@$2" "/cfg/users.js"
60-
msh "adm:pass@$2" "/cfg/scripts/deprioritize.js"
58+
msh "$2" "/cfg/scripts/deprioritize.js"
6159
}

0 commit comments

Comments
 (0)