[go: up one dir, main page]

scripts: Fix edge case in build-debian-packages-dependencies.sh

pietrorequested to merge
nomadic-labs/tezos:abate@fix-build-inspect into master

What

Fix a problem in the script build-debian-packages-dependencies.sh that arise when the architecture of the image is the same of the architecture we want to add.

Why

Sometimes we can get these kind of errors.

Appending to image manifesto for registry.gitlab.com/tezos/tezos/build-debian-bookworm:abate-reduce-packages-tests-matrix with new amd64 image
Digest for other registry.gitlab.com/tezos/tezos/build-debian-bookworm:abate-reduce-packages-tests-matrix: 
ERROR: failed to parse source "registry.gitlab.com/tezos/tezos/build-debian-bookworm@", valid sources are digests, references and descriptors: invalid reference format

How

I simply check if this is the case and I skip the append command.

Manually testing the MR

This is complicated since I saw this error happening once in a while. make sure the script still works in a test release pipeline like nomadic-labs/tezos@552cbfc3.110

Checklist

  • Document the interface of any function added or modified (see the coding guidelines)
  • Document any change to the user interface, including configuration parameters (see node configuration)
  • Provide automatic testing (see the testing guide).
  • For new features and bug fixes, add an item in the appropriate changelog (docs/protocols/alpha.rst for the protocol and the environment, CHANGES.rst at the root of the repository for everything else).
  • Select suitable reviewers using the Reviewers field below.
  • Select as Assignee the next person who should take action on that MR

Merge request reports