Skip to content

Conversation

@Vinzius
Copy link

@Vinzius Vinzius commented Nov 25, 2024

Exemple d'input:

philibea/[email protected]
  with:
    type: deploy
    scw_access_key: ***
    scw_secret_key: ***
    scw_registry: rg.fr-par.scw.cloud/my-namespace-name/console:latest

Actuellement GetContainerName retourne "latest" plutot que "console"

=> Cela créé un container avec le nom "latest". Et cela bug si on met un tag qui n'est pas un bon format.

Mais au passage, c'est "by design" que le container name reprenne le nom de l'image registry ?

@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@philibea
Copy link
Owner

Exemple d'input:

philibea/[email protected]
  with:
    type: deploy
    scw_access_key: ***
    scw_secret_key: ***
    scw_registry: rg.fr-par.scw.cloud/my-namespace-name/console:latest

Actuellement GetContainerName retourne "latest" plutot que "console"

=> Cela créé un container avec le nom "latest". Et cela bug si on met un tag qui n'est pas un bon format.

Mais au passage, c'est "by design" que le container name reprenne le nom de l'image registry ?

C'est le comportement qui est souhaité car cela permet de créer plusieurs container en fonction du tag. Comment gérer du multi-deploy lié au tag sans ça ?

Mais au passage, c'est "by design" que le container name reprenne le nom de l'image registry ?
oui car avec cette gestion de nom on peut retrouver l'id du container pour le teardown , changer cette stratégie risque de casser plusieurs worflow.

@Vinzius
Copy link
Author

Vinzius commented Nov 26, 2024

C'est le comportement qui est souhaité car cela permet de créer plusieurs container en fonction du tag. Comment gérer du multi-deploy lié au tag sans ça ?

Ce que j'entends, c'est qu'actuellement le nom de ton svl container dépend du nom de ton registry container, right ? On pourrait avoir un champ séparé pour le nom du svl container, non ?

J'ai l'impression que c'est lié à cette issue : #24

Sinon par rapport à ma question d'origine, on est d'accord que y'a un soucis sur le fonctionnement de GetContainerName qui retourne le tag et non le name, right ?

@NicolasRoehm
Copy link

J'aurais également besoin de dissocier les deux informations : le tag et le nom du conteneur

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants