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 Needs Updating to Latest Version #101

Open
RichieRogers opened this issue Aug 26, 2024 · 3 comments
Open

Helm Chart Needs Updating to Latest Version #101

RichieRogers opened this issue Aug 26, 2024 · 3 comments

Comments

@RichieRogers
Copy link

Hi,
I'm migrating over to Kubernetes now and this error has resurfaced.

helm repo add charlie-haley http://charts.charliehaley.dev
helm repo update
helm install omada-exporter charlie-haley/omada-exporter --set omada.host=https://omada.rartec.net --set omada.username=viewer-user --set omada.password=my-password --set omada.site=Default -n monitoring

panic: runtime error: slice bounds out of range [1:0]

goroutine 1 [running]:
github.com/charlie-haley/omada_exporter/pkg/api.(*Client).getCid(0xc0000b2680)
github.com/charlie-haley/omada_exporter@v0.13.0/pkg/api/auth.go:64 +0x1fc
github.com/charlie-haley/omada_exporter/pkg/api.Configure(0xd28700)
github.com/charlie-haley/omada_exporter@v0.13.0/pkg/api/api.go:51 +0x8f
github.com/charlie-haley/omada_exporter/cmd.run(0xc0000c6640?)
github.com/charlie-haley/omada_exporter@v0.13.0/cmd/exporter.go:91 +0x136
github.com/urfave/cli/v2.(*App).RunContext(0xc000082820, {0xa0ac68?, 0xc00002e040}, {0xc000022230, 0x1, 0x1})
github.com/urfave/cli/v2@v2.3.0/app.go:322 +0x97c
github.com/urfave/cli/v2.(*App).Run(...)
github.com/urfave/cli/v2@v2.3.0/app.go:224
github.com/charlie-haley/omada_exporter/cmd.Run()
github.com/charlie-haley/omada_exporter@v0.13.0/cmd/exporter.go:60 +0xc25
main.main()
github.com/charlie-haley/omada_exporter@v0.13.0/main.go:8 +0x17
panic: runtime error: slice bounds out of range [1:0]

It looks like the helm chart might not be up to date.
Can this be resolved?

Thanks,
Richie

@dvonessen
Copy link

I see the same error.
@RichieRogers did find a workaround for this?

@RichieRogers
Copy link
Author

I see the same error. @RichieRogers did find a workaround for this?

Hi,
Sadly not.
I'm still hoping that the helm chart will get updated to the latest version.

Thanks,
Richie

@dvonessen
Copy link

dvonessen commented Oct 3, 2024

Hi,
you can override the image.tag in the omada-exporter values.yml.
Just add following to your values yaml.

image:
  tag: 0.13.1

I already opened a PR charlie-haley/charts#11 to increase the appVersion in the helm chart.

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

No branches or pull requests

3 participants
@dvonessen @RichieRogers and others