-
Notifications
You must be signed in to change notification settings - Fork 0
Set berserker config via url #95
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
base: main
Are you sure you want to change the base?
Changes from all commits
b1811f8
7ea0587
db0dcc8
1ad4bea
dffc558
20e0b8c
bd96551
3398aa7
b441426
539b3d1
3037952
8beb1e8
b807df8
bdf8975
ccef454
ef1ef36
b45d630
57d6d2b
39c0d04
708f4b9
52bbe98
a5a9456
e338d6b
b7b0526
12ea674
52e0e0f
dd7426a
7cea873
71a82d9
9c83977
6be36f9
a1998f3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -19,7 +19,26 @@ else | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| echo "Using ACS pre-4.11 secured cluster setup (version: ${version_major_minor})" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| fi | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| # Create namespace and image pull secrets BEFORE running sensor.sh | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| kubectl create namespace stackrox || true | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| kubectl -n stackrox create secret docker-registry stackrox \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --docker-server=quay.io \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --docker-username="${REGISTRY_USERNAME}" \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --docker-password="${REGISTRY_PASSWORD}" || true | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| kubectl -n stackrox create secret docker-registry secured-cluster-services-main \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --docker-server=quay.io \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --docker-username="${REGISTRY_USERNAME}" \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --docker-password="${REGISTRY_PASSWORD}" || true | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| kubectl -n stackrox create secret docker-registry secured-cluster-services-collector \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --docker-server=quay.io \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --docker-username="${REGISTRY_USERNAME}" \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --docker-password="${REGISTRY_PASSWORD}" || true | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+22
to
+39
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
Suggested fix-kubectl create namespace stackrox || true
+kubectl create namespace stackrox --dry-run=client -o yaml | kubectl apply -f -
-kubectl -n stackrox create secret docker-registry stackrox \
- --docker-server=quay.io \
- --docker-username="${REGISTRY_USERNAME}" \
- --docker-password="${REGISTRY_PASSWORD}" || true
+kubectl -n stackrox create secret docker-registry stackrox \
+ --docker-server=quay.io \
+ --docker-username="${REGISTRY_USERNAME}" \
+ --docker-password="${REGISTRY_PASSWORD}" \
+ --dry-run=client -o yaml | kubectl apply -f -(repeat for the other two secrets) 📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| "${STACKROX_DIR}/deploy/k8s/sensor.sh" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| kubectl -n stackrox create secret generic access-rhacs \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --from-literal="username=${ROX_ADMIN_USERNAME}" \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --from-literal="password=${ROX_ADMIN_PASSWORD}" \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
@@ -28,6 +47,9 @@ kubectl -n stackrox create secret generic access-rhacs \ | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| # Create the collector-config ConfigMap in order to enable external IPs | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| kubectl create -f "${SCRIPT_DIR}/collector-config.yaml" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| # Patch the collector DaemonSet to configure fact container | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| kubectl -n stackrox set env daemonset/collector FACT_PATHS="/tmp/data/**/*" FACT_LOGLEVEL="info" -c fact | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| echo "Deploying Monitoring..." | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| monitoring_values_file="${COMMON_DIR}/../charts/monitoring/values.yaml" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔒 Security & Privacy | 🟡 Minor | ⚡ Quick win
Checkout missing
persist-credentials: false.Static analysis (zizmor) flags this checkout for credential persistence via
GITHUB_TOKENon disk (artipacked). Since artifacts/workspace can be exposed downstream, consider disabling credential persistence unless it's needed by a later step.Suggested fix
- name: Check out berserker config repository code uses: actions/checkout@v4 with: repository: ${{ inputs.berserker-config-repo }} path: .berserker-config ref: ${{ inputs.berserker-config-ref }} + persist-credentials: false📝 Committable suggestion
🧰 Tools
🪛 zizmor (1.26.1)
[warning] 483-489: credential persistence through GitHub Actions artifacts (artipacked): does not set persist-credentials: false
(artipacked)
🤖 Prompt for AI Agents
Source: Linters/SAST tools