chore(argo-cd): Update public.ecr.aws/bitnami/redis-exporter Docker tag to v1.67.0

[argoproj-renovate]

This PR contains the following updates:

Package	Update	Change
public.ecr.aws/bitnami/redis-exporter (source)	minor	1.58.0 -> 1.67.0

---

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR has been generated by Renovate Bot.

[yu-croco]

#3072 made this PR. It seems that we needed to enhance https://github.com/argoproj/argo-helm/blob/main/renovate.json at the same time. 🙃

[yu-croco]

🫠

[tico24]

Bitnami’s charts went oci-only about a month ago. Did we update to reflect that? If not, this would explain the error.

[yu-croco]

There was a PR to run Renovate against tags in values.yaml and we needed to handle scripts for it.

But here is the result. 🫠
Need to modify scripts for tags in values.yaml (or stop supporting Renovate to tags in values.yaml).

[tico24]

It looks like https://github.com/argoproj/argo-helm/blob/main/scripts/renovate-bump-version.sh Is only designed to look at our internal charts, not third party dependencies. It looks like renovate it running this script expecting to be able to change the upstream chart.yaml.

[tico24]

The script needs to be refactored so that it understands that it’s changing our argocd values file and not the third party chart.

[tico24]

The issue lies in the depname variable and how it’s then used.

[yu-croco]

I will fix the script in #3106 🙋

[mkilchhofer]

The initial PR (#3072) about dependencies inside values.yaml introduced a new PR naming pattern.

Our idea was to name PRs according to conventional commits using the chart names as the scopes:

• feat(argo-cd): Bump Argo CD
• chore(argo-cd): Bump redis-exporter
• chore(argo-workflows): Blah blah

Now we have the dependency in the conventional commit scope. I think we should address this as well.

Ref in contributors guide:
https://github.com/argoproj/argo-helm/blob/main/CONTRIBUTING.md#pull-request-title-linting