Skip to content

Conversation

yyzxw
Copy link

@yyzxw yyzxw commented Jun 24, 2025

wait upstream pr: kubernetes-sigs/apiserver-runtime#122


Summary by cubic

Upgraded Kubernetes dependencies to v1.32.2 and updated related Go modules for compatibility.

  • Dependencies

    • Bumped k8s.io and related modules to v0.32.2.
    • Updated other dependencies to match new Kubernetes requirements.
  • Refactors

    • Replaced usage of ctx.StopCh with ctx.Context.Done() in loopback client initialization.

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 2 issues across 3 files. Review them in cubic.dev

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

k8s.io/kube-aggregator v0.27.2
k8s.io/kube-openapi v0.0.0-20240228011516-70dd3763d340
k8s.io/utils v0.0.0-20240711033017-18e509b52bc8
k8s.io/kube-openapi v0.0.0-20241105132330-32ad38e42d3f
Copy link

Choose a reason for hiding this comment

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

The pseudo-version contains a future timestamp (20241105), risking build failures because Go may refuse modules dated after the current time.

sigs.k8s.io/apiserver-network-proxy/konnectivity-client v0.30.3
sigs.k8s.io/apiserver-runtime v1.1.2-0.20221102045245-fb656940062f
sigs.k8s.io/apiserver-network-proxy/konnectivity-client v0.31.0
sigs.k8s.io/apiserver-runtime v1.1.2-0.20250117204231-9282f514a674
Copy link

Choose a reason for hiding this comment

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

The pseudo-version timestamp (20250117) lies in the future, which can break reproducible builds and go mod tidy on machines whose clocks are correct because Go considers versions with future timestamps invalid or unreachable.

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.

1 participant