https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2018-12-02T02:47:22ZCeph Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1255642018-12-02T02:47:22Ztao ningningt0509@gmail.com
<ul></ul><p>Operation steps: <br />1. Down and out an OSD; <br />2. Execute balancer plan</p> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1255682018-12-03T06:51:18Zxie xingguo258156334@qq.com
<ul></ul><p>I guess it should be fixed by <a class="external" href="https://github.com/ceph/ceph/pull/22630">https://github.com/ceph/ceph/pull/22630</a>, can you try that?</p> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1255692018-12-03T06:51:42Zxie xingguo258156334@qq.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>4</i></li></ul> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1255702018-12-03T07:08:19Ztao ningningt0509@gmail.com
<ul></ul><p>xie xingguo wrote:</p>
<blockquote>
<p>I guess it should be fixed by <a class="external" href="https://github.com/ceph/ceph/pull/22630">https://github.com/ceph/ceph/pull/22630</a>, can you try that?</p>
</blockquote>
<p>I have included this PR, otherwise how could such an assert appear?</p> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1255712018-12-03T07:41:47Zxie xingguo258156334@qq.com
<ul></ul><p>tao ning wrote:</p>
<blockquote>
<p>xie xingguo wrote:</p>
<blockquote>
<p>I guess it should be fixed by <a class="external" href="https://github.com/ceph/ceph/pull/22630">https://github.com/ceph/ceph/pull/22630</a>, can you try that?</p>
</blockquote>
<p>I have included this PR, otherwise how could such an assert appear?</p>
</blockquote>
<p>I guess the pool-type is EC? Can you post a complete log? (With log level 20, probably)</p> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1255722018-12-03T07:49:04Ztao ningningt0509@gmail.com
<ul></ul><p>xie xingguo wrote:</p>
<blockquote>
<p>tao ning wrote:</p>
<blockquote>
<p>xie xingguo wrote:</p>
<blockquote>
<p>I guess it should be fixed by <a class="external" href="https://github.com/ceph/ceph/pull/22630">https://github.com/ceph/ceph/pull/22630</a>, can you try that?</p>
</blockquote>
<p>I have included this PR, otherwise how could such an assert appear?</p>
</blockquote>
<p>I guess the pool-type is EC? Can you post a complete log?</p>
</blockquote>
<p>Yeah, the log might not be there, so let me try to reproduce that</p> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1256682018-12-06T09:53:15Ztao ningningt0509@gmail.com
<ul><li><strong>File</strong> <a href="/attachments/download/3840/Mgr%20balancer%E7%9B%B8%E5%85%B3PR%EF%BC%88From%2012.2.7%EF%BC%89.txt">Mgr balancer相关PR(From 12.2.7).txt</a> <a class="icon-only icon-magnifier" title="View" href="/attachments/3840/Mgr%20balancer%E7%9B%B8%E5%85%B3PR%EF%BC%88From%2012.2.7%EF%BC%89.txt">View</a> added</li><li><strong>File</strong> <a href="/attachments/download/3842/osd.dump.txt">osd.dump.txt</a> <a class="icon-only icon-magnifier" title="View" href="/attachments/3842/osd.dump.txt">View</a> added</li><li><strong>File</strong> <a href="/attachments/download/3841/osd.tree.txt">osd.tree.txt</a> <a class="icon-only icon-magnifier" title="View" href="/attachments/3841/osd.tree.txt">View</a> added</li></ul><p>tao ning wrote:</p>
<blockquote>
<p>xie xingguo wrote:</p>
<blockquote>
<p>tao ning wrote:</p>
<blockquote>
<p>xie xingguo wrote:</p>
<blockquote>
<p>I guess it should be fixed by <a class="external" href="https://github.com/ceph/ceph/pull/22630">https://github.com/ceph/ceph/pull/22630</a>, can you try that?</p>
</blockquote>
<p>I have included this PR, otherwise how could such an assert appear?</p>
</blockquote>
<p>I guess the pool-type is EC? Can you post a complete log?</p>
</blockquote>
<p>Yeah, the log might not be there, so let me try to reproduce that</p>
</blockquote>
<p>I found a way to do that, so I just put OSD out, not down</p> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1256692018-12-06T09:53:53Ztao ningningt0509@gmail.com
<ul><li><strong>File</strong> <a href="/attachments/download/3843/mgr.x.log">mgr.x.log</a> <a class="icon-only icon-magnifier" title="View" href="/attachments/3843/mgr.x.log">View</a> added</li></ul> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1256712018-12-06T11:49:13Ztao ningningt0509@gmail.com
<ul></ul><p>Please see if there is any problem with this modification</p>
<p><code>@ -4212,10 +4216,12 </code>@ int OSDMap::calc_pg_upmaps(<br /> if (q.second == osd) {<br /> ldout(cct, 10) << " dropping pg_upmap_items " << pg<br /> << " " << p->second << dendl;<br />- for (auto i : p->second) {<br />- pgs_by_osd[i.second].erase(pg);<br />- pgs_by_osd[i.first].insert(pg);<br />- }<br />+ for (auto i : p->second) {<br />+ if (pgs_by_osd.count(i.second)) {<br />+ pgs_by_osd[i.second].erase(pg);<br />+ }<br />+ pgs_by_osd[i.first].insert(pg);<br />+ }</p> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1264652018-12-21T01:11:36Zxie xingguo258156334@qq.com
<ul></ul><p><a class="external" href="https://github.com/ceph/ceph/pull/25630">https://github.com/ceph/ceph/pull/25630</a></p> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1265152018-12-22T00:34:32Zxie xingguo258156334@qq.com
<ul><li><strong>Status</strong> changed from <i>4</i> to <i>Pending Backport</i></li><li><strong>Backport</strong> set to <i>luminous,mimic</i></li></ul> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1265312018-12-22T14:32:36ZNathan Cutlerncutler@suse.cz
<ul><li><strong>Subject</strong> changed from <i>luminous:Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))</i> to <i>Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))</i></li></ul> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1265442018-12-22T14:33:59ZNathan Cutlerncutler@suse.cz
<ul><li><strong>Copied to</strong> <i><a class="issue tracker-9 status-3 priority-4 priority-default closed" href="/issues/37743">Backport #37743</a>: luminous: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))</i> added</li></ul> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1265462018-12-22T14:34:07ZNathan Cutlerncutler@suse.cz
<ul><li><strong>Copied to</strong> <i><a class="issue tracker-9 status-3 priority-4 priority-default closed" href="/issues/37744">Backport #37744</a>: mimic: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))</i> added</li></ul> Ceph - Bug #37501: Mgr: OSDMap.cc: 4140: FAILED assert(osd_weight.count(i.first))https://tracker.ceph.com/issues/37501?journal_id=1283062019-01-30T12:53:08ZNathan Cutlerncutler@suse.cz
<ul><li><strong>Status</strong> changed from <i>Pending Backport</i> to <i>Resolved</i></li></ul>