Skip to content

Commit

Permalink
Merge pull request #509 from kool-dev/build-args-envs
Browse files Browse the repository at this point in the history
Build args environment variables support
  • Loading branch information
fabriciojs authored Feb 29, 2024
2 parents 9b31f94 + 3917d20 commit 6511719
Show file tree
Hide file tree
Showing 3 changed files with 36 additions and 2 deletions.
13 changes: 13 additions & 0 deletions docs/02-Kool-Cloud/03-Building-Images-to-Deploy.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,19 @@ services:
Your image will be built locally when running the `kool` CLI for a deploy and then pushed securely to the Kool.dev Cloud registry to a repository dedicated to your app environment.

### Build arguments (`args`) variables

As stated above the args provided to Docker when building the image will come from the `services.<service>.build.args` configuration entry.

It's a common need to have different values for different environments (i.e staging vs production). Kool Cloud supports two different ways for you to have a single `kool.cloud.yml` definition and still use different values per environment:

- **Environment Variables**: we will parse environment variables before passing the build args to Docker so you can use the common syntax `FOO: "$FOO"` and the value will be interpolated to the current value of `FOO` when running the `kool cloud deploy` command.
- **Kool Cloud Environment Variables**: you can define the variables under the Environment on Kool.dev Cloud panel, and then use the special `FOO: {{FOO}}` syntax to have the value interpolated with the web panel managed value for `FOO`.

#### Escaping variables

If the value you want to use contains `$` sign that could lead to trouble on having the sign mistakenly parsed as a variable marker. To scape it you need to double it: `FOO=$$bar` will have the desired effect of getting the actual `$bar` string as the value of `FOO` environment variable.

### Using a Private Registry

You may already have or use your own private registry for handling images. You are welcome to hold the build process apart from the `kool cloud deploy` step and just use the already built images in your `kool.cloud.yml` file:
Expand Down
2 changes: 1 addition & 1 deletion docs/02-Kool-Cloud/05-Environment-Variables.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
Most applications and frameworks nowadays rely on environment variables to configure important aspects of their functions, mainly providing credentials and other secrets your app needs to work and access other resources.
Most applications and frameworks will rely on environment variables to configure important aspects of their functions, specially for providing credentials and other secrets your app needs to work and access other resources.

Kool.dev Cloud supports a few different ways you can define your environment variables for a deploying container, so pick the one that best suits you.

Expand Down
23 changes: 22 additions & 1 deletion services/cloud/build.go
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ import (
"kool-dev/kool/core/environment"
"kool-dev/kool/core/shell"
"kool-dev/kool/services/cloud/api"
"os"
"path/filepath"
"strings"

Expand Down Expand Up @@ -51,7 +52,7 @@ func BuildPushImageForDeploy(service string, config *DeployConfigService, deploy

if buildConfig.Args != nil {
for k, v := range *buildConfig.Args {
dockerBuild.AppendArgs("--build-arg", fmt.Sprintf("%s=%s", k, v))
dockerBuild.AppendArgs("--build-arg", fmt.Sprintf("%s=%s", k, parseDeployEnvs(v, deploy.Deploy.Environment.Env)))
}
}

Expand Down Expand Up @@ -124,3 +125,23 @@ func parseBuild(build interface{}) (config *DeployConfigBuild, err error) {
err = yaml.Unmarshal(b, config)
return
}

func parseDeployEnvs(i interface{}, env interface{}) string {
// workaround to allow for escaping $ with a double $$
_ = os.Setenv("$", "$")

var value = os.ExpandEnv(fmt.Sprintf("%s", i))

if strings.Contains(value, "{{") && strings.Contains(value, "}}") {
// we have something to replace!
if recs, ok := env.(map[string]interface{}); ok {
for k, v := range recs {
if strings.Contains(value, "{{"+k+"}}") {
value = strings.ReplaceAll(value, "{{"+k+"}}", fmt.Sprintf("%v", v))
}
}
}
}

return value
}

0 comments on commit 6511719

Please sign in to comment.