Skip to content

Conversation

PushparajShetty
Copy link

@PushparajShetty PushparajShetty commented Jun 30, 2025

Summary by cubic

Upgraded Go to 1.23.8 and Kubernetes dependencies to the latest supported versions. Updated related workflows, CRDs, and tools to ensure compatibility.

  • Dependencies
    • Updated Go, Kubernetes libraries, and related tools in go.mod, Dockerfile, Makefile, and workflow files.
    • Refreshed CRDs and codegen outputs for compatibility with new versions.

Pushparaj Shetty K S added 7 commits June 30, 2025 10:20
Signed-off-by: Pushparaj Shetty K S <[email protected]>
Signed-off-by: Pushparaj Shetty K S <[email protected]>
Signed-off-by: Pushparaj Shetty K S <[email protected]>
Signed-off-by: Pushparaj Shetty K S <[email protected]>
Signed-off-by: Pushparaj Shetty K S <[email protected]>
Signed-off-by: Pushparaj Shetty K S <[email protected]>
@PushparajShetty PushparajShetty force-pushed the chore/upgrade-go-k8s-version branch from 1475ff8 to ee52ee1 Compare June 30, 2025 04:50
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic found 7 issues across 16 files. Review them in cubic.dev

React with 👍 or 👎 to teach cubic. Tag @cubic-dev-ai to give specific feedback.


# Produce CRDs that work back to Kubernetes 1.11 (no version conversion)
CRD_OPTIONS ?= "crd:trivialVersions=true"
CRD_OPTIONS ?= "crd"
Copy link

Choose a reason for hiding this comment

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

Removing trivialVersions=true from CRD_OPTIONS may cause issues with CRD compatibility for clusters older than Kubernetes 1.13, as version conversion will not be handled. Ensure this is intentional and compatible with your supported Kubernetes versions.

Suggested change
CRD_OPTIONS ?= "crd"
CRD_OPTIONS ?= "crd:trivialVersions=true"

@anoop2811 anoop2811 merged commit ee52ee1 into kubevela:master Jul 7, 2025
11 of 15 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.

2 participants