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.

Sign up for GitHub

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

Jump to bottom

influxdb_user and influxdb_database states need virtual functions #24301

Closed
iggy opened this issue Jun 1, 2015 · 1 comment
Closed

influxdb_user and influxdb_database states need virtual functions #24301

iggy opened this issue Jun 1, 2015 · 1 comment
Labels
Bug broken, incorrect, or confusing behavior help-wanted Community help is needed to resolve this P1 Priority 1 Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-low 4th level, cosemtic problems, work around exists State-Module
Milestone
Approved

Comments

@iggy
Copy link
Contributor

iggy commented Jun 1, 2015

They should probably not load if the influx module isn't loaded.
@jfindlay jfindlay added Bug broken, incorrect, or confusing behavior severity-low 4th level, cosemtic problems, work around exists help-wanted Community help is needed to resolve this P1 Priority 1 labels Jun 5, 2015
@jfindlay jfindlay added this to the Approved milestone Jun 5, 2015
@jfindlay
Copy link
Contributor

jfindlay commented Jun 5, 2015

@iggy, thanks for the report.

@jfindlay jfindlay added State-Module Platform Relates to OS, containers, platform-based utilities like FS, system based apps labels Jun 5, 2015
@wwentland wwentland mentioned this issue Jun 30, 2015
Merged
rallytime pushed a commit to rallytime/salt that referenced this issue Jul 6, 2015
@wwentland
Add virtual functions to influxdb state modules
1eeefbd 
Fixes saltstack#24301
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug broken, incorrect, or confusing behavior help-wanted Community help is needed to resolve this P1 Priority 1 Platform Relates to OS, containers, platform-based utilities like FS, system based apps severity-low 4th level, cosemtic problems, work around exists State-Module
Projects
None yet
Milestone
Approved
Development

No branches or pull requests
2 participants
@iggy @jfindlay