https://tracker.ceph.com/https://tracker.ceph.com/favicon.ico2015-09-14T21:53:40ZCeph Ceph - Bug #13089: mon: check for store writeablility before participating in electionhttps://tracker.ceph.com/issues/13089?journal_id=585342015-09-14T21:53:40ZGreg Farnumgfarnum@redhat.com
<ul></ul><p>Mmm, I'm not sure we want to do the naive thing (write a file and fsync) on every election as that will slow them down a lot in the common case. Probably if we just check store accessibility on bootup, flip a flag if we ever fail one, and check for it on every election?</p> Ceph - Bug #13089: mon: check for store writeablility before participating in electionhttps://tracker.ceph.com/issues/13089?journal_id=589862015-09-22T14:58:03ZSage Weilsage@newdream.net
<ul></ul><p>Greg Farnum wrote:</p>
<blockquote>
<p>Mmm, I'm not sure we want to do the naive thing (write a file and fsync) on every election as that will slow them down a lot in the common case. Probably if we just check store accessibility on bootup, flip a flag if we ever fail one, and check for it on every election?</p>
</blockquote>
<p>Yeah I think a check on boot is sufficient.</p> Ceph - Bug #13089: mon: check for store writeablility before participating in electionhttps://tracker.ceph.com/issues/13089?journal_id=589882015-09-22T14:59:36ZKefu Chaitchaikov@gmail.com
<ul><li><strong>Assignee</strong> set to <i>Kefu Chai</i></li></ul> Ceph - Bug #13089: mon: check for store writeablility before participating in electionhttps://tracker.ceph.com/issues/13089?journal_id=589922015-09-22T16:28:09ZJoao Eduardo Luis
<ul></ul><p>Can we have a bit more context please? What's the rationale behind this? The monitor should not start if leveldb is not open, and since it's not open it's not writeable. Is this in case the disk is full or the fs is read-only? Or something else I'm missing?</p> Ceph - Bug #13089: mon: check for store writeablility before participating in electionhttps://tracker.ceph.com/issues/13089?journal_id=590292015-09-23T02:40:18ZKefu Chaitchaikov@gmail.com
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Fix Under Review</i></li><li><strong>Assignee</strong> changed from <i>Kefu Chai</i> to <i>Sage Weil</i></li></ul><p><a class="external" href="https://github.com/ceph/ceph/pull/6040">https://github.com/ceph/ceph/pull/6040</a></p> Ceph - Bug #13089: mon: check for store writeablility before participating in electionhttps://tracker.ceph.com/issues/13089?journal_id=590302015-09-23T02:50:28ZKefu Chaitchaikov@gmail.com
<ul><li><strong>Tags</strong> set to <i>hammer</i></li></ul> Ceph - Bug #13089: mon: check for store writeablility before participating in electionhttps://tracker.ceph.com/issues/13089?journal_id=590312015-09-23T02:50:46ZKefu Chaitchaikov@gmail.com
<ul><li><strong>Tags</strong> deleted (<del><i>hammer</i></del>)</li><li><strong>Backport</strong> set to <i>hammer</i></li></ul> Ceph - Bug #13089: mon: check for store writeablility before participating in electionhttps://tracker.ceph.com/issues/13089?journal_id=591662015-09-25T12:54:17ZSage Weilsage@newdream.net
<ul><li><strong>Status</strong> changed from <i>Fix Under Review</i> to <i>Pending Backport</i></li></ul> Ceph - Bug #13089: mon: check for store writeablility before participating in electionhttps://tracker.ceph.com/issues/13089?journal_id=594892015-10-02T11:22:11ZLoïc Dacharyloic@dachary.org
<ul><li><strong>Backport</strong> changed from <i>hammer</i> to <i>hammer,firefly</i></li></ul> Ceph - Bug #13089: mon: check for store writeablility before participating in electionhttps://tracker.ceph.com/issues/13089?journal_id=613092015-11-10T11:15:06ZLoïc Dacharyloic@dachary.org
<ul><li><strong>Status</strong> changed from <i>Pending Backport</i> to <i>Resolved</i></li></ul> Ceph - Bug #13089: mon: check for store writeablility before participating in electionhttps://tracker.ceph.com/issues/13089?journal_id=653092016-02-05T09:59:28ZKefu Chaitchaikov@gmail.com
<ul></ul><p>please note that the background compaction fails, if leveldb can not create new tables or remove merged tables.</p>
<p>so, if we "chmod -w store.db". and leveldb complains in this case, but it does not return error on Write() unless "leveldb_paranoid" is true, but this option is "false" by default. so "failed to write to db" assertion is not triggered, even if compaction fails with the default settings.</p>