search

SharebookBR / sharebook-backend

Projeto backend de código livre para o app Sharebook.
117 stars 75 forks source link

Develop #385

Closed danilobbezerra closed 3 years ago

danilobbezerra commented 3 years ago

Tomei a liberdade de criar as automações do github de actions de build e criei modifiquei a estrutura de onde fica o dockerfile docker compose e até adicionei na solução um projeto de docker compose.

image

Adicionei no arquivo readme as informações de build image

Acredito que isto permitirá maior eficiência no desenvolvimento diário do projeto.

Obs.: Desculpe o monte de commit, mas apanhei um pouco para fazer funcionar no github apenas os actions.

AppVeyorBot commented 3 years ago

:x: Build Sharebook-Backend: PR Check 1.0.83-rgbuxkic failed (commit https://github.com/SharebookBR/sharebook-backend/commit/32db084db7 by @danilobbezerra)

danilobbezerra commented 3 years ago

Acredito que o problema é que os testes de código não estão para versão 5 seria isso?

raffacabofrio commented 3 years ago

@danilobbezerra Vou investigar e te dou um feedback daqui a pouco.

raffacabofrio commented 3 years ago

@danilobbezerra Salve man. Desculpe a demora. Nosso pipeline estava quebrado por conta de uma atualização no sonar cloud que agora exige uma nova versão do Java. Eu corrigi o script de build ajustando a variável de ambiente JAVA_HOME e está tudo certo agora. Está tudo certo em relação aos testes unitários também. Ufaaaa.

Com relação a sua PR, tenho duas perguntas. Sem pegadinha. Pergunta inocente mesmo...

danilobbezerra commented 3 years ago

@danilobbezerra Salve man. Desculpe a demora. Nosso pipeline estava quebrado por conta de uma atualização no sonar cloud que agora exige uma nova versão do Java. Eu corrigi o script de build ajustando a variável de ambiente JAVA_HOME e está tudo certo agora. Está tudo certo em relação aos testes unitários também. Ufaaaa.

Com relação a sua PR, tenho duas perguntas. Sem pegadinha. Pergunta inocente mesmo...

  • Por que vc moveu o dockerfile para a pasta api? Na raíz ele não teria mais visibilidade pra quem está chegando no projeto?
  • Por que vc removeu a parte de sql server do docker composer?

Opa e ai blz, relaxa quanto a demora, sem problemas. Bom quanto as dúvidas vamos lá...

Mudança do docker file de diretório: Eu fiz isto baseado na minha experiência e pesquisei os conceitos, posso estar enganado mas segue minha lógica... Eu tenho 1 projeto que tem o docker compose na raiz do projeto onde fica o .sln, esse docker compose sobe 3 caras ao mesmo tempo, sendo Web, Api e Console.

Quando se trabalha na raiz você teria que criar diversos dockerfiles com nomes diferentes do nome "DockerFile" e no docker compose colocar os nomes/caminhos corretos para que subam corretamente essas aplicações seguindo o que está no dockerfile de nome diferente.

A minha lógica é se o dockerFile pertence a um determinado project que fica em um determinado diretório esse arquivo DockerFile deveria ficar lá, desta forma você mantem o nome padrão do arquivo e mantem ele no local que ele pertence, caso no futuro você precise subir mais 1 projeto junto como por ex outra api, web etc... basta criar a mesma lógica que informei no outro projeto e colocar no compose.

Quanto a remover o SQL do docker compose Não entendi o que ocorreu, acho que comi bola, porque tinha feito na master depois fui fazer na branch dev, vou dar uma revisada no que eu posso ter feito aqui.

raffacabofrio commented 3 years ago

@danilobbezerra Salve man. Desculpe a demora. Nosso pipeline estava quebrado por conta de uma atualização no sonar cloud que agora exige uma nova versão do Java. Eu corrigi o script de build ajustando a variável de ambiente JAVA_HOME e está tudo certo agora. Está tudo certo em relação aos testes unitários também. Ufaaaa. Com relação a sua PR, tenho duas perguntas. Sem pegadinha. Pergunta inocente mesmo...

  • Por que vc moveu o dockerfile para a pasta api? Na raíz ele não teria mais visibilidade pra quem está chegando no projeto?
  • Por que vc removeu a parte de sql server do docker composer?

Opa e ai blz, relaxa quanto a demora, sem problemas. Bom quanto as dúvidas vamos lá...

Mudança do docker file de diretório: Eu fiz isto baseado na minha experiência e pesquisei os conceitos, posso estar enganado mas segue minha lógica... Eu tenho 1 projeto que tem o docker compose na raiz do projeto onde fica o .sln, esse docker compose sobe 3 caras ao mesmo tempo, sendo Web, Api e Console.

Quando se trabalha na raiz você teria que criar diversos dockerfiles com nomes diferentes do nome "DockerFile" e no docker compose colocar os nomes/caminhos corretos para que subam corretamente essas aplicações seguindo o que está no dockerfile de nome diferente.

A minha lógica é se o dockerFile pertence a um determinado project que fica em um determinado diretório esse arquivo DockerFile deveria ficar lá, desta forma você mantem o nome padrão do arquivo e mantem ele no local que ele pertence, caso no futuro você precise subir mais 1 projeto junto como por ex outra api, web etc... basta criar a mesma lógica que informei no outro projeto e colocar no compose.

Quanto a remover o SQL do docker compose Não entendi o que ocorreu, acho que comi bola, porque tinha feito na master depois fui fazer na branch dev, vou dar uma revisada no que eu posso ter feito aqui.

Bacana man. Entendi. Só ajusta então a questão do composer que eu aprovo. E se possível manda uma PR pra release também. Vlw 👊

AppVeyorBot commented 3 years ago

:x: Build Sharebook-Backend: PR Check 1.0.83-cxwtgeiq failed (commit https://github.com/SharebookBR/sharebook-backend/commit/dbc9994931 by @danilobbezerra)

AppVeyorBot commented 3 years ago

:x: Build Sharebook-Backend: PR Check 1.0.83-liinwlfq failed (commit https://github.com/SharebookBR/sharebook-backend/commit/faa9c6c59b by @danilobbezerra)

AppVeyorBot commented 3 years ago

:x: Build Sharebook-Backend: PR Check 1.0.83-vncqsvrl failed (commit https://github.com/SharebookBR/sharebook-backend/commit/5d52473b62 by @danilobbezerra)

AppVeyorBot commented 3 years ago

:x: Build Sharebook-Backend: PR Check 1.0.83-fnoopuil failed (commit https://github.com/SharebookBR/sharebook-backend/commit/6226003248 by @danilobbezerra)

danilobbezerra commented 3 years ago

@raffacabofrio cara fiz as correções mas ainda está quebrando pode dar uma validada.

raffacabofrio commented 3 years ago

@raffacabofrio cara fiz as correções mas ainda está quebrando pode dar uma validada.

T1001: .UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe3 in position 564: invalid continuation byte [C:\projects\sharebook-backend\ShareBook\docker-compose.dcproj]

sonarcloud[bot] commented 3 years ago

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

AppVeyorBot commented 3 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.83-wfoqkmwb completed (commit https://github.com/SharebookBR/sharebook-backend/commit/6aa597251d by @danilobbezerra)

danilobbezerra commented 3 years ago

@raffacabofrio cara fiz as correções mas ainda está quebrando pode dar uma validada.

T1001: .UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe3 in position 564: invalid continuation byte [C:\projects\sharebook-backend\ShareBook\docker-compose.dcproj]

Correção efetuada.