Helm chart missing extraContainers support and publishing to registry

I took a deeper look into ArtifactHub and it requires a URL where the files can be served from. This medium article mentioned "Although Artifact Hub is a distribution service for resources like Helm charts, it is not a file storage service. Consequently, we need to find some other place to store our chart online. For this purpose, we can leverage GitHub pages".

Since Github pages isn't an option, an alternative is using Github Container Registry. Another article showcases its usage and it would be shown under the packages section right below releases. It's free for public repos. We can setup the CI to only publish the helm chart package whenever a change is detected in that directory. How does this approach sound?

it sounds ok to me!

Decided to try publishing the packaged helm chart to Docker Hub as they support it and casdoor already uses Docker Hub. https://github.com/casbin/casdoor/pull/2543

it seems to have bugs

Can you test it on your fork first?

Based on the github action error, seems to be just a permission issue: Error: server message: insufficient_scope: authorization failed . The docker forums state to use a Personal Access Token (PAS) instead as that provides fine grained controls to provide it access. The token should probably be created by the maintainers of casdoor

casdoor repo already pushes to dockerhub its images, did you use that credential name?

I tested more on my personal fork and discovered the issue. Turns out permissions were not the issue. It was just pointing to the wrong docker hub location. Tested on personal fork and was able to upload to docker hub on personal account with no issues. Created fresh PR https://github.com/casbin/casdoor/pull/2546. The new dockerhub location will be casbin/casdoor-helm-charts

does casbin/casdoor-helm-charts work now?

I saw https://hub.docker.com/repository/docker/casbin/casdoor-helm-charts/general was released. But the version is "0.3.0". Can we use the same version number as Casdoor?

Sure created fix https://github.com/casbin/casdoor/pull/2549

Lets deploy the helm chart only when it is changed. I see a lot of projects pick this option. The helm chart allows developers to enter the app version they want to use so they can pick any version. I will create a PR for the documentation later this week ✅

I'm not an expert of helm, also don't care much about how to use it in helm. The question here is that should not mess with version numbers in the git files. If you are hacking with updating version numbers in frequent PRs again and again, it's not a correct way to do it

Specific version numbers should be out of the git controlled file system

Created Github issue alongside PR fixes for it:
Issue
Casdoor Fix
Casdoor Helm Fix

merged

@tinder71 can you take a look at: https://github.com/casdoor/casdoor-helm/issues/8?

@tinder71 already fixed

Semantic Versioning
Issue
helm pr
casdoor pr

plz fix: https://github.com/casdoor/casdoor-helm/actions/runs/7515929174/job/20460434532

I see the issue. I thought semantic-release was language agnostic but it seems its really designed for nodeJS projects that contain a package.json. The original issue was to utilize semantic versioning. It may be better to manually create a release using gh tool

No. semantic-release can be used by other languages, see Casdoor (which is go): https://github.com/casbin/casdoor/releases

we use semantic-release for all languages (go, java, python, .net), etc., it's not for Node.js only

I had seen this error in the posted logs but there must be a way to configure it to not view the package.json SemanticReleaseError: Missing `package.json` file

I had copied most of the same format from casdoor originally