Remove the double @sha256 in case the image name is already digest base

[lveyde]

Due to the issue in podman it seems to incorrectly return digests with
double @sha256 in the name.

This patch fixes the issue in the bootkube.sh to remove double occurances
of @256 and replace it with single occurence, as normally expected.

I.e.
quay.io/openshift-release-dev/ocp-release@sha256@sha256:e237499d3b118e25890550daad8b17274af93baf855914a9c6f8f07ebc095dea

Reference to the issue:
#933

Signed-off-by: Lev Veyde lveyde@redhat.com

[openshift-ci-robot]

Hi @lveyde. Thanks for your PR.

I'm waiting for a openshift member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[wking]

I think we can probably wait for containers/podman#2106 to percolate through into RHCOS (probably on Friday), but will leave open in case other maintainers want to land a stopgap here.

[wking]

0.11.0 bumped to RHCOS 47.280 with Podman 1.0, so I think this is obsolete.

/close

[openshift-ci-robot]

@wking: Closed this PR.

In response to this:

0.11.0 bumped to RHCOS 47.280 with Podman 1.0, so I think this is obsolete.

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[deads2k]

I'm seeing lots of stuck pulls. If this gets past the bootkube stage, i think we should merge it.

[abhinavdahiya]

/ok-to-test

[wking]

/lgtm

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lveyde, wking

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 [wking]

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

[deads2k]

button-ing in to help our queues

[lveyde]

I'm seeing lots of stuck pulls. If this gets past the bootkube stage, i think we should merge it.

Thanks David.