From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sage Weil Subject: RE: wip-proxy-write and (non-idempotent) client ops Date: Tue, 20 Jan 2015 18:30:12 -0800 (PST) Message-ID: References: <06E7D85B3BA36C4DB207FEDE871C5348979125@SHSMSX101.ccr.corp.intel.com> <06E7D85B3BA36C4DB207FEDE871C53489792FD@SHSMSX101.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Return-path: Received: from mx1.redhat.com ([209.132.183.28]:34056 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753448AbbAUCaP (ORCPT ); Tue, 20 Jan 2015 21:30:15 -0500 In-Reply-To: <06E7D85B3BA36C4DB207FEDE871C53489792FD@SHSMSX101.ccr.corp.intel.com> Sender: ceph-devel-owner@vger.kernel.org List-ID: To: "Wang, Zhiqiang" Cc: "sjust@redhat.com" , "ceph-devel@vger.kernel.org" On Wed, 21 Jan 2015, Wang, Zhiqiang wrote: > Is it sufficient to only preserve the list of osd_reqid_t? It's able to > match dup ops. But it can't tell if the op is already completed, acked > or still undergoing. > > However, maybe we could say these ops have completed since they are from > the base tier and we just do a RWORDERED promotion. That is, all the ops > before initiating the promotion have completed in base tier, and all the > ops after initiating the promotion are requeued after the promotion. > Sounds right? Yeah exactly. Except I think there shouldn't be any write ops after the promotion starts since the cache tier won't do that (it will start blocking writes once a promotion is in progress). Either way, I think it's 1- add vector to object_info_t, populate it on write, and check it for dups when we check the pg log. make a config tunable and/or a pg_pool_t tunable to control how many to keep. 2- add it to the object_copy_data_t so that promote and flush can preserve it ? sage > > -----Original Message----- > From: Sage Weil [mailto:sweil@redhat.com] > Sent: Tuesday, January 20, 2015 11:06 PM > To: Wang, Zhiqiang > Cc: sjust@redhat.com; ceph-devel@vger.kernel.org > Subject: RE: wip-proxy-write and (non-idempotent) client ops > > On Tue, 20 Jan 2015, Wang, Zhiqiang wrote: > > Do we have any proposed solutions for this problem? Copy the needed > > info from base tier to cache tier during promotion? I see it has been > > there for over 6 months. > > Yeah... > > 1. keep a list of osd_reqid_t's in each object_info_t and match against that for dup ops (i forget if the patch for this already went in?). > there should probably be a tunable for the max list len and age cutoff. > > 2. preserve that list on copy-from when a flag is specified so that we preserve it for both promote and flush. > > sage > > > > > > -----Original Message----- > > From: Sage Weil [mailto:sweil@redhat.com] > > Sent: Tuesday, January 20, 2015 12:51 AM > > To: Wang, Zhiqiang; sjust@redhat.com; ceph-devel@vger.kernel.org > > Subject: wip-proxy-write and (non-idempotent) client ops > > > > Consider: > > > > 1- primary rx client delete > > proxy delete to base pool > > 2- primary initiate promote (list-snaps, copy-from) > > 3- primary rx delete reply > > 4- primary tx client reply > > 5- socket failure drops client reply > > 6- primary rx promote completion (enoent), writes a whiteout > > 7- client resents delete > > 8- primary replies with ENOENT > > > > i.e., the problem seems to be that delete is not idempotent and we can't tell that the same client op is what triggered the delete. > > > > We could special case delete since that is where this is noticeable, but I think the bigger problem is that the op history that is used for dup op detection is not preserved across the cache and base tier. That is, this is another variation on this ticket: > > > > http://tracker.ceph.com/issues/8935 > > > > I have this sinking feeling we need to properly address that problem before we can do the write proxying... > > > > sage > > -- > > To unsubscribe from this list: send the line "unsubscribe ceph-devel" > > in the body of a message to majordomo@vger.kernel.org More majordomo > > info at http://vger.kernel.org/majordomo-info.html > > > > > >