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

Undocumented chart in helm repo #45

Closed
brianclemens opened this issue Jun 14, 2023 · 12 comments
Closed

Undocumented chart in helm repo #45

brianclemens opened this issue Jun 14, 2023 · 12 comments

Comments

@brianclemens
Copy link

There appears to be a chart with an older chart version (but newer Woodpecker version?) in the repo. Probably best to remove it to avoid confusion.

image
@pat-s
Copy link
Contributor

pat-s commented Jun 16, 2023

0.1.5 is the most recent chart and WP 0.15.9 the most recent "release".

The older versions with chart version "0.15.6" didn't properly distinguish between chart and app version.

@Rojikku
Copy link

Rojikku commented Jun 18, 2023

I did, in fact, notice the lack of proper distinguishment when I spent the last two hours trying to deploy woodpecker.

https://woodpecker-ci.org/docs/administration/kubernetes

But why, might I ask, weren't the docs updated?
Why would you do a breaking change and not update the docs?!

I'm going to try switching to the root chart from the individual agent and server charts, because the agent chart appears to be addicted to dind containers and refuses to just not deploy them like I ask and the settings are set to.

@6543
Copy link
Member

6543 commented Jun 18, 2023

the helm charts did got split out from the main repo and so they had new tagging ... we have to remove the old refs now ...

@6543
Copy link
Member

6543 commented Jun 18, 2023

... but I'm not sure if this is kind a breaking etc ...

@Rojikku
Copy link

Rojikku commented Jun 18, 2023

the helm charts did got split out from the main repo and so they had new tagging ... we have to remove the old refs now ...

Ah, is that what happened? That makes more sense.

@pat-s
Copy link
Contributor

pat-s commented Jul 2, 2023

@6543 Given how new the k8s support is and that it only really works in next, I guess removing the old releases would be fine here. Would also clear confusion...
I don't think there are many users atm and even if, they should be on 0.1.5 by now or be "forced" to migrate.

If a "fresh start" would help, now would be a good time I guess.

I can check if permissions allow me to do so but feel free to go ahead yourself.
I won't do it until either you or @anbraten confirm this action.

@anbraten
Copy link
Member

anbraten commented Jul 2, 2023

@pat-s give it a try. otherwise feel free to tag me, so I can have a look

@pat-s
Copy link
Contributor

pat-s commented Jul 2, 2023

@anbraten
Copy link
Member

anbraten commented Jul 2, 2023

How can I remove it. Should I just remove the lines of woodpecker-server & woodpecker-agent from the index.yaml?

@pat-s
Copy link
Contributor

pat-s commented Jul 2, 2023

Yes remove them from the index.yaml and maybe also delete the GH releases and assets

@anbraten
Copy link
Member

anbraten commented Jul 3, 2023

removed the old charts from the index.

@anbraten anbraten closed this as completed Jul 3, 2023
@Rojikku
Copy link

Rojikku commented Jul 3, 2023

@anbraten You uh, tested this right? Because I'm getting a lot of angry error messages from my helm bot right now about woodpecker.
I'll go on in a minute and see if maybe deleting and re-adding to clear a cache or something helps... But since I've never had this happen before I assume something has gone wrong.

failed to load Helm repository from index YAML: failed to load index: error unmarshaling JSON: while decoding JSON: json: cannot unmarshal string into Go struct field IndexFile.entries of type repo.ChartVersions

Continued evidence:

helm repo add woodpecker https://woodpecker-ci.org/                                                                                                                                                    Mon 03 Jul 2023 07:30:58 AM EDT
Error: looks like "https://woodpecker-ci.org/" is not a valid chart repository or cannot be reached: error unmarshaling JSON: while decoding JSON: json: cannot unmarshal string into Go struct field IndexFile.entries of type repo.ChartVersions

Edit: found it

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

5 participants