Add comments about the hotfix

ooni · Jan 16, 2025 · cfea627 · cfea627
1 parent 0b6bfd1
commit cfea627
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/tf/modules/ooniapi_frontend/main.tf b/tf/modules/ooniapi_frontend/main.tf
@@ -190,8 +190,6 @@ resource "aws_lb_listener_rule" "ooniapi_oonifindings_rule" {
 }
 
 resource "aws_lb_listener_rule" "ooniapi_oonifindings_rule_host" {
-  count = var.ooniapi_oonimeasurements_target_group_arn != null ? 1 : 0
-
   listener_arn = aws_alb_listener.ooniapi_listener_https.arn
   priority     = 131
 
@@ -207,6 +205,9 @@ resource "aws_lb_listener_rule" "ooniapi_oonifindings_rule_host" {
 }
 
 resource "aws_lb_listener_rule" "ooniapi_oonimeasurements_rule_1" {
+  # hotfix: to allow us to deploy the frontend without the measurements service
+  count = var.ooniapi_oonimeasurements_target_group_arn != null ? 1 : 0
+
   listener_arn = aws_alb_listener.ooniapi_listener_https.arn
   priority     = 140
 
@@ -229,6 +230,7 @@ resource "aws_lb_listener_rule" "ooniapi_oonimeasurements_rule_1" {
 }
 
 resource "aws_lb_listener_rule" "ooniapi_oonimeasurements_rule_2" {
+  # hotfix: to allow us to deploy the frontend without the measurements service
   count = var.ooniapi_oonimeasurements_target_group_arn != null ? 1 : 0
 
   listener_arn = aws_alb_listener.ooniapi_listener_https.arn
@@ -252,6 +254,7 @@ resource "aws_lb_listener_rule" "ooniapi_oonimeasurements_rule_2" {
 }
 
 resource "aws_lb_listener_rule" "ooniapi_oonimeasurements_rule_host" {
+  # hotfix: to allow us to deploy the frontend without the measurements service
   count = var.ooniapi_oonimeasurements_target_group_arn != null ? 1 : 0
 
   listener_arn = aws_alb_listener.ooniapi_listener_https.arn