Examples: query, "exact match", wildcard*, wild?ard, wild*rd
Fuzzy search: cake~ (finds cakes, bake)
Term boost: "red velvet"^4, chocolate^2
Field grouping: tags:(+work -"fun-stuff")
Escaping: Escape characters +-&|!(){}[]^"~*?:\ with \, e.g. \+
Range search: properties.timestamp:[1587729413488 TO *] (inclusive), properties.title:{A TO Z}(excluding A and Z)
Combinations: chocolate AND vanilla, chocolate OR vanilla, (chocolate OR vanilla) NOT "vanilla pudding"
Field search: properties.title:"The Title" AND text
Answered
I'M Trying To Configure The Glue Agent To Use Aws Ecr Via Helm Charts. Below Is My Configuration. It Is Not Pulling The Image Though, It Is Failing With

I'm trying to configure the glue agent to use AWS ECR via helm charts. Below is my configuration. It is not pulling the image though, it is failing with K8S glue error: ImagePullBackOff . Does this look correct ?
` imageCredentials:

-- Use private authentication mode

enabled: true

-- If this is set, chart will not generate a secret but will use what is defined here

registry: <aws_account>.dkr.ecr.us-east-1.amazonaws.com

-- Registry username

username: AWS

-- Registry password

password: <password> `

  
  
Posted one year ago
Votes Newest

Answers 30


Can you fix this or should I open a PR. I'm blocked by this.

  
  
Posted one year ago

Seems like its just missing the brackets

  
  
Posted one year ago

Yes please 🙂
BTW: I originally thought the double quotes (in your PR) were also a bug, this is why I was asking, wdyt?

  
  
Posted one year ago

You guys are the maintainers of this repo

  
  
Posted one year ago

Its clearly a bug

  
  
Posted one year ago

I think the quotes don't effect the yaml

  
  
Posted one year ago

I can open a PR

  
  
Posted one year ago

Thanks everyone!

  
  
Posted one year ago

Test it on your local setup (I would hate to push a broken fix)
Is that possible?

  
  
Posted one year ago

I learned helm a few days ago

  
  
Posted one year ago

I dont know how to do that

  
  
Posted one year ago

Thanks! PR merged 🚀

  
  
Posted one year ago

For instance, quotes are used

  
  
Posted one year ago

Shouldn't this be a real value and not a template

you mean value being pulled to the pod that failed ?

  
  
Posted one year ago

I'm not familiar with helm that well to clone this, fix it, and then test

  
  
Posted one year ago

(once you verify PR the fix, I'll make sure it is merged)

  
  
Posted one year ago

ouch, I think you are correct, can you test a fix?

  
  
Posted one year ago

Can you fix locally, just to verify ?

  
  
Posted one year ago

Yea this is a bug

  
  
Posted one year ago

Yea that is a similar bug, needs the same fix

  
  
Posted one year ago

Do you want me to PR that fix?

  
  
Posted one year ago

I cannot test it at the moment, hence my question.
JuicyFox94 any chance you can blindly approve ?

  
  
Posted one year ago

Yes! Thanks so much for the quick turnaround

  
  
Posted one year ago

Merged, is it working for you now?

  
  
Posted one year ago

Hi everyone, I just fixed releases so new charts containing this fix are published. ty!

  
  
Posted one year ago

I made the PR here JuicyFox94 AgitatedDove14 https://github.com/allegroai/clearml-helm-charts/pull/106

  
  
Posted one year ago

Yes! Thanks so much for the quick turnaround

My pleasure 🙂

BTW: did you see this (it seems like the same bug?!)
https://github.com/allegroai/clearml-helm-charts/blob/0871e7383130411694482468c228c987b0f47753/charts/clearml-agent/templates/agentk8sglue-configmap.yaml#L14

  
  
Posted one year ago