-
Notifications
You must be signed in to change notification settings - Fork 113
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
[ BUG ] panic: runtime error: index out of range [1] with length 1 #124
Comments
Can confirm this is a bug in the latest tag of the docker container and most likely the v1.2.1 tag as they appear to have been uploaded at the same time. https://quay.io/repository/helmpack/chart-releaser?tab=tags My issue was resolved by using the 1.2.0 tag. Will leave this issue open for tracking and debugging by the devs but i am no longer blocked. |
thanks, @sOblivionsCall for the report, I don't have time this week to look over this, but I can try in my weekend, I need to set up an env for that. |
hello @sOblivionsCall sorry for the delay I was able to reproduce this issue only when my remote was configured as ssh like (git@github.com:cpanato/charts.git) then I got the exact error, when I convert that to be https (https://github.com/cpanato/charts.git) it worked can you check how your remote looks like? for ssh it will not work, the chart-releaser expect a HTTPS remote + token |
Ran into same issue in my CI so add little workaround which will set [[ $(git config --get remote.origin.url) == *"git@github.com"* ]] && \
SSH_REPO=true && \
PROJECT_USERNAME=$(git config --get remote.origin.url | sed 's/git\@github\.com\:\|\.git\|https\:\/\/github\.com\///g' | awk -F\/ '{printf $1}') && \
PROJECT_REPONAME=$(git config --get remote.origin.url | sed 's/git\@github\.com\:\|\.git\|https\:\/\/github\.com\///g' | awk -F\/ '{printf $2}')
[[ ${SSH_REPO} == *"true"* ]] && \
git remote set-url origin https://github.com/${PROJECT_USERNAME}/${PROJECT_REPONAME}
cr index --push
[[ ${SSH_REPO} == *"true"* ]] && \
git remote set-url origin git@github.com:${PROJECT_USERNAME}/${PROJECT_REPONAME}.git |
the chart release expect the remote to be HTTPS and not SSH |
This will set |
Experiencing the same bug. |
## Which problem is this PR solving? With the bump to chart releaser 1.5.0 we lost the ability to do `cr index` with an `ssh` git remote; the tool now requires us to use an `https` remote. For the `v1.2.3-secure-tenancy` release I had to do the `cr index` step manually from my laptop. See and helm/chart-releaser#124. The PR solves this problem and also removes some deprecated flags and unused env vars. ## Short description of the changes - set git remote to https url for `cr index` step. - remove unused `--charts-repo` flag - remove unused variables. ## How to verify that this has the expected result I couldn't :/ but I'll try to do a honeycomb chart release once this is merged to test it.
Hit this as well. Needed to read source code to find out what the problem is.... |
I have a CircleCi pipeline that triggers on commit to the main branch. CR is ran using docker with the latest tag:
On the index step i am seeing:
Obviously it's an issue with an out of bounds index but i don't know what index it is referring to. This pipeline has ran flawlessly for months without any changes so i'm fairly certain it's not something i did. Can anyone provide some insight as to what might be going on?
The text was updated successfully, but these errors were encountered: