Evelyn Liu

Results 21 comments of Evelyn Liu

Let me try to rephrase to see if I understand correctly: `my/image:1.0` can exist in backend `a`, `b`, `c`, but you want to distribute the pull requests by adding a...

>> Adding a prefix notion in the top level config can be tricky when one day there is a need to, instead of trimming prefix, adding a prefix for backends....

> This is going to break all of our existing monitoring. Is there a way to make this backwards compatible? +1, maybe its better to wrap promethues metrics to add...

also http_ usually implies its related to http endpoints...you should just use kraken_

could you remove `lib/backend/artifactorybackend/security` ? I assume you have merged all you need in httputil?

@niklaskorz `"{\"$CI_REGISTRY\":{\".*\":{\"basic\":{\"username\":\"$CI_REGISTRY_USER\",\"password\":\"$CI_REGISTRY_PASSWORD\"}}}}"` doesnt look like the correct [config](https://github.com/uber/makisu#configuring-docker-registry)...could you try adding `security`?

@hatemosphere which version of makisu are you using? 0.1.1 and later should use https by default.

@hatemosphere ah you are right! Dockerhub returns unauthorized with a different https location for handling its credential magic on the ping request. I think maybe your registry enforces https on...

@hatemosphere https probably makes more sense as most basic auth implies you use TLS.

@hatemosphere It modifies the scheme with https. This probably happens when the https failed and fell back to http. Could you verify if there was a https request to the...