GoogleContainerTools/skaffold

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

Ouverte

#4 321 ouverte le 15 juin 2020

Voir sur GitHubGuide contributeur
 (2 commentaires) (4 réactions) (0 personne assignée)Go (1 416 forks)batch import
area/default-repobugbash/q4-2022fixithelp wantedjuly-chillkind/feature-requestneeds-design-proposalpriority/p2

Métriques du dépôt

Stars
 (12 822 étoiles)
Métriques de merge PR
 (Merge moyen 3j 6h) (16 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez comment ` default repo` est implémenté dans l'analyse des arguments de ligne de commande de Skaffold (probablement dans `cmd/skaffold/app/cmd/build.go`). Le nouveau drapeau ` repo prefix` doit appliquer le préfixe au nom de l'image sans modifier les barres obliques en tirets bas. Vérifiez les drapeaux de la commande `build` et la gestion du nom de l'image dans `pkg/skaffold/build/...` pour vous assurer que le préfixe est préfixé tel quel. La proposition est simple ; vérifiez qu'il n'y a pas de PR existants traitant de cela.
Stack technique
go
Domaine
clidevops
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70