Skip to content

chore(chart-deps): update harbor to version 1.19.2 - #3504

Draft
svcAPLBot wants to merge 2 commits into
mainfrom
ci-update-harbor-to-1.19.2
Draft

chore(chart-deps): update harbor to version 1.19.2#3504
svcAPLBot wants to merge 2 commits into
mainfrom
ci-update-harbor-to-1.19.2

Conversation

@svcAPLBot

Copy link
Copy Markdown
Contributor

This PR updates the dependency harbor to version 1.19.2.

Copilot AI lite review requested due to automatic review settings August 7, 2026 08:48
@svcAPLBot svcAPLBot added the chart-deps Auto generated helm chart dependencies label Aug 7, 2026

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

This PR bumps the vendored Harbor Helm chart dependency to 1.19.2 (Harbor 2.15.2) and aligns the chart’s pinned default image tags/repositories with that release.

Changes:

  • Update Harbor chart version to 1.19.2 and appVersion to 2.15.2.
  • Bump Harbor component image tags from v2.15.1 → v2.15.2.
  • Switch the internal “redis” image default from redis-photon to valkey-photon (and reflect this in the README row that was touched).

Reviewed changes

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

Show a summary per file
File Description
charts/harbor/values.yaml Updates pinned Harbor component image tags to v2.15.2 and switches internal redis image repo to valkey.
charts/harbor/README.md Updates the documented default internal redis image repository.
charts/harbor/Chart.yaml Bumps chart version and appVersion to match Harbor 2.15.2 / chart 1.19.2.
charts/dependencies.yaml Updates the Harbor dependency version to 1.19.2.
apps.yaml Updates the Harbor appVersion displayed in app metadata to 2.15.2.

Comment thread charts/harbor/values.yaml
Comment on lines 1136 to +1140
type: internal
internal:
image:
repository: docker.io/goharbor/redis-photon
tag: v2.15.1
repository: docker.io/goharbor/valkey-photon
tag: v2.15.2
Comment thread charts/harbor/README.md
| **Redis** | | |
| `redis.type` | If external redis is used, set it to `external` | `internal` |
| `redis.internal.image.repository` | Repository for redis image | `goharbor/redis-photon` |
| `redis.internal.image.repository` | Repository for redis image | `goharbor/valkey-photon` |
Copilot AI review requested due to automatic review settings August 7, 2026 09:05

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 5 out of 5 changed files in this pull request and generated no new comments.

Suppressed comments (2)

charts/harbor/README.md:352

  • The README parameter description still says this is a "redis" image repository, but the default has been switched to goharbor/valkey-photon. Updating the wording to mention Valkey (Redis-compatible) would avoid confusion for chart consumers.
| `redis.internal.image.repository`                                    | Repository for redis image                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                          | `goharbor/valkey-photon`                                                 |

apps.yaml:83

  • apps.yaml still links Harbor documentation for 2.6.0, which is inconsistent with the updated Harbor appVersion: 2.15.2. Consider pointing at the unversioned docs root (or updating the versioned link) to keep this metadata accurate.
    appVersion: 2.15.2
    repo: https://github.com/goharbor/harbor
    maintainers: Project Harbor
    relatedLinks:
      - https://goharbor.io/docs/2.6.0/

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.

2 participants