Skip to content

Commit 525c046

Browse files
committed
MB-66959 Fix invocation of fetching collections manifest
Get collections manifest request validates that the vbid given is 0. But the cb_evict_key program is meant to specify some vbucket and hence it fails: ./cb_evict_key.py Administrator password default 100 colA_-5835 _default.FunctionCollection1 mc_bin_client.ErrorEinval: Memcached error #4: EINVAL : Invalid packet : {"error":{"context":"Request vbucket id must be 0"}} The fix is same as what we do for get_collection_id. Post fixing: ./cb_evict_key.py Administrator password default 100 colA_-5835 _default.FunctionCollection1 (211893125, 0, b'Ejected.') Change-Id: Ic35e6e03c6bd47b37c2e5c3a91165fd1395973f1 Reviewed-on: https://review.couchbase.org/c/kv_engine/+/230628 Reviewed-by: Mohammad Zaeem <[email protected]> Tested-by: Build Bot <[email protected]>
1 parent 76d972a commit 525c046

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

engines/ep/management/mc_bin_client.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1058,11 +1058,14 @@ def set_collections(self, manifest):
10581058
self._update_collection_map(manifest)
10591059

10601060
def get_collections(self, update_map=False):
1061+
tmpVbucketId = self.vbucketId
1062+
self.vbucketId = 0
10611063
rv = self._doCmd(memcacheConstants.CMD_COLLECTIONS_GET_MANIFEST,
10621064
'',
10631065
'')
10641066
if update_map:
10651067
self._update_collection_map(rv[2])
1068+
self.vbucketId = tmpVbucketId
10661069
return rv
10671070

10681071
def get_collection_id(self, path):

0 commit comments

Comments
 (0)