use only tags with prefix v for version determination

[cgroschupp]

Description

git describe uses the wrong tag for the container images.

Checklist

• Unit tests updated
• End user documentation updated

[johngmyers]

I think we should make the same change to scripts/releaser.sh. It doesn't appear to be used by Helm chart releases.

[cgroschupp]

I think we should make the same change to scripts/releaser.sh. It doesn't appear to be used by Helm chart releases.

done

[johngmyers]

/ok-to-test
/lgtm
/assign @szuecs

[szuecs]

/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cgroschupp, szuecs

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:

• OWNERS [szuecs]

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

[cgroschupp]

@szuecs @johngmyers I need some help to understand why Prow pushes the image with the wrong tag: https://storage.googleapis.com/kubernetes-jenkins/logs/post-external-dns-push-images/1675907490161627136/artifacts/build.log

[johngmyers]

@cgroschupp I believe that is a question to ask in https://github.com/kubernetes-sigs/image-builder

[johngmyers]

@cgroschupp It's because of line 90 in Makefile.

[johngmyers]

Found it. It's from https://github.com/kubernetes/test-infra/blob/5c915df0de03a462981cfa2a5dfa225e083b8109/images/builder/main.go#L95

[cgroschupp]

@johngmyers Can you look over there, please? kubernetes/test-infra#30006