Code Monkey home page Code Monkey logo

Comments (8)

jackzampolin avatar jackzampolin commented on July 18, 2024

@mlieberman85 It would need to be refactored to make all the keys under .Values.config.{{ outputs,inputs }} an array instead.

Its something I've been meaning to add but haven't had time. Would love to see a PR for that.

from tick-charts.

amoskyler avatar amoskyler commented on July 18, 2024

@mlieberman85 @jackzampolin I've created a pull request which accomplishes this against the kubernetes/charts repo.

A maintainer closed it however as the one in kubernetes/charts is deprecated. I can look at porting it to this repository. Or of course...it can be updated in the other one. It would be nice to get a link between the kubernetes/charts repo and this one.

helm/charts#4170

from tick-charts.

jackzampolin avatar jackzampolin commented on July 18, 2024

Thanks @amoskyler! I don't have access on this repo anymore so I would cc @timhallinflux on this. This would be a great feature.

from tick-charts.

maddisondavid avatar maddisondavid commented on July 18, 2024

+1 for this. We have a configuration with multiple InfluxDB outputs depending on the different tags. Currently I'm unable to express this in a way that the current chart will accept (since we need multiple [[outputs.influxdb]] sections.

At the very least the chart could have something like an extraConfig section that allows the user to express an extra config block that is just appended to the end of the generated telegraf.conf

from tick-charts.

maddisondavid avatar maddisondavid commented on July 18, 2024

@amoskyler do you have any plans to port your PR over to this repo? I think it represents a big missing feature and it would be a shame for your work to get lost.

from tick-charts.

amoskyler avatar amoskyler commented on July 18, 2024

Hey @maddisondavid I was just thinking about this yesterday. I definitely want to get this added. I will port it over here and submit a PR in the next day or two.

from tick-charts.

amoskyler avatar amoskyler commented on July 18, 2024

@maddisondavid I've created a PR here #28

@timhallinflux Would you be up to take a look and give feedback?

from tick-charts.

amoskyler avatar amoskyler commented on July 18, 2024

@timhallinflux additionally, has there been any consideration of merging this chart downstream to the kubernetes/charts repository? I know the charts over there were deprecated, but I don't understand the reasoning.

from tick-charts.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.