Skip to content

Conversation

@samdoran
Copy link
Contributor

@samdoran samdoran commented Apr 9, 2024

Container images may only contain [a-zA-Z0-9][a-zA-Z0-9_.-]. It is somewhat common to use '/' in branch names, which will create an invalid container image name.

A more complete solution using tr is possible but this is a much simpler change to start.

Example from a job output:

17:20:34 docker: Error response from daemon: Invalid container name (cost-management-ci/reenable-coverage-683fd72-1712683220), only [a-zA-Z0-9][a-zA-Z0-9_.-] are allowed.

Container images may only contain [a-zA-Z0-9][a-zA-Z0-9_.-]. It is
somewhat common to use '/' in branch names, which will create
an invalid container image name.

A more complete solution using tr is possible but this is a much
simpler change to start.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant