[SERVER-4237] Enable the full js test suite to run with authentication enabled Created: 09/Nov/11  Updated: 22/Sep/16  Resolved: 11/Feb/13

Status: Closed
Project: Core Server
Component/s: Security, Testing Infrastructure
Affects Version/s: None
Fix Version/s: 2.4.0-rc1

Type: Task Priority: Major - P3
Reporter: Spencer Brody (Inactive) Assignee: Spencer Brody (Inactive)
Resolution: Done Votes: 0
Labels: authentication, shell, test
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
depends on SERVER-4250 db.currentOp and db.killOp on mongos ... Closed
depends on SERVER-4241 Shell connections don't re-authentica... Closed
depends on SERVER-4245 clone command does not support auth Closed
depends on SERVER-4251 db.killOp doesn't work right on mongo... Closed
depends on SERVER-4252 Auth doesn't work on SyncClusterConne... Closed
depends on SERVER-4256 When running with authorization enabl... Closed
depends on SERVER-4277 Creating new shell while fsync locked... Closed
depends on SERVER-4278 Slave tracking cache is cleaned up to... Closed
depends on SERVER-4280 sharding/sync3.js failure when run wi... Closed
depends on SERVER-4281 isMaster command hangs if db is fsync... Closed
depends on SERVER-6327 Attempting to write to config or admi... Closed
depends on SERVER-7637 Fix jstests/drop2.js when run with auth Closed
depends on SERVER-4254 benchRun doesn't work with authentica... Closed
depends on SERVER-6368 Make ToolTest run with auth when runn... Closed
depends on SERVER-6388 Make tests that use benchRun work whe... Closed
depends on SERVER-7317 Make dur tests pass with auth Closed
depends on SERVER-7356 Make disk tests pass with auth Closed
depends on SERVER-4156 fix read only user support with sharding Closed
depends on SERVER-7357 Clean up how the shell launches proce... Closed
is depended on by SERVER-8191 master1.js in master/slave test suite... Closed
Related
related to SERVER-26249 replsetfreeze.js should wait for repl... Closed
Participants:

 Description   

We should be able to run our entire existing test suite with authentication enabled to ensure that authentication doesn't break any other core functionality.
This ticket encompasses both creating the test infrastructure to be able to run the tests with authentication enabled, as well as getting all the tests passing with authentication, if any bugs are discovered.



 Comments   
Comment by auto [ 08/Feb/13 ]

Author:

{u'date': u'2013-02-08T20:22:22Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: Skip jstests/sharding/read_pref.js when running test with auth SERVER-6972 SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/288df8991ba54a02e034b7a45068acf1972071c9

Comment by auto [ 07/Feb/13 ]

Author:

{u'date': u'2013-01-29T23:34:25Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Make jstests/repl/repl7.js work with auth
Branch: master
https://github.com/mongodb/mongo/commit/f556f67442af30bec0a9f6c10f63f17cfcb53882

Comment by auto [ 24/Jan/13 ]

Author:

{u'date': u'2013-01-24T20:20:04Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 SERVER-6972 Skip jstests/sharding/read_pref_cmd.js when run with auth
Branch: master
https://github.com/mongodb/mongo/commit/4621e405d98606b869421a4bdc7a8aed63b6616e

Comment by auto [ 17/Jan/13 ]

Author:

{u'date': u'2013-01-16T18:59:15Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 SERVER-8191 Enable master/slave replication tests to run with auth
Branch: master
https://github.com/mongodb/mongo/commit/0ad5610b28aaeebab3f3713c517f468fbb6eee17

Comment by auto [ 15/Jan/13 ]

Author:

{u'date': u'2013-01-15T21:12:27Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Add logging to jstests/drop2.js to help investigate auth failure on buildbot
Branch: master
https://github.com/mongodb/mongo/commit/a34cee844d3f61c6bd9e9db053cbd8aade9fb251

Comment by auto [ 14/Jan/13 ]

Author:

{u'date': u'2013-01-14T18:16:22Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Make ReplSetTest.awaitSecondaryNodes reload who slaves are after each attempt
Branch: master
https://github.com/mongodb/mongo/commit/13801770f1633e976854fc0c2a1c75a3a8352c33

Comment by auto [ 11/Jan/13 ]

Author:

{u'date': u'2012-11-27T19:14:32Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Make ReplicaSetTest wait for nodes to stabilize after initialization
Branch: master
https://github.com/mongodb/mongo/commit/03ae388bb6e855a7988fe227dfc84797fbe0129c

Comment by auto [ 24/Dec/12 ]

Author:

{u'date': u'2012-12-23T23:14:31Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Make jstests/shard/read_pref.js work with auth
Branch: master
https://github.com/mongodb/mongo/commit/19c0d71a949273a719e477c920fce95d71602609

Comment by Ian Whalen (Inactive) [ 21/Dec/12 ]

Even with the above changes (as well as https://github.com/mongodb/mongo/commit/fdbf689f653c3dd203c5f57372dc1859d31600bc), sync_passive2.js seems to still be failing:

http://buildbot.mongodb.org/builders/Linux%2064-bit%20debug%20dur%20off/builds/2011
http://buildbot.mongodb.org/builders/FreeBSD%2064-bit/builds/959

Comment by auto [ 20/Dec/12 ]

Author:

{u'date': u'2012-12-20T20:08:37Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Make sync_passive2.js more robust
Branch: master
https://github.com/mongodb/mongo/commit/458796c7cca9f299ee2f23b4dec25162d3235b29

Comment by auto [ 21/Nov/12 ]

Author:

{u'date': u'2012-11-21T21:02:40Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 make jstests/replsets/fsync_lock_read_secondaries.js more robust
Branch: master
https://github.com/mongodb/mongo/commit/bfec1e9b891fd2160eb7b38fc6585d58f4aaaa5a

Comment by auto [ 09/Nov/12 ]

Author:

{u'date': u'2012-11-09T20:50:46Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: Make authenticate actually retry on failure. SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/d6ece0d1c9784d1fd0268cd45f852266b1778cfd

Comment by auto [ 07/Nov/12 ]

Author:

{u'date': u'2012-11-05T20:34:24Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: Make authenticate retry on socket exception SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/bfbc29331726fdd2663bdcb6eeae44dce2bed200

Comment by auto [ 07/Nov/12 ]

Author:

{u'date': u'2012-11-07T01:48:16Z', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 SERVER-6972 Skip gle_with_conf_servers when running tests with auth
Branch: master
https://github.com/mongodb/mongo/commit/57f1be7c72d00eb78c3470f46bfcb9c0e7c0af20

Comment by auto [ 05/Nov/12 ]

Author:

{u'date': u'2012-11-02T19:41:17Z', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}

Message: Try to make ShardingTest more robust. SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/dcc1f2404856344885a3f5b1ef25c40ea3a105fb

Comment by auto [ 23/Oct/12 ]

Author:

{u'date': u'2012-10-19T15:26:21-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: Make ShardingTest add admin users to all its shards when running with auth SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/972c32cd09c6dba54b8976f7b78da22146c0078b

Comment by auto [ 19/Oct/12 ]

Author:

{u'date': u'2012-10-19T10:20:10-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Make sure key file have the correct permissions before running test with auth
Branch: master
https://github.com/mongodb/mongo/commit/fd3f282059ab92907f22038914f034fffcf83dbb

Comment by auto [ 15/Oct/12 ]

Author:

{u'date': u'2012-10-15T08:26:29-07:00', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}

Message: SERVER-7356 SERVER-4237 Skip disk tests when running with auth
Branch: master
https://github.com/mongodb/mongo/commit/3342a6ec9cb3a79794587c8d2927257617fbf60d

Comment by auto [ 10/Oct/12 ]

Author:

{u'date': u'2012-10-10T12:52:14-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-7317 SERVER-4237 Skip dur tests when running with auth
Branch: master
https://github.com/mongodb/mongo/commit/3028cda31d04098d95b70ece6186a883aaa8ff47

Comment by auto [ 05/Oct/12 ]

Author:

{u'date': u'2012-10-05T14:16:26-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Make disk/repair5.js use startMongodTest instead of startMongoProgram so running with auth works
Branch: master
https://github.com/mongodb/mongo/commit/9f6d290ba2800658e2e74f207fdc992a7c0d3737

Comment by auto [ 28/Sep/12 ]

Author:

{u'date': u'2012-09-26T13:09:30-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Make directoryperdb.js use startMongodTest instead of startMongoProgram so running with auth works
Branch: master
https://github.com/mongodb/mongo/commit/f0748473dbc00a197b7a886a5002410d15974d2a

Comment by auto [ 28/Sep/12 ]

Author:

{u'date': u'2012-09-26T13:09:53-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Don't try to auth when restoring directly to data files
Branch: master
https://github.com/mongodb/mongo/commit/736e1b29a68b83c40ed79cabfd9fccbb28047a0e

Comment by auto [ 26/Sep/12 ]

Author:

{u'date': u'2012-09-10T15:19:39-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Make code to skip tests when running with auth work on any platform
Branch: master
https://github.com/mongodb/mongo/commit/ad0cb17b58dfbcaf54c5e77e0ed77e78f303cdf8

Comment by auto [ 10/Sep/12 ]

Author:

{u'date': u'2012-09-10T14:12:48-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Only set db in spawned shell process if db is currently defined
Branch: master
https://github.com/mongodb/mongo/commit/243de0b208795923fd094a7390c0a748d8b5916d

Comment by auto [ 10/Sep/12 ]

Author:

{u'date': u'2012-09-10T13:34:42-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: Put '+' line continuation on previous line to avoid javascript line termination ambiguity. SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/4621f058b4546b3ad41877a784e6158cc4a27c02

Comment by auto [ 10/Sep/12 ]

Author:

{u'date': u'2012-09-07T13:23:07-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-6972 SERVER-4237 Skip sharding/read_pref_rs_client.js when running with auth
Branch: master
https://github.com/mongodb/mongo/commit/bbc53eca04200a1c16d8e202886e4ffe0f595759

Comment by auto [ 10/Sep/12 ]

Author:

{u'date': u'2012-08-15T09:17:42-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: Make tests that use startParallelShell work with auth. SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/a977464bfe226dd466aaef73cc63949394d3e3f8

Comment by auto [ 10/Sep/12 ]

Author:

{u'date': u'2012-09-07T13:23:07-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-6972 SERVER-4237 Skip sharding/read_pref_rs_client.js when running with auth
Branch: parallelShellPlusNoReadPref
https://github.com/mongodb/mongo/commit/bbc53eca04200a1c16d8e202886e4ffe0f595759

Comment by auto [ 10/Sep/12 ]

Author:

{u'date': u'2012-08-15T09:17:42-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: Make tests that use startParallelShell work with auth. SERVER-4237
Branch: parallelShellPlusNoReadPref
https://github.com/mongodb/mongo/commit/a977464bfe226dd466aaef73cc63949394d3e3f8

Comment by auto [ 04/Sep/12 ]

Author:

{u'date': u'2012-08-15T12:44:00-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: Skip client tests when running with auth. SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/91b22ce9bbf990ea4aa67cf7371335d2e9057781

Comment by auto [ 06/Aug/12 ]

Author:

{u'date': u'2012-08-03T11:55:20-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Move auth handling out of gridfs.js and into test infrastructure
Branch: master
https://github.com/mongodb/mongo/commit/8cf562cc844a83849ca6d91f2680761c8f70d739

Comment by auto [ 01/Aug/12 ]

Author:

{u'date': u'2012-07-31T14:26:33-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: Rename repl11.js to repl_auth.js. SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/3589f9c1073becaee72abd5dcdcc8f5e3c3ba68e

Comment by auto [ 01/Aug/12 ]

Author:

{u'date': u'2012-07-27T16:21:57-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: Make shellspawn.js pass with auth. SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/b793369a637d4263b4dd957d559dc08b8b6064d9

Comment by auto [ 01/Aug/12 ]

Author:

{u'date': u'2012-07-27T16:23:57-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Skip tests that use benchRun when running with auth
Branch: master
https://github.com/mongodb/mongo/commit/440ea570396eae6481c8fe1134aaa5a43aea1a9c

Comment by auto [ 30/Jul/12 ]

Author:

{u'date': u'2012-07-30T09:53:02-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Make replsetfreeze.js more stable by freezing the secondary as well
Branch: master
https://github.com/mongodb/mongo/commit/624bc33260275d1b35c19bd8899ec64de113b2de

Comment by auto [ 24/Jul/12 ]

Author:

{u'date': u'2012-07-20T11:38:16-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 make slaveDelay2.js use the standard waitForAllMembers function
Branch: master
https://github.com/mongodb/mongo/commit/6dbeb708c8a5d316d790a59e12513f903e4b6287

Comment by auto [ 24/Jul/12 ]

Author:

{u'date': u'2012-07-20T11:37:26-07:00', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}

Message: SERVER-4237 make waitForAllMembers function retry replSetGetStatus on socket exception
Branch: master
https://github.com/mongodb/mongo/commit/5b3d2178975bdae2ca67df14cd4a6bcd93203d8b

Comment by auto [ 24/Jul/12 ]

Author:

{u'date': u'2012-07-20T11:36:14-07:00', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}

Message: SERVER-4237 Don't mark a connection as authenticated unless the authentication was successful
Branch: master
https://github.com/mongodb/mongo/commit/85044432be8067c764349cfb64d1c595527d9d54

Comment by auto [ 24/Jul/12 ]

Author:

{u'date': u'2012-07-06T14:40:38-07:00', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}

Message: SERVER-4237: Wait longer for addUser to replicate when running tests with auth
Branch: master
https://github.com/mongodb/mongo/commit/55f708333231bd11120e32f9ca364e406f001d99

Comment by auto [ 24/Jul/12 ]

Author:

{u'date': u'2012-07-06T15:18:38-07:00', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}

Message: SERVER-4237 Make reconnect function wait for authentication to succeed
Branch: master
https://github.com/mongodb/mongo/commit/4b497d5a05e4ce80b0cd275e2e53ca82b8411e22

Comment by auto [ 10/Jul/12 ]

Author:

{u'date': u'2012-07-08T17:40:30-07:00', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}

Message: Make shardingtest work with auth when using 3 config servers. SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/ebcc8c1b3924e617127efa7426caa0ea585031df

Comment by auto [ 10/Jul/12 ]

Author:

{u'date': u'2012-07-09T17:56:26-07:00', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}

Message: Don't run tests with auth that we know will fail. SERVER-4254 SERVER-6327 SERVER-6368 SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/781f22c175e41e30ff217aa87fe93f4c478ede38

Comment by auto [ 06/Jul/12 ]

Author:

{u'date': u'2012-07-06T09:05:44-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Don't run client suite with authentication
Branch: master
https://github.com/mongodb/mongo/commit/da0e72a1004a4abd69e21b2acc5153629de8ac2e

Comment by auto [ 06/Jul/12 ]

Author:

{u'date': u'2012-07-05T14:49:52-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Make gridfs.js pass username and password into mongofiles when running tests with auth
Branch: master
https://github.com/mongodb/mongo/commit/14e19a5c502ae28e9db3c61aab3d91c458c2acaf

Comment by auto [ 05/Jul/12 ]

Author:

{u'date': u'2012-07-05T13:33:34-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237: Make drop_configdb.js use ShardingTest
Branch: master
https://github.com/mongodb/mongo/commit/a73395b046808de8dd2f45b0c60ee6d168e827f2

Comment by auto [ 05/Jul/12 ]

Author:

{u'date': u'2012-07-05T08:25:02-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: Only authenticate connections when using authentication. SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/d0aa1e841770adb612bb63482def974ad341a64d

Comment by Eric Milkie [ 05/Jul/12 ]

But the Linux64 weekly test suite failed on an auth phase. Should we really be turning on all the builders?

Comment by auto [ 05/Jul/12 ]

Author:

{u'date': u'2012-07-05T10:40:58-07:00', u'email': u'dcrosta@10gen.com', u'name': u'Dan Crosta'}

Message: SERVER-4237: use a default for --keyFile if only --auth was given to smoke.py
Branch: master
https://github.com/mongodb/mongo/commit/09f05b4e72a82a856f125e14bed7df87d8b8d0af

Comment by Daniel Crosta [ 05/Jul/12 ]

All weeklies will now run the triggered test suites with --auth after running the slowWeekly suite.

Comment by Daniel Crosta [ 04/Jul/12 ]

Enabled these tests on Linux64 weekly test suite. If that works, will then enable on all weeklies.

Comment by auto [ 03/Jul/12 ]

Author:

{u'date': u'2012-07-03T10:27:29-07:00', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}

Message: SERVER-4237 When running tests with auth, skip tests that run with auth normally
Branch: master
https://github.com/mongodb/mongo/commit/e104c3d1fa47383b0bbee7daf188b2668ad3b75d

Comment by auto [ 03/Jul/12 ]

Author:

{u'date': u'2012-07-03T08:57:42-07:00', u'email': u'spencer@10gen.com', u'name': u'Spencer T Brody'}

Message: SERVER-4237 Dont authenticate as __system when running tests with auth
Branch: master
https://github.com/mongodb/mongo/commit/e1f1c959713287df2a6f109ccb441d55d9f048cb

Comment by Spencer Brody (Inactive) [ 24/Feb/12 ]

The tests should run with auth second, after running without auth, so that it doesn't mask other failures while we work on getting the auth tests fully passing.

Also, the tests should be started with "--keyFile jstests/libs/authTestsKey" in addition to --auth.

Comment by auto [ 22/Nov/11 ]

Author:

{u'login': u'stbrody', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}

Message: Skip failing SyncClusterConnection tests when running tests with auth. SERVER-4237
Branch: master
https://github.com/mongodb/mongo/commit/220776dff8d5a1de1ceb5e98691122822fe65c76

Comment by Spencer Brody (Inactive) [ 09/Nov/11 ]

That commit gives the infrastructure to run the tests with authentication. There are a few tests failing that need to be investigated. I think most of them are problems with the tests rather than real bugs, but we need to look into them further to be sure.

Comment by auto [ 09/Nov/11 ]

Author:

{u'login': u'stbrody', u'name': u'Spencer T Brody', u'email': u'spencer@10gen.com'}

Message: Enable running our full js test suite with authentication. SERVER-4237.
Branch: master
https://github.com/mongodb/mongo/commit/f2d426df7031ea8aa85ae967b9165851507dcd31

Generated at Thu Feb 08 03:05:23 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.