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

[GDB-9925] Module restructuring #30

Merged
1 commit merged into from
Apr 1, 2024
Merged

[GDB-9925] Module restructuring #30

1 commit merged into from
Apr 1, 2024

Conversation

ghost
Copy link

@ghost ghost commented Mar 28, 2024

Description

Module restructuring in order to be more closer to the structure of the Azure Terraform Module.
Also added toggle for the monitoring to have an option to enable it or disable it.
Added definitions for the was provider in order to split region-based resources related to the monitoring.

Changes

  1. Module restructuring
  2. Monitoring toggle
  3. AWS Provider definition changes
  4. Removed unused variables.

Checklist

  • I have tested these changes thoroughly.
  • My code follows the project's coding style.
  • I have added appropriate comments to my code, especially in complex areas.
  • All new and existing tests passed locally.

@ghost ghost requested review from yaskoo, mihailradkov and viktor-ribchev March 28, 2024 14:54
@ghost ghost assigned viktor-ribchev Mar 28, 2024
main.tf Outdated Show resolved Hide resolved
modules/graphdb/iam.tf Outdated Show resolved Hide resolved
modules/graphdb/variables.tf Show resolved Hide resolved
modules/graphdb/variables.tf Show resolved Hide resolved
modules/vpc/variables.tf Show resolved Hide resolved
modules/graphdb/variables.tf Outdated Show resolved Hide resolved
@ghost ghost force-pushed the GDB-9925-Module-Restructuring branch from e8ef7df to 9d0cdf1 Compare April 1, 2024 08:02
@ghost ghost merged commit 4e04795 into main Apr 1, 2024
2 checks passed
@ghost ghost deleted the GDB-9925-Module-Restructuring branch April 1, 2024 08:05
This pull request was closed.
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.

4 participants