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
