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

attempt to fix #3 (allow to configure service manually) #12

Merged
merged 1 commit into from
Mar 11, 2016

Conversation

feniix
Copy link
Contributor

@feniix feniix commented Jun 14, 2015

we would like to add the ability to completely skip the service creation/install and be able to set the state to stopped or running.

@feniix feniix changed the title attempt to fix #3 attempt to fix #3 (allow to configure service manually) Jun 14, 2015
@jyaworski
Copy link
Member

Hello:

What's the status of this PR?

@bastelfreak
Copy link
Member

@feniix hi, are you still interested in this PR? could you rebase it?

@feniix
Copy link
Contributor Author

feniix commented Mar 10, 2016

@bastelfreak I just rebased and pushed again

@feniix
Copy link
Contributor Author

feniix commented Mar 10, 2016

I am looking into the failing tests

@dhoppe
Copy link
Member

dhoppe commented Mar 10, 2016

@bastelfreak Not so fast. A lot of these files will change with PR #46.

@bastelfreak
Copy link
Member

@feniix sorry to ping you again :(

@feniix feniix force-pushed the master branch 2 times, most recently from a1975b6 to b50f939 Compare March 10, 2016 23:10
@feniix
Copy link
Contributor Author

feniix commented Mar 11, 2016

Here we go again I also added a test for the use case
🎈 @bastelfreak

jyaworski added a commit that referenced this pull request Mar 11, 2016
attempt to fix #3 (allow to configure service manually)
@jyaworski jyaworski merged commit abe36d7 into voxpupuli:master Mar 11, 2016
elmendalerenda pushed a commit to elmendalerenda/puppet-kafka that referenced this pull request Mar 30, 2018
attempt to fix voxpupuli#3 (allow to configure service manually)
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.

4 participants