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

(#110) Add support for customizing realm.properties #111

Merged
merged 1 commit into from
Oct 20, 2015

Conversation

patcadelina
Copy link

Using realm_template, realm.properties could be customized. By default, rundeck/realm.properties.erb is used.

@patcadelina patcadelina changed the title Add support for customizing realm.properties (#110) Add support for customizing realm.properties Jun 16, 2015
@dandunckelman
Copy link
Contributor

What's the advantage of managing the template yourself vs writing the configuration of the template that's already present?

@liamjbennett
Copy link
Member

The reason this sort of pattern appears again and again in modules is mostly due to the fact that the module (this module) does not update the configuration as new versions are released. If we could keep up with the new configuration in rundeck and if we were managing it all as per documentation then this would not be required, however that is unrealistic so it's valid for us to do.

Having said all that. This pattern does actually scale to more than a couple of templates so we need to be careful about it's use.

@patcadelina
Copy link
Author

This pull request (along with 109) unhacks our puppet-rundeck module. @liamjbennett can you share some guidelines as to which templates you believe is worthy of landing in master?

@liamjbennett
Copy link
Member

realm.properties is already a pretty generic template. What is it that you are trying to add by using your own? Or it it a UX thing?

@jyaworski
Copy link
Member

@liamjbennett IMO it's a good idea to be as generic as possible; we can't account for all possibilities of the use of this module.

@patcadelina please rebase.

@liamjbennett
Copy link
Member

Ok, will merge this once rebased.

@patcadelina
Copy link
Author

Rebased.

@igalic
Copy link
Contributor

igalic commented Oct 20, 2015

@patcadelina thank you very much for this patch!
same as #109: 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

Yep, please go ahead @igalic.

igalic added a commit that referenced this pull request Oct 20, 2015
(#110) Add support for customizing realm.properties
@igalic igalic merged commit 3895f5a into voxpupuli:master Oct 20, 2015
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.

5 participants