Skip to content

Bump network controller#489

Merged
Vicente-Cheng merged 1 commit intoharvester:releasefrom
starbops:bump-network-controller
Mar 12, 2026
Merged

Bump network controller#489
Vicente-Cheng merged 1 commit intoharvester:releasefrom
starbops:bump-network-controller

Conversation

@starbops
Copy link
Copy Markdown
Member

@starbops starbops commented Mar 11, 2026

Problem:

Solution:

Related Issue(s):

harvester/harvester#9140

Test plan:

Additional documentation or context

Signed-off-by: Zespre Chang <zespre.chang@suse.com>
@starbops starbops marked this pull request as ready for review March 11, 2026 04:47
Copy link
Copy Markdown

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

Updates the Harvester network controller Helm chart to support an additional link state value in the LinkMonitor CRD, aligning schema validation with the controller’s reported status values.

Changes:

  • Extend LinkMonitor.status.linkStatus[*].state CRD validation enum to include unknown.
  • Bump harvester-network-controller chart version from 1.8.0-dev.2 to 1.8.0-dev.3.

Reviewed changes

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

File Description
charts/harvester-network-controller/templates/crds/network.harvesterhci.io_linkmonitors.yaml Adds unknown as an allowed value for link state in the CRD schema.
charts/harvester-network-controller/Chart.yaml Increments chart version to reflect template/CRD change.

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

@starbops starbops force-pushed the bump-network-controller branch from 1795d7f to d5d6405 Compare March 11, 2026 06:22
@starbops
Copy link
Copy Markdown
Member Author

The chart version bump commit has been dropped.

@Vicente-Cheng Vicente-Cheng merged commit 0bc7572 into harvester:release Mar 12, 2026
4 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