use kustomize instead of sed for environment configuration by hahn-kev · Pull Request #1785 · sillsdev/web-languageforge
Still reading through this, but one comment I have is that you have commands like kubectl --context aws-rke in the Makefile. That's a good idea, but we'll need to add something to the README saying "You need to name your kubectl contexts by these names" or else people who have a different name in their .kubectl will have the command fail by default. And I'd also suggest changing the context names to something that includes "languageforge" or "lf" in it, because in my .kubectl I have the following:
- context:
cluster: aws-rke
namespace: languagedepot
user: aws-rke
name: languagedepot-rke
- context:
cluster: aws-rke
namespace: languageforge
user: aws-rke
name: languageforge-rke
I should actually rename those to something like language{forge,depot}-prod or {lf,ld}-prod but haven't gotten around to it. The point is that the kubectl context can carry a default namespace along with it, and I use that so I don't see languageforge pods when I'm deploying languagedepot, and vice versa.