-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Join to existing active directory #18
Comments
I'll try to incorporate this into the module, if you give me some pointers. What should I set the role to be (it can't be 'dc') or how do I detect the domain already exists? |
At first glance, the best way to implement it would be to rename the "dc" class in a "domain" class and make it more generic (exposing the samba-tool command line arguments, more options, etc. In theory, the "dc" class already permits to override most parameters in smb.conf). Then, make specialized classes, one reimplementing the dc class with the current API, an other implementing a "Domain Member" class for example. A generic class could be interesting for other use cases like replicas. For the server role, it's probably member/member server, but I've never deployed this kind of setup. On how to detect if the domain already exists, in the dc class I check if the /var/lib/samba/states/sysvol/ directory exists (note: /var/lib/samba could change, it's only the default). I don't know if the same check could be done for a domain member. For testing if the join is OK, in the Classic class, I run net ads testjoin. I lack the infrastructure to test these kind of setups, specially, I don't have a Windows AD to play with. |
I'll try to take a look at this as I want to solve the problem for my One thought I had is that, in samba terms, you are either a dc or a member I'm also short on infrastructure and only have a samba environment, no On Tue, 9 Aug 2016 at 09:50 Carpentier Pierre-Francois <
|
Looks like only need to change the samba-tool cmd.
https://wiki.samba.org/index.php/Join_an_additional_Samba_DC_to_an_existing_Active_Directory
The text was updated successfully, but these errors were encountered: