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

Do not use legacy version of the ENV command #3553

Closed
1 task done
bschmalhofer opened this issue Jun 28, 2024 · 1 comment
Closed
1 task done

Do not use legacy version of the ENV command #3553

bschmalhofer opened this issue Jun 28, 2024 · 1 comment
Assignees
Labels
docker Docker related issues tidying Tidying of the code
Milestone

Comments

@bschmalhofer
Copy link
Contributor

bschmalhofer commented Jun 28, 2024

Doing a build with Docker version 27.0.2 I noticed warnings like:

2 warnings found (use --debug to expand):

  • LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 25)
  • LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 26)

These were due to the lines:

grep -n ENV otobo.elasticsearch.dockerfile
25:ENV LC_ALL C.UTF-8
26:ENV LANG C.UTF-8

Note that there are no = sign between key and value.

The explanation of the warnings is at https://docs.docker.com/reference/build-checks/legacy-key-value-format/ . The documentation of the ENV command confirms that. See https://docs.docker.com/reference/dockerfile/#env .

Note that e.g. otobo.nginx-kerberos.dockerfile already used the = signs.

ENV SPNEGO_AUTH_COMMIT_ID=v1.1.1
ENV SPNEGO_AUTH_COMMIT_ID_FILE=1.1.1

TODO:

  • add the = sign to the ENV commands in rel-10_0, rel-10_1, rel-11_0, rel-11_1 branches
@bschmalhofer bschmalhofer added tidying Tidying of the code docker Docker related issues labels Jun 28, 2024
@bschmalhofer bschmalhofer added this to the OTOBO 10.0 milestone Jun 28, 2024
@bschmalhofer bschmalhofer self-assigned this Jun 28, 2024
bschmalhofer added a commit that referenced this issue Jun 28, 2024
the remaining cases that were not merged from rel-10_0
bschmalhofer added a commit that referenced this issue Jun 28, 2024
the remaining cases that were not merged from rel-10_0
@bschmalhofer
Copy link
Contributor Author

Done. The Docker image builds in the branches were fine. A run of the test suite in rel-10_0 was fine too. Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docker Docker related issues tidying Tidying of the code
Projects
None yet
Development

No branches or pull requests

1 participant