Skip to content

Commit

Permalink
Drop the FGC namespace from the fedora-toolbox images
Browse files Browse the repository at this point in the history
The FGC namespace used to be required by the Fedora container
guidelines, but not anymore [1].

[1] https://pagure.io/ContainerSIG/container-sig/issue/2

containers#615
  • Loading branch information
HarryMichal authored and debarshiray committed Feb 12, 2021
1 parent b27795a commit d575318
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 6 deletions.
2 changes: 1 addition & 1 deletion images/fedora/f32/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ ENV NAME=fedora-toolbox VERSION=32
LABEL com.github.containers.toolbox="true" \
com.github.debarshiray.toolbox="true" \
com.redhat.component="$NAME" \
name="$FGC/$NAME" \
name="$NAME" \
version="$VERSION" \
usage="This image is meant to be used with the toolbox command" \
summary="Base image for creating Fedora toolbox containers" \
Expand Down
2 changes: 1 addition & 1 deletion images/fedora/f33/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ ENV NAME=fedora-toolbox VERSION=33
LABEL com.github.containers.toolbox="true" \
com.github.debarshiray.toolbox="true" \
com.redhat.component="$NAME" \
name="$FGC/$NAME" \
name="$NAME" \
version="$VERSION" \
usage="This image is meant to be used with the toolbox command" \
summary="Base image for creating Fedora toolbox containers" \
Expand Down
2 changes: 1 addition & 1 deletion images/fedora/f34/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ ENV NAME=fedora-toolbox VERSION=34
LABEL com.github.containers.toolbox="true" \
com.github.debarshiray.toolbox="true" \
com.redhat.component="$NAME" \
name="$FGC/$NAME" \
name="$NAME" \
version="$VERSION" \
usage="This image is meant to be used with the toolbox command" \
summary="Base image for creating Fedora toolbox containers" \
Expand Down
12 changes: 9 additions & 3 deletions src/pkg/utils/utils.go
Original file line number Diff line number Diff line change
Expand Up @@ -98,8 +98,8 @@ var (
"fedora-toolbox",
parseReleaseFedora,
"registry.fedoraproject.org",
"f%s",
true,
"",
false,
},
"rhel": {
"rhel-toolbox",
Expand Down Expand Up @@ -345,7 +345,13 @@ func GetFullyQualifiedImageFromDistros(image, release string) (string, error) {
repository = distroObj.Repository
}

imageFull := distroObj.Registry + "/" + repository + "/" + image
imageFull := distroObj.Registry

if repository != "" {
imageFull = imageFull + "/" + repository
}

imageFull = imageFull + "/" + image

logrus.Debugf("Resolved image %s to %s", image, imageFull)

Expand Down

0 comments on commit d575318

Please sign in to comment.