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-10418 Single Node Deployment #53

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

viktor-ribchev
Copy link
Contributor

Description

Added the ability to deploy a single node GraphDB

Related Issues

GDB-10418

Changes

Added support for single node deployment.
Removed hardcoded values for DNS zone in userdata scripts
Updated the NSGs ports based on graphdb_node_count
Updated user_data.tf
Changed the name of the LB target group to avoid conflicts when scaling from 1 to 3 AZs
Updated the monitoring, to not deploy cluster alarms when a single node is deployed
Updated the availability_tests
Updated how the VPC azs are calculated based on the graphdb_node_count
Added calculations for the subnets based on the graphdb_node_count

Screenshots (if applicable)

N/A

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.

@viktor-ribchev viktor-ribchev requested review from mihailradkov and a user July 2, 2024 07:31
@viktor-ribchev viktor-ribchev force-pushed the GDB-10418_single_node_deployment branch 4 times, most recently from 8782970 to 3920e97 Compare July 3, 2024 10:07
CHANGELOG.md Outdated Show resolved Hide resolved
main.tf Outdated Show resolved Hide resolved
modules/graphdb/templates/00_functions.sh Outdated Show resolved Hide resolved
@ghost ghost self-requested a review July 3, 2024 12:41
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

LGTM

modules/graphdb/nsg.tf Outdated Show resolved Hide resolved
modules/load_balancer/main.tf Show resolved Hide resolved
main.tf Outdated Show resolved Hide resolved
@viktor-ribchev viktor-ribchev force-pushed the GDB-10418_single_node_deployment branch 2 times, most recently from bc1c3d1 to 1a6a048 Compare July 5, 2024 06:59
@viktor-ribchev viktor-ribchev force-pushed the GDB-10418_single_node_deployment branch from 1a6a048 to 05547c0 Compare July 9, 2024 05:17
Removed hardcoded values for DNS zone in userdata scripts
Updated the NSGs ports based on graphdb_node_count
Updated user_data.tf
Changed the name of the LB target group to avoid conflicts when scaling from 1 to 3 AZs
Updated the monitoring, to not deploy cluster alarms when a single node is deployed
Updated the availability_tests
Updated how the VPC azs are calculated based on the graphdb_node_count
Added calculations for the subnets based on the graphdb_node_count
Removed route53_availability_content_match from modules\monitoring
Moved route53_availability_http_string_type to root level.
Changed the availability test to support single node deployment.
Added dynamic change of the availability tests http string type based on tls_enabled
Updated the README.md
Updated CHANGELOG.md
Made duplicate code into functions and moved it to 00_functions.sh
Updated README.md example for existing subnets
Renamed 10_start_graphdb_services.sh.tpl to 10_start_single_graphdb_services.sh.tpl
@viktor-ribchev viktor-ribchev force-pushed the GDB-10418_single_node_deployment branch from 05547c0 to e4ae245 Compare July 9, 2024 05:18
@viktor-ribchev viktor-ribchev merged commit 3932c42 into main Jul 9, 2024
3 checks passed
@viktor-ribchev viktor-ribchev deleted the GDB-10418_single_node_deployment branch July 9, 2024 05:41
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.

2 participants