stackrox / stackrox

The StackRox Kubernetes Security Platform performs a risk analysis of the container environment, delivers visibility and runtime alerts, and provides recommendations to proactively improve security by hardening the environment.
Apache License 2.0
1.12k stars 142 forks source link

Fix shell scripts to pass shellcheck linter #3544

Open janisz opened 1 year ago

janisz commented 1 year ago

In scripts/style/shellcheck_skip.txt we have a list of not linted files. The goal of this issue is to get them fixed to pass shellcheck. To run shellcheck use make shell-style. Ideally every PR should fix one file:

lavanyajain commented 1 year ago

@janisz created PR to Fix the shell check linter for deploy/common/deploy.sh https://github.com/stackrox/stackrox/pull/3547/files

dastgirp commented 1 year ago

@janisz created PR to fix below listed files: dev-tools/helmdiff.sh scripts/k8s/cleanup.sh scripts/k8s/get-lb-ip.sh local/expand-db.sh

Please check if it's correct or any modifications are required.

rachana-dani commented 1 year ago

Created a PR for this issue -> https://github.com/stackrox/stackrox/pull/3578. Please review Files updated are deploy/k8s/sensor.sh and deploy/k8s/central.sh. @janisz Can you please approve the PR checks as well?

someoneme commented 1 year ago

Created the PR for this issue -> #3582. Files updated are deploy/k8s/deploy.sh and deploy/k8s/deploy-local.sh @janisz, Please verify this and approve the PR checks too.

someoneme commented 11 months ago

Created PR for this issue -> https://github.com/stackrox/stackrox/pull/8266 Files updated are deploy/openshift/deploy-local.sh @janisz Please verify and approve the PR checks