Update test case

It's too difficult to simulate power outage during multi-block transaction on 
disk, so the way I test this patch is to simulate a multi-block transaction and 
trigger a decode error in
try {
      auto p = bl.cbegin();
      decode(t, p);
      seen_recs = true;
    }

Add the following line right after decode(t, p) to throw an error
throw buffer::malformed_input("error test");

According to the patch description https://github.com/ceph/ceph/pull/42830
this error will be considered as a normal bluefs log reply stop condition and 
will *not* prevent OSD from starting
After the error test is triggered, OSD can still be started normally


** Description changed:

  [Impact]
  Multi-block transaction could fail during unexpected power down
  in this case, it should be stop replaying this log instead of throwing 
unrecoverable error
  
  [Test Case]
- need to simulate power outage to reproduce this issue
+ It's too difficult to simulate power outage during multi-block transaction on 
disk, so the way I test this patch is to simulate a multi-block transaction and 
trigger a decode error in
+ try {
+       auto p = bl.cbegin();
+       decode(t, p);
+       seen_recs = true;
+     }
+ 
+ Add the following line right after decode(t, p) to throw an error
+ throw buffer::malformed_input("error test");
+ 
+ According to the patch description https://github.com/ceph/ceph/pull/42830
+ this error will be considered as a normal bluefs log reply stop condition and 
will *not* prevent OSD from starting
+ After the error test is triggered, OSD can still be started normally
  
  [Where problems could occur]
  This upstream PR was created 2 months ago and Luminous was EOL upstream for a 
while
  so Luminous Bluefs code is outdated, in order to backport this commit
  it might also need to backport some dependencies
  
  [Other Info]
  upstream tracker: https://tracker.ceph.com/issues/52079
  PR: https://github.com/ceph/ceph/pull/42830

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

Title:
  [SRU] accept undecodable multi-block bluefs transactions on log

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


-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to