-
Notifications
You must be signed in to change notification settings - Fork 5
Remove arquivos desnecessários e corrige configurações #16
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
Remove arquivos desnecessários e corrige configurações #16
Conversation
There was a problem hiding this 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
@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.