GoogleContainerTools/skaffold

Dockerfile file analysis should support Buildkit's `RUN --mount`

Ouverte

#6 091 ouverte le 28 juin 2021

Voir sur GitHubGuide contributeur
 (1 commentaire) (3 réactions) (0 personne assignée)Go (1 416 forks)batch import
area/buildbuild/dockerhelp wantedkind/feature-requestpriority/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

(From the #skaffold Slack channel)

Buildkit supports run --mount=type=bind,source=xxx,target=yyy which behaves similar to a COPY xxx yyy. The type=bind can be omitted as this is the default mount type.

The doc does say that this is only valid with dockerfile syntax 1.2 or above:

To use this flag set Dockerfile version to at least 1.2

# syntax=docker/dockerfile:1.2

Expected behavior

Skaffold would detect changes to the source in xxx and trigger a rebuild.

Actual behavior

Skaffold doesn't detect the source changes.

Information

  • Skaffold version: unspecified

Workaround

Insert manual COPY statements.

Guide contributeur

Direction de recherche
Étudiez comment Skaffold analyse actuellement les Dockerfiles et suit les dépendances de fichiers. Recherchez le code qui gère les instructions `COPY` et étendez le pour reconnaître également les instructions `RUN mount=type=bind`. Le code pertinent se trouve probablement dans la logique de détection des modifications de fichiers. Prenez également en compte l'exigence de version de la syntaxe Dockerfile (1.2 ou supérieure). Examinez les tests existants et le répertoire `pkg/skaffold/docker` pour la logique d'analyse.
Stack technique
go
Domaine
backenddevopsbuild system
Type d'issue
Bug
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
DockerGoSkaffold
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70