Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(docker setup): support the new docker compose command #1315

Merged
merged 1 commit into from
Mar 11, 2023

Conversation

FabricioFFC
Copy link
Contributor

Uma pequena adição no package.json, para suportar o novo comando docker compose, que substitui o antigo docker-compose, a partir da versão 20.10 (19-01-2023).

A adição suporta ambas versões do comando.

The `docker compose` command is the newer one that was introduced in Docker version 20.10
(2023-01-19).
@vercel
Copy link

vercel bot commented Mar 10, 2023

@FabricioFFC is attempting to deploy a commit to the TabNews Team on Vercel.

A member of the Team first needs to authorize it.

@aprendendofelipe
Copy link
Collaborator

Fala @FabricioFFC, obrigado pelo PR! 💪

É mesmo necessário manter a versão antiga? Não acha melhor deixar somente docker compose?

@FabricioFFC
Copy link
Contributor Author

FabricioFFC commented Mar 10, 2023

Opa @aprendendofelipe. É uma boa pergunta.

Por enquanto acho bom manter a antiga, pois por ser recente essa versão, acredito que muitas pessoas ainda usam a versão com o comando docker-compose.

Mas também posso atualizar a seção de Dependências globais do README, colocando a versão base do Docker a 20.10. O que acha melhor?

@aprendendofelipe
Copy link
Collaborator

Por enquanto acho bom manter a antiga, pois por ser recente essa versão, acredito que muitas pessoas ainda usam a versão com o comando docker-compose.

O comando docker-compose não vai deve parar de funcionar, pois o aliase vai ser mantido. O que deve acontecer logo é que essa opção abaixo, que permite forçar a utilização da V1, não vai mais existir e o docker-compose vai sempre direcionar para a V2:

image

Para continuar deixando a decisão entre compose V1 ou V2 na mão do dev, não precisamos mudar nada.

Só se quiséssemos por algum motivo forçar a utilização da V2, então faria sentido mudar o comando para docker compose, mas nesse caso não poderíamos manter as duas alternativas. Então como está no PR não tem efeito prático nenhum.

Mas também posso atualizar a seção de Dependências globais do README, colocando a versão base do Docker a 20.10. O que acha melhor?

Por que exatamente a 20.10? Por enquanto não vejo necessidade de mudar a versão mínima compatível.

Eu também acho que não é necessário alterar o comando, mas, caso eu esteja enganado, por favor me ajude a entender a razão para alterar. 👍

Resumindo. O que vai parar de funcionar é o compose V1, mas não o aliase docker-compose.

@FabricioFFC
Copy link
Contributor Author

FabricioFFC commented Mar 11, 2023

@aprendendofelipe vou então precisar investigar melhor, pois a motivação da PR foi bem simples, o docker-compose não funciona localmente aqui

image

(estou no Ubuntu 22.04 - Docker 23.0.1)

Assim que tiver novidades te aviso, obrigado pelas informações. Talvez quando eu instalei o Docker, fiz algo diferente, se bem que pelo que lembro segui a doc oficial.

@FabricioFFC
Copy link
Contributor Author

FabricioFFC commented Mar 11, 2023

Entendi agora. O alias que você mencionou, é um recurso do Docker Desktop. Como eu não estou usando ele, acabo não tendo esse alias.

Neste caso, acredito que a exceção seja por enquanto o meu setup, ainda mais tendo em vista que o time do docker quer manter a compatibilidade entre os comandos, não vejo mais necessidade desse PR, portanto, estou fechando. Desculpa pelo incômodo.

EDIT: Só um extra, para caso alguém esteja no mesmo cenário que tenho, onde uso o Docker Engine, sem o Docker Desktop.

Para funcionar o docker-compose eu criei um pequeno script com o nome docker-compose, numa pasta que está no PATH, chamando o docker compose e passando os argumentos para ele. Abaixo, o passo a passo.

  1. Criar um arquivo numa pasta que está no seu PATH, exemplo: vim ~/apps/docker-compose
  2. Adicionar o seguinte conteúdo no arquivo criado: docker compose --compatibility "$@"
  3. Fazer o script executável: chmod +x ~/apps/docker-compose

Um simples alias não funciona, pois um alias funciona apenas para sua sessão sh, quando você roda o npm run, ele abre outra sessão sem o seu alias.

Referências:

@aprendendofelipe
Copy link
Collaborator

Boa @FabricioFFC, então pode sim valer a pena aprovar o seu PR para não precisar exigir o Docker Desktop como dependência.

O que acha?

@vercel
Copy link

vercel bot commented Mar 11, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
tabnews ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 11, 2023 at 3:04AM (UTC)

@FabricioFFC
Copy link
Contributor Author

Tudo bem, irá ser útil quando não usar ele mesmo, e é mais fácil do que a alternativa que fiz aqui hehe

@aprendendofelipe aprendendofelipe merged commit 2a12cbc into filipedeschamps:main Mar 11, 2023
@aprendendofelipe
Copy link
Collaborator

Em produção! 🚀🚀🚀

@FabricioFFC, bem vindo à Turma de contribuidores do TabNews! 👏👏👏

@aprendendofelipe
Copy link
Collaborator

@FabricioFFC você foi citado no post comemorativo: https://www.tabnews.com.br/FelipeBarso/tabnews-exibir-ocultar-respostas-e-mais-outras-melhorias 🎉

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