Fix helm chart URLs

[janorn]

SUMMARY

The URL in the last change of the helm chart index generation is incorrect.
This will fix the URL and also put releases in relevant subdirs to make the final URL in the chart correct.

fixes #1200

ISSUE TYPE

• Bug, Docs Fix or other nominal change

ADDITIONAL INFORMATION

Download URL in the current index.yaml file doesn't point to the release file.

[janorn]

If someone could help with testing the workflow I would appreciate it. I have tested the command in itself and it works.

$ find helmdir
helmdir
helmdir/1.1.3
helmdir/1.1.3/awx-operator-1.1.3.tgz
helmdir/1.1.4
helmdir/1.1.4/awx-operator-1.1.4.tgz

$ helm repo index helmdir --url https://github.com/ansible/awx-operator/releases/download/
$ find helmdir
helmdir
helmdir/1.1.3
helmdir/1.1.3/awx-operator-1.1.3.tgz
helmdir/1.1.4
helmdir/1.1.4/awx-operator-1.1.4.tgz
helmdir/index.yaml

helmdir/index.yaml

apiVersion: v1
entries:
  awx-operator:
  - apiVersion: v2
    appVersion: 1.1.4
    created: "2023-01-27T15:12:40.5238315+01:00"
    description: A Helm chart for the AWX Operator
    digest: 0d2ddc1eb31a3e61fca1b37d6be957423aba79edc2f8343e2f4e27fd6030a3e7
    name: awx-operator
    type: application
    urls:
    - https://github.com/ansible/awx-operator/releases/download/1.1.4/awx-operator-1.1.4.tgz
    version: 1.1.4
  - apiVersion: v2
    appVersion: 1.1.3
    created: "2023-01-27T15:12:40.5223565+01:00"
    description: A Helm chart for the AWX Operator
    digest: 0ea2087d0201e790db8aa30f4680d0ad2f7773520759ee6641c1d0c5e4ac2aa8
    name: awx-operator
    type: application
    urls:
    - https://github.com/ansible/awx-operator/releases/download/1.1.3/awx-operator-1.1.3.tgz
    version: 1.1.3

[rooftopcellist]

@janorn I tested this out locally, it throws a syntax error because ln 384 is missing a ;\ at the end of it. Otherwise, this change works. Could you update the PR, then I can merge?

[rooftopcellist]

@janorn Using your changes, I created this PR to update the gh-pages branch.

• Updated index.yaml latest release #1205

[rooftopcellist]

Sorry @janorn I merged that a bit pre-maturely. Here is the patch for fixing that syntax error I mentioned:

• Fix minor helm make target syntax error #1206

This is the error I was getting running locally:

	if [ ${result} -eq 0 ]; then\
		echo "Downloaded ${dl_url}";\
		mkdir -p .cr-release-packages/${tag}
/bin/sh: -c: line 9: syntax error: unexpected end of file

[janorn]

You beat me to it 😂

[rooftopcellist]

Thanks @janorn !

It looks like it's working for me, can you confirm that the updated index.yaml is working for you as expected?

[janorn]

@rooftopcellist i just took of. I can test later tonight but at the moment I'm on the move.

[janorn]

@rooftopcellist
I have tested and it seems to work as it should now!