Skip to content

Conversation

@CarlosSMA
Copy link

@CarlosSMA CarlosSMA commented Dec 21, 2025

Workaround que implementei ao encontrar a Issue #171.

Permite que o usuário defina a tag das imagens linuxtips/girus-frontend e linuxtips/girus-backend e mantém latest como padrão. Acho interessante ser implementado porque evita a modificação dos manifestos YAML dos deployments, e permite o uso do girus caso haja algum problema nas imagens mais recentes.

Teste

❯ ./dist/girus create cluster --help             
cluster - Cria o cluster Girus

[...]
Flags:
      --tag string                Define quais tags do frontend e backend serão usadas na criação do cluster (default "latest")
  -v, --verbose                   Modo detalhado com output completo em vez da barra de progresso
[...]
❯ ./dist/girus create cluster --tag 0.1 --verbose

[...]
Sobreescrevendo imagem dos deployments
Executando comando kubectl set image deployment/girus-frontend frontend=linuxtips/girus-frontend:0.1 -n girus
Executando comando kubectl set image deployment/girus-backend backend=linuxtips/girus-backend:0.1 -n girus

[...]

Abrindo navegador com o Girus...

────────────────────────────────────────────────────────────
GIRUS PRONTO PARA USO!
────────────────────────────────────────────────────────────
PRÓXIMOS PASSOS:
  • Acesse o Girus no navegador:
    http://localhost:8000

  • Para aplicar mais templates de laboratórios com o Girus:
    girus create lab -f caminho/para/lab.yaml

  • Para ver todos os laboratórios disponíveis:
    girus list labs
────────────────────────────────────────────────────────────
❯ k describe deploy girus-{frontend,backend} | rg -i image
    Image:        linuxtips/girus-frontend:0.1
    Image:      linuxtips/girus-backend:0.1

Dúvidas

  1. Devo adicionar tradução em espanhol para a flag e logs neste PR?
  2. Devo adicionar esta configuração no ~/.girus/config.yaml?

@ricardosilva86
Copy link
Collaborator

@CarlosSMA muito bom seu PR!
Só alguns pontos do meu lado, não sei se o @badtuxx vai concordar: dei uma olhada e aparentemente pode ter sido uma falha no build da versão amd64 para aquela build especificamente, chequei agora mesmo e tem a versão amd disponível na última latest. Gosto da ideia de poder passar uma tag para podermos testar uma versão específica, mas talvez ser mais verboso para ser explícito o que estamos querendo fazer: se pá usar --front-image-tag e --back-image-tag pra não ficar ambíguo.

@CarlosSMA
Copy link
Author

@ricardosilva86 Obrigado pelo feedback!

Fiz push do commit 813a0be, que contém a tua sugestão de termos flags diferentes. Se o Jef optar por manter como uma flag só, podemos fazer um revert desse commit somente.

 ./dist/girus create cluster --backend-tag 0.4.0 --frontend-tag 0.1.0  --verbose
────────────────────────────────────────────────────────────────────────────────
GIRUS CREATE
────────────────────────────────────────────────────────────────────────────────
[...]
Sobreescrevendo imagem dos deployments...
Executando comando kubectl set image deployment/girus-backend backend=linuxtips/girus-backend:0.4.0 -n girus
Executando comando kubectl set image deployment/girus-frontend frontend=linuxtips/girus-frontend:0.1.0 -n girus

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.

2 participants