Helm chart missing extraContainers support and publishing to registry

Yes, that's why I ask you to fork this repo and test in your own place, so we don't waste PRs in official repo

Or we have to see 10+ failed PRs which seem to try different combinations of config

When your test in your repo is done, make a final PR to the official repo

I've tried a few variations in my personal repo as you suggested but seems to get one error after the next. Might be better to just revert the PR in the casdoor-helm until someone more knowledgable is able to address the semantic version feature. The casdoor pr that was merged is fine since it just pushes up git tags.

ok

@tinder71 from what I can see, you changed the chart name to "casdoor" which is the same name of Docker image, so it has replaced the original docker image. As I said, please don't mess up with the original docker image. https://github.com/casdoor/casdoor-helm/commit/bb89dcd693fa7c34cd9b60b8f3d6251f1063b0aa

And this one: https://github.com/casdoor/casdoor-helm/commit/8305b54a47a406504ade853013af2fa6ce3cb6ee is definitely wrong, you are creating new entry for that

It is really becoming a mess now

already fixed

I didn't anticipate the name change would cause a collision after it was published to the hub. The helm chart and casdoor docker image are in a good state now. I won't be making any CI related changes to them

@tinder71 more powerful invitation code online now: https://github.com/casbin/casdoor/pull/2628

Add docs PR for helm chart table