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

(#108) Add support for customizing rundeck-config #109

Merged
merged 1 commit into from
Oct 20, 2015

Conversation

patcadelina
Copy link

There are customizations to rundeck-config that I would like to do. Using the rdeck_config_template, I could specify a template for it. By default, it would use rundeck/rundeck-config.erb.

PS: My first contribution to puppet-rundeck, please comment on my diff.

@patcadelina patcadelina changed the title Add support for customizing rundeck-config #108 Add support for customizing rundeck-config Jun 16, 2015
@patcadelina patcadelina changed the title #108 Add support for customizing rundeck-config (#108) Add support for customizing rundeck-config Jun 16, 2015
@patcadelina patcadelina force-pushed the rundeck-config-template branch from 001d4fc to d6976ff Compare June 16, 2015 09:02
@patcadelina
Copy link
Author

@liamjbennett any update?

@jyaworski
Copy link
Member

Hello:

Can you rebase?

@liamjbennett
Copy link
Member

Now happy for custom templates to be merged. So please rebased this and we'll get it in.

@patcadelina patcadelina force-pushed the rundeck-config-template branch from d6976ff to 2834f18 Compare October 20, 2015 06:52
@patcadelina
Copy link
Author

Rebased.

@igalic
Copy link
Contributor

igalic commented Oct 20, 2015

@patcadelina thank you very much for this patch!
unless you want magic internet points, and want to still change the commit author on this pr so github recognizes it's you i'll merge this as-is:)

@patcadelina
Copy link
Author

Please go ahead @igalic.

igalic added a commit that referenced this pull request Oct 20, 2015
(#108) Add support for customizing rundeck-config
@igalic igalic merged commit 7301d7e into voxpupuli:master Oct 20, 2015
@igalic
Copy link
Contributor

igalic commented Oct 20, 2015

thanks @patcadelina!

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