Skip to content
This repository was archived by the owner on Jan 3, 2026. It is now read-only.

Fixed nested modules import#37

Open
spawluk wants to merge 18 commits intoryanspletzer:devfrom
spawluk:dev
Open

Fixed nested modules import#37
spawluk wants to merge 18 commits intoryanspletzer:devfrom
spawluk:dev

Conversation

@spawluk
Copy link

@spawluk spawluk commented Apr 10, 2017

When I defined custom database names I got error that New-SqlConnectionString cmdlet is not found.

@spawluk
Copy link
Author

spawluk commented Apr 10, 2017

My PR unfortunately did not fixed issue with CommandNotFoundExecption

@ryanspletzer
Copy link
Owner

Thanks for taking a crack at it, I've been a little swamped lately... These things are tricky to nail down. Part of me wishes I'd stuck with non class-based resources, but I'm determined to make it work!

@spawluk
Copy link
Author

spawluk commented Apr 10, 2017

I can confirm that some parts of it works on Windows Server 2016.

I'm little surprised of your determination, because even MS stuck to MOF based resources.

Maybe they know that class based resources are hard to debug and maintain? ;)

@ryanspletzer
Copy link
Owner

Well from what I understand... it's because they started with the classic approach for certain products so in order to not severely "shake up" those repo's and maintain compatibility they've continued with the same approach. They'd probably have to increment a major version number to start over with a class-based approach for things like Exchange, SharePoint etc... which would be a lot of work + disruption.

I drank the Kool-Aid when I watched Helmick's + Snover's DSC classes on MVA and they were saying class-based was the future, so I went that route. :)

Funny enough, Service Bus for Windows Server and Workflow Manager (which I understand because they're sort of deprecated?) and BizTalk Server are the few MSFT products I haven't seen DSC resources for from them... Most of the rest of their products have a DSC offering from Microsoft.

@spawluk
Copy link
Author

spawluk commented Apr 11, 2017

Finally i managed to fix Set, Get and Test methods. I can tomorrow verify on fresh server if all resources are working as expected

@ryanspletzer
Copy link
Owner

Greatly appreciate your efforts!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants