Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update README.md #2

Closed
wants to merge 1 commit into from
Closed

Update README.md #2

wants to merge 1 commit into from

Conversation

cdurf1
Copy link
Contributor

@cdurf1 cdurf1 commented Nov 30, 2016

Minor edits to readme file

Minor edits to readme file
@johannlombardi
Copy link
Contributor

Already landed.

@cdurf1 cdurf1 deleted the patch-1 branch October 14, 2019 20:49
brianjmurrell added a commit that referenced this pull request Mar 11, 2020
Quick-build: true
Skip-checkpatch: true
Skip-run_test: true
Signed-off-by: Brian J. Murrell <[email protected]>
brianjmurrell added a commit that referenced this pull request Mar 17, 2020
Quick-build: true
Skip-checkpatch: true
Skip-run_test: true
Signed-off-by: Brian J. Murrell <[email protected]>
jolivier23 pushed a commit that referenced this pull request Mar 23, 2020
- Fixes to most critical issues found in scan #11
- CRT_BULK_NULL moved from internal to external header

Change-Id: Ib14f5b67ed98d60ce2addf0e295686288fa3a775
Signed-off-by: Alex Oganezov <[email protected]>
Reviewed-on: https://review.whamcloud.com/31579
Tested-by: Maloo <[email protected]>
Reviewed-by: Ashley Pittman <[email protected]>
Reviewed-by: Yulu Jia <[email protected]>
brianjmurrell added a commit that referenced this pull request Jun 2, 2020
Quick-build: true
Skip-checkpatch: true
Skip-run_test: true
Signed-off-by: Brian J. Murrell <[email protected]>
mallove79 added a commit that referenced this pull request Aug 16, 2020
Clean up (hopefully unneccessary) cart.log printers. Put cart.log's into
default DAOS logs dir from the very beginning.

Competing approaches:
  1. Find where cart.log's are currently, and move them to artifacts
     dir.
  2. Put cart.log's into artifacts dir from the start.

Up to now, this branch has used approach #1. #2 appears to be the far
simpler approach; so KISS, reduce complexity -- try #2.

Skip-build-leap15-rpm: true
Skip-build-ubuntu-clang: true
Skip-build-leap15-icc: true
Skip-coverity-test: true
Skip-checkpatch: true
Quick-build: true
Skip-run_test: true
Test-tag: all_cart
Skip-func-test-leap15: true
Skip-func-hw-test-small: true
Skip-func-hw-test-medium: true
Skip-func-hw-test-large: true

Signed-off-by: Ethan Mallove <[email protected]>
tanabarr added a commit that referenced this pull request Sep 22, 2020
frostedcmos added a commit that referenced this pull request Nov 3, 2020
Signed-off-by: Alexander A Oganezov <[email protected]>
sylviachanoiyee pushed a commit that referenced this pull request Nov 3, 2020
Signed-off-by: Alexander A Oganezov <[email protected]>
mallove79 added a commit that referenced this pull request Jan 8, 2021
 * Instead of --shutdown_rank, use --rank $rank_id --shut_only.
 * Instead of --delete_from_ranklist (black-list sense),
   use cart_ctl-style white-list: --rank 0,1 (to e.g., remove
   rank #2 from rank list).

Skip-build-leap15-rpm: true
Skip-build-ubuntu-clang: true
Skip-build-leap15-icc: true
Skip-coverity-test: true
Skip-checkpatch: true
Quick-build: true
Test-tag: cart
Skip-func-test-leap15: true
Skip-func-hw-test-small: true
Skip-func-hw-test-medium: true
Skip-func-hw-test-large: true
Skip-unit-test-memcheck: true

Signed-off-by: Ethan Mallove <[email protected]>
jolivier23 added a commit that referenced this pull request Jan 27, 2021
For these directories and their sub_dirs:
daos/src/object
daos/src/mgmt
daos/src/iosrv
daos/src/include
daos/src/gurt
daos/src/dtx
daos/src/placement
daos/src/pool
daos/src/proto
daos/src/rdb
daos/src/rebuild
daos/src/rsvc

Signed-off-by: Hua Kuang <[email protected]>
Co-authored-by: Jeff Olivier <[email protected]>
ravalsam added a commit that referenced this pull request Feb 18, 2021
Fixed Coverity issue in test code.
313028(#2 of 2) and 313033(#1 of 1).

Signed-off-by: Samir Raval <[email protected]>
mallove79 added a commit that referenced this pull request Feb 24, 2021
alloc_fn: Storage is returned from allocation function malloc.
  2. var_assign: Assigning: swim_seq = storage returned from malloc(1024UL).
     CID 318948 (#2 of 2): Resource leak (RESOURCE_LEAK)
  4. overwrite_var: Overwriting swim_seq in swim_seq = "" leaks the storage that
     swim_seq points to.

Skip-build-leap15-rpm: true
Skip-func-test-leap15: true
Skip-build-ubuntu-clang: true
Skip-build-leap15-icc: true
Skip-coverity-test: true
Quick-build: true
Test-tag: cart
Skip-unit-tests: true
Skip-nlt: true
Skip-unit-test: true
Skip-unit-test-memcheck: true
Skip-coverity-test: true
Skip-func-hw-test-small: true
Skip-func-hw-test-medium: true
Skip-func-hw-test-large: true

Signed-off-by: Ethan Mallove <[email protected]>
sylviachanoiyee pushed a commit that referenced this pull request Feb 26, 2021
Fixed Coverity issue in test code.
313028(#2 of 2) and 313033(#1 of 1).

Signed-off-by: Samir Raval <[email protected]>
sylviachanoiyee pushed a commit that referenced this pull request Mar 11, 2021
Fix coverity issue:
  CID 318952 (#3 of 3): Explicit null dereferenced (FORWARD_NULL)
  15. var_deref_op: Dereferencing null pointer rank_list.

alloc_fn: Storage is returned from allocation function malloc.
  2. var_assign: Assigning: swim_seq = storage returned from malloc(1024UL).
     CID 318948 (#2 of 2): Resource leak (RESOURCE_LEAK)
  4. overwrite_var: Overwriting swim_seq in swim_seq = "" leaks the storage that
     swim_seq points to.

Fix coverity STRING_OVERFLOW issue:
  CID 318951 (#1 of 1): Copy into fixed size buffer (STRING_OVERFLOW) 39.
  fixed_size_dest: You might overrun the 8-character fixed-size string exp_status
  by copying cursorCopy + groupArray[g].rm_so without checking the length.

Signed-off-by: Ethan Mallove <[email protected]>
jolivier23 pushed a commit that referenced this pull request Jul 19, 2022
Makito and Samir observed the following assertion failure after
restarting engines.

  #0  raise () from /lib64/libc.so.6
  #1  abort () from /lib64/libc.so.6
  #2  __assert_fail_base () from /lib64/libc.so.6
  #3  __assert_fail () from /lib64/libc.so.6
  #4  pool_map_get_version (map=0x0) at src/common/pool_map.c:2852
  #5  ds_pool_get_version (pool=0x7f0ca063c690, pool=0x7f0ca063c690) at
      src/include/daos_srv/pool.h:296
  #6  pc=rpc@entry=0x7f0ca0998d30, p_rpt=p_rpt@entry=0x7f0ca83a77b0) at
      src/rebuild/srv.c:2101
  #7  rebuild_tgt_scan_handler (rpc=0x7f0ca0998d30) at
      src/rebuild/scan.c:954
  #8  crt_handle_rpc (arg=0x7f0ca0998d30) at src/cart/crt_rpc.c:1654
  #9  ABTD_ythread_func_wrapper (p_arg=0x7f0ca83a78a0) at
      arch/abtd_ythread.c:21
  #10 make_fcontext () from /usr/lib64/libabt.so.1
  #11 ?? ()

The ds_pool_get_version call passed a NULL map argument to
pool_map_get_version. The ds_pool.sp_map field may be NULL after the
pool is started but before the pool receives the initial pool map from
the pool service. This patch fixes ds_pool_get_version to return 0,
which is less than all valid pool map versions, when sp_map is NULL,
resulting in rebuild retries like this:

  Rebuild [queued] (pool=3bf68c9c ver=2) tgts=2
  Rebuild [started] (pool 3bf68c9c ver=2)
  Rebuild [failed] (pool 3bf68c9c ver=2 status=DER_BUSY(-1012): 'Device
    or resource busy')
  Rebuild [queued] (pool=3bf68c9c ver=2) tgts=2
  Rebuild [started] (pool 3bf68c9c ver=2)
  Rebuild [failed] (pool 3bf68c9c ver=2 status=DER_BUSY(-1012): 'Device
    or resource busy')
  Rebuild [queued] (pool=3bf68c9c ver=2) tgts=2
  Rebuild [started] (pool 3bf68c9c ver=2)
  Rebuild [scanning] (pool 3bf68c9c ver=2, toberb_obj=0, rb_obj=0, [...]
  Rebuild [scanning] (pool 3bf68c9c ver=2, toberb_obj=0, rb_obj=0, [...]
  Rebuild [completed] (pool 3bf68c9c ver=2, toberb_obj=0, rb_obj=0,[...]
  Target[2] (rank 2 idx 0 status 16 ver 1) is excluded.

Also, this patch removes some rebuild code that handles NULL
ds_pool.sp_group fields. Those can not happen as we always initialize
sp_group (as well as sp_iv_ns) before putting a ds_pool object into the
LRU.

Signed-off-by: Li Wei <[email protected]>
liw added a commit that referenced this pull request Jul 20, 2022
Makito and Samir observed the following assertion failure after
restarting engines.

  #0  raise () from /lib64/libc.so.6
  #1  abort () from /lib64/libc.so.6
  #2  __assert_fail_base () from /lib64/libc.so.6
  #3  __assert_fail () from /lib64/libc.so.6
  #4  pool_map_get_version (map=0x0) at src/common/pool_map.c:2852
  #5  ds_pool_get_version (pool=0x7f0ca063c690, pool=0x7f0ca063c690) at
      src/include/daos_srv/pool.h:296
  #6  pc=rpc@entry=0x7f0ca0998d30, p_rpt=p_rpt@entry=0x7f0ca83a77b0) at
      src/rebuild/srv.c:2101
  #7  rebuild_tgt_scan_handler (rpc=0x7f0ca0998d30) at
      src/rebuild/scan.c:954
  #8  crt_handle_rpc (arg=0x7f0ca0998d30) at src/cart/crt_rpc.c:1654
  #9  ABTD_ythread_func_wrapper (p_arg=0x7f0ca83a78a0) at
      arch/abtd_ythread.c:21
  #10 make_fcontext () from /usr/lib64/libabt.so.1
  #11 ?? ()

The ds_pool_get_version call passed a NULL map argument to
pool_map_get_version. The ds_pool.sp_map field may be NULL after the
pool is started but before the pool receives the initial pool map from
the pool service. This patch fixes ds_pool_get_version to return 0,
which is less than all valid pool map versions, when sp_map is NULL,
resulting in rebuild retries like this:

  Rebuild [queued] (pool=3bf68c9c ver=2) tgts=2
  Rebuild [started] (pool 3bf68c9c ver=2)
  Rebuild [failed] (pool 3bf68c9c ver=2 status=DER_BUSY(-1012): 'Device
    or resource busy')
  Rebuild [queued] (pool=3bf68c9c ver=2) tgts=2
  Rebuild [started] (pool 3bf68c9c ver=2)
  Rebuild [failed] (pool 3bf68c9c ver=2 status=DER_BUSY(-1012): 'Device
    or resource busy')
  Rebuild [queued] (pool=3bf68c9c ver=2) tgts=2
  Rebuild [started] (pool 3bf68c9c ver=2)
  Rebuild [scanning] (pool 3bf68c9c ver=2, toberb_obj=0, rb_obj=0, [...]
  Rebuild [scanning] (pool 3bf68c9c ver=2, toberb_obj=0, rb_obj=0, [...]
  Rebuild [completed] (pool 3bf68c9c ver=2, toberb_obj=0, rb_obj=0,[...]
  Target[2] (rank 2 idx 0 status 16 ver 1) is excluded.

Also, this patch removes some rebuild code that handles NULL
ds_pool.sp_group fields. Those can not happen as we always initialize
sp_group (as well as sp_iv_ns) before putting a ds_pool object into the
LRU.

Signed-off-by: Li Wei <[email protected]>
jolivier23 pushed a commit that referenced this pull request Jul 21, 2022
Makito and Samir observed the following assertion failure after
restarting engines.

  #0  raise () from /lib64/libc.so.6
  #1  abort () from /lib64/libc.so.6
  #2  __assert_fail_base () from /lib64/libc.so.6
  #3  __assert_fail () from /lib64/libc.so.6
  #4  pool_map_get_version (map=0x0) at src/common/pool_map.c:2852
  #5  ds_pool_get_version (pool=0x7f0ca063c690, pool=0x7f0ca063c690) at
      src/include/daos_srv/pool.h:296
  #6  pc=rpc@entry=0x7f0ca0998d30, p_rpt=p_rpt@entry=0x7f0ca83a77b0) at
      src/rebuild/srv.c:2101
  #7  rebuild_tgt_scan_handler (rpc=0x7f0ca0998d30) at
      src/rebuild/scan.c:954
  #8  crt_handle_rpc (arg=0x7f0ca0998d30) at src/cart/crt_rpc.c:1654
  #9  ABTD_ythread_func_wrapper (p_arg=0x7f0ca83a78a0) at
      arch/abtd_ythread.c:21
  #10 make_fcontext () from /usr/lib64/libabt.so.1
  #11 ?? ()

The ds_pool_get_version call passed a NULL map argument to
pool_map_get_version. The ds_pool.sp_map field may be NULL after the
pool is started but before the pool receives the initial pool map from
the pool service. This patch fixes ds_pool_get_version to return 0,
which is less than all valid pool map versions, when sp_map is NULL,
resulting in rebuild retries like this:

  Rebuild [queued] (pool=3bf68c9c ver=2) tgts=2
  Rebuild [started] (pool 3bf68c9c ver=2)
  Rebuild [failed] (pool 3bf68c9c ver=2 status=DER_BUSY(-1012): 'Device
    or resource busy')
  Rebuild [queued] (pool=3bf68c9c ver=2) tgts=2
  Rebuild [started] (pool 3bf68c9c ver=2)
  Rebuild [failed] (pool 3bf68c9c ver=2 status=DER_BUSY(-1012): 'Device
    or resource busy')
  Rebuild [queued] (pool=3bf68c9c ver=2) tgts=2
  Rebuild [started] (pool 3bf68c9c ver=2)
  Rebuild [scanning] (pool 3bf68c9c ver=2, toberb_obj=0, rb_obj=0, [...]
  Rebuild [scanning] (pool 3bf68c9c ver=2, toberb_obj=0, rb_obj=0, [...]
  Rebuild [completed] (pool 3bf68c9c ver=2, toberb_obj=0, rb_obj=0,[...]
  Target[2] (rank 2 idx 0 status 16 ver 1) is excluded.

Also, this patch removes some rebuild code that handles NULL
ds_pool.sp_group fields. Those can not happen as we always initialize
sp_group (as well as sp_iv_ns) before putting a ds_pool object into the
LRU.

Signed-off-by: Li Wei <[email protected]>
johannlombardi added a commit that referenced this pull request Jan 24, 2023
Tag a new 2.4 test build (#2).

Signed-off-by: Johann Lombardi <[email protected]>
phender pushed a commit that referenced this pull request Jan 25, 2023
Tag a new 2.4 test build (#2).

Signed-off-by: Johann Lombardi <[email protected]>
phender pushed a commit that referenced this pull request Jan 25, 2023
Tag a new 2.4 test build (#2).

Required-githooks: true

Signed-off-by: Johann Lombardi <[email protected]>
phender added a commit that referenced this pull request Jan 25, 2023
Tag a new 2.4 test build (#2).

Required-githooks: true

Signed-off-by: Johann Lombardi <[email protected]>
sylviachanoiyee pushed a commit that referenced this pull request Jan 27, 2023
Tag a new 2.4 test build (#2).

Signed-off-by: Johann Lombardi <[email protected]>
liw added a commit that referenced this pull request Mar 8, 2023
Commit 53d66d7 might have introduced the following "epi != NULL"
assertion failure in crt_context_req_untrack.

  #0  raise () from /usr/lib64/libc.so.6
  #1  abort () from /usr/lib64/libc.so.6
  #2  __assert_fail_base.cold.0 () from /usr/lib64/libc.so.6
  #3  __assert_fail () from /usr/lib64/libc.so.6
  #4  crt_context_req_untrack (rpc_priv=rpc_priv@entry=0x55a254b83150)
      at src/cart/crt_context.c:1358
          crt_ctx = 0x55a25465f500
          epi = 0x0
          submit_list = {next = 0x0, prev = 0x4}
          tmp_rpc = <optimized out>
          rc = <optimized out>
          __func__ = "crt_context_req_untrack"
          __PRETTY_FUNCTION__ = "crt_context_req_untrack"
  #5  crt_hg_req_send_cb (hg_cbinfo=<optimized out>) at
      src/cart/crt_hg.c:1287
          rpc_pub = <optimized out>
          rpc_priv = <optimized out>
          hg_ret = HG_SUCCESS
          rc = 0
          __func__ = "crt_hg_req_send_cb"
          __PRETTY_FUNCTION__ = "crt_hg_req_send_cb"
          __rc = <optimized out>

The RPC might be an outgoing URI_LOOKUP request, which we don't track.
Hence its crp_epi field was NULL as expected. Commit 53d66d7 should have
asserted "epi != NULL" _after_ making sure that the RPC is not an
URI_LOOKUP request. This patch does that and removes a few useless lines
from crt_context_req_untrack_internal.

Test-tag: pr dynamic_server_pool OSAOfflineExtend NvmePoolExtend
Signed-off-by: Li Wei <[email protected]>
Required-githooks: true
frostedcmos pushed a commit that referenced this pull request Mar 9, 2023
Commit 53d66d7 might have introduced the following "epi != NULL"
assertion failure in crt_context_req_untrack.

  #0  raise () from /usr/lib64/libc.so.6
  #1  abort () from /usr/lib64/libc.so.6
  #2  __assert_fail_base.cold.0 () from /usr/lib64/libc.so.6
  #3  __assert_fail () from /usr/lib64/libc.so.6
  #4  crt_context_req_untrack (rpc_priv=rpc_priv@entry=0x55a254b83150)
      at src/cart/crt_context.c:1358
          crt_ctx = 0x55a25465f500
          epi = 0x0
          submit_list = {next = 0x0, prev = 0x4}
          tmp_rpc = <optimized out>
          rc = <optimized out>
          __func__ = "crt_context_req_untrack"
          __PRETTY_FUNCTION__ = "crt_context_req_untrack"
  #5  crt_hg_req_send_cb (hg_cbinfo=<optimized out>) at
      src/cart/crt_hg.c:1287
          rpc_pub = <optimized out>
          rpc_priv = <optimized out>
          hg_ret = HG_SUCCESS
          rc = 0
          __func__ = "crt_hg_req_send_cb"
          __PRETTY_FUNCTION__ = "crt_hg_req_send_cb"
          __rc = <optimized out>

The RPC might be an outgoing URI_LOOKUP request, which we don't track.
Hence its crp_epi field was NULL as expected. Commit 53d66d7 should have
asserted "epi != NULL" _after_ making sure that the RPC is not an
URI_LOOKUP request. This patch does that and removes a few useless lines
from crt_context_req_untrack_internal.

Signed-off-by: Li Wei <[email protected]>
Michael-Hennecke added a commit that referenced this pull request Apr 24, 2023
mkdocs.yml debug/fix #2

Doc-only: true
Signed-off-by: Michael Hennecke <[email protected]>
mlawsonca pushed a commit that referenced this pull request May 16, 2023
Remove license header from yaml config files.
Use "engines:" instead of "servers:" in daos_server.yml.
Bump crt_timeout to 200s

Signed-off-by: Johann Lombardi <[email protected]>
osalyk pushed a commit to osalyk/daos that referenced this pull request Oct 12, 2023
jolivier23 added a commit that referenced this pull request Jul 8, 2024
Signed-off-by: Jeff Olivier <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants