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

Pacotes plonegovbr deveriam ter mais de um job de testes #11

Closed
idgserpro opened this issue Jun 17, 2016 · 3 comments
Closed

Pacotes plonegovbr deveriam ter mais de um job de testes #11

idgserpro opened this issue Jun 17, 2016 · 3 comments
Assignees
Labels
Milestone

Comments

@idgserpro
Copy link
Member

Da forma como o buildout.cfg dos pacotes hoje é feito, ficamos presos a um release e só temos integração contínua "de verdade" quando um novo release está pra ser lançado e trocamos a versão.

A desvantagem disso é que pode ocorrer de, no lançamento de um release, o pacote ter um erro e isso tem de ser consertado: isso meio que invalida a lógica de integração contínua, que é a de visualizar os erros quando eles aparecem. Ao mesmo tempo, é útil sim testarmos com o próximo release pois alguma pinagem pode influenciar nos testes e também deve ser visto.

Assim, propomos que todos os pacotes tenham dois jobs: o tradicional, que pega do master, e um que utilize a url do úlltimo release no extends. collective.cover possui mais de um job e pode ser usado como referência. Com uso de sed, é possível adicionar uma linha numa linha x, seria possível o padrão ser o master e o do release com o uso dessa ferramenta (pode-se até pensar numa lógica de pegar o último release em portalpadrao.release ao invés de pinar manualmente a versão que sempre é alterada).

@idgserpro idgserpro added this to the 1.1.5 milestone Aug 8, 2016
@idgserpro idgserpro self-assigned this Aug 8, 2016
@idgserpro
Copy link
Member Author

Pensamos num processo mais simples.

https://raw.githubusercontent.com/plonegovbr/portal.buildout/master/buildout.d/versions.cfg passa a conter as pinagens que deverão constar num novo release. Dessa forma, todos os pacotes devem ter, como última linha do extends, essa url. Esse é o job padrão: referencia sempre as versões de um release ainda pendente.

O segundo job, via sed, remove essa linha, dessa forma, pega diretamente do master. O que pega do master pode falhar, ver como o collective.cover faz isso.

idgserpro added a commit to plonegovbr/brasil.gov.portal that referenced this issue Sep 8, 2016
idgserpro added a commit to plonegovbr/brasil.gov.portal that referenced this issue Sep 8, 2016
idgserpro added a commit to plonegovbr/brasil.gov.portal that referenced this issue Sep 9, 2016
@idgserpro
Copy link
Member Author

idgserpro commented Sep 9, 2016

A "prova de conceito" envolveu brasil.gov.portal e portal.buildout, feito nas branches (já removidas) em https://github.com/plonegovbr/portal.buildout/tree/new_use_base_versions_cfg/ e https://github.com/plonegovbr/brasil.gov.portal/tree/issue_11_portalpadrao_release.

Os pull requests de ambas as provas de conceitos já estão no master e a mesma idéia deve ser aplicada nos demais pacotes plonegovbr:

idgserpro added a commit to plonegovbr/brasil.gov.portal that referenced this issue Sep 26, 2016
Como plonegovbr/portal.buildout#53 já foi aprovado, podemos alterar a referência à branch para ser direto do master.
idgserpro added a commit to plonegovbr/brasil.gov.portal that referenced this issue Sep 27, 2016
idgserpro added a commit to plonegovbr/brasil.gov.paginadestaque that referenced this issue Oct 6, 2016
Melhorar a estrutura de testes de dependências em futuros releases, evitando que muitos erros só apareçam quando é lançado o release.

Implementa plonegovbr/portalpadrao.release#11
idgserpro added a commit to plonegovbr/brasil.gov.tiles that referenced this issue Oct 6, 2016
Melhorar a estrutura de testes de dependências em futuros releases, evitando que muitos erros só apareçam quando é lançado o release

plonegovbr/portalpadrao.release#11
idgserpro added a commit to plonegovbr/portal.buildout that referenced this issue Oct 6, 2016
O padrão fica um pouco diferente dos demais (como em brasil.gov.portal) uma vez que a estrutura dos cfgs do buildout dele é diferente.
idgserpro added a commit to plonegovbr/brasil.gov.temas that referenced this issue Oct 6, 2016
Melhorar a estrutura de testes de dependências em futuros releases, evitando que muitos erros só apareçam quando é lançado o release

Resolve plonegovbr/portalpadrao.release#11
idgserpro added a commit to plonegovbr/portal.buildout that referenced this issue Oct 6, 2016
O padrão fica um pouco diferente dos demais (como em brasil.gov.portal) uma vez que a estrutura dos cfgs do buildout dele é diferente.
idgserpro added a commit to plonegovbr/portal.buildout that referenced this issue Oct 6, 2016
@idgserpro idgserpro reopened this Oct 7, 2016
idgserpro added a commit to plonegovbr/brasil.gov.portal that referenced this issue Oct 7, 2016
Além de travis.cfg não existir mais devido a

plonegovbr/portalpadrao.release#11

as configurações aqui definidas foram implementadas em .travis.yml.
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Oct 7, 2016
Para poder ter dois jobs.

Melhorar a estrutura de testes de dependências em futuros releases, evitando
que muitos erros só apareçam quando é lançado o release.
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Oct 7, 2016
Para poder ter dois jobs ou mais jobs.

Melhorar a estrutura de testes de dependências em futuros releases, evitando
que muitos erros só apareçam quando é lançado o release.
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Oct 7, 2016
Para poder ter dois jobs ou mais jobs.

Melhorar a estrutura de testes de dependências em futuros releases, evitando
que muitos erros só apareçam quando é lançado o release.
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Oct 7, 2016
Para poder ter dois jobs ou mais jobs.

Melhorar a estrutura de testes de dependências em futuros releases, evitando
que muitos erros só apareçam quando é lançado o release.
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Oct 10, 2016
Para atender plonegovbr/portalpadrao.release#11

preciso corrigir o code analysis de alguns arquivos.
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Oct 10, 2016
Para atender plonegovbr/portalpadrao.release#11

preciso corrigir o code analysis de alguns arquivos.
idgserpro added a commit to plonegovbr/brasil.gov.barra that referenced this issue Oct 10, 2016
Atualiza também o bootstrap.py.
idgserpro added a commit to plonegovbr/brasil.gov.barra that referenced this issue Oct 11, 2016
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Oct 11, 2016
Para atender plonegovbr/portalpadrao.release#11

preciso corrigir o code analysis de alguns arquivos.
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Oct 11, 2016
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Oct 13, 2016
O pacote agora tem mais de um job em integração contínua.

Por causa das modificações feitas no buildout.cfg e no .travis.yml,
também foi necessário corrigir o code-analysis.
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Oct 13, 2016
O pacote agora tem mais de um job em integração contínua.

Por causa das modificações feitas no buildout.cfg e no .travis.yml,
também foi necessário corrigir o code-analysis.
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Oct 13, 2016
@idgserpro
Copy link
Member Author

Todos os pacotes agora possuem mais de um job de testes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant