acmcsufoss / acm-server

Terraform deployment files for acmCSUF.
MIT License
6 stars 2 forks source link

Replace NetworkManager-wait-online.service with a ping service #25

Open diamondburned opened 3 months ago

diamondburned commented 3 months ago

NetworkManager-wait-online.service is very unreliable, in that when NetworkManager unexpectedly changes, it stops working.

Ideally, something like a service that pings ecs.fullerton.edu would be far more reliable as an alternative.

diamondburned commented 3 months ago

As a side note, our healthcheck.service already does this, but it's not running at the same frequency. It can certainly be modified to do so, however.

amyipdev commented 3 months ago

Sidenote: healthcheck currently prevents Terraform applys that cause cs306.acmcsuf.com to return anything that isn't 200.

diamondburned commented 3 months ago

Interesting, what's the behavior of that? Does it reboot the server?

healthcheck currently disables itself if there's a user logged into the server. Does that help bypass this issue?

amyipdev commented 3 months ago

Interesting, what's the behavior of that? Does it reboot the server?

Doesn't appear to. This came up when I was trying to test Watch306 - upon running terraform apply, it saw that healthcheck would fail, and didn't appear to actually apply - that or it instantly rolled back after applying.

healthcheck currently disables itself if there's a user logged into the server. Does that help bypass this issue?

Let me test real quick.