Inline getTags into its only user

This also eliminates a duplicate call to reference.Parse() and
removes an impossible erorr path.

Should not change behavior.

Signed-off-by: Miloslav Trmač <mitr@redhat.com>

Closes: #1112
Approved by: rhatdan
This commit is contained in:
Miloslav Trmač
2018-07-18 22:32:56 +02:00
committed by Atomic Bot
parent e8bc914134
commit 014b3dcc35
2 changed files with 1 additions and 14 deletions

View File

@ -28,10 +28,7 @@ func decompose(input string) (imageParts, error) {
if err != nil {
return parts, err
}
ntag, isTagged, err := getTags(input)
if err != nil {
return parts, err
}
ntag, isTagged := imgRef.(reference.NamedTagged)
if !isTagged {
tag = "latest"
if strings.Contains(input, "@sha256:") {

View File

@ -12,16 +12,6 @@ import (
"github.com/pkg/errors"
)
func getTags(nameInput string) (reference.NamedTagged, bool, error) {
inputRef, err := reference.Parse(nameInput)
if err != nil {
return nil, false, errors.Wrapf(err, "unable to obtain tag from input name")
}
tagged, isTagged := inputRef.(reference.NamedTagged)
return tagged, isTagged, nil
}
// findImageInRepotags takes an imageParts struct and searches images' repotags for
// a match on name:tag
func findImageInRepotags(search imageParts, images []*Image) (*storage.Image, error) {