Skip to content

chore(chart-deps): update policy-reporter to version 3.8.1 - #3434

Closed
svcAPLBot wants to merge 22 commits into
mainfrom
ci-update-policy-reporter-to-3.8.1
Closed

chore(chart-deps): update policy-reporter to version 3.8.1#3434
svcAPLBot wants to merge 22 commits into
mainfrom
ci-update-policy-reporter-to-3.8.1

Conversation

@svcAPLBot

Copy link
Copy Markdown
Contributor

This PR updates the dependency policy-reporter to version 3.8.1.

@svcAPLBot svcAPLBot added the chart-deps Auto generated helm chart dependencies label Jul 9, 2026
Copilot AI review requested due to automatic review settings July 29, 2026 13:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the vendored policy-reporter Helm chart to 3.8.1, aligning chart metadata and dependency pinning while incorporating upstream template/value changes (HTTPRoute backendRefs explicit typing and a new ServiceMonitor enable toggle).

Changes:

  • Bump policy-reporter chart version/appVersion to 3.8.1 and update the chart-index dependency pin accordingly.
  • Update related image tags (UI + plugins) to match the newer chart release.
  • Add explicit group/kind to Gateway API HTTPRoute backendRefs, and gate ServiceMonitor rendering behind monitoring.serviceMonitor.enabled (defaulted on).

Reviewed changes

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

Show a summary per file
File Description
charts/policy-reporter/values.yaml Updates image tag defaults and introduces monitoring.serviceMonitor.enabled (default true).
charts/policy-reporter/templates/ui/httproute.yaml Makes backendRefs explicitly reference a core Service.
charts/policy-reporter/templates/plugins/kyverno/httproute.yaml Makes backendRefs explicitly reference a core Service.
charts/policy-reporter/templates/monitoring/servicemonitor.yaml Adds a template guard requiring monitoring.serviceMonitor.enabled.
charts/policy-reporter/templates/httproute.yaml Makes backendRefs explicitly reference a core Service.
charts/policy-reporter/README.md Updates chart/app versions and documents updated default tags and the new ServiceMonitor toggle (currently with an empty description).
charts/policy-reporter/Chart.yaml Bumps chart version and appVersion to 3.8.1.
chart/chart-index/Chart.yaml Pins the policy-reporter dependency version to 3.8.1.

Comment on lines 1872 to 1874
serviceMonitor:
enabled: true
# -- HonorLabels chooses the metrics labels on collisions with target labels
Copilot AI review requested due to automatic review settings July 29, 2026 14:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comments suppressed due to low confidence (2)

charts/policy-reporter/values.yaml:1873

  • monitoring.serviceMonitor.enabled was added with default true, but it lacks the usual # -- (...) documentation comment used throughout this values file. Adding the comment keeps the chart values self-documenting and aligns with the README table generation.
  serviceMonitor:
    enabled: true

charts/policy-reporter/README.md:636

  • The new monitoring.serviceMonitor.enabled value is documented in the parameters table but the description cell is empty, which makes the generated documentation unclear. Add a short description (and ideally mention it is gated by monitoring.enabled).
| monitoring.serviceMonitor.enabled | bool | `true` |  |

Copilot AI review requested due to automatic review settings July 30, 2026 13:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comments suppressed due to low confidence (2)

charts/policy-reporter/values.yaml:1873

  • monitoring.serviceMonitor.enabled was added without a # -- doc comment, which results in an empty description in the generated/maintained values documentation. Add a short comment describing what the flag does and that it is gated by monitoring.enabled.
  serviceMonitor:
    enabled: true

charts/policy-reporter/README.md:636

  • The README values table row for monitoring.serviceMonitor.enabled has an empty description, which makes the rendered chart docs less useful. Fill in a short description consistent with values.yaml.
| monitoring.serviceMonitor.enabled | bool | `true` |  |

Copilot AI review requested due to automatic review settings July 31, 2026 11:45

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Suppressed comments (2)

charts/policy-reporter/values.yaml:1875

  • monitoring.serviceMonitor.enabled was added, but it lacks a # -- doc comment in values.yaml. This causes the README table entry to have an empty description and makes the option harder to discover/understand.
  serviceMonitor:
    enabled: true
    # -- HonorLabels chooses the metrics labels on collisions with target labels
    honorLabels: false

charts/policy-reporter/README.md:636

  • The new monitoring.serviceMonitor.enabled row in the values table has an empty description, which makes the generated documentation less useful. Add a short description consistent with surrounding rows.
| monitoring.serviceMonitor.enabled | bool | `true` |  |

@merll

merll commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

#3461

@merll merll closed this Aug 3, 2026
@merll
merll deleted the ci-update-policy-reporter-to-3.8.1 branch August 3, 2026 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chart-deps Auto generated helm chart dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants