Remove arquivos desnecessários e corrige configurações#16
Remove arquivos desnecessários e corrige configurações#16pitangainnovare merged 14 commits intoscieloorg:mainfrom
Conversation
robertatakenaka
left a comment
There was a problem hiding this comment.
@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 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. |
|
@robertatakenaka yaml kubernetes foram movidos para kubernetes/hml. |
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:
Dockerfilenão utilizado, substituído por deployment via Kubernetes.deployment-markapi-hml-django.ymldeployment-markapi-hml-celeryworker.ymldeployment-markapi-hml-celerybeat.ymlconfigmap-markapi-hml.ymlMakefilepara uso dedocker compose(sem hífen).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.