[Bug 1953426] Re: [SRU] configurable threshold for reaping dead connections and add a unit test

[gerald.yang]

** Description changed:

  [Impact]
  add a configurable threshold for reaping dead connections and fix cephfs test case
  
  [Test plan]
  run ceph qa tests in: qa/tasks/cephfs/test_sessionmap.py
  this also verifies the configurable threshold
  
  steps to run:
  1. build ceph from source folder
  2. go to build folder
  3. run all tests in test_sessionmap.py, e.g.
  python ../qa/tasks/vstart_runner.py tasks.cephfs.test_sessionmap.TestSessionMap.test_mount_conn_close
  python ../qa/tasks/vstart_runner.py tasks.cephfs.test_sessionmap.TestSessionMap.test_tell_session_drop
  ...
  
  [Where problems could occur]
  these patches only add a configuration and fix the test case
- it can also be tested by ceph qa tests explained above
- and they also passed all the qa tests upstream and been merged by upstream, I don't think there will be any regression
+ the only problem I can imagine is someone set this config to a non-sense value
  
  [Other info]
  from upstream:
  octopus backport tracker
  https://tracker.ceph.com/issues/51199
  
  octopus backport PR
  https://github.com/ceph/ceph/pull/43310

** Description changed:

  [Impact]
  add a configurable threshold for reaping dead connections and fix cephfs test case
  
  [Test plan]
  run ceph qa tests in: qa/tasks/cephfs/test_sessionmap.py
  this also verifies the configurable threshold
  
  steps to run:
  1. build ceph from source folder
  2. go to build folder
  3. run all tests in test_sessionmap.py, e.g.
  python ../qa/tasks/vstart_runner.py tasks.cephfs.test_sessionmap.TestSessionMap.test_mount_conn_close
  python ../qa/tasks/vstart_runner.py tasks.cephfs.test_sessionmap.TestSessionMap.test_tell_session_drop
  ...
  
  [Where problems could occur]
  these patches only add a configuration and fix the test case
- the only problem I can imagine is someone set this config to a non-sense value
+ the only problem I can imagine is someone sets this config to a non-sense value
  
  [Other info]
  from upstream:
  octopus backport tracker
  https://tracker.ceph.com/issues/51199
  
  octopus backport PR
  https://github.com/ceph/ceph/pull/43310

-- 
You received this bug notification because you are a member of Ubuntu
OpenStack, which is subscribed to ceph in Ubuntu.
https://bugs.launchpad.net/bugs/1953426

Title:
  [SRU] configurable threshold for reaping dead connections and add a
  unit test

Status in ceph package in Ubuntu:
  In Progress
Status in ceph source package in Focal:
  In Progress

Bug description:
  [Impact]
  add a configurable threshold for reaping dead connections and fix cephfs test case

  [Test plan]
  run ceph qa tests in: qa/tasks/cephfs/test_sessionmap.py
  this also verifies the configurable threshold

  steps to run:
  1. build ceph from source folder
  2. go to build folder
  3. run all tests in test_sessionmap.py, e.g.
  python ../qa/tasks/vstart_runner.py tasks.cephfs.test_sessionmap.TestSessionMap.test_mount_conn_close
  python ../qa/tasks/vstart_runner.py tasks.cephfs.test_sessionmap.TestSessionMap.test_tell_session_drop
  ...

  [Where problems could occur]
  these patches only add a configuration and fix the test case
  the only problem I can imagine is someone sets this config to a non-sense value

  [Other info]
  from upstream:
  octopus backport tracker
  https://tracker.ceph.com/issues/51199

  octopus backport PR
  https://github.com/ceph/ceph/pull/43310

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/ceph/+bug/1953426/+subscriptions