Re: Tenant Implementation in Sling

2014-09-05 Thread Vineet
with configuration factory service even? -- View this message in context: http://apache-sling.73963.n3.nabble.com/Tenant-Implementation-in-Sling-tp4031217p4039070.html Sent from the Sling - Dev mailing list archive at Nabble.com.

Re: Tenant Implementation in Sling

2014-09-02 Thread Felix Meschberger
to ConfigAdmin as well so that an osgi service could have different configurations for different tenants, just like we have based on run-modes? -- View this message in context: http://apache-sling.73963.n3.nabble.com/Tenant-Implementation-in-Sling-tp4031217p4038579.html Sent from the Sling

Re: Tenant Implementation in Sling

2014-08-29 Thread Vineet
Hi all, Is there any support added to ConfigAdmin as well so that an osgi service could have different configurations for different tenants, just like we have based on run-modes? -- View this message in context: http://apache-sling.73963.n3.nabble.com/Tenant-Implementation-in-Sling

Re: Tenant Implementation in Sling

2014-02-25 Thread Bertrand Delacretaz
On Tue, Feb 25, 2014 at 2:41 AM, Andreas Schaefer Sr. schaef...@me.com wrote: ...it might a good idea to add the getSearchPathExtension() to the Tenant... Shouldn't that rather be called getAdditionalSearchPaths? In Sling, extension means the .html at the end of a URL, IMO we should avoid

Re: Tenant Implementation in Sling

2014-02-25 Thread Andreas Schaefer Sr.
I still like my TenantAware approach :) as by this you explicitely mark your component to handle things Tenant specific. Do you mean with that that we place the Tenant on a Thread Local which then can be obtained from the Servlet Resolver? If so do you intent to place the Tenant into the

Re: Tenant Implementation in Sling

2014-02-25 Thread Andreas Schaefer Sr.
Ok, will adjust the wiki page. - Andy On Feb 25, 2014, at 1:11 AM, Bertrand Delacretaz bdelacre...@apache.org wrote: On Tue, Feb 25, 2014 at 2:41 AM, Andreas Schaefer Sr. schaef...@me.com wrote: ...it might a good idea to add the getSearchPathExtension() to the Tenant... Shouldn't that

Re: Tenant Implementation in Sling

2014-02-25 Thread Carsten Ziegeler
I think using a thread local is an implementation detail and TenantAware has a single method like TenantResolver getTenantResolver() while TenantResolver has a single method Tenant getCurrentTenant() (Don't quote me on the names, its just the first option which came to my mind) And then

Re: Tenant Implementation in Sling

2014-02-24 Thread Bertrand Delacretaz
On Fri, Feb 21, 2014 at 5:19 PM, Andreas Schaefer Sr. schaef...@me.com wrote: ...My confluence name is ‘schaefera’ Ok, I have added you to the sling-contributors group, you should now have write access to https://cwiki.apache.org/confluence/display BTW I saw that there is a page called

Re: Tenant Implementation in Sling

2014-02-24 Thread Andreas Schaefer Sr.
Hi Thanks. I’ll created the page here: https://cwiki.apache.org/confluence/display/SLING/Multitenancy+Support+Integration and also created a ticket for it here: https://issues.apache.org/jira/browse/SLING-3414 - Andy On Feb 24, 2014, at 5:08 AM, Bertrand Delacretaz bdelacre...@apache.org

Re: Tenant Implementation in Sling

2014-02-24 Thread Andreas Schaefer Sr.
Hi Carsten Even if a Search Path Extension is not used outside of the Tenants I still think it would be great to keep things separate. Based on my tests if the Search Path Extension is set when the Resource Resolver of a request is created and the Servlet resolver’s own Resource Resolver is

Re: Tenant Implementation in Sling

2014-02-24 Thread Carsten Ziegeler
If you want the method to get the search paths extensions to be on the resource resolver and make it independent of a tenant, then why do we need this method at all? In the case of a tenant getSearchPath would return the normal search path with the tenant specific one prepended. Carsten

Re: Tenant Implementation in Sling

2014-02-24 Thread Andreas Schaefer Sr.
If we have the search path extension separate and it is settable we do the following with it: 1) We can set it with a Servlet Filter instead of having it to hook into the Engine 2) In the Servlet Resolver the Search Path that did deliver the Servlet can be used in order to manage the cache.

Re: Tenant Implementation in Sling

2014-02-24 Thread Carsten Ziegeler
Right, 2) and also 3) are good reasons to keep it separate. I think it makes sense to link the search path extension somehow to the tenant. In the old proposal ( https://cwiki.apache.org/confluence/display/SLING/Multitenancy+Support) we suggested to have a special method on the resource resolver

Re: Tenant Implementation in Sling

2014-02-21 Thread Bertrand Delacretaz
Hi, On Fri, Feb 21, 2014 at 7:20 AM, Andreas Schaefer Sr. schaef...@me.com wrote: ...I started to look into how to add tenant support to Sling I haven't looked at the details but my feeling is that tenant means different things to different people, so it would be good to have a list of use

Re: Tenant Implementation in Sling

2014-02-21 Thread Andreas Schaefer Sr.
Hi Carsten First I want to clarify that the Search Path Extension has nothing to do with Tenants per se but is a “per-call” extension of the search path which could be used without tenants.In order to make the overlays work the Search Path Extension must be set early on. For this part the

Re: Tenant Implementation in Sling

2014-02-21 Thread Andreas Schaefer Sr.
Hi Yeah, I can do that. My confluence name is ‘schaefera’. BTW I saw that there is a page called Multitenancy Support” which talks about the Tenant. I would like to create a page called Multitenancy Integration” where I would talk about how to use Tenants within Sling. - Andy On Feb 21,

Re: Tenant Implementation in Sling

2014-02-21 Thread Carsten Ziegeler
Hi Andy, I'm not sure if we need search path extensions which are not related to tenants. The adaption to a tenant does not tie it to the resource resolver module, the adapter factory can live in the tenant module and therefore the resource resolver module is totally unaware of tenant handling.

Re: Tenant Implementation in Sling

2014-02-20 Thread Andreas Schaefer Sr.
Hi I started to look into how to add tenant support to Sling. This is my test scenario: I have an /apps/foo/bar/html.esp and a /apps/tenant1/foo/bar/html.esp and a /content/mynode entry. The title of the second html.esp is different so that I know which one is loaded. Changes to the code:

Re: Tenant Implementation in Sling

2014-02-20 Thread Andreas Schaefer Sr.
Sorry, I forgot to mention that the Search Path Extension is prepended to the Search Path when it is requested (in the Resource Resolver). - Andy On Feb 20, 2014, at 10:20 PM, Andreas Schaefer Sr. schaef...@me.com wrote: Hi I started to look into how to add tenant support to Sling. This

Re: Tenant Implementation in Sling

2014-02-20 Thread Carsten Ziegeler
Hi Andy, this sounds interesting - and I guess a patch would be great. Now I just would like to present my idea again - just for the sake of discussion :) I think over time there will be more components than just the servlet resolver which make use of the tenant and the extended search path, so

Tenant Implementation in Sling

2014-02-17 Thread Andreas Schaefer Sr.
Hi I am working for a client which needs support for tenants and because the current implementation of the Tenants in Sling is just that but no integration I started to code a workaround. For now I have a patch that does the trick but it is not clean because I use a Servlet Filter to place the

Re: Tenant Implementation in Sling

2014-02-17 Thread Henry Saginor
Hi Andy, Thank you for bringing this up. I have a similar requirement. I don’t see any way of integrating Tenants other than patching Servlet Resolver. This is what I had done for my customer but for a really specific case (they are not truly multi-tenant). I also had to use ThreadLocal. I