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

Add package resource for unzip to init.pp #4

Merged
merged 2 commits into from
Nov 26, 2019

Conversation

hsnodgrass
Copy link
Contributor

  • Adds package resource declaration to ensure that the unzip package
    is installed.

* Adds package resource declaration to ensure that the `unzip` package
  is installed.
@gabe-sky
Copy link
Owner

Can you add a class parameter, something like "manage_unzip" defaulting to false, that wraps this up and only does it if the user opts in?

* Adds parameter manage_unzip, defaults to false, that toggles whether
  puppet-ngrok manages a package resource for unzip, a dependency of
  the archive resource.
* Also adds a before statement to the unzip package resource to ensure
  that, if unzip is managed, the package resource is resolved before
  the archive resource.
@hsnodgrass
Copy link
Contributor Author

@gabe-sky PR has been updated with a parameter to toggle whether puppet-ngrok will manage unzip. I also added explicit ordering to the unzip package resource if it's managed.

@gabe-sky gabe-sky merged commit 9385576 into gabe-sky:master Nov 26, 2019
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.

2 participants