Remove outdated junk

dns.tf

@@ -22,17 +22,6 @@ resource "aws_route53_record" "email_dkim_hubspot_records" {
   ]
 }
 
-# TODO remove this; we do DKIM-based verification instead.
-resource "aws_ses_domain_identity" "email_domain_identity" {
-  domain = var.email_domain_name
-}
-
-# TODO remove this. It's unnecessary because the domain is verified.
-# It's still here only to make a refactoring `plan` clean.
-resource "aws_ses_email_identity" "email" {
-  email = "sre@seagl.org"
-}
-
 resource "aws_route53_record" "cloud-a" {
   zone_id = module.production_env.zone_id
   name    = "cloud.seagl.org"

env/dns.tf

@@ -1,4 +1,5 @@
-# TODO create NS and SOA records
+# Note: we intentionally do not create NS and SOA records and just let Route 53 create them automatically.
+# This is Probably Fine(tm) because these records are basically never updated.
 
 resource "aws_route53_zone" "apex" {
   name = var.zone_name

env/nextcloud-office.tf

@@ -5,6 +5,5 @@ module "nc-office-vm" {
   instance_type = "m1.small"
   network       = "general_servers2"
   disk_size     = 15
-  # TODO replace this with a real Route 53 zone object when we control that in Terraform
   dns_zone = aws_route53_zone.apex
 }