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.

Sign up for GitHub

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

Jump to bottom

Disponibilizar no docker hub #47

Closed
PauloGoncalvesBH opened this issue Jun 12, 2020 · 22 comments · Fixed by #146 or #142
Closed

Disponibilizar no docker hub #47

PauloGoncalvesBH opened this issue Jun 12, 2020 · 22 comments · Fixed by #146 or #142
Assignees
@eliasreis54
Labels
documentation Improvements or additions to documentation enhancement New feature or request Hacktoberfest released on @latest released on v2.12.0

Comments

@PauloGoncalvesBH
Copy link
Member

PauloGoncalvesBH commented Jun 12, 2020 •
edited

Disponibilizar o ServeRest no docker hub.

Estando no docker hub será possível que usuários tenham 2 formas de utilizar o serverest, sem serem obrigados instalarem node.

  • Incluir informação de uso com docker no README.md
  • OPCIONAL: Incluir na entrega contínua o build e disponibilização no docker hub. Pré-requisito do job será criação de release.

Documentação:

  • Revisar documentação do CONTRIBUTING.md
  • Revisar Dockerfile
  • Revisar MAKEFILE
@PauloGoncalvesBH PauloGoncalvesBH added enhancement New feature or request new issue and removed new issue labels Jun 12, 2020
@PauloGoncalvesBH PauloGoncalvesBH mentioned this issue Jun 12, 2020
Closed
@PauloGoncalvesBH PauloGoncalvesBH added Hacktoberfest documentation Improvements or additions to documentation labels Sep 29, 2020
@eliasreis54
Copy link
Contributor

@PauloGoncalvesBH para essa issue aqui seria legal configurar um travis já né. O que você acha?

@PauloGoncalvesBH
Copy link
Member Author

@eliasreis54 o projeto está todo configurado com o github actions.
Só de commitar na master é feita publicação automática no NPM, gera changelog, coloca label e comentário na issue e PR de tudo que foi publicado.
A ideia era complementar o que existe atualmente no arquivo .releaserc, configurando a action de publicar o docker após a etapa de npm.

@eliasreis54
Copy link
Contributor

eliasreis54 commented Oct 2, 2020 •
edited

Verdade, não tinha me atendado que ta com github actions.
Vou dar uma olhada como fazer essas paradas ai.
Se quiser, pode me colocar nessa task também 😄

@PauloGoncalvesBH
Copy link
Member Author

Coloquei como assignee. Essa parte de colocar na entrega contínua é um pouco confusa, custei a pegar. Se precisar de qualquer apoio só chamar.

@eliasreis54
Copy link
Contributor

eliasreis54 commented Oct 2, 2020 •
edited

Combinado!
Uma pergunta, você já configurou as variáveis de ambiente DOCKER_USERNAME e DOCKER_PASSWORD no github actions ou em algum lugar que possa configurar essas envs?

@PauloGoncalvesBH
Copy link
Member Author

Nada meu caro, irei configurar ambos com os dados do docker hub.

@PauloGoncalvesBH
Copy link
Member Author

PauloGoncalvesBH commented Oct 2, 2020 •
edited

Pronto.
Coloquei o secret com esse nome para ser fácil de identificar no futuro a que se refere.
Basta colocar na pipeline que a variável de ambiente DOCKER_USERNAME é o secret DOCKER_HUB_USERNAME.

Captura de tela de 2020-10-01 22-42-12

@PauloGoncalvesBH
Copy link
Member Author

@eliasreis54 vou te adicionar como colaborador temporariamente para poder criar branch aqui no repositório ao invés de fork. Acredito que vai ajudar a desenvolver essa feature.

@eliasreis54
Copy link
Contributor

Ok @PauloGoncalvesBH, mas acredito que consigo configurar no fork mesmo.
Como o push no docker hub só vai poder ser feito caso seja um push na trunk acredito que não ficar poluindo o repositório oficial é até melhor.
O que você acha?

@PauloGoncalvesBH
Copy link
Member Author

@eliasreis54 se for possível apagar o que enviarmos lá, pode poluir a vontade nos testes. Não sei se tem como excluir imagem postada.

@eliasreis54
Copy link
Contributor

Maravilha. Como vamos postar várias imagens com a mesma tag, vai acabar sobrescrevendo sempre a mesma imagem.
Mas vou fazendo os testes...

@PauloGoncalvesBH
Copy link
Member Author

PauloGoncalvesBH commented Oct 2, 2020 •
edited

Pra fazer esse teste de integração contínua vai ter que:
2. Excluir todos os plugins do .releaserc para que não publiquemos novas release por engano (para corrigir vai dar um trabalhão)
3. Deixar dentro do plugin do .releaserc apenas a action semantic-release-docker

Recomendo fortemente você usar a branch beta, pois a entrega contínua está configurado para rodar nela e vai demandar menos configuração para os testes.

@PauloGoncalvesBH
Copy link
Member Author

PauloGoncalvesBH commented Oct 2, 2020 •
edited

@eliasreis54 a entrega contínua do docker funcionou 🎉
https://hub.docker.com/repository/docker/paulogoncalvesbh/serverest
vou ter que ajustar o README do hub.docker

Agora só precisamos verificar se o makefile (se ajustar será preciso ajustar a seção do makefile no contributing) e o dockerfile estão corretos.
A parte do README.md de usar o docker pode ajustar também, colocando badge caso exista. Se quiser eu ajusto essa parte específica.

@eliasreis54
Copy link
Contributor

Que legal @PauloGoncalvesBH maravilha que deu certo.
Lembrando que precisa voltar os plugins do semantic-release que removemos. Certo?
Vou testar fazer um pull da imagem aqui e subir localmente.

@eliasreis54
Copy link
Contributor

Uma pergunta, vocês tem algum servidor no discord ou no slack que a gente possa conversar por chat?

@PauloGoncalvesBH
Copy link
Member Author

@all-contributors please add @eliasreis54 for infra

@allcontributors allcontributors bot mentioned this issue Oct 2, 2020
Merged
@allcontributors
Copy link
Contributor

@PauloGoncalvesBH

I've put up a pull request to add @eliasreis54! 🎉

@PauloGoncalvesBH
Copy link
Member Author

tenho nenhum dos 2 Elias, consigo ter acesso a eles apenas à noite. Por ora pode me contatar no linkedin: https://www.linkedin.com/in/paulo-goncalves/

@eliasreis54 eliasreis54 mentioned this issue Oct 2, 2020
Merged
3 tasks
@eliasreis54
Copy link
Contributor

@PauloGoncalvesBH eu voltei os plugins do semantic release que removemos para testar e também tentei adicionar algumas instruções de como rodar com docker no README do projeto.

@PauloGoncalvesBH PauloGoncalvesBH linked a pull request Oct 2, 2020 that will close this issue
Add docker on readme #146
Merged
3 tasks
@PauloGoncalvesBH PauloGoncalvesBH removed a link to a pull request Oct 2, 2020
Add docker on readme #146
Merged
3 tasks
@PauloGoncalvesBH PauloGoncalvesBH linked a pull request Oct 2, 2020 that will close this issue
Add docker on readme #146
Merged
3 tasks
PauloGoncalvesBH pushed a commit that referenced this issue Oct 2, 2020
@eliasreis54
feat: add ServeRest on docker hub (#146)
4d750b8 
* feat(dockerhub): add plugins to semantic-release

* fix(dockerhub): add dev dependency

* fix(dockerhub): add semantic-release-docker in devDependencies

* fix(dockerhub): move semantic-release-docker to devDependencies

* fix(dockerhub): remove all semantic release plugins

* fix(dockerhub): add env vars

* fix: incluir análise de commit para triggar release

* fix: nome deve estar em lowercase

* fix: esqueci de ajustar outro nome em lowercase

* fix(dockerhub): return the semantic-release plugins

* docs(dockerhub): update readme to use docker

* docs(dockerhub): update readme

* docs(dockerhub): update readme

* docs(dockerhub): update readme

* docs(dockerhub): update readme

* docs(contributing): ajustar seção de publicação no NPM

* docs(readme): inserir badge de docker pulls

* Update README.md

Co-authored-by: Paulo Gonçalves <paulorochag@hotmail.com>

* Update README.md

Co-authored-by: Paulo Gonçalves <paulorochag@hotmail.com>

* Update README.md

Co-authored-by: Paulo Gonçalves <paulorochag@hotmail.com>

* Update README.md

Co-authored-by: Paulo Gonçalves <paulorochag@hotmail.com>

* Update README.md

Co-authored-by: Paulo Gonçalves <paulorochag@hotmail.com>

Co-authored-by: Paulo Gonçalves <paulorochag@hotmail.com>

Closes #47
@PauloGoncalvesBH
Copy link
Member Author

🎉 This issue has been resolved in version 2.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@eliasreis54
Copy link
Contributor

@PauloGoncalvesBH foi muito legal poder contribuir com seu projeto!
Vou acompanhando a criação das issues, pra continuar colaborando =)
Agora ta sem nenhuma =(

@PauloGoncalvesBH
Copy link
Member Author

@eliasreis54 esse fim de semana vou procurar pontos de melhoria. Se abrir alguma te falo.
Muito obrigado pelas contribuições, sua ajuda foi inestimável.

@PauloGoncalvesBH PauloGoncalvesBH linked a pull request Oct 2, 2020 that will close this issue
Feature/push docker hub #142
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@eliasreis54 eliasreis54

Labels
documentation Improvements or additions to documentation enhancement New feature or request Hacktoberfest released on @latest released on v2.12.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add docker on readme
Feature/push docker hub
2 participants
@PauloGoncalvesBH @eliasreis54