[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: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| 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. |
| 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 |
| 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: |
| 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: |
| 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: |
| 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: |
| 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: |
| 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: |
| 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: |
| 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 |
| 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: |
| 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: |
| 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. |
| 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 |
| 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: |
| 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. |
| 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 |
| 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: |
| 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: |
| 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: |
| 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: |
| 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: |
| 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: |
| 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: |
| 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: |
| 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. |
| 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: |
| 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. |
| 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: |
| 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. |
| 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. |
| 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: |
| 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. |
| 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. |
| 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: |
| 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: |
| 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: |
| 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: |
| 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: |
| 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: |
| 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: |
| 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. |
| 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. |
| 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: |
| 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: |
| 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: |
| 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. |
| 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: |
| 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: |
| 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: |
| 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. |
| 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. |