<div><div dir="auto">Pretty sure that's true and mostly optimistic on the part of the db replicator which is more in the data path for replication - than say rsync object replication. </div><div dir="auto"><br></div><div dir="auto">If you look at ssync or ec reconstructor you'll see it's quite possible for them to trip built in DiskFile quarantine behavior during replication as well!</div><div dir="auto"><br></div><div dir="auto">I wouldn't say it's "by design" but we've never been shy to quarantine anytime we encounter invalid data - the storage wsgi servers can quarantine for example - if you happen to hit the right object at the right time. </div><div dir="auto"><br></div><div dir="auto">Why?  ;)</div><div dir="auto"><br></div><div dir="auto">-Clay</div><br><div class="gmail_quote"><div>On Wed, Jun 14, 2017 at 6:16 PM Mark Kirkwood <<a href="mailto:mark.kirkwood@catalyst.net.nz">mark.kirkwood@catalyst.net.nz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This was highlighted to me recently, and after doing some experiments it<br>
seems that the 'job descriptions' for replicators vs auditors is<br>
different for objects vs containers (and probably accounts).<br>
<br>
For objects:<br>
<br>
- replicator will move objects to where they should go<br>
<br>
- auditor will check validity + quarantine as required<br>
<br>
<br>
For containers (and possibly accounts - haven't checked but code looks<br>
similar)<br>
<br>
- replicator will move objects to where they should go and check<br>
validity + quarantine as required<br>
<br>
- auditor will check validity and quarantine as required<br>
<br>
<br>
This looks a bit like the container replicator is treading on the<br>
auditors toes - is this intended and/or am I missing something?<br>
<br>
<br>
In terms of *why* this happens, it seems to be due to:<br>
<br>
- ContainerBroker.get_info using a self.get which resolves to<br>
DatabaseBroker.get<br>
<br>
- DatabaseBroker.get calls self.possibly_quarantine<br>
<br>
So anywhere that these brokers call 'get_info' results in a potential<br>
quarantine...this *looks* like it might be accidental function creepage<br>
- thoughts?<br>
<br>
(This code analysis done on Mitaka/2.7.0)<br>
<br>
<br>
regards<br>
<br>
Mark<br>
<br>
<br>
<br>
_______________________________________________<br>
Mailing list: <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack</a><br>
Post to     : <a href="mailto:openstack@lists.openstack.org" target="_blank">openstack@lists.openstack.org</a><br>
Unsubscribe : <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack</a><br>
</blockquote></div></div>