Skip to content

Conversation

@starbops
Copy link
Member

Problem:

We currently couple the kubeovn-operator and kubeovn-operator-crd charts, so it's impossible to use different versions of them. As a result, we need to bump the kubeovn-operator-crd chart version to match kubeovn-operator's.

Solution:

Bump the kubeovn-operator-crd chart version

Related Issue(s):

harvester/harvester#9787

Test plan:

Additional documentation or context

Copy link
Member

@w13915984028 w13915984028 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the quick fix.

Copy link

Copilot AI left a comment

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 kubeovn-operator-crd Helm chart version to align with the kubeovn-operator chart version, as both charts are currently coupled and must use the same versions. The chart version is incremented from 1.14.10-dev.0 to 1.14.10-dev.2, and the appVersion is bumped from v1.14.10-dev.0 to v1.14.10-dev.1.

Changes:

  • Updated chart version from 1.14.10-dev.0 to 1.14.10-dev.2
  • Updated appVersion from v1.14.10-dev.0 to v1.14.10-dev.1

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

# follow Semantic Versioning. They should reflect the version the application is using.
# It is recommended to use it with quotes.
appVersion: v1.14.10-dev.0
appVersion: v1.14.10-dev.1
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

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

The appVersion (v1.14.10-dev.1) is incrementing to dev.1 while the chart version is jumping to dev.2, creating a version mismatch. Based on the kubeovn-operator chart which has both version and appVersion at the same dev level (1.14.10-dev.2 and v1.14.10-dev.1 respectively), this appears correct. However, the version number jump from dev.0 to dev.2 suggests dev.1 was skipped. Consider verifying if the version number progression is intentional or if an intermediate version was missed.

Copilot uses AI. Check for mistakes.
@starbops starbops merged commit 4107ebf into harvester:release-v1.7 Jan 16, 2026
10 checks passed
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