Handle multiple apps

Makes:

• production app use url: http://$CI_PROJECT_NAME.$KUBE_DOMAIN
• staging app use url: http://$CI_PROJECT_NAME-staging.$KUBE_DOMAIN
• review apps use url: http://$CI_PROJECT_NAME-$CI_ENVIRONMENT_SLUG.$KUBE_DOMAIN

Also uses $CI_BUILD_REF_SLUG in environment name instead of $CI_BUILD_REF_NAME, which may have odd characters (including more slashes).

e.g. foo.example.com, foo-staging.com, and foo-27193-handle-4f99a2.example.com

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/27193

@ayufan Man, the documentation for $CI_ENVIRONMENT_SLUG is really unclear. How many characters does it truncate to? I thought it was 24 characters, but the example shows https://100-do-the-4f99a2.example.com, which is only 17 characters. Does it really strip out review/?

And I really wonder if we should use http://$CI_PROJECT_NAME-$CI_BUILD_REF_SLUG.$KUBE_DOMAIN instead since the URL has less character limits.

@ayufan Can environment.url reference a variable declared in variables? Because I don't like repeating the URL in both places.

added 1 commit

• c4d9a1a0 - Use slug for environment name

Compare with previous version

I think the app name is automatically based on $CI_ENVIRONMENT_SLUG, so I guess that's a good reason to use that in the URL as well. If that turns into review-BRANCH-SOMETHINGRANDOM, then the URL will be PROJECT-review-BRANCH-SOMETHINGRANDOM.KUBE_DOMAIN, which is consistent.

FYI, it's only WIP because it hasn't been tested and I'd like someone to actually spin up a cluster and use this script before committing it. It feels pretty safe, but you never know.

Also, you have an syntax error.

added 1 commit

• 07518aa7 - Fix one more env name

Compare with previous version

assigned to @markpundsack

I fixed the syntax error and tested autodeploy/Kubernetes.gitlab-ci.yml and it worked. The others follow the same thing, so this seems pretty safe now.

unmarked as a Work In Progress

assigned to @zj

Only a few renames and this should be good to go!

assigned to @markpundsack

added 7 commits

• 07518aa7...377fcb81 - 6 commits from branch master
• fee771b0 - Handle multiple apps

Compare with previous version

added 1 commit

• bdc0b0d6 - Handle multiple apps

Compare with previous version

Approved this merge request

@markpundsack I was renaming the rest anyway, to keep that up to date, so updated this too. It looked good to me, so I'm just going to merge this.

mentioned in commit 054bf206

merged