https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2016-12-08T07:05:56ZCeph Ceph - Backport #18193: jewel: transient jerasure unit test failureshttps://tracker.ceph.com/issues/18193?journal_id=827362016-12-08T07:05:56ZLoïc Dacharyloic@dachary.org
<ul><li><strong>Copied from</strong> <i><a class="issue tracker-1 status-3 priority-6 priority-high2 closed" href="/issues/17762">Bug #17762</a>: transient jerasure unit test failures</i> added</li></ul> Ceph - Backport #18193: jewel: transient jerasure unit test failureshttps://tracker.ceph.com/issues/18193?journal_id=848332017-01-26T04:04:59ZNathan Cutlerncutler@suse.cz
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Need More Info</i></li><li><strong>Assignee</strong> set to <i>Kefu Chai</i></li></ul><p>Kefu: one of the commits in this patchset removes "int main()" from a bunch of test programs. I suspect that this is a cmake-specific change and that "int main()" is still needed in jewel, but I'm not sure so I'm assigning to you to clarify.</p>
<p>Also, do you think this is really worth backporting to jewel? The changes are pretty extensive.</p> Ceph - Backport #18193: jewel: transient jerasure unit test failureshttps://tracker.ceph.com/issues/18193?journal_id=878302017-03-22T06:39:01ZKefu Chaitchaikov@gmail.com
<ul></ul><p>Nathan, sorry, i just noticed your inquiry.</p>
<p>regarding to the removed main(), please see <a class="external" href="https://github.com/ceph/ceph/pull/11733/commits/cb1cda96713b2ec0f6418c4cbe3d964c2020729c#diff-f20df8ef28538009594974bd56764285">https://github.com/ceph/ceph/pull/11733/commits/cb1cda96713b2ec0f6418c4cbe3d964c2020729c#diff-f20df8ef28538009594974bd56764285</a></p>
<p>"unit.h" is included in the files where main() is removed, so it will take care of this. and it's not the a cmake-specific change.</p>
<p>right, the change is pretty extensive. and it fixes the problem we run into when the program exits. so i am fine if you think we should not backport it. but the test failure is always annoying, every time we spot it (even in an released branch), we need to root cause it or at least try to recall if it's a known one. and this unit test failure will stay with us for around two years (two LTS releases).</p> Ceph - Backport #18193: jewel: transient jerasure unit test failureshttps://tracker.ceph.com/issues/18193?journal_id=878312017-03-22T06:39:26ZKefu Chaitchaikov@gmail.com
<ul><li><strong>Assignee</strong> changed from <i>Kefu Chai</i> to <i>Nathan Cutler</i></li></ul> Ceph - Backport #18193: jewel: transient jerasure unit test failureshttps://tracker.ceph.com/issues/18193?journal_id=878342017-03-22T08:32:41ZNathan Cutlerncutler@suse.cz
<ul><li><strong>Status</strong> changed from <i>Need More Info</i> to <i>New</i></li></ul> Ceph - Backport #18193: jewel: transient jerasure unit test failureshttps://tracker.ceph.com/issues/18193?journal_id=878352017-03-22T08:34:49ZNathan Cutlerncutler@suse.cz
<ul></ul><p>@Kefu If there is no "more minimal" way to fix the test failure in jewel, then I guess we'll have to take it? I guess it's OK since the changes are only in unit test code and are tested by running make check?</p> Ceph - Backport #18193: jewel: transient jerasure unit test failureshttps://tracker.ceph.com/issues/18193?journal_id=900872017-04-20T22:28:53ZNathan Cutlerncutler@suse.cz
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/90087/diff?detail_id=87112">diff</a>)</li><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li></ul> Ceph - Backport #18193: jewel: transient jerasure unit test failureshttps://tracker.ceph.com/issues/18193?journal_id=904022017-04-26T21:28:28ZNathan Cutlerncutler@suse.cz
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Resolved</i></li><li><strong>Target version</strong> set to <i>v10.2.8</i></li></ul>