ryankert01 opened a new pull request, #201:
URL: https://github.com/apache/yunikorn-web/pull/201
### What is this PR for?
1 warning found:
- RedundantTargetPlatform: Setting platform to predefined $TARGETPLATFORM
in FROM is redundant as this is the default behavior (line 17)
Setting platform to predefined $TARGETPLATFORM in FROM is redundant as this
is the default behavior
More info:
https://docs.docker.com/go/dockerfile/rule/redundant-target-platform/
Dockerfile:17
--------------------
15 | # See the License for the specific language governing permissions
and
16 | # limitations under the License.
17 | >>> FROM --platform=$TARGETPLATFORM scratch
18 | COPY --chown=0:0 LICENSE NOTICE third-party-licenses.md
yunikorn-scheduler /
19 | USER 4444:4444
--------------------
from docker version 27.1.1
suggested change: remove `--platform=$TARGETPLATFORM` in dockerfile
ref:
https://docs.docker.com/reference/build-checks/redundant-target-platform/#examples
change:
1. shim: RedundantTargetPlatform
2. web:
- RedundantTargetPlatform: Setting platform to predefined $TARGETPLATFORM
in FROM is redundant as this is the default behavior (line 32)
- LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy
"ENV key value" format (line 36)
- FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 20)
- InvalidDefaultArgInFrom: Default value for ARG
node:${NODE_VERSION}-alpine results in empty or invalid base image name (line
20)
### What type of PR is it?
* [ ] - Bug Fix
* [x] - Improvement
* [ ] - Feature
* [ ] - Documentation
* [ ] - Hot Fix
* [ ] - Refactoring
### Todos
* [ ] - Task
### What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2799
### How should this be tested?
### Screenshots (if appropriate)
### Questions:
* [ ] - The licenses files need update.
* [ ] - There is breaking changes for older versions.
* [ ] - It needs documentation.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]