Jon Edvald

Results 50 comments of Jon Edvald

I found a pretty good workaround for my use case. Instead of `require("jsonpath")` use `require("jsonpath/jsonpath.min")`. The latter is basically the browserified version, and that one doesn't use the monkey-patching trickery...

@AASanchezA I'd expect you could mount or otherwise write the `/kaniko/.docker/config.json` JSON file as suggested, and just have two keys under `auths`, one for each registry. Did you try that?...

FYI, the way we've solved this for [ourselves](https://github.com/garden-io/garden) is this patch, applied with https://www.npmjs.com/package/patch-package: ``` diff --git a/node_modules/pkg/prelude/bootstrap.js b/node_modules/pkg/prelude/bootstrap.js index 7285b7c..27d3975 100644 --- a/node_modules/pkg/prelude/bootstrap.js +++ b/node_modules/pkg/prelude/bootstrap.js @@ -1870,7 +1870,7 @@...

I'm not quite sure about this one. This is effectively a breaking API change, even though it never worked as expected. Another option here would be to copy the secret...

I see, that makes sense. In that case, I'd prefer to update the docs for the field instead of removing it, and emitting a log warning when it's being used,...

Thanks @mattpolzin! We'll be making several enhancements relating to sync error reporting/logging, leaving this open to make sure this is captured.

Wanna revive this one @thsig?

@thsig Tests seem to be legitimately failing. Please make sure those are okay before merging.

Hey @diasjorge! That's good feedback. We've done some work to make placement of code+configs more flexible, but we do currently assume source files for a module to be within the...

For this case, the advised approach is to place the `garden.yml` file for each Dockerfile next to the Dockerfile. So it'd be: ``` . ├── api │ └── Dockerfile │...