Organising apiserver facades

[Menno Smits]

+1

This helps us get to towards the requested feature of having client and
agent facades served from different network spaces.

Having controller agent facades only served on localhost would also be nice
from a security perspective.

On 7 Jul 2017 11:17 am, "Andrew Wilkins" <andrew.wilkins at canonical.com>
wrote:

> On Thu, Jul 6, 2017 at 7:09 PM John Meinel <john at arbash-meinel.com> wrote:
>
>> I'd really like to see us split apart the facades-by-purpose. So we'd
>> collect the facades for Agents separately from facades for Users (and
>> possibly also facades for Controller).
>> I'm not sure if moving things just into 'facades' just moves the problem
>> around and leaves us with just a *different* directory that is a bit
>> cluttered.  But I'm +1 on things that would help organize the layout.
>>
>
> Cool. I was considering controller vs. agent already, separating client
> off sounds good to me too. I'll send a PR soon.
>
>
>> John
>> =:->
>>
>> On Thu, Jul 6, 2017 at 1:55 PM, Andrew Wilkins <
>> andrew.wilkins at canonical.com> wrote:
>>
>>> The juju/apiserver package currently has a whole lot of facade packages
>>> within it, as well as some other packages related to authentication,
>>> logging, and other bits and bobs. I find it difficult to navigate and tell
>>> what's what a lot of the time.
>>>
>>> I'd like to move the apiserver facade packages into a common "facades"
>>> sub-directory:
>>>   apiserver/facades/application
>>>   apiserver/facades/client
>>>   apiserver/facades/controller
>>>   etc.
>>>
>>> Any objections? Or alternative suggestions?
>>>
>>> Cheers,
>>> Andrew
>>>
>>> --
>>> Juju-dev mailing list
>>> Juju-dev at lists.ubuntu.com
>>> Modify settings or unsubscribe at: https://lists.ubuntu.com/
>>> mailman/listinfo/juju-dev
>>>
>>>
> --
> Juju-dev mailing list
> Juju-dev at lists.ubuntu.com
> Modify settings or unsubscribe at: https://lists.ubuntu.com/
> mailman/listinfo/juju-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/juju-dev/attachments/20170707/a62b91a5/attachment.html>