change DNS configuration to use proxy and update EIP state for subql

[DaMandal0rian]

PR Type

Enhancement

---

Description

• Updated DNS TTL values to 1 for faster propagation.

• Enabled proxying for Cloudflare DNS records for added security.

• Applied changes across multiple Cloudflare DNS records configurations.

---

Changes walkthrough 📝

	Relevant files
Enhancement	dns.tf Adjusted DNS TTL and enabled proxying                                        templates/terraform/subql/base/dns.tf Changed TTL value from 3600 to 1 for faster updates. Added proxied = true to enable Cloudflare proxying. Applied changes to multiple DNS records (blue, green, live, nova). +12/-6

---

Need help?

Type /help how to ... in the comments thread for any questions about PR-Agent usage.

Check out the documentation for more information.

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review TTL Configuration The TTL value for all DNS records has been reduced to 1. This change should be reviewed to ensure it aligns with the desired DNS propagation behavior and does not introduce unnecessary overhead or performance issues. ttl = "1" proxied = true } resource "cloudflare_record" "subql-green" { count = var.green-subql-node-config.instance-count-green > 0 ? var.green-subql-node-config.instance-count-green : 0 zone_id = data.cloudflare_zone.cloudflare_zone.id name = "${var.green-subql-node-config.domain-prefix}.${var.green-subql-node-config.network-name}" value = local.green_subql_node_ip_v4[count.index] type = "A" ttl = "1" proxied = true } resource "cloudflare_record" "subql-live" { count = var.blue-subql-node-config.instance-count-blue > 0 ? var.blue-subql-node-config.instance-count-blue : 0 zone_id = data.cloudflare_zone.cloudflare_zone.id name = "subql.${var.network_name}" value = local.blue_subql_node_ip_v4[count.index] type = "A" ttl = "1" proxied = true } resource "cloudflare_record" "nova-subql-blue" { count = var.nova-blue-subql-node-config.instance-count-blue > 0 ? var.nova-blue-subql-node-config.instance-count-blue : 0 zone_id = data.cloudflare_zone.cloudflare_zone.id name = "${var.nova-blue-subql-node-config.domain-prefix}.${var.network_name}" value = local.blue_subql_node_ip_v4[count.index] type = "A" ttl = "1" proxied = true } resource "cloudflare_record" "nova-subql-green" { count = var.nova-green-subql-node-config.instance-count-green > 0 ? var.nova-green-subql-node-config.instance-count-green : 0 zone_id = data.cloudflare_zone.cloudflare_zone.id name = "${var.nova-green-subql-node-config.domain-prefix}.${var.network_name}" value = local.nova_green_subql_node_ip_v4[count.index] type = "A" ttl = "1" proxied = true } resource "cloudflare_record" "nova-subql-live" { count = var.nova-blue-subql-node-config.instance-count-blue > 0 ? var.nova-blue-subql-node-config.instance-count-blue : 0 zone_id = data.cloudflare_zone.cloudflare_zone.id name = "nova.subql.${var.network_name}" value = local.nova_blue_subql_node_ip_v4[count.index] type = "A" ttl = "1" proxied = true Proxying Enabled Proxying has been enabled for all DNS records. This change should be validated to ensure it does not conflict with the intended functionality or introduce unexpected behavior. ttl = "1" proxied = true } resource "cloudflare_record" "subql-green" { count = var.green-subql-node-config.instance-count-green > 0 ? var.green-subql-node-config.instance-count-green : 0 zone_id = data.cloudflare_zone.cloudflare_zone.id name = "${var.green-subql-node-config.domain-prefix}.${var.green-subql-node-config.network-name}" value = local.green_subql_node_ip_v4[count.index] type = "A" ttl = "1" proxied = true } resource "cloudflare_record" "subql-live" { count = var.blue-subql-node-config.instance-count-blue > 0 ? var.blue-subql-node-config.instance-count-blue : 0 zone_id = data.cloudflare_zone.cloudflare_zone.id name = "subql.${var.network_name}" value = local.blue_subql_node_ip_v4[count.index] type = "A" ttl = "1" proxied = true } resource "cloudflare_record" "nova-subql-blue" { count = var.nova-blue-subql-node-config.instance-count-blue > 0 ? var.nova-blue-subql-node-config.instance-count-blue : 0 zone_id = data.cloudflare_zone.cloudflare_zone.id name = "${var.nova-blue-subql-node-config.domain-prefix}.${var.network_name}" value = local.blue_subql_node_ip_v4[count.index] type = "A" ttl = "1" proxied = true } resource "cloudflare_record" "nova-subql-green" { count = var.nova-green-subql-node-config.instance-count-green > 0 ? var.nova-green-subql-node-config.instance-count-green : 0 zone_id = data.cloudflare_zone.cloudflare_zone.id name = "${var.nova-green-subql-node-config.domain-prefix}.${var.network_name}" value = local.nova_green_subql_node_ip_v4[count.index] type = "A" ttl = "1" proxied = true } resource "cloudflare_record" "nova-subql-live" { count = var.nova-blue-subql-node-config.instance-count-blue > 0 ? var.nova-blue-subql-node-config.instance-count-blue : 0 zone_id = data.cloudflare_zone.cloudflare_zone.id name = "nova.subql.${var.network_name}" value = local.nova_blue_subql_node_ip_v4[count.index] type = "A" ttl = "1" proxied = true

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Score
General	Increase TTL to reduce DNS load Setting the TTL to "1" may result in excessive DNS queries and increased load on the DNS server. Consider using a higher TTL value to reduce the frequency of DNS lookups while balancing the need for quick updates. templates/terraform/subql/base/dns.tf [11] -ttl = "1" +ttl = "300" Suggestion importance[1-10]: 9 Why: The suggestion addresses a critical issue where a TTL value of "1" could lead to excessive DNS queries, increasing server load and potentially causing performance issues. Increasing the TTL to "300" is a reasonable compromise between reducing DNS load and maintaining timely updates.	9
	Make proxying behavior configurable Ensure that enabling the proxied option aligns with the intended behavior, as it routes traffic through Cloudflare's proxy, potentially affecting latency and caching. templates/terraform/subql/base/dns.tf [12] -proxied = true +proxied = var.enable_proxy Suggestion importance[1-10]: 7 Why: The suggestion to make the proxied option configurable improves flexibility and ensures that the behavior aligns with the intended use case. However, it requires additional context to verify if var.enable_proxy is defined and used correctly in the configuration.	7

[marc-aurele-besner]

LGTM