GoogleContainerTools/skaffold

New flag --repo-prefix to backfill --default-repo functionality

Offen

#4.321 geöffnet am 15.06.2020

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (4 Reaktionen) (0 zugewiesene Personen)Go (1.416 Forks)batch import
area/default-repobugbash/q4-2022fixithelp wantedjuly-chillkind/feature-requestneeds-design-proposalpriority/p2

Repository-Metriken

Stars
 (12.822 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 3T 6h) (16 gemergte PRs in 30 T)

Beschreibung

Expected behavior

Previously I had been using --default-repo more like --repo-prefix this all came to a grinding halt when I attempted to use the below skaffold.yaml. When I ran it with

skaffold build --default-repo=abc.123.com

I expected to see:

building abc.123.com/bases/go/1.13

building abc.123.com/bases/go/1.14

Actual behavior

building abc.123.com/bases_go_1.13

building abc.123.com/bases_go_1.14

Information

  • Skaffold version: v1.10.1
  • Operating system: Linux
  • Contents of skaffold.yaml:
apiVersion: skaffold/v1beta15
kind: Config

build:
  artifacts:
    - image: bases/go/1.13
      context: bases/go/multi
      docker:
        buildArgs:
          GO_VERSION:     "1.13"
          ALPINE_VERSION: "3.12"
    - image: bases/go/1.14
      context: bases/go/multi
      docker:
        buildArgs:
          GO_VERSION:     "1.14"
          ALPINE_VERSION: "3.12"

Proposal

We introduce a new flag --repo-prefix which doesn't mangle the image names for Docker Hub and simply applies the prefix.

Contributor Guide

Research-Richtung
Schauen Sie sich an, wie ` default repo` in der Befehlszeilenargumentverarbeitung von Skaffold implementiert ist (wahrscheinlich in `cmd/skaffold/app/cmd/build.go`). Das neue Flag ` repo prefix` sollte das Präfix auf den Imagenamen anwenden, ohne Schrägstriche in Unterstriche zu ändern. Überprüfen Sie die Flags des `build` Befehls und die Behandlung des Imagenamens in `pkg/skaffold/build/...`, um sicherzustellen, dass das Präfix unverändert vorangestellt wird. Der Vorschlag ist unkompliziert; stellen Sie sicher, dass es keine bestehenden PRs gibt, die dies behandeln.
Tech Stack
go
Domain
clidevops
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Keine
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70