Usability of 'samba-tool domain provision'

[Rowland Penny]

On 04/05/16 23:01, Andrew Bartlett wrote:
> On Wed, 2016-05-04 at 11:09 +0100, Rowland Penny wrote:
>> On 04/05/16 10:29, Peter C. Kelly wrote:
>>> Andreas said:
>>>
>>>> This documents issues I
>>>> found during provisioning. samba-tool is far away from being user
>>>> friendly.
>>>> a) It isn't clear what domain is from 'samba-tool domain provison
>>>> --help'
>>> I have raised bug 11894, and attached a patch which addresses this
>>> point as per Rowland's suggested wording.
>>>
>>>> b) The --realm option is essential but at the end of the help ...
>>> The reason that the realm is way down the bottom is that it is
>>> under a
>>> heading "Samba Common Options", along with configFile debuglevel
>>> and
>>> option.
>>>
>>> What is currently holding me up from developing the second trivial
>>> patch is a need for a bit more clarity on how this --help output
>>> should look.  It seems to me that if the --domain parameter is
>>> compulsory, then the heading text should say so,
>>> and that if a realm has to be specified (or read in from an
>>> existing
>>> conf file) then perhaps that should be mentioned in the
>>> introductory
>>> text as well.
>>>
>>> As I am a first time contributor I would be happy to make these
>>> trivial changes, if I can get further input on what changes are
>>> desirable.
>>>
>>> Thanks,
>>>
>>> Peter
>>>
>> Hi, I did some work on this and sent Andreas a new domain.py that
>> addressed all of the issues he raised, I am still awaiting a reply
>> from
>> Andreas (hint, hint)
> Rowland,
>
> Can you please post the patches to the mailing list?
>
> I'm planning on merging the trivial typos in the meantime, it makes no
> sense to block a new contributor while you look at the larger issues.
>
> Thanks,
>
> Andrew Bartlett
> -- 
> Andrew Bartlett
> https://samba.org/~abartlet/
> Authentication Developer, Samba Team         https://samba.org
> Samba Development and Support, Catalyst IT
> https://catalyst.net.nz/services/samba
>
>
>
>
>
>

OK, here is the patch I am working on, it changes the provision command 
somewhat, you can now only provision a DC, for instance.

It is a work in progress, I have not tested it fully yet.

Rowland
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Samba-tool-domain-provision.patch
Type: text/x-diff
Size: 10341 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20160505/0b7ea1ea/Samba-tool-domain-provision.diff>