Skip to content

Fix Helm deployments#30

Merged
vladbologa merged 1 commit into
mainfrom
vb/fix-helm-install
Dec 18, 2025
Merged

Fix Helm deployments#30
vladbologa merged 1 commit into
mainfrom
vb/fix-helm-install

Conversation

@vladbologa

@vladbologa vladbologa commented Dec 18, 2025

Copy link
Copy Markdown
Collaborator

Fixes Helm deployment failure caused by invalid resource specification for scanner-db.

The scanner.dbResources field was incorrectly wrapped in an extra resources object,
producing dbResources.resources.requests instead of dbResources.requests.

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 fixes an incorrect nesting structure in the Helm values for scanner database resources. The changes remove an unnecessary intermediate resources key that was causing the scanner's database resources to be incorrectly structured.

  • Corrected the structure of scanner.dbResources to directly map to resource definitions instead of nesting them under an additional resources key
  • Applied the fix to both small and medium resource configurations

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

@vladbologa vladbologa merged commit 3cd2e2c into main Dec 18, 2025
9 of 10 checks passed
@mclasmeier mclasmeier deleted the vb/fix-helm-install branch May 18, 2026 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants