Skip to content
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

xADDomain and xADDomainController: Ability to enable/disable Global Catalog #75

Closed
PlagueHO opened this issue Mar 9, 2016 · 2 comments · Fixed by #255
Closed

xADDomain and xADDomainController: Ability to enable/disable Global Catalog #75

PlagueHO opened this issue Mar 9, 2016 · 2 comments · Fixed by #255
Labels
enhancement The issue is an enhancement request.

Comments

@PlagueHO
Copy link
Member

PlagueHO commented Mar 9, 2016

Currently all DC's promoted using xADDomain and xADDomainController are set as GC's.

This is fine for the first DC on in a domain (or site), but usually undesirable for additional DC's added to a domain (usually only one GC per site is recommended).

Could we add a parameter to disable GC when promoting a DC? If the DC is already promoted then this switch would need to disable GC on the DC by calling something like this:
http://blogs.msmvps.com/richardsiddaway/2012/03/18/enable-global-catalog/

This probably only needs to be added to the xADDomainController resource because there must always be at least one GC in a domain so the xADDomain should always create a GC, although, if the first DC on the domain (that was promoted using xADDomain) needed to have the GC removed then xADDomain should also support enabling/disabling the GC. Thoughts?

I don't think that this needs to be a tri-state switch (like the DNS one).

@kwirkykat kwirkykat added enhancement The issue is an enhancement request. help wanted The issue is up for grabs for anyone in the community. labels Aug 18, 2016
@Merto410
Copy link
Contributor

Merto410 commented Jan 8, 2017

Couple comments. Be careful about GC placement without planning your FSMO holders. The scenario you paint would be bad in a multi-domain forest since the infra master would be a GC and your other DC's would not.
I think this should be an option, but also I think your info on recommendations is outdated. In single domain forests it is expected that all DC's will be GC's (there is no impact either way really so the option isn't necessary in that scenario), and even in multi-domain forests the recommendation is to have all DC's be GC's unless specific conditions apply. VIA https://technet.microsoft.com/en-us/library/cc732877%28v=ws.10%29.aspx?f=255&MSPPError=-2147217396
"
In most cases, it is recommended that you include the global catalog when you install new domain controllers. The following exceptions apply:

Limited bandwidth: In remote sites, if the wide area network (WAN) link between the remote site and the hub site is limited, you can use universal group membership caching in the remote site to accommodate the logon needs of users in the site.

Infrastructure operations master role incompatibility: Do not place the global catalog on a domain controller that hosts the infrastructure operations master role in the domain unless all domain controllers in the domain are global catalog servers or the forest has only one domain.

"

@Merto410
Copy link
Contributor

Merto410 commented Jan 8, 2017

I am submitting a pull request with the IsGlobalCatalog option for the xAdDomainController.

@johlju johlju added in progress The issue is being actively worked on by someone. and removed help wanted The issue is up for grabs for anyone in the community. labels May 8, 2018
@johlju johlju changed the title xADDomain and xADDomainController - ability to enable/disable Global Catalogue xADDomain and xADDomainController: Ability to enable/disable Global Catalog May 8, 2018
johlju added a commit to johlju/ActiveDirectoryDsc that referenced this issue Apr 19, 2019
  - Added new parameter to disable or enable the Global Catalog (GC) (issue dsccommunity#75).
johlju added a commit to johlju/ActiveDirectoryDsc that referenced this issue Apr 19, 2019
  - Added new parameter to disable or enable the Global Catalog (GC) (issue dsccommunity#75).
johlju added a commit to johlju/ActiveDirectoryDsc that referenced this issue Apr 19, 2019
  - Added new parameter to disable or enable the Global Catalog (GC) (issue dsccommunity#75).
johlju added a commit to johlju/ActiveDirectoryDsc that referenced this issue Apr 19, 2019
  - Added new parameter to disable or enable the Global Catalog (GC) (issue dsccommunity#75).
johlju added a commit to johlju/ActiveDirectoryDsc that referenced this issue Apr 19, 2019
  - Added new parameter to disable or enable the Global Catalog (GC) (issue dsccommunity#75).
johlju added a commit to johlju/ActiveDirectoryDsc that referenced this issue Apr 19, 2019
  - Added new parameter to disable or enable the Global Catalog (GC) (issue dsccommunity#75).
johlju added a commit to johlju/ActiveDirectoryDsc that referenced this issue Apr 19, 2019
  - Added new parameter to disable or enable the Global Catalog (GC) (issue dsccommunity#75).
johlju added a commit to johlju/ActiveDirectoryDsc that referenced this issue Apr 20, 2019
  - Added new parameter to disable or enable the Global Catalog (GC) (issue dsccommunity#75).
johlju added a commit to johlju/ActiveDirectoryDsc that referenced this issue Apr 28, 2019
  - Added new parameter to disable or enable the Global Catalog (GC) (issue dsccommunity#75).
johlju added a commit to johlju/ActiveDirectoryDsc that referenced this issue Apr 30, 2019
  - Added new parameter to disable or enable the Global Catalog (GC) (issue dsccommunity#75).
johlju added a commit that referenced this issue May 2, 2019
- Changes to xADDomainController
  - Added new parameter to disable or enable the Global Catalog (GC) (issue #75).
  - Fixed a bug with the parameter `InstallationMediaPath` that it would
    not be added if it was specified in a configuration. Now the parameter
    `InstallationMediaPath` is correctly passed to `Install-ADDSDomainController`.
  - Refactored the resource with major code cleanup and localization.
  - Updated unit tests to latest unit test template, and refactored the
    tests for the function 'Set-TargetResource'.
  - Improved test code coverage.
@kwirkykat kwirkykat removed the in progress The issue is being actively worked on by someone. label May 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement The issue is an enhancement request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants