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 support for customizing profile #174

Merged
merged 1 commit into from
Feb 27, 2016

Conversation

danifr
Copy link
Member

@danifr danifr commented Feb 12, 2016

Following #109, I've created a patch but this time for customizing profile.
By default, it will still use rundeck/profile.erb

@jyaworski
Copy link
Member

Please rebase.

@danifr danifr force-pushed the rdeck_profile_template branch 2 times, most recently from 2dfcc2e to 8a5fe45 Compare February 17, 2016 09:21
@danifr
Copy link
Member Author

danifr commented Feb 17, 2016

I rebased and added this https://github.com/voxpupuli/puppet-rundeck/pull/174/files#diff-0ac866172efd29d0ddc2aa75ae6f182dR62 but tests are still failing.

Undefined variable "::aio_agent_version"; Undefined variable "aio_agent_version" at /home/travis/build/voxpupuli/puppet-rundeck/spec/fixtures/modules/archive/manifests/params.pp:24

@danifr danifr force-pushed the rdeck_profile_template branch 2 times, most recently from d5ebe08 to 1ebc4b3 Compare February 23, 2016 08:42
@danifr
Copy link
Member Author

danifr commented Feb 23, 2016

Ping

@jyaworski
Copy link
Member

@danifr sorry. Please rebase again. :)

@danifr danifr force-pushed the rdeck_profile_template branch from 1ebc4b3 to 3c8b6de Compare February 27, 2016 09:25
@danifr
Copy link
Member Author

danifr commented Feb 27, 2016

@jyaworski done :) I'll wait for this one to be merged until I rebase the other PR.

bastelfreak added a commit that referenced this pull request Feb 27, 2016
Add support for customizing profile
@bastelfreak bastelfreak merged commit 0b78428 into voxpupuli:master Feb 27, 2016
@danifr danifr deleted the rdeck_profile_template branch March 9, 2016 08:53
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