Yes, I think we should also the scrub_interval. OSD::sched_scrub() { if ((double)diff < cct->_conf->osd_scrub_min_interval) { dout(10) << "sched_scrub " << pgid << " at " << t << ": " << (double)diff << " < min (" << cct->_conf->osd_scrub_min_interval << " seconds)" << dendl; break; } if ((double)diff < cct->_conf->osd_scrub_max_interval && !load_is_low) { // save ourselves some effort dout(10) << "sched_scrub " << pgid << " high load at " << t << ": " << (double)diff << " < max (" << cct->_conf->osd_scrub_max_interval << " seconds)" << dendl; break; } } 2015-10-29 8:26 GMT+08:00 David Zafman <dzafman@xxxxxxxxxx>: > > Good point. In my previous response I did "echo garbage > > ........./foo__head_7FC1F406__1" to corrupt a replica. > > David > > > On 10/28/15 5:13 PM, Sage Weil wrote: >> >> Becuse you *just* wrote the object, and the FileStore caches open file >> handles. Vim renames a new inode over the old one so the open inode is >> untouched. >> >> If you restart the osd and then scrub you'll see the error. >> >> sage >> > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Regards, xinze -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html