]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
mon/test_mon_osdmap_prune: Use first_pinned instead of first_committed 63341/head
authorAishwarya Mathuria <amathuri@redhat.com>
Wed, 13 Nov 2024 13:48:30 +0000 (19:18 +0530)
committerAishwarya Mathuria <amathuri@redhat.com>
Mon, 19 May 2025 07:26:40 +0000 (12:56 +0530)
commit912de99e840d06621a8eb2c1e8eef8be65ed191f
tree439a932a2de734ff2150694d545d21d41df9965e
parentff66c31ef18e27d145586818b26d61277c7c3e10
mon/test_mon_osdmap_prune: Use first_pinned instead of first_committed

In the test, we assume the trimming of osdmaps is done if osdmap_first_committed is equal to the epoch of the map upto which we had to trim.
osdmap_first_committed however returns the value of cached_first_committed which is updated before the Paxos proposal for trim goes through.
Normally, this is not a problem as when a refresh() happens, cached_first_committed will be corrected to the value before the trim was proposed if the Paxos proposal failed.
But for this test, we encounter a failure in this case as the osdmap_first_committed will not be equal to the osdmap_manifest.first_pinned as the trim never went through.
Changing the test to verify that osdmap_manifest.first_pinned has changed to the correct value is a better way to signify that the trim is complete for the purpose of this test.

Fixes: https://tracker.ceph.com/issues/47838
Signed-off-by: Aishwarya Mathuria <amathuri@redhat.com>
(cherry picked from commit 1e2f3758118b7af6edf8d7153fdb6463fe2e177d)
qa/workunits/mon/test_mon_osdmap_prune.sh