deis / deis/workflow-cli

Consider disabling automatic Procfile lookup for deis pull

Ouverte
#307 2 commentaires 1 réaction 0 personnes assignées Voir sur GitHub
bug proposal
Langage dominant
Go
Étoiles
31
Forks
42
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Currently the deis workflow cli looks for a `Procfile` in the pwd and uses it to declare proc types when doing a `deis pull`.

This feature is very error prone, because it can happen very easily that one does a `deis create foo --no-remote` followed by `deis pull` while in the git worktree of another workflow app "bar", eg. a rails app with buildpack deploy and a `Procfile`.

If one makes this error, deis will create the new app "foo", but it wil use the bogus `Procfile` to setup a "web" process type along with wrong labels on the "foo"-service which will cause routing to be broken. The app will also not start because the "web" proc type is scaled to 1 and cmd to 0.

To fix such an app, one has to manually scale up cmd and scale down web, edit the deis database to remove the bogus proctype and edit the "foo"-service selector to point to the "cmd" type.

In short it is a major pain to fix for a feature that is probably very rarely used and it is hard to even know what caused this without in-depth knowledge of deis workflow and kubernetes.

I propose to instead change the behavior of the `--procfile` switch to accept either a string or a path to a procfile and remove the automatic lookup for a `Procfile` in the pwd.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par suivre le point d’entrée de `deis pull` et la gestion de l’option `--procfile`, puis identifiez où le `Procfile` du répertoire courant est découvert et analysé. La tâche est terminée lorsqu’un `Procfile` implicite n’est plus utilisé, tandis qu’une chaîne ou un chemin de procfile fourni explicitement continue de déclarer les types de processus prévus sans affecter les autres apps.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
go, kubernetes
Domaine
cli, devops
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.