As we see in the diff above

   if (ev >= 17) {
-    decode(max_xattr_size, p);
+    decode(bal_rank_mask, p);
   }
 
   if (ev >= 18) {
-    decode(bal_rank_mask, p);
+    decode(max_xattr_size, p);
+  }
+

these two decode() calls were swapped. Let's find out why.
To do so we need to clone the upstream repo and run git blame on the file to 
see when and why the lines were changed:

> git clone https://github.com/ceph/ceph ceph-upstream
> cd ceph-upstream/
> git blame src/mds/MDSMap.cc
...
e134c8907013 (Yongseok Oh      2022-10-11 20:47:32 +0900  963)   if (ev >= 17) {
78abfeaff27f (Patrick Donnelly 2024-02-15 10:28:32 -0500  964)     
decode(bal_rank_mask, p);
36ee8e7ed365 (Venky Shankar    2023-12-01 04:32:20 -0500  965)   }
36ee8e7ed365 (Venky Shankar    2023-12-01 04:32:20 -0500  966) 
36ee8e7ed365 (Venky Shankar    2023-12-01 04:32:20 -0500  967)   if (ev >= 18) {
78abfeaff27f (Patrick Donnelly 2024-02-15 10:28:32 -0500  968)     
decode(max_xattr_size, p);
e134c8907013 (Yongseok Oh      2022-10-11 20:47:32 +0900  969)   }
...

We see that both decode() functions where changed in the same commit. If
we look at it with

> git show 78abfeaff27f

we'll see that this is what we were looking for. A link to the commit:
https://github.com/ceph/ceph/commit/78abfeaff27fee343fb664db633de5b221699a73.

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

Title:
  MON and MDS crash upgrading  CEPH  on ubuntu 24.04 LTS

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


-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to