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 InfluxDB2 exports #313

Closed
wants to merge 6 commits into from

Conversation

brettmiller
Copy link

Proposed Changes

Add configuration options to support export to InfluxDB2. If both InfluxDB and InfluxDB2 are configured export to both. Because only a single interval is supported, in cases of running exports to both InfluxDB and InfluxDB2 the higher interval value is used (resolves one of the issues with #217)

Related Issues

#217

@sinclairpaul
Copy link
Member

sinclairpaul commented Dec 29, 2022

#311 would suggest working on the same PR

@brettmiller
Copy link
Author

brettmiller commented Dec 29, 2022

@sinclairpaul I opened a new PR because my solution is completely different from #311. Mine allows exports of either or both InfluxDB and InfluxDB2 and keeps the InfluxDB and InfluxDB2 configs completely separate, where as #311 is one or the other and mixes the config options all under "influxdb"

@kamontat
Copy link
Contributor

kamontat commented Jan 2, 2023

Yes, My solution I based on how home-assistant config their influxdb

@github-actions
Copy link

github-actions bot commented Feb 1, 2023

There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.

@github-actions github-actions bot added the stale There has not been activity on this issue or PR for quite some time. label Feb 1, 2023
@github-actions github-actions bot closed this Feb 8, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Feb 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
stale There has not been activity on this issue or PR for quite some time.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants