Update daos_rebuild_*.c per Dalton's comments, also retest daos_test …

…for stability.

Test-tag: daos_test
Skip-func-hw-test-medium-md-on-ssd: false
Skip-func-hw-test-medium: false
Skip-func-hw-test-large-md-on-ssd: false
Skip-func-hw-test-large: false
Allow-unstable-test: true
Doc-only: false
Required-githooks: true
Signed-off-by: Ding Ho <ding-hwa.ho@intel.com>

src/tests/suite/daos_rebuild_common.c

@@ -1205,9 +1205,6 @@ rebuild_sub_setup_common(void **state, daos_size_t pool_size, int node_nr, uint3
  rc = test_setup(state, SETUP_POOL_CONNECT, true,
  pool_size, node_nr, NULL);
  if (rc) {
- /* Let's skip for this case, since it is possible there
- * is not enough ranks here.
- */
  print_message("It can not create the pool, probably due"
  " to not enough ranks %d\n", rc);
  return rc;

src/tests/suite/daos_rebuild_ec.c

@@ -265,9 +265,6 @@ rebuild_ec_setup(void **state, int number, uint32_t rf)
  rc = test_setup(state, SETUP_POOL_CONNECT, true,
  REBUILD_POOL_SIZE, number, NULL);
  if (rc) {
- /* Let's skip for this case, since it is possible there
- * is not enough ranks here.
- */
  print_message("It can not create the pool with %d ranks"
  " probably due to not enough ranks %d\n",
  number, rc);

src/tests/suite/daos_rebuild_simple.c

@@ -927,9 +927,6 @@ rebuild_small_pool_n4_setup(void **state)
  rc = test_setup(state, SETUP_CONT_CONNECT, true,
  REBUILD_SMALL_POOL_SIZE, 4, NULL);
  if (rc) {
- /* Let's skip for this case, since it is possible there
- * is not enough ranks here.
- */
  print_message("It can not create the pool with 4 ranks"
  " probably due to not enough ranks %d\n", rc);
  return rc;