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

Build e push de imagem OCI #8

Merged
merged 104 commits into from
Jul 28, 2021
Merged

Build e push de imagem OCI #8

merged 104 commits into from
Jul 28, 2021

Conversation

PedroRegisPOAR
Copy link
Contributor

@PedroRegisPOAR PedroRegisPOAR commented Jul 27, 2021

Resumo

Closes #5

Este PR faz o build da imagem de dev usando o podman rootless instalado via nix e faz push para o github registry.

Detalhes

É necessário usar um PAT para se logar, após logar é possível fazer:

podman pull docker.pkg.github.com/imobanco/docker-python-image/python:dev-latest

A flag --format docker

Após fazer o build e push, ao tentar fazer o pull localmente recebi o seguinte erro:
unsupported docker v2s2 media type: ""

Depois de algumas horas tentando entender o que se passava, descobri uma solução (apresentada para o podman 1.6.x).
A solução foi usar --format docker, https://bugzilla.redhat.com/show_bug.cgi?id=1794167, no podman build.

Algumas discussões relevantes sobre o erro:

Outros detalhes

Sobre a desinstalação do podman: containers/podman#9345

Copy link
Member

@rodrigondec rodrigondec left a comment

Choose a reason for hiding this comment

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

🤔

.github/workflows/push.yml Outdated Show resolved Hide resolved
.github/workflows/push.yml Outdated Show resolved Hide resolved
@@ -0,0 +1,31 @@
FROM python:3.8.3-slim-buster
Copy link
Member

Choose a reason for hiding this comment

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

está igual à de dev no momento?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Quase, apenas ENV PIP_DISABLE_PIP_VERSION_CHECK 1 que está diferente, em prod está desabilitando a warn do pip que aparece toda vez que existe versão mais recente.

Copy link
Member

Choose a reason for hiding this comment

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

Eu diria então pra deixar a mesma e termos uma imagem só 🤔

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Eu diria então pra deixar a mesma e termos uma imagem só thinking

Não sei, não está muito claro o que vai ocorrer mais a frente, pode ser que ainda exista alguma diferença que não estamos vendo ainda.

Copy link
Member

Choose a reason for hiding this comment

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

Justo

Makefile Outdated Show resolved Hide resolved
.github/workflows/push.yml Outdated Show resolved Hide resolved
.github/workflows/tests.yml Outdated Show resolved Hide resolved
PedroRegisPOAR and others added 5 commits July 28, 2021 16:28
Co-authored-by: Rodrigo Castro <[email protected]>
@rodrigondec rodrigondec merged commit 7c5ad5f into master Jul 28, 2021
@rodrigondec rodrigondec deleted the feature/build-and-push branch July 28, 2021 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tipo: Refatoração ♻️ Uma mudança que não é nem uma correção nem uma funcionalidade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Buildar imagem e dar PUSH para o registry do GITHUB
2 participants