Add dev-proxmox to UA Apps cluster list#356
Conversation
ffionalys
left a comment
There was a problem hiding this comment.
@KashDev00 @Ziggiyzoo We had problems with deploying to multiple clusters when the fallback cluster was introduced. e.g. traffic was still delivered to both even when one of them was down, observability was difficult as logs were split across multiple clusters.
Are we going to see similar problems with deploying dev to 2 different clusters?
The traffic redirection is done at the reverse proxy level. Ideally, we should make changes to the reverse proxy so it doesn't direct traffic to the cluster that is down. For observability, we can add fields to OpenTelemetry data, such as the cluster name from which the data is emitted. These are separate issues. This PR just deploys the apps to the new cluster. There are no changes to the reverse proxy, so no traffic is directed to the new cluster. We are just going to deploy apps to the new cluster. |
Ok that's fine with me, if we are going to have prod deploy to both then I think we need to make the changes to the reverse proxy first, so that traffic doesn't get directed to a cluster that's down. |
Closes: #355