remove usage of digest from helm catalog item uid

[debsmita1]

Fixes:
https://issues.redhat.com/browse/ODC-6077

Root analysis:

• digest is an optional property and the uid of helm chart depends on the digest's value. As a result, charts with no digest property will have the uid as <provider-name>--undefined so all the charts that belong to the same provider will have the same uid and the last one in the lot is considered.

Solution description:

• using the chart name and version as the value of uid.

Screenshots:

Before

After

Test setup:

apiVersion: helm.openshift.io/v1beta1
kind: HelmChartRepository
metadata:
  name: sample-repo
spec:
  connectionConfig:
    url:  https://raw.githubusercontent.com/debsmita1/charts/remove-digest/repo/community/index.yaml

[debsmita1]

/kind bug
/assign @rohitkrai03

[rohitkrai03]

This will not be unique when same chart is coming from two different chart repositories. I think we need to use chart URLs to make it always unique. Check #tmp-odc-helm channel on slack for related conversation around this.

[rohitkrai03]

This might clash again since the same chart can come from multiple chart repositories.

[debsmita1]

How about if we go with name + randomString ?

[rohitkrai03]

This should be good enough.

Suggested change

	uid: `${chartURL}`,
	uid: `${chartRepoName}--${chartURL}`,

[rohitkrai03]

This should be good enough.

Suggested change

	uid: `${chartURL}`,
	uid: `${chartRepoName}--${chartURL}`,

[rohitkrai03]

/lgtm
/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: debsmita1, rohitkrai03

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• frontend/packages/helm-plugin/OWNERS [rohitkrai03]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment