Skip to content

Gera o relatório do Pages dentro do job de publicação - #72

Merged
ThomasTDS merged 1 commit into
masterfrom
ci/historico-allure
Oct 26, 2026
Merged

ThomasTDS merged 1 commit into
masterfrom
ci/historico-allure

Conversation

@ThomasTDS

@ThomasTDS ThomasTDS commented Oct 26, 2026 •

Copy link
Copy Markdown
Owner

Resumo

  • Problema: com dois merges quase juntos, as duas execuções da master baixavam o mesmo history.jsonl no job integration, cada uma acrescentava só a sua linha, e a segunda publicação apagava a da primeira. Aconteceu nos merges do Atualiza o print do relatório para o Allure #63 e do Adiciona testes de segurança leves, baseados no OWASP API Top 10 #67: o histórico ficou com 2 execuções em vez de 3
  • Correção: na master, o job integration (3.12) só guarda os resultados brutos (allure-results, artifact de 1 dia). O job deploy-pages, que já roda um de cada vez (concurrency: pages, do Enfileira as publicações do GitHub Pages #51), baixa os resultados e o histórico publicado, gera o relatório e publica. Como tudo isso acontece dentro do job enfileirado, a segunda publicação parte do histórico que a primeira publicou
  • O relatório de artifact dos PRs continua sendo gerado no job integration, como antes
  • O deploy-pages ganha contents: read, para o checkout do package.json e do allurerc.mjs

Limite conhecido

Com três ou mais merges quase simultâneos, o GitHub mantém só uma publicação na fila e cancela a intermediária. Os testes daquele commit rodam normalmente, mas a linha dele não entra no histórico. Não há corrupção, só uma execução de fora.

Test plan

  • YAML válido; ordem dos passos conferida nos dois jobs
  • CI deste PR: geração do relatório de artifact segue igual; o passo "Guardar os resultados" é pulado em PR, como esperado
  • Depois do merge: o histórico publicado deve passar de 6 para 7 execuções

Baixar o histórico, gerar o relatório e publicar passam a acontecer no job deploy-pages, que já roda um de cada vez. Antes, duas execuções seguidas da master baixavam o mesmo histórico no job de integração, e a segunda publicação apagava a linha da primeira.
@codecov

codecov Bot commented Oct 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@ThomasTDS
ThomasTDS merged commit e7a7cac into master Oct 26, 2026
8 checks passed
@ThomasTDS
ThomasTDS deleted the ci/historico-allure branch October 26, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant