search

Desenho2018-1 / pan-pan

Projeto Pan Pan da disciplina de Arquitetura e Desenho de Software - FGA, UnB 2018.1
MIT License
3 stars 0 forks source link

Organizing files and improving scripts #64

Closed alaxalves closed 6 years ago

hugonxc commented 6 years ago

Percebi que você alterou a porta do mysql no compose e definiu variáveis de ambiente, mas não aplicou essas mudanças na configuração do backend. Por favor, corrija também no application.properties essas informações.

josutk commented 6 years ago

Com essas mudanças será necessário baixar novamente todas as dependências do docker em quem já tem o ambiente rodando?

alaxalves commented 6 years ago

Na verdade eu só alterei a porta porque o meu mysql local já estava utilizando a 3306. Voce acha que tem necessidade de mudar no yml? Porque isso pode acontecer com mais integrantes

alaxalves commented 6 years ago

Vai sim @josutk usando o docker-compose up --build

hugonxc commented 6 years ago

E @alaxalves esse choque na porta acontece comigo também eu solucionei parando meu serviço. Acho sua maneira mais elegante. Me equivoquei aqui na hora de associar ao application.properties, pois os containers conversam em rede local, logo o mapeamento interno não precisa refletir essa mudança. Mas peço que faça um teste rodando a aplicação só para fins de validação antes do pr.

alaxalves commented 6 years ago

@hugonxc atualizei o commit usando apenas a "3306". Nesse abordagem o docker irá escolher uma porta ociosa pra se conectar. Referencia

alaxalves commented 6 years ago

@hugonxc Eu rodei a aplicação e funcionou normalmente, mas posso ter esquecido algo. Sobe o ambiente aí também e confere pra gnt ficar mais seguro kkk

alaxalves commented 6 years ago

@hugonxc Atualizei o MR ;)