boleto_cnab_api icon indicating copy to clipboard operation
boleto_cnab_api copied to clipboard

[UPD] Atualizando OS Alpine 3.20.2, README com informações sobre o Alpine e padronização do Dockerfile

Open mbcosta opened this issue 1 year ago • 3 comments

Atualizando OS Alpine 3.20.2, README com informações sobre o Alpine e padronização do Dockerfile

Parece que em versões recentes o Docker está retornando Warning devido alguns parâmetros, ao fazer um build com debug

$ docker --debug build ..........

está retornando

 2 warnings found:
 - MaintainerDeprecated: Maintainer instruction is deprecated in favor of using label (line 2)
The MAINTAINER instruction is deprecated, use a label instead to define an image author
More info: https://docs.docker.com/go/dockerfile/rule/maintainer-deprecated/
Dockerfile:2
--------------------
   1 |     FROM alpine:3.20.0
   2 | >>> MAINTAINER "[email protected]"
   3 |     
   4 |     WORKDIR /usr/src/app
--------------------

 - JSONArgsRecommended: JSON arguments recommended for CMD to prevent unintended behavior related to OS signals (line 31)
JSON arguments recommended for ENTRYPOINT/CMD to prevent unintended behavior related to OS signals
More info: https://docs.docker.com/go/dockerfile/rule/json-args-recommended/
Dockerfile:31
--------------------
  29 |     EXPOSE 9292
  30 |     USER app
  31 | >>> CMD bundle exec puma config.ru
  32 |     
--------------------

Segui as referencias que retorna https://docs.docker.com/reference/build-checks/maintainer-deprecated/ https://docs.docker.com/reference/build-checks/json-args-recommended/

cc @rvalyi

mbcosta avatar Aug 01 '24 21:08 mbcosta

ai viu @mbcosta agora eu botei uns testes com emissão de boleto pelo container. Aind vou olhar o PR, mas ja da para ficar mais tranquilo.

rvalyi avatar Aug 01 '24 21:08 rvalyi

certo eu vi @rvalyi , é algo semelhante que do pode ser feito na Localização? Porque se for incluído lá acredito que podemos alterar a imagem para latest e deixar o BRCobranca na branch master, já que os testes serão mais frequentes, assim tornamos desnecessários esses PRs de atualização

mbcosta avatar Aug 01 '24 22:08 mbcosta

então agora que ta no Alpine a gente pode sim ver de botar na OCA tb. Mas eu diria que seria bom começar na branch 16 e talvez fazer um backport depois. Ai como escrevi la eu acho que vale a pena segurar a migração do l10n_br_account_payment_brcobranca para depois do l10n_br_account para nao atrasar a migração do l10n_br_account. Ou seja em breve vamos pensar nisso sim, mas talvez hoje temos outras prioridades como destravar a migração na v16...

rvalyi avatar Aug 01 '24 22:08 rvalyi