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

xDFSReplicationGroup - Currently requires DomainName even though README states optional #28

Open
solbirn opened this issue Apr 26, 2017 · 2 comments
Assignees
Labels
bug The issue is a bug. help wanted The issue is up for grabs for anyone in the community.

Comments

@solbirn
Copy link

solbirn commented Apr 26, 2017

Set-TargetResource populates $ExisitingMembers via FQDN ($ExistingMembers = (Get-DfsrMember @Splat -ErrorAction Stop).DnsName), which causes an error when $DomainName parameter is not populated and $Members parameter is not populated via FQDN.

PowerShell DSC resource MSFT_xDFSReplicationGroup  failed to execute Set-TargetResource functionality with error message: The running command
stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: A computer with the specified name
already exists and cannot be added.
    + CategoryInfo          : InvalidOperation: (:) [], CimException
    + FullyQualifiedErrorId : ProviderOperationExecutionFailure
    + PSComputerName        : localhost

I can think of the following options to resolve:

  • Update documentation to mention that DomainName is required when FQDN is not supplied in Members.
  • Require Members to be populated via FQDN.
  • Update Get-FQDNMemberName helper function to append computer's current domain name if not already included in member name instead of returning the computer name.
@PlagueHO PlagueHO self-assigned this Apr 27, 2017
@PlagueHO PlagueHO added bug The issue is a bug. help wanted The issue is up for grabs for anyone in the community. labels Apr 27, 2017
@PlagueHO
Copy link
Member

Thanks @solbirn - great job submitting this! I'll take a look at this over the weekend.

@PlagueHO
Copy link
Member

Hi @solbirn - I'll work on this once this PR is merged (should be over the next few days).

@PlagueHO PlagueHO 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 22, 2018
@SteveL-MSFT SteveL-MSFT added this to In progress in powershell/dscresources May 14, 2019
@SteveL-MSFT SteveL-MSFT removed this from In progress in powershell/dscresources Nov 27, 2019
@johlju johlju added help wanted The issue is up for grabs for anyone in the community. and removed in progress The issue is being actively worked on by someone. labels Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug The issue is a bug. help wanted The issue is up for grabs for anyone in the community.
Projects
None yet
Development

No branches or pull requests

3 participants