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

Enhancements for Terraform InfluxDB Module: Compatibility, Performance, and Consistency Updates #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

xe-nvdk
Copy link
Contributor

@xe-nvdk xe-nvdk commented Nov 26, 2024

This pull request includes a series of updates and improvements to ensure compatibility, enhance performance, and improve code readability and consistency.

Key Updates

Syntax Updates:

  • Updated syntax to support newer versions of Terraform, ensuring compatibility with the latest Terraform features and standards.

Variable Fixes:

  • Corrected typos in variable names for better clarity.
  • Updated default options for improved usability and consistency.

Instance Type Upgrade:

  • Upgraded instances from r4/r5 to r6i, leveraging the latest generation for better performance and cost efficiency.

Disk Type Upgrade:

  • Changed EBS volume type from io1 to io2 for increased durability, better performance, and lower cost.

Additional Improvements:

  • Enhanced code readability and organization for maintainability.
  • Improved consistency across resources (e.g., security group attachments).
  • Dynamic handling of resource dependencies (e.g., disk mounting and availability zones).

…tion bits from 1.8 to 1.11.x, syntax update to match new versions of tf
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.

1 participant