selkies-project / selkies-operator

K8s operator for per-user stateful workloads
https://selkies.io
Apache License 2.0
74 stars 13 forks source link

Change default/placeholder regions to `us-central1` #28

Closed JanCVanB closed 2 years ago

JanCVanB commented 2 years ago

(Waiting for #22)

us-central1 is a better default region than us-west1, as it should have better worst-case performance across US users. Outside of the US, users should be changing the default anyways.

danisla commented 2 years ago

@JanCVanB is the title and description of this PR accurate to the changes?

JanCVanB commented 2 years ago

@danisla Yes, it's just the last commit (rebased onto the commits in #22), which is finished in my eyes. It conflicts mildly with the README changes in #22, so I figured I'd wait to submit this one. However, we could merge this commit alone first, then rebase #22 to deconflict.

danisla commented 2 years ago

@JanCVanB i would rather review this without all of the other changes.

JanCVanB commented 2 years ago

Sounds good, I'll rebase it now

JanCVanB commented 2 years ago

Rebase complete! Is Selkies viable in all regions? If so, what "supported" regions can we suggest to users?