** 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]
  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;
-     }
+       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
+ This upstream PR was created 2 months ago and Luminous was EOL upstream for a 
while, so no backport and test by upstream
+ In order to backport this commit, it also needs 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