-
Notifications
You must be signed in to change notification settings - Fork 54
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
Cluster: When using distinguished naming of cluster, second pass of DSC fails. #256
Comments
In the first node you have:
In the second node you have:
|
By "second pass" do you mean that the first node creates the CNO in the OU, but when the configuration runs Test-TargetResource it fails (when it enforces the configuration and checks current state against desired state)? |
Yes, and that is on the first pass of cluster node one and two. On the first pass it all works. Ont the second pass the first node falls. The one with a distibgusihed name
…________________________________
From: Johan Ljunggren ***@***.***>
Sent: Monday, April 12, 2021 5:58:25 PM
To: dsccommunity/xFailOverCluster ***@***.***>
Cc: Ilian Nilsson ***@***.***>; Author ***@***.***>
Subject: Re: [dsccommunity/xFailOverCluster] xCluster: When using distinguished naming of cluster, second pass of DSC fails. (#256)
By "second pass" do you mean that the first node creates the CNO in the OU, but when the configuration runs Test-TargetResource it fails (when it enforces the configuration and checks current state against desired state)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#256 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AID6F4DROTJ2EPCHMSQ4VH3TIMKCDANCNFSM42Y7CW5Q>.
|
I have used pre-staged accounts so never seen this. Sounds like there is an issue when using the full distinguished name. |
Happy to review a pull request that fixes this. |
I’ll try powershell in DSC format is not my thing.
…________________________________
From: Johan Ljunggren ***@***.***>
Sent: Monday, April 12, 2021 9:08:48 PM
To: dsccommunity/xFailOverCluster ***@***.***>
Cc: Ilian Nilsson ***@***.***>; Author ***@***.***>
Subject: Re: [dsccommunity/xFailOverCluster] xCluster: When using distinguished naming of cluster, second pass of DSC fails. (#256)
Happy to review a pull request that fixes this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#256 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AID6F4CF3OMUWUXVLVECU53TINAMBANCNFSM42Y7CW5Q>.
|
Well I've made my best, but when i try to push my branch i get this: E:\IGEL\Source\xFailoverCluster [fix_to_use_distinguised_name]> git push --set-upstream origin fix_to_use_distinguised_name And yes i got a browser to autenticate with, and it said, Success :( |
You need to fork the main repository. On the repository page there is a button “Fork” at top right. Once you forked you need to make a remote Read how to do it here https://dsccommunity.org/guidelines/getting-started/#forking-a-repository-on-github |
Thats intendent, when you do create-cluster it is valid to use distinguished name, not when searching to add nodes
…________________________________
From: Johan Ljunggren ***@***.***>
Sent: Saturday, April 17, 2021 5:33:55 PM
To: dsccommunity/xFailOverCluster ***@***.***>
Cc: Ilian Nilsson ***@***.***>; Author ***@***.***>
Subject: Re: [dsccommunity/xFailOverCluster] xCluster: When using distinguished naming of cluster, second pass of DSC fails. (#256)
You need to fork the main repository. On the repository page there is a button “Fork” at top right. Once you forked you need to make a remote my to the fork, in the local repository folder. Then you push to the remote. From there you make the pull request.
Read how to do it here https://dsccommunity.org/guidelines/getting-started/#forking-a-repository-on-github
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#256 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AID6F4ESMBTD4SUJ4GAUKVDTJGS6HANCNFSM42Y7CW5Q>.
|
Here we go, a fork IlleNilsson/xFailOverCluster. There is at least one thing I'm not happy with. I added a function named Convert-DistinguishedNameToSimpleName. I could not find a common place to put it, so it is in all Resources, not optimal. |
Great! Now you need to go to your fork an create a pull request. Got to https://github.com/IlleNilsson/xFailOverCluster/pulls and click on new pull request. Base repository should be
We could add a new common module |
You said "we could", does that mean me? I'm not up to speed on building DSC Resources. Can we do it togheter? |
We could, as in the community could. That can be resolved in another PR, or I could see if I have time to add an empty module where we can move the code. |
I would like to get it done, l’m changing assignment and I’m afraid the customer won’t resolve this issue without me. So whatever I can do, let me know.
…________________________________
From: Johan Ljunggren ***@***.***>
Sent: Thursday, April 22, 2021 7:25:21 PM
To: dsccommunity/xFailOverCluster ***@***.***>
Cc: Ilian Nilsson ***@***.***>; Author ***@***.***>
Subject: Re: [dsccommunity/xFailOverCluster] xCluster: When using distinguished naming of cluster, second pass of DSC fails. (#256)
We could, as in the community could. That can be resolved in another PR, or I could see if I have time to add an empty module where we can move the code.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#256 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AID6F4BX5D2FZLGYSOHS3LLTKBLYDANCNFSM42Y7CW5Q>.
|
Done this, getting...
Tests completed in 77.33s
Tests Passed: 289, Failed: 0, Skipped: 4, Pending: 0, Inconclusive: 0
Code coverage report:
Covered 99.38% of 649 analyzed Commands in 8 Files.
Missed commands:
File Class Function Line Command
---- ----- -------- ---- -------
Modules\FailoverCluster.Common\FailoverCluster.Common.psm1 Convert-DistinguishedNameToSimpleName 29 $returnV...
Modules\FailoverCluster.Common\FailoverCluster.Common.psm1 Convert-DistinguishedNameToSimpleName 29 (($Disti...
Modules\FailoverCluster.Common\FailoverCluster.Common.psm1 Convert-DistinguishedNameToSimpleName 29 ($Distin...
Modules\FailoverCluster.Common\FailoverCluster.Common.psm1 Convert-DistinguishedNameToSimpleName 29 $Disting...
Any hints for me to resolve this?
From: Johan Ljunggren ***@***.***>
Sent: Sunday, 18 April 2021 12:51
To: dsccommunity/xFailOverCluster ***@***.***>
Cc: Ilian Nilsson ***@***.***>; Author ***@***.***>
Subject: Re: [dsccommunity/xFailOverCluster] xCluster: When using distinguished naming of cluster, second pass of DSC fails. (#256)
Great! Now you need to go to your fork an create a pull request. Got to https://github.com/IlleNilsson/xFailOverCluster/pulls and click on new pull request. Base repository should be dsccommunity/xFailOverCluster and base should be main. The head repository should be IlleNilsson/xFailOverCluster and compare should be the working branch where you pushed the changes.
I could not find a common place to put it, so it is in all Resources, not optimal.
We could add a new common module FailoverCluster.Common similar to here: https://github.com/dsccommunity/DnsServerDsc/tree/main/source/Modules/DnsServerDsc.Common
We also need to copy it during build by changing build.yaml:
https://github.com/dsccommunity/DnsServerDsc/blob/97f6ef720def5fc0d920c63607f3ed2f8e03f9fb/build.yaml#L8
The we just need to import the module in the resource code like line 2 and 5 here:
https://github.com/dsccommunity/DnsServerDsc/blob/97f6ef720def5fc0d920c63607f3ed2f8e03f9fb/source/DSCResources/DSC_DnsServerADZone/DSC_DnsServerADZone.psm1#L2-L5
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#256 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AID6F4D2SWOBBDH5VLWG2VTTJK2STANCNFSM42Y7CW5Q>.
|
I want to create the AD Cluster object in a specific OU
Verbose logs showing the problem
Suggested solution to the issue
The DSC configuration that is used to reproduce the issue (as detailed as possible)
--First Nodes MOF
--Second Nodes MOF
The operating system the target node is running
OsName : Microsoft Windows Server 2019 Datacenter
OsOperatingSystemSKU : DatacenterServerEdition
OsArchitecture : 64-bit
WindowsVersion : 1809
WindowsBuildLabEx : 17763.1.amd64fre.rs5_release.180914-1434
OsLanguage : en-US
OsMuiLanguages : {en-US}
Version and build of PowerShell the target node is running
SVersion 5.1.17763.1490
PSEdition Desktop
PSCompatibleVersions {1.0, 2.0, 3.0, 4.0...}
BuildVersion 10.0.17763.1490
CLRVersion 4.0.30319.42000
WSManStackVersion 3.0
PSRemotingProtocolVersion 2.3
SerializationVersion 1.1.0.1
Version of the DSC module that was used
1.16.0
The text was updated successfully, but these errors were encountered: