Skip to content

Conversation

@pitangainnovare
Copy link
Contributor

Este PR corrige a release v0.1.0, que não está buildando devido ao arquivo Dockerfile, que não é ser utilizado (por padrão, usamos aqueles da pasta compose, mas por algum motivo, tínhamos esse Dockerfile extra na raiz). As seguintes alterações são realizadas:

  • Remoção do Dockerfile não utilizado, substituído por deployment via Kubernetes.
  • Adição dos arquivos de configuração para deploy no ambiente HML:
    • deployment-markapi-hml-django.yml
    • deployment-markapi-hml-celeryworker.yml
    • deployment-markapi-hml-celerybeat.yml
    • configmap-markapi-hml.yml
  • Atualização do Makefile para uso de docker compose (sem hífen).
  • Substituição do SQLite por PostgreSQL como base de dados.

Como testar

Baixa o código-fonte do PR e faça make build, seguido de make up. Acesse a seção admin do wagtail, após criar superusuário e fazer as migrações.

Copy link
Member

@robertatakenaka robertatakenaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pitangainnovare o Dockerfile é necessário para criar a imagem

@pitangainnovare
Copy link
Contributor Author

@pitangainnovare o Dockerfile é necessário para criar a imagem

@robertatakenaka o Dockerfile utilizado para construir a imagem está em compose/local/django/Dockerfile e compose/production ...

@pitangainnovare pitangainnovare linked an issue Jun 6, 2025 that may be closed by this pull request
@robertatakenaka
Copy link
Member

@pitangainnovare Dockerfile não está no PR. Além disso isso fica fora do padrão que temos adotado, colocar na raiz, criar Dockefile-dev também na raiz. Os kubernets não deveriam ficar públicos. Então estou assumindo que são só um template.

@pitangainnovare
Copy link
Contributor Author

pitangainnovare commented Jun 6, 2025

@pitangainnovare Dockerfile não está no PR. Além disso isso fica fora do padrão que temos adotado, colocar na raiz, criar Dockefile-dev também na raiz. Os kubernets não deveriam ficar públicos. Então estou assumindo que são só um template.

@robertatakenaka Entendo. Como o repo está sem README, assumi, lendo a estrutura de códigos, que foi algo derivado do scms-template. Por padrão, ao fazer make build, tenta-se subir vários serviços, inclusive solr (sem haver um schema), um postgresql, etc. Um SQLite tbm é usado. Enfim.

Sobre o Dockerfile que está na raiz, esse arquivo está fazendo com que a imagem não seja construída (veja erro em https://hub.docker.com/repository/registry-1.docker.io/scieloorg/markapi/builds/cfa4de0a-f064-4eb9-b0f8-98cc0cda1501). Ele procura um requirements.txt que não existe. Ou seja, a tag v0.1.0 não aponta para uma imagem válida, assim, nenhum yaml do kubernetes irá encontra-lá - isso me motivou a "corrigir" o repo.

Este projeto está no padrão do scms-template, como no scms-oca, usage, scms-upload, e demais, que não contêm Dockerfile na raiz, mas sim nas pastas compose/*, conforme for dev/local ou production.

Gerei os yaml que achei que eram os corretos, baseando-me na estrutura já existente do usage e que tenho acesso via Lens.

@pitangainnovare
Copy link
Contributor Author

@robertatakenaka yaml kubernetes foram movidos para kubernetes/hml.

@pitangainnovare pitangainnovare merged commit 9892267 into scieloorg:main Jun 16, 2025
3 checks passed
@pitangainnovare pitangainnovare deleted the major-fixes-v0.1.0 branch June 16, 2025 14:21
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.

Criar yaml/kubernetes

2 participants