Skip to content
This repository was archived by the owner on Jun 5, 2026. It is now read-only.

Mandronic kubescape helm charts 711 fix#68

Merged
slashben merged 2 commits into
masterfrom
mandronic-kubescape-helm-charts-711-fix
Nov 4, 2025
Merged

Mandronic kubescape helm charts 711 fix#68
slashben merged 2 commits into
masterfrom
mandronic-kubescape-helm-charts-711-fix

Conversation

@Bezbran

@Bezbran Bezbran commented Nov 4, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

mandronic and others added 2 commits November 1, 2025 02:19
…(refs kubescape/helm-charts#711)

Signed-off-by: Mihail Andronic <104365774+mandronic@users.noreply.github.com>
host-scanner daemonset installation failure fix
@slashben
slashben requested a review from Copilot November 4, 2025 08:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes the procMount: Unmasked security context setting from the host-scanner DaemonSet configuration. The change simplifies the security context configuration by removing this specific mount option, allowing Kubernetes to use the default procMount behavior.

  • Removed procMount: Unmasked from the host-scanner container security context in the deployment YAML
  • Updated the README documentation to reflect the removal of this setting

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
deployment/host-scanner.yaml Removed procMount: Unmasked from the container security context configuration
README.md Updated the documentation example to remove the procMount: Unmasked setting

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions

github-actions Bot commented Nov 4, 2025

Copy link
Copy Markdown

Summary:

  • License scan: failure
  • Credentials scan: failure
  • Vulnerabilities scan: success
  • Unit test: success
  • Go linting: failure

@kubescape kubescape deleted a comment from Copilot AI Nov 4, 2025
@slashben

slashben commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

@matthyx, did we need this unmasked setting somewhere?

@matthyx

matthyx commented Nov 4, 2025

Copy link
Copy Markdown
Contributor

@matthyx, did we need this unmasked setting somewhere?

no it's fine and in line with the other PR kubescape/helm-charts#756

@slashben
slashben merged commit 553faee into master Nov 4, 2025
25 checks passed
@Bezbran
Bezbran deleted the mandronic-kubescape-helm-charts-711-fix branch November 4, 2025 08:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants