Skip to content

Updated credentials.pp and created carbon.pp.#3

Open
ouberlord wants to merge 1 commit intoinsentia:masterfrom
ouberlord:master
Open

Updated credentials.pp and created carbon.pp.#3
ouberlord wants to merge 1 commit intoinsentia:masterfrom
ouberlord:master

Conversation

@ouberlord
Copy link

This is to address Issue #2, where if multiple services are configured the catalog compilation fails due to the "carbon.dll" resource being declared twice. Many thanks to @Areson for their help.

This is to address the issue where trying to configure more than one
service results in an error due to the module declaring the carbon.dll
File resource more than once.  Many thanks to Areson for their help.
@ouberlord ouberlord closed this Dec 29, 2016
@ouberlord
Copy link
Author

ouberlord commented Dec 29, 2016

Functions, but I am finding upon further testing that the module now always has to create and then delete the pair of .ps1 files. Cancelling this pull request.

Later Note: This comment was meant for an entirely different thread. Disregard.

@Areson
Copy link

Areson commented Dec 29, 2016

You don't have any .ps1 files in the module, so it isn't the module. Earlier versions of the Powershell provider create temporary files on disk before executing the scripts. So, the exec resources that you have that use that provider will cause those files to be created. Newer versions of the provider no longer do that and execute the code in memory entirely. I'd check what version you have on your test system.

@ouberlord
Copy link
Author

Well, this is awkward. I'm pretty sure I intended to close a different request for a different project with that comment, and did so on this one instead.

Don't mind the GitHub newbie over here. Reopening this, as I believe this pull request is worth consideration.

@ouberlord ouberlord reopened this Jan 12, 2017
@cmachler
Copy link

I hope this project isn't dead, maybe good candidate to transfer over to @voxpupuli community. Even though I see it is in progress at Puppet as a feature request. https://tickets.puppetlabs.com/browse/PUP-1289

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants