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

Single node ES instance #102

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Single node ES instance #102

wants to merge 1 commit into from

Conversation

egorksv
Copy link
Contributor

@egorksv egorksv commented Jan 2, 2021

  • No load balancers
  • No autoscaling
  • Output set to single node IP address

terraform-aws/outputs.tf Outdated Show resolved Hide resolved
@egorksv egorksv force-pushed the feature/single-node branch 4 times, most recently from 2204189 to bd1eaf6 Compare January 7, 2021 11:35
- No load balancers
- No autoscaling
- Output set to single node IP address
fixed issue with singlenode instance being provisioned in clustered mode
Single-node network interface attachment to ensure connectivity without load balancer
updated gitignore

Kibana image

restored file

file restored
name = "${var.es_cluster}-alb"
count = local.singlenode_mode ? 0 : 1

name = "${var.environment}-${var.es_cluster}-alb"
internal = ! var.public_facing
Copy link
Contributor

Choose a reason for hiding this comment

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

The variable public_facing is not accounted anymore for single-nodes.

@@ -1,41 +1,76 @@
#!/bin/bash
set +e

echo "Testing AMI Builder if it works properly"
Copy link
Contributor

Choose a reason for hiding this comment

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

Cleanup

})
}

resource "aws_network_interface" "single_node" {
Copy link
Contributor

Choose a reason for hiding this comment

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

Has to be conditional, only created when singlenode_mode is true.

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