Skip to content

chore : fix Dockerfile to use fully qualified image name#92

Open
rohanKanojia wants to merge 1 commit into
devtools-qe-incubator:mainfrom
rohankanojia-forks:pr/fix-dockerfile
Open

chore : fix Dockerfile to use fully qualified image name#92
rohanKanojia wants to merge 1 commit into
devtools-qe-incubator:mainfrom
rohankanojia-forks:pr/fix-dockerfile

Conversation

@rohanKanojia

Copy link
Copy Markdown

Description

When I was trying to build container image using podman, I was getting this error:

podman build -t quay.io/devtools-qe-incubator/eventmanager:0.0.4 -f images/builder/Dockerfile .
[1/2] STEP 1/5: FROM golang:1.18 AS builder
Error: creating build container: short-name "golang:1.18" did not resolve to an alias and no unqualified-search registries are defined in "/etc/containers/registries.conf"
  • Update Dockerfile to use fully qualified base image name with registry prefix in order to be used by podman.
  • Update runner image to use Red Hat UBI9 image
  • Replace Entrypoint instruction to use exec form instead of shell form

@rohanKanojia
rohanKanojia marked this pull request as ready for review April 1, 2025 14:42
Comment thread images/builder/Dockerfile Outdated
# Build the manager binary
# FROM registry.access.redhat.com/ubi8/go-toolset:1.17.7 as builder
FROM golang:1.18 as builder
FROM docker.io/golang:1.18 AS builder

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May can we switch to ubi9 go toolset?

+ Update Dockerfile to use fully qualified base image name with registry prefix in
  order to be used by podman.
+ Update runner image to use Red Hat UBI9 image
+ Replace Entrypoint instruction to use exec form instead of shell form

Signed-off-by: Rohan Kumar <rohaan@redhat.com>

@adrianriobo adrianriobo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

2 participants