Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Helm Chart - Add on Values option to add Grafana Init Image Repository #889

Open
rodrigosandrin opened this issue Jul 5, 2024 · 1 comment

Comments

@rodrigosandrin
Copy link

rodrigosandrin commented Jul 5, 2024

Good morning!

I would like to request if is possible to add some way to change the grafana init container repository.

I need to change all images to point to an internal repository we have this option in values.yaml:

image: 
    repository: "projects.registry.vmware.com/antrea/theia-grafana"
    pullPolicy: "IfNotPresent"
    tag: "9.1.6"

However in build/charts/theia/templates/grafana/deployment.yaml the init container is hardcode.

Also, maybe in this request we could add an option to change only the repository and keep the images in the other values, because to make this change we need to configure all images like this:

clickhouse:
  image: 
    repository: "myinternalrepo.local/docker/antrea/theia-clickhouse-server"
  monitor:
    image:
      repository: "myinternalrepo.local/docker/antrea/theia-clickhouse-monitor"
  cluster:
    installZookeeper: 
      image:
        repository: "myinternalrepo.local/docker/antrea/theia-zookeeper"
grafana:
  image:
    repository: "myinternalrepo.local/docker/antrea/theia-grafana"
sparkOperator:
  enable: true
  image: 
    repository: "myinternalrepo.local/docker/antrea/theia-spark-operator"
theiaManager:
  image:
    repository: "myinternalrepo.local/docker/antrea/theia-manager"

I suggest to have something like "defaultRegistry".

Thanks!

Copy link

github-actions bot commented Oct 4, 2024

This issue is stale because it has been open 90 days with no activity. Remove stale label or comment, or this will be closed in 90 days

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant