Skip to content

Comments

#2680 Grant delete permissions for scheduled scans in auto-discovery#2871

Merged
J12934 merged 3 commits intosecureCodeBox:mainfrom
BorisShek:scb-2680_autodiscovery_lacks_delete_permissions
Jan 28, 2025
Merged

#2680 Grant delete permissions for scheduled scans in auto-discovery#2871
J12934 merged 3 commits intosecureCodeBox:mainfrom
BorisShek:scb-2680_autodiscovery_lacks_delete_permissions

Conversation

@BorisShek
Copy link
Contributor

@BorisShek BorisShek commented Jan 27, 2025

Add missing delete permissions for ScheduledScans in auto-discovery. Update .helmignore to exclude unused files, reducing Helm container size for local deployments.

Description

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

…to-discovery

Add missing delete permissions for ScheduledScans in auto-discovery.
Update `.helmignore` to exclude unused files, reducing Helm container size for local deployments.

Signed-off-by: Boris Shek <[email protected]>
@netlify
Copy link

netlify bot commented Jan 27, 2025

Deploy Preview for docs-securecodebox canceled.

Name Link
🔨 Latest commit ccb82e7
🔍 Latest deploy log https://app.netlify.com/sites/docs-securecodebox/deploys/6798f8a54673bb0008db83de

…scovery

Previously, the Container Auto-Discovery controller logged an error when trying
to fetch a ScheduledScan that was already deleted. This change introduces a check
for `IsNotFound` errors, treating them as expected behavior instead of logging
them as errors.
Now, if a ScheduledScan is not found, an informational log message is recorded
instead of an error. This prevents unnecessary noise in logs.

Signed-off-by: Boris Shek <[email protected]>
@sonarqubecloud
Copy link

@J12934 J12934 merged commit 2ff9acc into secureCodeBox:main Jan 28, 2025
53 checks passed
@BorisShek BorisShek self-assigned this Jan 30, 2025
@BorisShek
Copy link
Contributor Author

Resolves #2680

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants