vendor: bump c/common to PR c/common@2339

[flouthoc]

Does this PR introduce a user-facing change?

None

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: flouthoc
Once this PR has been reviewed and has the lgtm label, please assign ygalblum for approval. For more information see the Code Review Process.

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

Needs approval from an approver in each of these files:

• OWNERS

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

[flouthoc]

Test containers/common#2339

[flouthoc]

I think podman save --multi-image-archive (untagged images) and podman image scp transfer are expecting wrong output before this new c/common patch podman was attempting to save images to tar or copy to remote machine via id then while loading both test expect the images to not have names but now since we work directly with images so we can always get their original names back.

@mtrmac Any thoughts, while debugging I think both of these test are expecting wrong outputs.

[mtrmac]

I think the podman image scp transfer test is definitely correct and the code is buggy: It should not say “Loaded image” for a name which was not present at the source at all.

The podman save --multi-image-archive (untagged images) case seems to be similar: (Assuming the created archive does not contain tags) podman load should say which image names it loaded, not what it was deduplicated to.

See the review comments about reporting all of Image.Names() and not just the actually-loaded values.

[mtrmac]

… the c/common review comments that is

[openshift-merge-robot]

PR needs rebase.

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-sigs/prow repository.