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

remove InfluxDB and ElasticSearch mentions from the docs #1591

Merged
merged 3 commits into from
Feb 14, 2024

Conversation

Serpentiel
Copy link
Contributor

@Serpentiel Serpentiel commented Feb 13, 2024

About this change—what it does

  • removes influxdb mentions from the docs (replaced with m3db)
  • removes elasticsearch mentions from the docs (replaced with opensearch)

Why this way

  • influxdb is no longer supported
  • elasticsearch is replaced with opensearch

@Serpentiel Serpentiel added documentation Improvements or additions to documentation no changelog No changelog entries are required for this PR labels Feb 13, 2024
@Serpentiel Serpentiel requested a review from a team February 13, 2024 12:21
Copy link
Contributor

@byashimov byashimov left a comment

Choose a reason for hiding this comment

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

Could you please run acc tests?

@Serpentiel Serpentiel force-pushed the aleks-docs-remove-influxdb-mentions branch from cb9f908 to 5a2bfcd Compare February 13, 2024 16:36
@Serpentiel Serpentiel requested a review from byashimov February 13, 2024 16:36
@Serpentiel
Copy link
Contributor Author

Examples tests: https://github.com/aiven/terraform-provider-aiven/actions/runs/7888989212

Ran the changed acceptance test locally, it succeeds.

Copy link
Contributor

@byashimov byashimov left a comment

Choose a reason for hiding this comment

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

lgtm

@Serpentiel Serpentiel merged commit b22979d into main Feb 14, 2024
11 checks passed
@Serpentiel Serpentiel deleted the aleks-docs-remove-influxdb-mentions branch February 14, 2024 10:13
@Serpentiel Serpentiel assigned Serpentiel and unassigned byashimov Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation no changelog No changelog entries are required for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants