https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2017-06-25T08:01:04ZCeph CephFS - Backport #20412: test_remote_update_write (tasks.cephfs.test_quota.TestQuota) fails in Jewel 10.2.8 integration testinghttps://tracker.ceph.com/issues/20412?journal_id=937262017-06-25T08:01:04ZNathan Cutlerncutler@suse.cz
<ul><li><strong>Description</strong> updated (<a title="View differences" href="/journals/93726/diff?detail_id=90891">diff</a>)</li></ul> CephFS - Backport #20412: test_remote_update_write (tasks.cephfs.test_quota.TestQuota) fails in Jewel 10.2.8 integration testinghttps://tracker.ceph.com/issues/20412?journal_id=937612017-06-26T19:49:46ZPatrick Donnellypdonnell@redhat.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>In Progress</i></li><li><strong>Assignee</strong> set to <i>Patrick Donnelly</i></li></ul><p>I dug into the logs. It looks like the MDS is not sending a quota update to the client. From a brief look at the code, it appears the quota updates are only broadcast for parents of the updated inode:</p>
<p><a class="external" href="https://github.com/ceph/ceph/blob/7f5bd004b35e2a1b60f4951e8a70646f0ad08b3f/src/mds/MDCache.cc#L2078">https://github.com/ceph/ceph/blob/7f5bd004b35e2a1b60f4951e8a70646f0ad08b3f/src/mds/MDCache.cc#L2078</a></p>
<p>I'm not sure I see how this ever worked. I'll do more testing.</p> CephFS - Backport #20412: test_remote_update_write (tasks.cephfs.test_quota.TestQuota) fails in Jewel 10.2.8 integration testinghttps://tracker.ceph.com/issues/20412?journal_id=937642017-06-26T20:21:51ZJohn Sprayjcspray@gmail.com
<ul></ul><p>Aargh, I think this might just be failing because this is a new test that was written for luminous, where client_quota is true by default -- this might pass on jewel if we update the test config to set client_quota (borrow the override from cfuse_workunit_quota.yaml into quota.yaml)</p> CephFS - Backport #20412: test_remote_update_write (tasks.cephfs.test_quota.TestQuota) fails in Jewel 10.2.8 integration testinghttps://tracker.ceph.com/issues/20412?journal_id=937812017-06-27T02:07:38ZPatrick Donnellypdonnell@redhat.com
<ul><li><strong>Status</strong> changed from <i>In Progress</i> to <i>Fix Under Review</i></li></ul><p>John, that looks like the problem. Here's a PR:</p>
<p><a class="external" href="https://github.com/ceph/ceph/pull/15936">https://github.com/ceph/ceph/pull/15936</a></p> CephFS - Backport #20412: test_remote_update_write (tasks.cephfs.test_quota.TestQuota) fails in Jewel 10.2.8 integration testinghttps://tracker.ceph.com/issues/20412?journal_id=937822017-06-27T02:21:37ZPatrick Donnellypdonnell@redhat.com
<ul></ul><p>Also: <a class="external" href="https://github.com/ceph/ceph/pull/15937">https://github.com/ceph/ceph/pull/15937</a></p> CephFS - Backport #20412: test_remote_update_write (tasks.cephfs.test_quota.TestQuota) fails in Jewel 10.2.8 integration testinghttps://tracker.ceph.com/issues/20412?journal_id=946202017-07-07T04:47:16ZPatrick Donnellypdonnell@redhat.com
<ul><li><strong>Status</strong> changed from <i>Fix Under Review</i> to <i>Resolved</i></li></ul> CephFS - Backport #20412: test_remote_update_write (tasks.cephfs.test_quota.TestQuota) fails in Jewel 10.2.8 integration testinghttps://tracker.ceph.com/issues/20412?journal_id=948752017-07-11T15:42:01ZNathan Cutlerncutler@suse.cz
<ul><li><strong>Tracker</strong> changed from <i>Bug</i> to <i>Backport</i></li><li><strong>Description</strong> updated (<a title="View differences" href="/journals/94875/diff?detail_id=92053">diff</a>)</li><li><strong>Priority</strong> changed from <i>High</i> to <i>Normal</i></li><li><strong>Target version</strong> set to <i>v10.2.8</i></li><li><strong>Release</strong> set to <i>jewel</i></li></ul><a name="description"></a>
<h3 >description<a href="#description" class="wiki-anchor">¶</a></h3>
<p>This test fails reproducibly on the wip-jewel-backports branch: <code>fs/recovery/{clusters/4-remote-clients.yaml debug/mds_client.yaml dirfrag/frag_enable.yaml mounts/ceph-fuse.yaml tasks/quota.yaml xfs.yaml}</code></p>
<p>Test Run: smithfarm-2017-06-25_07:23:04-fs-wip-jewel-backports-distro-basic-smithi</p>
<p>info: <a class="external" href="http://pulpito.ceph.com/smithfarm-2017-06-25_07:23:04-fs-wip-jewel-backports-distro-basic-smithi/">http://pulpito.ceph.com/smithfarm-2017-06-25_07:23:04-fs-wip-jewel-backports-distro-basic-smithi/</a><br />logs: <a class="external" href="http://qa-proxy.ceph.com/teuthology/smithfarm-2017-06-25_07:23:04-fs-wip-jewel-backports-distro-basic-smithi/">http://qa-proxy.ceph.com/teuthology/smithfarm-2017-06-25_07:23:04-fs-wip-jewel-backports-distro-basic-smithi/</a></p>
<p>Log of failing test: <a class="external" href="http://qa-proxy.ceph.com/teuthology/smithfarm-2017-06-25_07:23:04-fs-wip-jewel-backports-distro-basic-smithi/1324503/teuthology.log">http://qa-proxy.ceph.com/teuthology/smithfarm-2017-06-25_07:23:04-fs-wip-jewel-backports-distro-basic-smithi/1324503/teuthology.log</a></p>
<p>The failure emanates from these lines: <a class="external" href="https://github.com/ceph/ceph/blob/jewel/qa/tasks/cephfs/test_quota.py#L100-L103">https://github.com/ceph/ceph/blob/jewel/qa/tasks/cephfs/test_quota.py#L100-L103</a></p>