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

Address warnings in Dockerfiles #516

Closed
emerkle826 opened this issue Jul 10, 2024 · 0 comments · Fixed by #519
Closed

Address warnings in Dockerfiles #516

emerkle826 opened this issue Jul 10, 2024 · 0 comments · Fixed by #519
Labels
done Issues in the state 'done'

Comments

@emerkle826
Copy link
Contributor

Recent updates to Docker have started showing some warnings with our current set of Dockerfiles, similar to this:

 10 warnings found (use --debug to expand):
 - ConsistentInstructionCasing: Command 'copy' should match the case of the command majority (uppercase) (line 99)
 - FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 8)
 - FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 60)
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 12)
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 13)
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 71)
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 72)
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 73)
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 114)
 - LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 115)

They are just warnings, and the images build correctly, but at some point they should be addressed.

@adejanovski adejanovski added the done Issues in the state 'done' label Jul 11, 2024
emerkle826 added a commit that referenced this issue Jul 11, 2024
* [FEATURE] [#517](#517) Add DSE 6.9.0 to the build matrix
* [ENHANCEMENT] [#516](#516) Address warnings in Dockerfiles
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done Issues in the state 'done'
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants