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 the sqlite backend, and for SUSE OSes #7

Merged
merged 1 commit into from May 12, 2017

Conversation

tampakrap
Copy link
Contributor

No description provided.

Copy link
Member

@aboe76 aboe76 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

@aboe76 aboe76 requested a review from kpostrup May 2, 2017 14:23
@aboe76
Copy link
Member

aboe76 commented May 8, 2017

@tampakrap could you take a look at the #8 see if this overlaps?

@bentwire
Copy link
Contributor

bentwire commented May 10, 2017

It does overlap a little. I went a bit farther than this one though and added DB initialization. And state and execution modules for pdnsapi...

@aboe76
Copy link
Member

aboe76 commented May 11, 2017

@tampakrap could you please see if you can adjust and remove the sqlitebackend part, because it was already merged....

@tampakrap
Copy link
Contributor Author

sure, I'll just remove it

@tampakrap
Copy link
Contributor Author

@aboe76 done

@aboe76 aboe76 merged commit dfa27b1 into saltstack-formulas:master May 12, 2017
@aboe76
Copy link
Member

aboe76 commented May 12, 2017

@tampakrap thanks.

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.

None yet

3 participants