Skip to content

bump app version [skip ci] - #1180

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
ci/bump-chart-appversions
Open

bump app version [skip ci]#1180
github-actions[bot] wants to merge 1 commit into
mainfrom
ci/bump-chart-appversions

Conversation

@github-actions

@github-actions github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

bump app version [skip ci]

bumped cortex: sha-91df27a0 -> sha-cd38777a
bumped cortex-shim: sha-b577e306 -> sha-cd38777a
noop cortex-postgres (sha-cf70e178)

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 Helm chart metadata for the helm/library/cortex application chart to point to a newer Cortex application SHA, keeping the deployment artifact aligned with the bumped underlying Cortex revision.

Changes:

  • Bumped appVersion in the Cortex Helm chart from sha-91df27a0 to sha-90034161.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot AI review requested due to automatic review settings August 31, 2026 11:30
@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from 1ff2b0d to a8392ee Compare August 31, 2026 11:30

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

Copilot AI review requested due to automatic review settings August 31, 2026 11:35
@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from a8392ee to 3052760 Compare August 31, 2026 11:35

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

Comment thread helm/library/cortex/Chart.yaml Outdated
Comment on lines +5 to +6
version: 0.4.0
appVersion: "sha-91df27a0"
appVersion: "sha-ccd47209"
Comment thread helm/library/cortex-shim/Chart.yaml Outdated
Copilot AI review requested due to automatic review settings September 1, 2026 08:49
@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from 3052760 to 4fcaa6f Compare September 1, 2026 08:49

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

Copilot AI review requested due to automatic review settings September 1, 2026 08:59
@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from 4fcaa6f to 7152c14 Compare September 1, 2026 08:59

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

Copilot AI review requested due to automatic review settings September 2, 2026 07:18
@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from 7152c14 to d8e270c Compare September 2, 2026 07:18

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.

🟡 Changes recommended

The modified Chart.yaml files are missing the SPDX license header that is consistently present in the repository’s other Helm charts.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 2
  • Review effort level: Lite

Comment thread helm/library/cortex-shim/Chart.yaml Outdated
Comment on lines +2 to +6
@@ -3,6 +3,6 @@ name: cortex-shim
description: A Helm chart to distribute cortex shims.
type: application
version: 0.1.15
appVersion: "sha-17e0a4fe"
appVersion: "sha-46aeea6b"
Comment thread helm/library/cortex/Chart.yaml Outdated
Comment on lines +2 to +6
@@ -3,6 +3,6 @@ name: cortex
description: A Helm chart to distribute cortex.
type: application
version: 0.4.0
appVersion: "sha-91df27a0"
appVersion: "sha-da763be7"
Copilot AI review requested due to automatic review settings September 2, 2026 14:24
Copilot AI review requested due to automatic review settings September 4, 2026 09:25
@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from c6edcbd to 54188b3 Compare September 4, 2026 09:25

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.

🟢 Approval recommended

The changes are a straightforward appVersion bump in Helm Chart.yaml files with no behavioral or structural modifications.

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

Copilot AI review requested due to automatic review settings September 4, 2026 09:34
@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from 54188b3 to ba55f66 Compare September 4, 2026 09:34

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.

🔵 Needs a closer look

The chart version fields were not bumped alongside appVersion, which can prevent Helm consumers from detecting/pulling the update.

Review details

Suppressed comments (2)

helm/library/cortex/Chart.yaml:6

  • Chart version is unchanged while appVersion is updated. Helm uses version (Chart.yaml) to detect chart updates; leaving it the same can prevent consumers from picking up the new app version via dependency updates/registry releases.
version: 0.4.0
appVersion: "sha-1a3f701f"

helm/library/cortex-shim/Chart.yaml:6

  • Chart version is unchanged while appVersion is updated. Helm uses version (Chart.yaml) to detect chart updates; leaving it the same can prevent consumers from picking up the new app version via dependency updates/registry releases.
version: 0.1.16
appVersion: "sha-261b4905"
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from ba55f66 to c5edc3c Compare September 4, 2026 09:43
Copilot AI review requested due to automatic review settings September 4, 2026 09:43
@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from c5edc3c to 670d25e Compare September 4, 2026 09: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.

🟡 Changes recommended

The modified Chart.yaml files are missing the repository license header and the chart versions are not bumped alongside the appVersion change, which can break dependency consumption in bundle charts.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details

Suppressed comments (1)

helm/library/cortex-shim/Chart.yaml:6

  • The chart's appVersion was updated, but the chart version was not bumped. Since bundle charts pin this dependency by chart version (e.g., helm/bundles/cortex-placement-shim/Chart.yaml:14 uses version 0.1.16), keeping version unchanged can prevent consumers from picking up the new appVersion unless the OCI tag is overwritten. Bump the chart version and update dependent bundle dependency versions accordingly.
version: 0.1.16
appVersion: "sha-0cf9e776"
  • Files reviewed: 2/2 changed files
  • Comments generated: 3
  • Review effort level: Lite

Comment thread helm/library/cortex/Chart.yaml Outdated
Comment on lines +5 to +6
version: 0.4.0
appVersion: "sha-91df27a0"
appVersion: "sha-261b4905"
Comment thread helm/library/cortex-shim/Chart.yaml Outdated
Comment on lines +2 to +6
@@ -3,6 +3,6 @@ name: cortex-shim
description: A Helm chart to distribute cortex shims.
type: application
version: 0.1.16
appVersion: "sha-b577e306"
appVersion: "sha-261b4905"
Comment thread helm/library/cortex/Chart.yaml Outdated
Comment on lines +2 to +6
@@ -3,6 +3,6 @@ name: cortex
description: A Helm chart to distribute cortex.
type: application
version: 0.4.0
appVersion: "sha-91df27a0"
appVersion: "sha-261b4905"
Copilot AI review requested due to automatic review settings September 4, 2026 09:46

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.

🟢 Approval recommended

The changes are limited to Helm chart metadata (appVersion) updates with no functional or structural modifications.

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

Copilot AI review requested due to automatic review settings September 4, 2026 09:53
@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from 670d25e to e392e33 Compare September 4, 2026 09:53

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.

🟢 Approval recommended

The changes are limited to Helm chart appVersion bumps with no functional logic modifications.

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

Copilot AI review requested due to automatic review settings September 4, 2026 11:01
@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from e392e33 to 53b3874 Compare September 4, 2026 11:01

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.

🔵 Needs a closer look

The modified Chart.yaml files are missing the repo-standard SPDX license header, which should be added for consistency and license compliance.

Review details

Suppressed comments (2)

helm/library/cortex/Chart.yaml:6

  • This Chart.yaml is missing the standard SPDX license header that other Helm charts in this repo include (e.g., helm/library/cortex-postgres/Chart.yaml:1-2, helm/bundles/cortex-nova/Chart.yaml:1-2). Since this file is being updated, please add the header for consistency and license compliance.
appVersion: "sha-0cf9e776"

helm/library/cortex-shim/Chart.yaml:6

  • This Chart.yaml is missing the standard SPDX license header that other Helm charts in this repo include (e.g., helm/library/cortex-postgres/Chart.yaml:1-2, helm/bundles/cortex-nova/Chart.yaml:1-2). Since this file is being updated, please add the header for consistency and license compliance.
appVersion: "sha-cd38777a"
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

bumped cortex: sha-91df27a0 -> sha-cd38777a
bumped cortex-shim: sha-b577e306 -> sha-cd38777a
noop cortex-postgres (sha-cf70e178)

Signed-off-by: PhilippMatthes <27271818+PhilippMatthes@users.noreply.github.com>
Copilot AI review requested due to automatic review settings September 4, 2026 11:10
@github-actions
github-actions Bot force-pushed the ci/bump-chart-appversions branch from 53b3874 to 0b66d54 Compare September 4, 2026 11:10

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.

🟢 Approval recommended

The changes are limited to Helm chart appVersion bumps with valid YAML and no functional chart logic modifications.

Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

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.

2 participants