-
Notifications
You must be signed in to change notification settings - Fork 6
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
Configure source repository #139
Comments
FWIW, |
@martinb3 My bad then, bad example maybe it worth a comment 😉 what do you think about the issue itself rather than this example? |
Based on what we are thinking with flags, we can enable yum repos based on flags: |
I think it'd be good to enable those kinds of things by default, but we'd probably need to review each stack and make sure EPEL and IUS don't conflict with any that are missing it. For example, I know in phpstack there were lots of issues with which repo PHP came from, so phpstack allows you to add any repo before invoking it, and then specify a list of packages. We might find turning on IUS or EPEL by default means we'd exclude the packages from another 3rd party repo that a customer is trying to install (making it impossible to install the packages from the other repo, until someone finds the I think as long as review what we're doing now for any issues, and document it, we'd be in good shape to include EPEL and IUS by default. |
Cool I do agree we just need to be careful with conflict, we could just default to no repo, it will allow to only use one attribute in stacks/client cookbook, and we will have to maintain stuff like https://github.com/AutomationSupport/2869487-cmonline_app/blob/master/recipes/yum.rb#L11-L30 only in one place. By example
Then in a recipe something like
And in a stack/client cookbook if I need a repo I just need to do |
👍 I like that idea. |
From what I've seen so far, a lot of our recipes are playing with yum/apt repo sources.
It creates a lot of duplication. At the same time I have no way to say I'd like to use the IUS repo for yum without adding custom code.
What about making it a part of platformstack?
https://github.com/search?utf8=%E2%9C%93&q=%22include_recipe+%27yum%27%22+++%22include_recipe+%27apt%27%22+user%3AAutomationSupport&type=Code&ref=searchresults
https://github.com/rackspace-cookbooks/platformstack/blob/master/recipes/default.rb#L27
platformstack/recipes/monitors.rb
Line 32 in f59c147
platformstack/recipes/monitors.rb
Line 24 in f59c147
https://github.com/AutomationSupport/2869487-cmonline_app/blob/master/recipes/yum.rb
https://github.com/AutomationSupport/phpstack/blob/9bc0901b0989c7a47a86d44909a67002be035ab4/recipes/mysql_holland.rb#L25
https://github.com/AutomationSupport/phpstack/blob/9bc0901b0989c7a47a86d44909a67002be035ab4/recipes/mysql_holland.rb#L33
The text was updated successfully, but these errors were encountered: