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

Use map.jinja as defaults store #66

Closed
refnode opened this issue Mar 26, 2015 · 6 comments
Closed

Use map.jinja as defaults store #66

refnode opened this issue Mar 26, 2015 · 6 comments

Comments

@refnode
Copy link

refnode commented Mar 26, 2015

This issue is a "duplicate" of #43 as I have also the problem to install MariaDB on CentOS 7.

The mysql formula should use map.as defaults store with grains.filter_by mechanism to provide a layered defaults store.

I'm working on a patch on my repo https://github.com/refnode/mysql-formula.git and send a pull request soon.

@vschum
Copy link
Contributor

vschum commented Mar 26, 2015

Have you tried using the mysql:lookup pillar key?

@refnode
Copy link
Author

refnode commented Mar 26, 2015

Sure, as a workaround and it's working.
Following the conventions in the apache formular, the layering of os_family, os and osmajorrelease makes sense for almost every network service formula and I think it would also reduce the redundant config values found in defaults.yaml.

I'll test my stuff now and push it asap.

@nmadhok
Copy link
Member

nmadhok commented Mar 26, 2015

@refnode MariaDb should be a formula on it's own and shouldn't been combined with mysql-formula. Mysql can still be installed on RHEL/CentOS 7 machines using the community repo. Obviously the support for that needs to be incorporated in this formula.

@refnode refnode closed this as completed Mar 26, 2015
@refnode
Copy link
Author

refnode commented Mar 26, 2015

Ok, thx for info.
I close this issue and prepare a separate formula for that.

@nmadhok
Copy link
Member

nmadhok commented Mar 26, 2015

@refnode Thanks! Let me know when you have created the formula so we can fork it under saltstack-formulas :)

@refnode
Copy link
Author

refnode commented Mar 26, 2015

Sure!

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

No branches or pull requests

3 participants