Skip to content

ci(alerts): memory limits, trivy ignores - #612

Merged
DerekRoberts merged 3 commits into
mainfrom
feat/fix-trivy-alerts
Jul 16, 2026
Merged

ci(alerts): memory limits, trivy ignores#612
DerekRoberts merged 3 commits into
mainfrom
feat/fix-trivy-alerts

Conversation

@DerekRoberts

@DerekRoberts DerekRoberts commented Jul 16, 2026

Copy link
Copy Markdown
Member

Thanks for the PR!

Any successful deployments (not always required) will be available below.
API available
Frontend available

Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

@DerekRoberts
DerekRoberts requested a review from mishraomp as a code owner July 16, 2026 20:09
Copilot AI review requested due to automatic review settings July 16, 2026 20:09

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 addresses Trivy configuration scan alerts by defining explicit Kubernetes resource limits for the Helm chart’s workloads and adding a Trivy ignore file for specific misconfiguration checks that are not actionable given the chart’s dynamic templating and OpenShift SCC behavior.

Changes:

  • Added CPU/memory limits alongside existing requests for API, frontend, and database (including the database initContainer) in the chart default values.
  • Added a root-level .trivyignore to suppress selected Trivy Kubernetes misconfiguration IDs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
charts/pubcode/values.yaml Adds default resource limits for API/frontend/database containers to satisfy config-scan requirements.
.trivyignore Introduces ignore rules for specific Trivy Kubernetes misconfiguration IDs.

@DerekRoberts DerekRoberts self-assigned this Jul 16, 2026
@DerekRoberts DerekRoberts moved this from New to Active in DevOps (NR) Jul 16, 2026
@DerekRoberts DerekRoberts changed the title ci(trivy): resolve configuration alerts by adding resource limits and ignore rules ci(alerts): resource limits, trivy ignores Jul 16, 2026
@DerekRoberts DerekRoberts moved this from Active to Waiting in DevOps (NR) Jul 16, 2026
@DerekRoberts
DerekRoberts enabled auto-merge (squash) July 16, 2026 20:23
@DerekRoberts DerekRoberts changed the title ci(alerts): resource limits, trivy ignores ci(alerts): memory limits, trivy ignores Jul 16, 2026
@DerekRoberts
DerekRoberts disabled auto-merge July 16, 2026 20:35
@DerekRoberts
DerekRoberts merged commit 69ac373 into main Jul 16, 2026
14 of 19 checks passed
@DerekRoberts
DerekRoberts deleted the feat/fix-trivy-alerts branch July 16, 2026 20:35
@github-project-automation github-project-automation Bot moved this from Waiting to Done in DevOps (NR) Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants