All of lore.kernel.org
 help / color / mirror / Atom feed
From: "J. Bruce Fields" <bfields@fieldses.org>
To: "Myklebust, Trond" <Trond.Myklebust@netapp.com>
Cc: Simo Sorce <simo@redhat.com>,
	"bfields@redhat.com" <bfields@redhat.com>,
	"linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH 0/4] Add support for new RPCSEC_GSS upcall mechanism for nfsd
Date: Fri, 13 Jul 2012 11:43:02 -0400	[thread overview]
Message-ID: <20120713154302.GA32660@fieldses.org> (raw)
In-Reply-To: <20120712221058.GC24162@fieldses.org>

On Thu, Jul 12, 2012 at 06:10:58PM -0400, J. Bruce Fields wrote:
> On Wed, Jul 11, 2012 at 05:49:09PM +0000, Myklebust, Trond wrote:
> > On Wed, 2012-07-11 at 13:27 -0400, J. Bruce Fields wrote:
> > > So, if the issue is: you want to be able to choose, in individual
> > > containers, which mechanism to use: whatever, we can probably make that
> > > work.
> > 
> > Please do then.
> > 
> > > If you're instead saying: it's not acceptable to use the gssproxy
> > > mechanism to authenticate v4.0 callbacks, the client must only ever use
> > > the existing mechanism: then I'd rather drop this entirely.  I don't
> > > want to merge an rpc server authentication upcall that's not acceptable
> > > for one of the rpc servers.
> > 
> > If it goes in, I want it to be optional so that it doesn't break working
> > setups. I also want upgrades/downgrades to be as painless as possible. 
> > 
> > See for instance the idmapper changes in 3.5 which are totally
> > transparent to the user: if the admin sets up /etc/request-key.conf to
> > use the new id_resolver, then that works, otherwise we just fall back to
> > using the old rpc.idmapd method. There are no module parameters etc
> > involved.
> 
> There's a module parameter here, but it's meant to be set by gssproxy to
> indicate it's listening for the new upcall.  In theory that should make
> the transition transparent.
> 
> However, if we need to make the choice per-namespace then we need to
> find some other mechanism (assuming there's no way to make module
> parameters per-namespace.)

So, perhaps something like: if the cache "channel" file for the
namespace associated with the current request isn't opened, then try the
new upcall.

--b.

  reply	other threads:[~2012-07-13 15:43 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-25 22:09 [PATCH 0/4] Add support for new RPCSEC_GSS upcall mechanism for nfsd Simo Sorce
2012-05-25 22:09 ` [PATCH 1/4] SUNRPC: conditionally return endtime from import_sec_context Simo Sorce
2012-05-25 22:09 ` [PATCH 2/4] SUNRPC: Document a bit RPCGSS handling in the NFS Server Simo Sorce
2012-05-25 22:09 ` [PATCH 3/4] SUNRPC: Add RPC based upcall mechanism for RPCGSS auth Simo Sorce
2012-05-25 22:09 ` [PATCH 4/4] SUNRPC: Use gssproxy upcall for nfsd's RPCGSS authentication Simo Sorce
2012-07-10 20:49 ` [PATCH 0/4] Add support for new RPCSEC_GSS upcall mechanism for nfsd J. Bruce Fields
2012-07-10 21:05   ` J. Bruce Fields
2012-07-12 12:39     ` J. Bruce Fields
2012-07-12 22:05       ` Simo Sorce
2012-07-12 22:42         ` J. Bruce Fields
2012-07-10 21:52   ` Myklebust, Trond
2012-07-10 21:56     ` J. Bruce Fields
2012-07-10 22:12       ` Myklebust, Trond
2012-07-10 22:25         ` Myklebust, Trond
2012-07-10 22:38           ` J. Bruce Fields
2012-07-10 22:58             ` Myklebust, Trond
2012-07-11 17:03               ` J. Bruce Fields
2012-07-11 17:27                 ` J. Bruce Fields
2012-07-11 17:49                   ` Myklebust, Trond
2012-07-12 22:10                     ` J. Bruce Fields
2012-07-13 15:43                       ` J. Bruce Fields [this message]
2012-08-08 19:36                         ` J. Bruce Fields
2012-08-08 19:43                           ` J. Bruce Fields
2012-08-08 20:12                             ` Stanislav Kinsbursky
2012-08-21 14:16                               ` J. Bruce Fields
2012-08-21 14:25                                 ` Myklebust, Trond
2012-08-21 14:29                                   ` J. Bruce Fields
2012-08-21 14:27                                 ` Stanislav Kinsbursky
2012-08-10 13:07                             ` Stanislav Kinsbursky
2012-07-11 11:15     ` Simo Sorce
2012-07-13 15:45 ` J. Bruce Fields
2012-07-13 15:55   ` Simo Sorce

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120713154302.GA32660@fieldses.org \
    --to=bfields@fieldses.org \
    --cc=Trond.Myklebust@netapp.com \
    --cc=bfields@redhat.com \
    --cc=linux-nfs@vger.kernel.org \
    --cc=simo@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.