Revert "test_filestore_idempotent: make newly created objects globally unique"

This reverts commit cdf5785c5e8b744f35ea34b2deb2a71edf8b9580.
Revert "test_filestore_idempotent: use obj name from source coll add"

This reverts commit 0bc731ea93ae198afcdcba285751a6201c3de8de.
test: idempotent filestore test failure

Remove obsolete use of collection_move()
Allow operations to be skipped if random selections don't make sense
Track total number of possible objects in m_num_objects
BUG: do_remove() was calling _do_touch() instead of _do_remove()
For ops that require an object, select from among existing objects in collection
Initialize m_num_objects unique objects across collections

touch: don't create an object that already exists in another collection
remove: Use remove_obj() to clear object from m_objects to have accurate tracking
clone/clone_range(): Select 2 existing objects in the collection
add: Skip operation if selected target object name exists in target collection
move: Removed this buggy operation that is only present for upgrades

Fixes: #5371
Fixes: #5240

Signed-off-by: David Zafman <david.zafman@inktank.com>
Merge branch 'wip-5371-next' into next

Reviewed-by: Samuel Just <sam.just@inktank.com>