Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Makefile container tool check only verifies binary exists, not runtime readiness

Aperta Adatta ai principianti
  • Qualcosa non va?
#2,249 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
78/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
docker
Ambito
build-system, devops

Direzione di ricerca

Inizia nel Makefile, al controllo della disponibilità dello strumento per container, e verifica come viene eseguito prima di target come deploy-olm e undeploy-olm. Verifica che il controllo testi la disponibilità a runtime dello strumento per container selezionato, quindi esegui i Make target pertinenti con il runtime arrestato per confermare che le operazioni distruttive non vengano raggiunte per prime.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

lifecycle/stale

Problem

The Makefile's container tool availability check uses command -v which only verifies the binary exists on PATH:

ifeq ($(shell command -v $(CONTAINER_TOOL) >/dev/null 2>&1 && echo found),)
  $(error The selected container tool '$(CONTAINER_TOOL)' is not available on this system.)
endif

On macOS with Podman, the podman binary is always present even when the Podman machine is stopped. This means the check passes, the Makefile proceeds with targets like deploy-olm (which first runs undeploy-olm), and only fails later when podman build is actually invoked. By that point, the existing deployment may already be torn down.

Expected behavior

The Makefile should detect that the container runtime is not functional before executing any destructive operations (like undeploy-olm). A check like podman info or docker info would verify the daemon/machine is actually running.

Suggested fix

Replace the command -v check with a runtime health check:

ifeq ($(shell $(CONTAINER_TOOL) info >/dev/null 2>&1 && echo found),)
  $(error The container tool '$(CONTAINER_TOOL)' is installed but not running. For podman on macOS, run: podman machine start)
endif

[!Note]
Responses generated with Claude

Lingua principale
Go
Stelle
92
Fork
93
Merge medio
21h 49m
PR unite (30g)
39

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di openshift/oadp-operator

Tutte le issue di openshift/oadp-operator

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.