User adjust interval for Speedtest-exporter

[dcwestra]

Is your feature request related to a problem?

Currently the chart sets the Speedtest interval to every hour.

Describe the solution you'd like

I would like it to be available for scale to adjust the interval.

The default is found on line 25 of charts/stable/speedtest-exporter/values.yaml.

please add as a variable to charts/stable/speedtest-exporter/questions.yaml.

Describe alternatives you've considered

I’ve submitted a ticket on discord and was told I needed to submit an enhancement

Additional context

No response

I've read and agree with the following

• [X] I've checked all open and closed issues and my request is not there.
• [X] I've checked all open and closed pull requests and my request is not there.

[dcwestra]

Add

- variable: interval
  label: "Interval"
  description: "This sets how often the speed test runs"
  schema:
    type: string
    default: 1h
    required: true

After line 76 in questions.yaml (# Include{metrics}), with proper indentation of course.

[stavros-k]

That’s definitely not an int

[dcwestra]

Good catch. Corrected. Sloppy copy paste edit other variable.

[PrivatePuffin]

You really don't have to tell us to do our jobs, thats pretty rude. But even more so, if you know it so well, just send a PR.

[dcwestra]

Sorry. I meant no disrespect. I'm new to truecharts, and relatively new to GitHub. I love what you all are doing with truecharts and was hoping to be able to contribute it some way.

I was just hoping that, as a community project, if I did some of the investigation work it would be helpful and appreciated. All of the documentation asks us, as end users, to do our due diligence before submitting enhancements or PRs. This was my attempt at that due diligence, and I'm sorry you found it offensive.

I haven't learned how to fork and do a pull request for only a part of a repo yet. I figured that if I suggested the edit in this way, it would save whoever made the edit time and work.

[PrivatePuffin]

Nah, we know how to do it, it's a time limit not a "don't know how" limit sadly enough. Most likely will be added around december.

[alfi0812]

Or you could open the pr (pull request) yourself. As you already know what needs to be changed. 😅 So creating a fork, making the change and then open the pull request. So it can be merged into the main repository

[PrivatePuffin]

Or you could open the pr (pull request) yourself. As you already know what needs to be changed. 😅 So creating a fork, making the change and then open the pull request. So it can be merged into the main repository

Hitting the edit button would already do all that though :-P

[PrivatePuffin]

Due to SCALE Apps deprecation closing this, as we're not going to improve the SCALE Apps GUI further.