fix: handle multiple colons in docker image

Xavier FRANCOISrequested to merge
xavier.francois/docker:fix-unhandled-colons into master

Presentation

Fixes #65 (closed)

Checklist

  • Documented:
    • README.md reflects any job, variable or whichever visible change
    • kicker.json reflects any job, variable or whichever visible change
  • Tested & examplified:
    • (url to a project sample successfully proving the merge request fixes the issue)

Proof of non regression (port unset)

Case 1 : Port unset and version unset

SEMREL_INFO_NEXT_VERSION=0.0.2
DOCKER_RELEASE_IMAGE=multirepo-oab.si.fr.intraorange/group/project

echo "$DOCKER_RELEASE_IMAGE" | sed "s/\(:.*\)\{0,1\}$/:$SEMREL_INFO_NEXT_VERSION/"
multirepo-oab.si.fr.intraorange/group/project:0.0.2

echo $(echo "$DOCKER_RELEASE_IMAGE" | sed -e 's/\(\/.*\):.*/\1/')$(echo :$SEMREL_INFO_NEXT_VERSION)
multirepo-oab.si.fr.intraorange/group/project:0.0.2

Case 2 : Port unset and version set

SEMREL_INFO_NEXT_VERSION=0.0.2
DOCKER_RELEASE_IMAGE=multirepo-oab.si.fr.intraorange/group/project:0.0.1

echo "$DOCKER_RELEASE_IMAGE" | sed "s/\(:.*\)\{0,1\}$/:$SEMREL_INFO_NEXT_VERSION/"
multirepo-oab.si.fr.intraorange/group/project:0.0.2

echo $(echo "$DOCKER_RELEASE_IMAGE" | sed -e 's/\(\/.*\):.*/\1/')$(echo :$SEMREL_INFO_NEXT_VERSION)
multirepo-oab.si.fr.intraorange/group/project:0.0.2

Proof of bugfixing (port set)

Case 1 : Port set and version unset

SEMREL_INFO_NEXT_VERSION=0.0.2
DOCKER_RELEASE_IMAGE=multirepo-oab.si.fr.intraorange:5002/group/project

echo "$DOCKER_RELEASE_IMAGE" | sed "s/\(:.*\)\{0,1\}$/:$SEMREL_INFO_NEXT_VERSION/"
multirepo-oab.si.fr.intraorange:0.0.2

echo $(echo "$DOCKER_RELEASE_IMAGE" | sed -e 's/\(\/.*\):.*/\1/')$(echo :$SEMREL_INFO_NEXT_VERSION)
multirepo-oab.si.fr.intraorange/group/project:0.0.2

Case 2 : Port set and version set

SEMREL_INFO_NEXT_VERSION=0.0.2
DOCKER_RELEASE_IMAGE=multirepo-oab.si.fr.intraorange:5002/group/project:0.0.1

echo "$DOCKER_RELEASE_IMAGE" | sed "s/\(:.*\)\{0,1\}$/:$SEMREL_INFO_NEXT_VERSION/"
multirepo-oab.si.fr.intraorange:0.0.2

echo $(echo "$DOCKER_RELEASE_IMAGE" | sed -e 's/\(\/.*\):.*/\1/')$(echo :$SEMREL_INFO_NEXT_VERSION)
multirepo-oab.si.fr.intraorange/group/project:0.0.2
Edited by Xavier FRANCOIS

Merge request reports