Clear out some optimistic changes from the base branch

ryapric · ryapric · commit 0b560c6d2b30 · 2025-01-12T16:11:45.000-06:00
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -0,0 +1,31 @@
+repos:
+- repo: https://github.com/pre-commit/pre-commit-hooks
+  rev: v4.6.0
+  hooks:
+    # Git style
+    - id: check-added-large-files
+    - id: check-merge-conflict
+    - id: check-vcs-permalinks
+    - id: forbid-new-submodules
+    - id: no-commit-to-branch
+
+    # Common errors
+    - id: end-of-file-fixer
+    - id: trailing-whitespace
+      args: [--markdown-linebreak-ext=md]
+      exclude: CHANGELOG.md
+    - id: check-yaml
+    - id: check-merge-conflict
+    - id: check-executables-have-shebangs
+
+- repo: https://github.com/jumanjihouse/pre-commit-hooks
+  rev: 3.0.0
+  hooks:
+    - id: shfmt
+      args: ['-l', '-i', '2', '-ci', '-sr', '-w']
+    - id: shellcheck
+- repo: https://github.com/antonbabenko/pre-commit-terraform
+  rev: v1.96.3
+  hooks:
+    - id: terraform_fmt
+    - id: terraform_docs
diff --git a/Makefile b/Makefile
@@ -1,4 +1,4 @@
-local_team_servers := $(shell vagrant status | grep -E -o 'team[0-9]+')
+local_team_servers = $(shell vagrant status | grep -E -o 'team[0-9]+')
 
 ci:
 	@bash ./scripts/ci.sh
diff --git a/instructions/challenge_2.md b/instructions/challenge_2.md
@@ -18,6 +18,6 @@ following:
 - Once there, the app there needs to be named `run-app`, not `app`
 
 - To ensure both locations always have the same version (e.g. during hotfixes),
-  make a reference from the binary location to the target location without
-  copying the actual file. This way, any update or change will reflect in both
+  make a reference from the binary location to the target location *without
+  copying the actual file*. This way, any update or change will reflect in both
   places at once without needing to manually sync them.
diff --git a/instructions/challenge_3.md b/instructions/challenge_3.md
@@ -15,5 +15,5 @@ Create that service, and get it running. You need to make sure it runs *even if
 the system reboots* -- meaning that if the server is restarted for any reason,
 the app service needs to start again *without you starting it manually*.
 
-If for any reason you need to check out more detailed logs of the service, you
-can also use the separate `journalctl` command to inspect them.
+If for any reason you need to check out more detailed logs of the service you're
+creating, you can also use the separate `journalctl` command to inspect them.
diff --git a/instructions/challenge_4.md b/instructions/challenge_4.md
@@ -24,5 +24,5 @@ look at the `/opt/app` directory again, you may have noticed that there is a
   correctly-installed app binary.
 
 Once you do that, be sure to check the `systemd` and/or `journald` logs of the
-new `app-deb.service` to make sure it's running successfully! (and, just like
-the last one, that it would keep running after a reboot)
+new `app-deb.service` to make sure it's running successfully (and, just like
+the last one, that it would keep running after a reboot).
diff --git a/instructions/challenge_6.md b/instructions/challenge_6.md
@@ -8,7 +8,7 @@ dissappeared while on a golfing trip a few weeks past. We were told that the
 code was about ready to deploy, just hadn't gotten the chance to merge it into
 the main branch. See if you can figure out how to get it up and running.
 
-The name of the app is 'carrot-cruncher'. The last dev got the repo set up
+The name of the app is "carrot-cruncher". The last dev got the repo set up
 somewhere on disk, but they never said where... hopefully you'll able to find
 it. When you do, supposedly there was a new working branch pushed to the remote
 repo, so you'll need to figure out how to authenticate to that repo.
diff --git a/scripts/ci.sh b/scripts/ci.sh
@@ -24,6 +24,7 @@ if [[ "$(</tmp/go-modules wc -l)" -gt 0 ]] ; then
   printf '> Installing CI checks for Go...\n'
   for pkg in \
     honnef.co/go/tools/cmd/staticcheck@latest \
+    github.com/mgechev/revive@latest \
     github.com/kisielk/errcheck@latest \
   ; do
     go install "${pkg}"
@@ -39,9 +40,11 @@ while read -r module ; do
     cd "${mod_dir}"
     printf '>> Running go vet...\n'
     go vet ./...
-    printf '>> Running linter...\n'
+    printf '>> Running staticcheck linter...\n'
     staticcheck ./...
-    printf '>> Running error checker...\n'
+    printf '>> Running revive linter...\n'
+    revive --set_exit_status ./...
+    printf '>> Running error-checker...\n'
     errcheck ./...
   )
 done < /tmp/go-modules
diff --git a/scripts/linux-workshop-admin.sh b/scripts/linux-workshop-admin.sh
@@ -35,13 +35,13 @@ _score-for-challenge() {
       log-info "Providing instruction to user for Challenge ${next_challenge}"
       cp "${wsroot}/instructions/challenge_${next_challenge}.md" /home/appuser/
       # Also broadcast message to user when challenge is complete
-      wall "Congrats on finishing Challenge ${which_challenge}! Be sure to check your home directory for any new instruction files! (hit any key to dismiss this message)"
+      wall "Congrats on finishing Challenge ${which_challenge}! Be sure to check your home directory for any new instruction files! (hit Enter to dismiss this message)"
     else
       log-info 'Team is done with the workshop!'
       cp "${wsroot}/instructions/congrats.md" /home/appuser/
       # This check suppresses an infinite loop of congratulations, lol
       if [[ ! -f "${wsroot}"/team_has_been_congratulated ]] ; then
-        wall "Congratulations -- you have completed ALL CHALLENGES! Be sure to read congrats.md in your home directory! (hit any key to dismiss this message)"
+        wall "Congratulations -- you have completed ALL CHALLENGES! Be sure to read congrats.md in your home directory! (hit Enter to dismiss this message)"
         touch "${wsroot}"/team_has_been_congratulated
       fi
     fi
diff --git a/terraform/.gitignore b/terraform/.gitignore
@@ -1,3 +1,4 @@
 .terraform/
 *.tfvars
 *tfstate*
+!main.auto.tfvars
diff --git a/terraform/.terraform.lock.hcl b/terraform/.terraform.lock.hcl
diff --git a/terraform/dns.tf b/terraform/dns.tf
@@ -0,0 +1,39 @@
+
+data "aws_route53_zone" "root_zone" {
+  count = var.create_dns ? 1 : 0
+  name  = var.zone_name
+
+}
+
+resource "aws_route53_zone" "workshop_zone" {
+  count = var.create_dns ? 1 : 0
+  name  = "${var.event_name}.${var.zone_name}"
+}
+
+resource "aws_route53_record" "workshop" {
+  count   = var.create_dns ? 1 : 0
+  zone_id = data.aws_route53_zone.root_zone[0].zone_id
+  name    = aws_route53_zone.workshop_zone[0].name
+  type    = "NS"
+  ttl     = "300"
+  records = aws_route53_zone.workshop_zone[0].name_servers
+}
+
+resource "aws_route53_record" "teams" {
+  count      = var.create_dns ? var.num_teams : 0
+  zone_id    = aws_route53_zone.workshop_zone[0].zone_id
+  name       = "team-${count.index + 1}"
+  type       = "A"
+  ttl        = 300 #5 mins
+  records    = [module.team_servers[count.index].public_ip]
+  depends_on = [aws_route53_record.hub]
+}
+
+resource "aws_route53_record" "hub" {
+  count   = var.create_dns ? 1 : 0
+  zone_id = aws_route53_zone.workshop_zone[0].zone_id
+  name    = "hub"
+  type    = "A"
+  ttl     = 300 #5 mins
+  records = [module.hub.public_ip]
+}
diff --git a/terraform/main.auto.tfvars b/terraform/main.auto.tfvars
@@ -0,0 +1,27 @@
+num_teams  = 2
+event_name = "codemash"
+custom_security_group_ingress = [{
+  from_port   = 2332,
+  to_port     = 2332,
+  protocol    = "tcp",
+  description = "ssh",
+  cidr_blocks = "0.0.0.0/0"
+  },
+  {
+    from_port   = 8080,
+    to_port     = 8080,
+    protocol    = "tcp",
+    description = "http",
+    cidr_blocks = "0.0.0.0/0"
+  }
+  # Wetty config
+  , {
+    from_port   = 0,
+    to_port     = 6556,
+    protocol    = "tcp",
+    description = "http",
+    cidr_blocks = "0.0.0.0/0"
+  }
+]
+zone_name  = "sbx.justindebo.com"
+create_dns = true
diff --git a/terraform/main.tf b/terraform/main.tf
@@ -20,8 +20,8 @@ module "vpc" {
   name = local.name
   cidr = "10.0.0.0/16"
 
-  azs            = [data.aws_availability_zones.available.names[0]]
-  public_subnets = ["10.0.1.0/24"]
+  azs            = [data.aws_availability_zones.available.names[0], data.aws_availability_zones.available.names[1]]
+  public_subnets = cidrsubnets("10.0.0.0/16", 8, 8)
 
   enable_nat_gateway = false
 
@@ -124,7 +124,7 @@ module "team_servers" {
   instance_type               = "t3a.micro"
   key_name                    = aws_key_pair.main.key_name
   vpc_security_group_ids      = [module.security_group.security_group_id]
-  subnet_id                   = module.vpc.public_subnets[0]
+  subnet_id                   = module.vpc.public_subnets[count.index % 2]
   associate_public_ip_address = true
 
   user_data = <<-EOF
diff --git a/terraform/outputs.tf b/terraform/outputs.tf
@@ -2,6 +2,10 @@ output "instance_ips" {
   value = { for instance in module.team_servers : instance.tags_all["Name"] => instance.public_ip }
 }
 
+output "instance_dns" {
+  value = { for dns in aws_route53_record.teams : "${dns.name}.${aws_route53_zone.workshop_zone[0].name}" => dns.records }
+}
+
 output "db_pub_ip" {
   value = module.db.public_ip
 }
diff --git a/terraform/variables.tf b/terraform/variables.tf
@@ -32,3 +32,15 @@ variable "ssh_local_key_path" {
   type        = string
   default     = "~/.ssh/id_rsa.pub"
 }
+
+variable "create_dns" {
+  description = "Whether to create a Route53 DNS zone for the workshop instances"
+  type        = bool
+  default     = false
+}
+
+variable "zone_name" {
+  description = "Route53 zone name to use for workshop instances"
+  type        = string
+  default     = null
+}

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-local_team_servers := $(shell vagrant status \| grep -E -o 'team[0-9]+')`
	`1`	`+local_team_servers = $(shell vagrant status \| grep -E -o 'team[0-9]+')`
`2`	`2`
`3`	`3`	`ci:`
`4`	`4`	`@bash ./scripts/ci.sh`