History log of /6.6.0/kv_engine/engines/ep/tests/ep_testsuite_common.cc (Results 1 - 25 of 69)
Revision (<<< Hide revision tags) (Show revision tags >>>) Date Author Comments
Revision tags: v7.0.2, v6.6.3, v7.0.1, v7.0.0, v6.6.2, v6.5.2, v6.6.1, v6.0.5, v6.6.0, v6.5.1
# b8209f09 04-Feb-2020 Dave Rigby <daver@couchbase.com>

MB-37096: Debug CRT: Reduce ep_testsuite default #vbuckets to 4

ep_testsuite tests run quite slowly under Windows Debug CRT, frequently
hitting the test suite timeout when run on CV mach

MB-37096: Debug CRT: Reduce ep_testsuite default #vbuckets to 4

ep_testsuite tests run quite slowly under Windows Debug CRT, frequently
hitting the test suite timeout when run on CV machines.

Profiling highlights that one reason for this is initializing and then
destroying each engine instance, specifically data for all 1024 vBuckets
which _could_ be used, however few fests use more than 1 or 2.

Change the default maximum number of vBuckets in ep_testsuite tests from
1024 to 4. This reduces the runtime of the ep_testsuite suite from 197s
to 134s under Windows Debug config.

Change-Id: I9bd656590189084f09fb3bb16b8d8efa4de31e8a
Reviewed-on: http://review.couchbase.org/121637
Well-Formed: Build Bot <build@couchbase.com>
Reviewed-by: Ben Huddleston <ben.huddleston@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


Revision tags: v6.0.4, v6.5.0
# 1c8db7e0 26-Sep-2019 Dave Rigby <daver@couchbase.com>

MB-34189: Audit configuration.json for non-dynamic params

A number of configuration parameters which are currently defined as
'dynamic' (can be changed at runtime once a bucket is initia

MB-34189: Audit configuration.json for non-dynamic params

A number of configuration parameters which are currently defined as
'dynamic' (can be changed at runtime once a bucket is initialized) are
in fact not. Change to dynamic==false:

- alog_block_size: Access log block size - not currently dynamic and
while it _could_ be made dynamic (for next Access log scan), doesn't
seem worthwhile to make it dynamic.

- backend: Cannot change storage backend without deleting / recreating
bucket.

- bucket_type: Cannot change between persistent / ephemeral bucket
types at runtime.

- chk_remover_stime: Checkpoint remover sleep time. Currently only
used at startup when task is created, _could_ make it dynamic but
don't see a current need.

- collections_enabled: Cannot turn collections on / off without
deleting & re-creating bucket.

- collections_max_size: Not possible to reduce max number of
collections at runtime, so for now just make it static.

- conflict_resolution_type: Cannot change without restarting bucket.

- couch_bucket: Cannot change bucket name without deleting /
re-creating it.

- data_traffic_enabled: Only read during initialization/warmup, hence
meaningless to be dynamic.

- dbname: Cannot change on-disk storage path after startup.

- failpartialwarmup: Only read during warmup, hence meaningless to be
dynamic.

- ht_locks: Cannot be changed once a vBucket is created.

- ht_size: Cannot be changed once a vBucket is created.

- item_eviction_policy: Cannot be changed without restarting the
bucket (see Instant Karma...).

- max_failover_entries: Not straightforward to change dynamically, and
doesn't seem worth the code changes to make it dynamic at this stage
(I cannot think of why we'd need to change it).

- max_vbuckets: Cannot be changed without deleting & re-creating the
bucket.

- max_threads: Used to explicitly set number of threads created
(instead of calculating based on CPU cores), hence cannot be changed
without restarting memcached process.

- max_num_shards: Cannot be changed with restarting the bucket.

- max_num_workers: Confusingly named param, actually used for bucket
priority, which cannot be changed dynamically.

- uuid: Set by ns_server and never changes.

- dcp_backfill_byte_limit: Not currently changable at runtime (set in
BackfillManager ctor), and doesn't seem worth the code changes to
make dynamic.

- dcp_flow_control_policy: Not currently changable at runtime (used in
EPEngine::initialize() to select which DcpFlowControlManager
subclass to construct.

- dcp_scan_byte_limit / dcp_scan_item_limit: Not currently changable
at runtime (set in BackfillManager ctor).

- magma_*: None of the Magma settings are currently dynamically
configurable.

- rocksdb_*: None of the RocksDB settings are currently dynamically
configurable.

- scopes_max_size: Not changable without restarting bucket.

- time_synchronization: No longer supported.

- warmup: Used to control if warmup runs; given that's a one-off event
at startup it's meaningless to be dynamic.

- warmup_batch_size: Used during warmup, hence no point in making
dynamic.

Change-Id: Ic13fb3345fcc58fd9374920b828bb55a1deb8fd1
Reviewed-on: http://review.couchbase.org/115422
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Trond Norbye <trond.norbye@couchbase.com>

show more ...


Revision tags: v6.0.3, v5.5.4
# 93bbefbe 11-Apr-2019 Dave Rigby <daver@couchbase.com>

MB-28031: Disable test_dcp_cursor_dropping_backfill while failing

Skip running test_dcp_cursor_dropping_backfill as it's currently
intermittently failing.

Change-Id: Ibc0ad50e12

MB-28031: Disable test_dcp_cursor_dropping_backfill while failing

Skip running test_dcp_cursor_dropping_backfill as it's currently
intermittently failing.

Change-Id: Ibc0ad50e12768dc8777c763393d470cdeee734f5
Reviewed-on: http://review.couchbase.org/107654
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Richard de Mellow <richard.demellow@couchbase.com>

show more ...


Revision tags: v5.5.5
# 62ba882c 29-Mar-2019 Scott Lashley <scott.lashley@couchbase.com>

MB-33463: Refactor test skip functions to include magma

Refactor skipping functions by creating a simple routine to look up any
configuration variable to see if its set or not.

MB-33463: Refactor test skip functions to include magma

Refactor skipping functions by creating a simple routine to look up any
configuration variable to see if its set or not.

Add magma skipping functions.

Change-Id: I08c1aaa03a7efd67a2da09aac6ea1172a423ddb5
Reviewed-on: http://review.couchbase.org/107021
Reviewed-by: Jim Walker <jim@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


# b42bb979 29-Mar-2019 Trond Norbye <trond.norbye@gmail.com>

Cleanup: Remove SunOS content

We don't build on Solaris and don't know the current status.

Change-Id: I9bcbb44dc5c7f67c05375ce4a3d328b62375cc6a
Reviewed-on: http://review.couchb

Cleanup: Remove SunOS content

We don't build on Solaris and don't know the current status.

Change-Id: I9bcbb44dc5c7f67c05375ce4a3d328b62375cc6a
Reviewed-on: http://review.couchbase.org/106995
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


Revision tags: v5.5.6
# fc7f5450 15-Mar-2019 Trond Norbye <trond.norbye@gmail.com>

Use cb::io::isFile instead of nonstandard access

Change-Id: I6f6e567787561e0530c174eff75f6e8169730f27
Reviewed-on: http://review.couchbase.org/106256
Tested-by: Trond Norbye <trond.n

Use cb::io::isFile instead of nonstandard access

Change-Id: I6f6e567787561e0530c174eff75f6e8169730f27
Reviewed-on: http://review.couchbase.org/106256
Tested-by: Trond Norbye <trond.norbye@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


Revision tags: v6.0.1
# c6185308 29-Nov-2018 Trond Norbye <trond.norbye@gmail.com>

Refactor: unknown_command should receive const Request&

The engine can't modify the request received from the client
so the object should be passed as const. While touching
everywher

Refactor: unknown_command should receive const Request&

The engine can't modify the request received from the client
so the object should be passed as const. While touching
everywhere we call unknown_command we should move it over
to the new Request class which contains getters for all
of the various fields in the packet.

Change-Id: I25e4f81a8e572fbee2bdb81c2a494cdece496530
Reviewed-on: http://review.couchbase.org/102279
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


Revision tags: v5.5.3
# 70213898 08-Nov-2018 Ben Huddleston <ben.huddleston@couchbase.com>

MB-31797: Make RocksDB assume deletions are for non-existing items

Correct the bugs currently in the code that are found by setting
the CB_DEVELOPMENT_ASSERTS flag to ON.

Modify

MB-31797: Make RocksDB assume deletions are for non-existing items

Correct the bugs currently in the code that are found by setting
the CB_DEVELOPMENT_ASSERTS flag to ON.

Modify the RocksDB commit callback to assume that any deletions are
for items that do not exist. This ensures that we overcount the
vBucket document counts under RocksDB (we won't decrement for any
deletions) as opposed to undercount it. This is required as this calls
the PersistenceCallback which would attempt to decrement the vBucket doc
counter to a value less than 0. Skip the now failing RocksDB tests that
relied on this vBucket document counter, or the other counters in the
vBucket that are now not set correctly due to the return value of the
RocksDB commitCallback.

Change-Id: Idc40e14d9f343f91042a2f926f88f98004454ecb
Reviewed-on: http://review.couchbase.org/101551
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


# 18d44712 01-Nov-2018 Trond Norbye <trond.norbye@gmail.com>

Refactor: Use the scoped enums cb::mcbp::ClientOpcode in ep-engine tests

Instead of the uint8_t definitions in PROTOCOL_BINARY_CMD_XXX

Change-Id: If1b5839ce6a3fcde3cf6c3a70f10288f54

Refactor: Use the scoped enums cb::mcbp::ClientOpcode in ep-engine tests

Instead of the uint8_t definitions in PROTOCOL_BINARY_CMD_XXX

Change-Id: If1b5839ce6a3fcde3cf6c3a70f10288f54c40ad5
Reviewed-on: http://review.couchbase.org/101270
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Jim Walker <jim@couchbase.com>

show more ...


Revision tags: v6.0.0
# 5eeaf3af 11-Oct-2018 Richard de Mellow <richard.demellow@couchbase.com>

MB-31510 Replace check marco for checkXX macros

Updated ep_test_apis, ep_testsuite_checkpoint and
ep_testsuite_common to use more relevant check macros
that give values in their fail

MB-31510 Replace check marco for checkXX macros

Updated ep_test_apis, ep_testsuite_checkpoint and
ep_testsuite_common to use more relevant check macros
that give values in their failure message. Also adding
checklt to provide less than checking and failure
messages.

Change-Id: Ic776ebc59da4a2fb275ec2587a2317ab425aa634
Reviewed-on: http://review.couchbase.org/100502
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Trond Norbye <trond.norbye@gmail.com>

show more ...


Revision tags: v5.1.3, v5.5.2
# d6a2e687 05-Sep-2018 Christopher Farman <christopher.farman@couchbase.com>

MB-30552: Convert type of vBucket id's from uint16_t to Vbid [8/n]

(Stage 8 of n) This commit changes to the type of Vbid for
ep's tests, including mock.
Vbid allows greater control

MB-30552: Convert type of vBucket id's from uint16_t to Vbid [8/n]

(Stage 8 of n) This commit changes to the type of Vbid for
ep's tests, including mock.
Vbid allows greater control over what operations can be performed
on a vBucket id, as well as ensuring the output formatting is
consistent. To support the splitting of this into multiple stages,
Vbid can currently be implicitly converted back to uint16_t to
cover elements that have not yet been converted. During this,
there may be logs or prints with "vb:vb:", which should be fixed
within this set of changes.

Change-Id: Ie86933239982b0dd882746fcc5ea0e490b79b3e7
Reviewed-on: http://review.couchbase.org/99627
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


# 06957872 30-Aug-2018 Christopher Farman <christopher.farman@couchbase.com>

MB-30552: Convert type of vBucket id's from uint16_t to Vbid [1/n]

(Stage 1 of n) This commit aims to change the usage type of vBucket
id's to the new type 'Vbid' within the inner workin

MB-30552: Convert type of vBucket id's from uint16_t to Vbid [1/n]

(Stage 1 of n) This commit aims to change the usage type of vBucket
id's to the new type 'Vbid' within the inner workings of each
function. This allows greater control over what operations can be
performed on a vBucket id, as well as ensuring the output formatting
is consistent. To support the splitting of this into multiple stages,
Vbid can currently be implicitly converted back to uint16_t to
cover elements that have not yet been converted.

This commit has been cherry-picked and then reverted, due to build
errors. It's previous changeID was
I0454882561e4fb7a91f5405f59e9489a5cfe92f1

Change-Id: I4a507179dd08cb8909e71aabd3ed19a40938a650
Reviewed-on: http://review.couchbase.org/99309
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


# bab57e37 06-Sep-2018 Jim Walker <jim@couchbase.com>

Revert "MB-30552: Convert type of vBucket id's from uint16_t to Vbid [1/n]"

Orginal patch is breaking OSX server builds (issue with hash template)

This reverts commit 3058aea20b75dc

Revert "MB-30552: Convert type of vBucket id's from uint16_t to Vbid [1/n]"

Orginal patch is breaking OSX server builds (issue with hash template)

This reverts commit 3058aea20b75dc27154feb81eb79f3aebf02eb40.

Change-Id: I95195ef2ef2c4218b980275a9eabfe3d476b3489
Reviewed-on: http://review.couchbase.org/99302
Reviewed-by: Trond Norbye <trond.norbye@gmail.com>
Tested-by: Trond Norbye <trond.norbye@gmail.com>

show more ...


# 3058aea2 30-Aug-2018 Christopher Farman <christopher.farman@couchbase.com>

MB-30552: Convert type of vBucket id's from uint16_t to Vbid [1/n]

(Stage 1 of n) This commit aims to change the usage type of vBucket
id's to the new type 'Vbid' within the inner workin

MB-30552: Convert type of vBucket id's from uint16_t to Vbid [1/n]

(Stage 1 of n) This commit aims to change the usage type of vBucket
id's to the new type 'Vbid' within the inner workings of each
function. This allows greater control over what operations can be
performed on a vBucket id, as well as ensuring the output formatting
is consistent. To support the splitting of this into multiple stages,
Vbid can currently be implicitly converted back to uint16_t to
cover elements that have not yet been converted.

Change-Id: I0454882561e4fb7a91f5405f59e9489a5cfe92f1
Reviewed-on: http://review.couchbase.org/99031
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Jim Walker <jim@couchbase.com>

show more ...


# 8d08d4b0 07-Aug-2018 Trond Norbye <trond.norbye@gmail.com>

MB-30149: Drop duplicate parameter to test functions [29/n]

This patch drops the second parameter to the test function which
used to be a pointer to the first interface version of the en

MB-30149: Drop duplicate parameter to test functions [29/n]

This patch drops the second parameter to the test function which
used to be a pointer to the first interface version of the engine
interface.

Change-Id: Ic42e7089c0d3b6c9167c9b7c5974ee1ffc4259fc
Reviewed-on: http://review.couchbase.org/97915
Reviewed-by: Dave Rigby <daver@couchbase.com>
Tested-by: Build Bot <build@couchbase.com>

show more ...


# f7728a93 07-Aug-2018 Trond Norbye <trond.norbye@gmail.com>

MB-30149: Drop duplicate parameter to test functions [26/n]

* wait_for_warmup_complete

Change-Id: Iafbdf48ca2757c2edd0cadac34f9b5c978a76aac
Reviewed-on: http://review.couchbas

MB-30149: Drop duplicate parameter to test functions [26/n]

* wait_for_warmup_complete

Change-Id: Iafbdf48ca2757c2edd0cadac34f9b5c978a76aac
Reviewed-on: http://review.couchbase.org/97903
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


# e117b542 06-Aug-2018 Trond Norbye <trond.norbye@gmail.com>

MB-30149: Drop duplicate parameter to test functions [23/n]

* wait_for_stat_to_be_gte
* wait_for_stat_to_be_lte

(and remove the h1 member in the TestDcpConsumer as it is unu

MB-30149: Drop duplicate parameter to test functions [23/n]

* wait_for_stat_to_be_gte
* wait_for_stat_to_be_lte

(and remove the h1 member in the TestDcpConsumer as it is unused
and cause compiler warning)

Change-Id: I1c9b808b0192a6e5cbd5f21d4e1571887ba23563
Reviewed-on: http://review.couchbase.org/97881
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


# 432be634 06-Aug-2018 Trond Norbye <trond.norbye@gmail.com>

MB-30149: Drop duplicate parameter to test functions [17/n]

* get_stat<>
* get_int_stat
* get_float_stat
* get_ul_stat
* get_ull_stat
* get_str_stat
* g

MB-30149: Drop duplicate parameter to test functions [17/n]

* get_stat<>
* get_int_stat
* get_float_stat
* get_ul_stat
* get_ull_stat
* get_str_stat
* get_bool_stat

Change-Id: I5ac5fbf3f6c01fb46be46ad6afda6d4c063faa35
Reviewed-on: http://review.couchbase.org/97874
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


# 15a41ba9 01-Aug-2018 Trond Norbye <trond.norbye@gmail.com>

MB-30149: Remove duplicate holders to engine iterface

Given that we've killed the two interface versions of the
engine interface we don't need to keep two identical values
around.

MB-30149: Remove duplicate holders to engine iterface

Given that we've killed the two interface versions of the
engine interface we don't need to keep two identical values
around.

Change-Id: Ic21fefb2ddec2aedbfb3c6a299da4fb64d579346
Reviewed-on: http://review.couchbase.org/97677
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


# d006897d 01-Aug-2018 Trond Norbye <trond.norbye@gmail.com>

MB-30149: Drop duplicate parameter to test functions [11/n]

* check_key_value

Change-Id: Iba19a421e20366a41cabecbc91088deb58fa8ae4
Reviewed-on: http://review.couchbase.org/976

MB-30149: Drop duplicate parameter to test functions [11/n]

* check_key_value

Change-Id: Iba19a421e20366a41cabecbc91088deb58fa8ae4
Reviewed-on: http://review.couchbase.org/97676
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


# ef4f227d 01-Aug-2018 Trond Norbye <trond.norbye@gmail.com>

MB-30149: Drop duplicate parameter to test functions [10/n]

* isCompressionEnabled
* isPassiveCompressionEnabled
* isActiveCompressionEnabled
* isWarmupEnabled
* is

MB-30149: Drop duplicate parameter to test functions [10/n]

* isCompressionEnabled
* isPassiveCompressionEnabled
* isActiveCompressionEnabled
* isWarmupEnabled
* isPersistentBucket
* isEphemeralBucket
* checkPersistentBucketTempItems

Change-Id: I1cacc6957b13d0aac1ac84510f2566e092c0cde0
Reviewed-on: http://review.couchbase.org/97675
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


# 57ad9160 01-Aug-2018 Trond Norbye <trond.norbye@gmail.com>

MB-30149: Drop duplicate parameter to test functions [5/n]

* allocate
* get
* store

Change-Id: Iea0517f78576fc4890bb271556ab013cbf947eae
Reviewed-on: http://review

MB-30149: Drop duplicate parameter to test functions [5/n]

* allocate
* get
* store

Change-Id: Iea0517f78576fc4890bb271556ab013cbf947eae
Reviewed-on: http://review.couchbase.org/97664
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


# aff078ba 01-Aug-2018 Trond Norbye <trond.norbye@gmail.com>

MB-30149: Drop duplicate parameter to test functions [4/n]

* set_vbucket_state
* get_all_vb_seqnos
* verify_all_vb_seqnos

Change-Id: I71d6e0e07c50068b2124cc8c541f9edb4

MB-30149: Drop duplicate parameter to test functions [4/n]

* set_vbucket_state
* get_all_vb_seqnos
* verify_all_vb_seqnos

Change-Id: I71d6e0e07c50068b2124cc8c541f9edb45f7b094
Reviewed-on: http://review.couchbase.org/97663
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Jim Walker <jim@couchbase.com>

show more ...


# 2a3f1b5e 01-Aug-2018 Trond Norbye <trond.norbye@gmail.com>

MB-30149: Remove duplicate EngineIface from test setup/teardown

The same variable was passed twice to the method

Change-Id: Idcf3057afa395002ea7611e2fd5d08009f978afc
Reviewed-on

MB-30149: Remove duplicate EngineIface from test setup/teardown

The same variable was passed twice to the method

Change-Id: Idcf3057afa395002ea7611e2fd5d08009f978afc
Reviewed-on: http://review.couchbase.org/97652
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


# 7b313091 01-Aug-2018 Trond Norbye <trond.norbye@gmail.com>

MB-30149: Move test_harness to virtual funcs

And remove unused methods and drop "duplicate" parameters
for EngineIface. When we move the actual test function
over to expect a single

MB-30149: Move test_harness to virtual funcs

And remove unused methods and drop "duplicate" parameters
for EngineIface. When we move the actual test function
over to expect a single EngineIface we can remove the
assignment "h1 = h" I added after the reload_engine.

Change-Id: I05c12d15bd66a142105944bd6999874493742705
Reviewed-on: http://review.couchbase.org/97651
Tested-by: Build Bot <build@couchbase.com>
Reviewed-by: Dave Rigby <daver@couchbase.com>

show more ...


123