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

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

Merged
merged 4 commits into from
Oct 7, 2024

Conversation

mbcosta
Copy link
Member

@mbcosta mbcosta commented Aug 1, 2024

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 "raphael.valyi@akretion.com"
   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

@rvalyi
Copy link
Member

rvalyi commented Aug 1, 2024

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.

@mbcosta
Copy link
Member Author

mbcosta commented Aug 1, 2024

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

@rvalyi
Copy link
Member

rvalyi commented Aug 1, 2024

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...

dependabot bot and others added 4 commits October 7, 2024 19:56
Bumps [puma](https://github.com/puma/puma) from 6.4.2 to 6.4.3.
- [Release notes](https://github.com/puma/puma/releases)
- [Changelog](https://github.com/puma/puma/blob/master/History.md)
- [Commits](puma/puma@v6.4.2...v6.4.3)

---
updated-dependencies:
- dependency-name: puma
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
…ronização que o Docker está recomendando como 'Warnings' ao fazer o 'docker build'.
@mbcosta mbcosta force-pushed the UPD-alpine_3_20_2-readme-padronizacao_dockerfile branch from 95c645e to ca4b26b Compare October 7, 2024 23:03
@rvalyi rvalyi merged commit c8c4cf4 into master Oct 7, 2024
1 check passed
@github-grap-bot github-grap-bot deleted the UPD-alpine_3_20_2-readme-padronizacao_dockerfile branch October 7, 2024 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants