https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2015-04-01T03:46:28ZCeph Ceph - Bug #11296: "FAILED assert(!old_value.deleted())" in upgrade:giant-x-hammer-distro-basic-multi runhttps://tracker.ceph.com/issues/11296?journal_id=502452015-04-01T03:46:28ZSamuel Justsjust@redhat.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>12</i></li><li><strong>Assignee</strong> set to <i>Samuel Just</i></li></ul><p>Huh, this is an old bug with cache tiers and snaps. I'll add more detail tomorrow, but the problem is in finish_promote. If we are promoting a snap which maps to the head object on the backend, we might find that all snaps which it maps to have been trimmed. Actually, that problem can still happen if we promoted a clone whose last snap was trimmed between when the backing tier responded to the promote and now. Either way, we have to filter out the removed clones and adjust the snapset as necessary.</p>
<p>Second, there is an off-by-one error in the handling of the snap->head case which resulted in 1b0 being included in the snaps vector (should have been empty and crashed due to the above case not being handled right).</p>
<p>The information for this is in ubuntu@teuthology:/a/teuthology-2015-03-30_17:05:01-upgrade:giant-x-hammer-distro-basic-multi/829317/remote/ceph-osd.burnupi2612242-86. Search backwards from the end for finish_promote.</p>
<p>See also grep 'make_writeable.*burnupi2612242-86\|\(<==\|-->\).*osd_op.*burnupi2612242-86\( \|@\)' ceph-osd.burnupi2612242-86 |less.</p>
<p>I'll look into a fix tomorrow. I don't think this is a blocker per-se, it's been around since firefly.</p> Ceph - Bug #11296: "FAILED assert(!old_value.deleted())" in upgrade:giant-x-hammer-distro-basic-multi runhttps://tracker.ceph.com/issues/11296?journal_id=521822015-05-19T20:10:31ZSamuel Justsjust@redhat.com
<ul><li><strong>Status</strong> changed from <i>12</i> to <i>7</i></li><li><strong>Regression</strong> set to <i>No</i></li></ul> Ceph - Bug #11296: "FAILED assert(!old_value.deleted())" in upgrade:giant-x-hammer-distro-basic-multi runhttps://tracker.ceph.com/issues/11296?journal_id=527972015-05-29T21:21:09ZSage Weilsage@newdream.net
<ul><li><strong>Target version</strong> set to <i>v9.0.3</i></li><li><strong>Backport</strong> set to <i>hammer</i></li></ul> Ceph - Bug #11296: "FAILED assert(!old_value.deleted())" in upgrade:giant-x-hammer-distro-basic-multi runhttps://tracker.ceph.com/issues/11296?journal_id=528002015-05-29T21:23:34ZSamuel Justsjust@redhat.com
<ul><li><strong>Status</strong> changed from <i>7</i> to <i>Pending Backport</i></li></ul> Ceph - Bug #11296: "FAILED assert(!old_value.deleted())" in upgrade:giant-x-hammer-distro-basic-multi runhttps://tracker.ceph.com/issues/11296?journal_id=543942015-07-01T17:45:39ZAbhishek Lekshmananabhishek.lekshmanan@gmail.com
<ul></ul><p>pull request (master): <a class="external" href="https://github.com/ceph/ceph/pull/4526">https://github.com/ceph/ceph/pull/4526</a></p> Ceph - Bug #11296: "FAILED assert(!old_value.deleted())" in upgrade:giant-x-hammer-distro-basic-multi runhttps://tracker.ceph.com/issues/11296?journal_id=543972015-07-01T17:49:05ZAbhishek Lekshmananabhishek.lekshmanan@gmail.com
<ul></ul><p>Hammer backport: <a class="external" href="https://github.com/ceph/ceph/pull/5121">https://github.com/ceph/ceph/pull/5121</a></p> Ceph - Bug #11296: "FAILED assert(!old_value.deleted())" in upgrade:giant-x-hammer-distro-basic-multi runhttps://tracker.ceph.com/issues/11296?journal_id=576292015-09-02T10:42:13ZLoïc Dacharyloic@dachary.org
<ul><li><strong>Status</strong> changed from <i>Pending Backport</i> to <i>Resolved</i></li></ul>