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

Atualizar Products.CMFPlone para 4.3.15. #331

Closed
idgserpro opened this issue Nov 30, 2016 · 11 comments · Fixed by plonegovbr/portal.buildout#77
Closed

Atualizar Products.CMFPlone para 4.3.15. #331

idgserpro opened this issue Nov 30, 2016 · 11 comments · Fixed by plonegovbr/portal.buildout#77
Milestone

Comments

@idgserpro
Copy link
Member

idgserpro commented Nov 30, 2016

Hoje estamos na 4.3.9.

Ao atualizar a versão do Plone, o relato #310 e #290 podem ser fechados.

Também seria interessante que os jobs de integração contínua que apontem para o master também passassem a quebrar a build quando falharem.

Além disso, com a atualização, podemos remover as pinagens da infra de testes presente em https://github.com/plonegovbr/portal.buildout/blob/9d9e21910c622c529c5d735dec037e11489da390/buildout.d/versions.cfg#L181,L187 pois praticamente todos estarão atualizados. Recomendamos deixar apenas robotframework-wavelibrary que não está pinado pelo Plone. Uma revisão no arquivo todo será interessante.

A pinagem em https://github.com/plonegovbr/brasil.gov.tiles/blob/2cd2026d8752dc4d7118788b348c25e1974c2e07/buildout.cfg e https://github.com/plonegovbr/brasil.gov.tiles/blob/2cd2026d8752dc4d7118788b348c25e1974c2e07/.travis.yml (foi necessária para passar o teste https://github.com/plonegovbr/brasil.gov.tiles/blob/2cd2026d8752dc4d7118788b348c25e1974c2e07/src/brasil/gov/tiles/tests/test_bannerrotativo_tile.robot#L59) também poderá ser removida, pois essa versão do selenium está disponível desde a 4.3.10 de Products.CMFPlone.

Ver também quais hotfixes podem ser removidos (também de portal.buildout) após essa atualização (ver https://plone.org/security/hotfixes).

Ver o impacto nas customizações feitas em https://github.com/plonegovbr/brasil.gov.portal/pull/270/files e que evoluiram para ae2054a após a atualização do plone.app.contenttypes. Acredito que não será necessário pois o impacto maior foi com o plone.app.contenttypes.

@hvelarde
Copy link
Member

só uma dúvida: o plone.subrequest está atualizado agora? lembrem que a versão pinada no Plone 4.3.9 estava com um memory leak.

@idgserpro
Copy link
Member Author

@hvelarde foi na versão 1.7.0 que corrigiram isso, não foi? Tomamos o cuidado de no release 1.1.5.2 já ter essa preocupação. Ver https://github.com/plonegovbr/portalpadrao.release/blob/9584fcdf5dee7793936328919bc0fbb71530ff2d/1.1.5.2/versions.cfg#L88

Ao atualizar o Plone, essa pinagem da gente inclusive pode ser removida.

@claytonc
Copy link
Contributor

claytonc commented Mar 7, 2017

@idgserpro eu achei um bug na versão 4.3.11 é melhor aguardar a correção.

plone/Products.CMFPlone#1959

@idgserpro idgserpro changed the title Atualizar Products.CMFPlone para 4.3.11. Atualizar Products.CMFPlone para 4.3.13. Mar 28, 2017
@idgserpro
Copy link
Member Author

@claytonc 4.3.13 parece resolver o problema. https://community.plone.org/t/plone-4-3-13-soft-released/3873. (Sendo mais específico, foi o pacote Products.CMFPlacefulWorkflow: 1.5.14 → 1.5.15). Depois se puder dar uma olhada, já vou alterar o título dessa issue.

@claytonc
Copy link
Contributor

@idgserpro além do problema da reinstalação do CMFPlacefulWorkflow também estava com problemas em adicionar os itens no portal_controlpanel [1].

Eu já estou utilizando a versão 4.3.12 em um novo projeto, apliquei o patch e atualizei o CMFPlacefulWorkflow, irei utilizar mas por causa do "Status check" [2].

Estava tendo problemas com o httpok aonde estava reiniciando as instâncias sem necessidade, ainda não consegui efetuar testes para comprovar se houve melhora ou não.

[1]1 plone/Products.CMFPlone@a86de30

[2] https://docs.plone.org/manage/deploying/production/status_check.html

@idgserpro idgserpro changed the title Atualizar Products.CMFPlone para 4.3.13. Atualizar Products.CMFPlone para 4.3.14. Apr 19, 2017
@claytonc
Copy link
Contributor

claytonc commented May 9, 2017

@idgserpro nesta versão vocês pretendem usar a plone.app.contenttypes 1.1.1 [1], que contém várias correções de bugs ?

Pergunto isso, pois no portalpadrao.release 1.1.5.2 esta pinado na versão 1.0 [2], sendo que o Plone 4.3.9 padrão é utilizar versão 1.1 [3].

Gostaria de sabe mais detalhes de erros sobre brasil.gov.agenda e brasil.gov.temas, pois em ambos produtos contém a pinagem plone.app.contenttypes<1.1a1'. Pergunto isso porque removi as pinagens de downgrade utilizadas em ambos estão e o portal esta funcionando corretamente, o único passo a mais que tive que executar foi um upgrade no plone.app.contenttypes em portal_setup.

[1] http://dist.plone.org/release/4.3.14/versions.cfg
[2] https://github.com/plonegovbr/portalpadrao.release/blob/master/1.1.5.2/versions.cfg#L78
[3] http://dist.plone.org/release/4.3.9/versions.cfg

@idgserpro
Copy link
Member Author

São issues diferentes, não precisa ser ao mesmo tempo. Sim, nos dois pacotes existem essas pinagens mas não temos o histórico da motivação. Em tese é como você falou, ao despinar não há muitos problemas, mas achamos que seria importante um teste exploratório após a atualização e por isso não foi feito ainda. De qualquer forma, bom saber que você já tem isso pinado e não há problemas aparentes.

@claytonc
Copy link
Contributor

claytonc commented May 9, 2017

@idgserpro na verdade estou tendo que fazer isso, por causa de vários bugs que os usuários provedores de conteúdos estão encontrando no IDG por causa do plone.app.contenttypes 1.0.

@idgserpro
Copy link
Member Author

@claytonc
Copy link
Contributor

claytonc commented May 9, 2017

@idgserpro testando aqui encontrei alguns probleminhas até agora nenhum grave todos são fáceis de resolver

Irei colocar os testes que fiz no #240

idgserpro added a commit to plonegovbr/brasil.gov.paginadestaque that referenced this issue Aug 23, 2017
https://travis-ci.org/plonegovbr/brasil.gov.paginadestaque/jobs/267740680

Corrige erro

AssertionError: Parent suite setup failed:
WebDriverException: Message: Can't load the profile. Profile Dir: %s If you specified a log_file in the FirefoxBinary constructor, check it for details.

Quando plonegovbr/brasil.gov.portal#331 for
atendido, podemos remover a pinagem do selenium em .travis.yml.
idgserpro added a commit to plonegovbr/portal.buildout that referenced this issue Aug 28, 2017
Corrige erro

AssertionError: Parent suite setup failed:
WebDriverException: Message: Can't load the profile. Profile Dir: %s If you specified a log_file in the FirefoxBinary constructor, check it for details.

Quando plonegovbr/brasil.gov.portal#331 for
atendido, podemos remover a pinagem do selenium em .travis.yml.
idgserpro added a commit that referenced this issue Sep 11, 2017
Corrige erro

AssertionError: Parent suite setup failed:
WebDriverException: Message: Can't load the profile. Profile Dir: %s If you specified a log_file in the FirefoxBinary constructor, check it for details.

Quando #331 for
atendido, podemos remover a pinagem do selenium em .travis.yml.
@idgserpro
Copy link
Member Author

idgserpro commented Sep 19, 2017

Ver o impacto da atualização devido às customizações presentes em https://github.com/plonegovbr/brasil.gov.portal/pull/270/files

idgserpro added a commit to plonegovbr/brasil.gov.agenda that referenced this issue Sep 21, 2017
Corrige erro

AssertionError: Parent suite setup failed:
WebDriverException: Message: Can't load the profile. Profile Dir: %s If you specified a log_file in the FirefoxBinary constructor, check it for details.

Quando plonegovbr/brasil.gov.portal#331 for
atendido, podemos remover a pinagem do selenium em .travis.yml.
idgserpro added a commit to plonegovbr/brasil.gov.tiles that referenced this issue Sep 22, 2017
Corrige erro

AssertionError: Parent suite setup failed:
WebDriverException: Message: Can't load the profile. Profile Dir: %s If you specified a log_file in the FirefoxBinary constructor, check it for details.

Quando plonegovbr/brasil.gov.portal#331 for
atendido, podemos remover a pinagem do selenium em .travis.yml.
idgserpro added a commit to plonegovbr/brasil.gov.paginadestaque that referenced this issue Sep 22, 2017
Corrige erro

AssertionError: Parent suite setup failed:
WebDriverException: Message: Can't load the profile. Profile Dir: %s If you specified a log_file in the FirefoxBinary constructor, check it for details.

Quando plonegovbr/brasil.gov.portal#331 for
atendido, podemos remover a pinagem do selenium em .travis.yml.
idgserpro added a commit to plonegovbr/brasil.gov.portlets that referenced this issue Sep 22, 2017
Corrige erro

AssertionError: Parent suite setup failed:
WebDriverException: Message: Can't load the profile. Profile Dir: %s If you specified a log_file in the FirefoxBinary constructor, check it for details.

Quando plonegovbr/brasil.gov.portal#331 for
atendido, podemos remover a pinagem do selenium em .travis.yml.
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Sep 22, 2017
Corrige erro

AssertionError: Parent suite setup failed:
WebDriverException: Message: Can't load the profile. Profile Dir: %s If you specified a log_file in the FirefoxBinary constructor, check it for details.

Quando plonegovbr/brasil.gov.portal#331 for
atendido, podemos remover a pinagem do selenium em .travis.yml.
idgserpro added a commit to plonegovbr/brasil.gov.vcge that referenced this issue Sep 22, 2017
Corrige erro

AssertionError: Parent suite setup failed:
WebDriverException: Message: Can't load the profile. Profile Dir: %s If you specified a log_file in the FirefoxBinary constructor, check it for details.

Quando plonegovbr/brasil.gov.portal#331 for
atendido, podemos remover a pinagem do selenium em .travis.yml.
@idgserpro idgserpro changed the title Atualizar Products.CMFPlone para 4.3.14. Atualizar Products.CMFPlone para 4.3.15. Oct 17, 2017
@hvelarde hvelarde added this to the 1.4 milestone Oct 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants