Skip to content

Update UI proxy flightctl go dep to v0.10.0-rc1#351

Merged
fzdarsky merged 1 commit into
flightctl:mainfrom
celdrake:update-proxy-go-flightctl-dependencies
Sep 29, 2025
Merged

Update UI proxy flightctl go dep to v0.10.0-rc1#351
fzdarsky merged 1 commit into
flightctl:mainfrom
celdrake:update-proxy-go-flightctl-dependencies

Conversation

@celdrake

@celdrake celdrake commented Sep 29, 2025

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Chores
    • Upgraded core platform and toolchain dependencies across the project.
    • Refreshed many transitive libraries: added needed packages and removed deprecated ones.
    • Updated Kubernetes-related components for improved compatibility and stability.
    • Applied upstream security, stability, and tooling fixes.
    • Simplified and reshaped the dependency graph to reduce maintenance overhead.
    • No user-facing feature or UI changes.

@coderabbitai

coderabbitai Bot commented Sep 29, 2025

Copy link
Copy Markdown

Walkthrough

Update of proxy/go.mod that bumps the top-level github.com/flightctl/flightctl dependency, updates the Go toolchain directive, performs widespread indirect dependency version bumps/additions/removals, and adjusts replace directives (notably around Kubernetes-related modules).

Changes

Cohort / File(s) Summary
Module & toolchain
proxy/go.mod
Bump github.com/flightctl/flightctl v0.2.0 → v0.10.0-rc1; update go directive/toolchain to Go 1.23 (go1.23.9).
Kubernetes & k8s ecosystem
proxy/go.mod
Upgrades to k8s modules (e.g., k8s.io/apimachinery v0.29.0→v0.32.3; k8s.io/klog/v2 v2.120.1→v2.130.1); updates sigs.k8s.io/* packages; removal of previous k8s.io/client-go replace pin.
Indirect dependency bumps & replacements
proxy/go.mod
Numerous indirect bumps (examples: felixge/httpsnoop v1.0.3→v1.0.4; getkin/kin-openapi v0.124.0→v0.131.0; go-logr/logr v1.4.1→v1.4.3; sigs.k8s.io/structured-merge-diff/v4 v4.4.1→v4.6.0) and several replace directive adjustments.
New indirect dependencies added
proxy/go.mod
Adds packages such as fxamacker/cbor/v2, github.com/go-chi/chi/v5, github.com/oasdiff/yaml & .../yaml3, sigs.k8s.io/yaml, robfig/cron/v3, samber/lo, and various golang.org/x/... entries.
Removed / replaced indirect dependencies
proxy/go.mod
Removes or replaces items like invopop/yaml, jinzhu/inflection, jinzhu/now, openshift/library-go and replaces older x/... modules with newer golang.org/x/... equivalents and other replacements.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

Suggested reviewers

  • gshilin-sdb
  • asafbss

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title succinctly describes the primary change of bumping the UI proxy’s flightctl Go dependency to version v0.10.0-rc1, making it clear and specific without extraneous details.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d10aea0 and 0a921c3.

⛔ Files ignored due to path filters (1)
  • proxy/go.sum is excluded by !**/*.sum
📒 Files selected for processing (1)
  • proxy/go.mod (2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
  • GitHub Check: integration-tests
  • GitHub Check: Build ocp plugin
  • GitHub Check: Build
  • GitHub Check: Lint

Comment thread proxy/go.mod Outdated
@celdrake celdrake force-pushed the update-proxy-go-flightctl-dependencies branch from 0a921c3 to 5c6def2 Compare September 29, 2025 09:40
@celdrake celdrake force-pushed the update-proxy-go-flightctl-dependencies branch from 5c6def2 to 2e89037 Compare September 29, 2025 09:50
Comment thread proxy/go.mod

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5c6def2 and 2e89037.

⛔ Files ignored due to path filters (1)
  • proxy/go.sum is excluded by !**/*.sum
📒 Files selected for processing (1)
  • proxy/go.mod (2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
  • GitHub Check: integration-tests
  • GitHub Check: Build
  • GitHub Check: Lint

Comment thread proxy/go.mod
@fzdarsky fzdarsky merged commit 0e004ac into flightctl:main Sep 29, 2025
6 checks passed
@celdrake celdrake deleted the update-proxy-go-flightctl-dependencies branch September 29, 2025 17:25
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