Skip to content

bump k8s-interface to v0.0.205 and update renamed constants#752

Closed
idohuber wants to merge 1 commit into
mainfrom
fix/bump-k8s-interface-renamed-constants
Closed

bump k8s-interface to v0.0.205 and update renamed constants#752
idohuber wants to merge 1 commit into
mainfrom
fix/bump-k8s-interface-renamed-constants

Conversation

@idohuber
Copy link
Copy Markdown

@idohuber idohuber commented Mar 17, 2026

Summary by CodeRabbit

  • Chores
    • Updated dependencies to the latest versions.
  • Bug Fixes
    • Improved accuracy of related-resource labels so displayed relationships are more reliable.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f08c2047-38f4-4354-96a5-a8c8f03f0330

📥 Commits

Reviewing files that changed from the base of the PR and between f00b41a and bcc99e3.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (2)
  • go.mod
  • pkg/objectcache/shared_container_data.go
🚧 Files skipped from review as they are similar to previous changes (1)
  • go.mod

📝 Walkthrough

Walkthrough

Dependency version bumps in go.mod (multiple packages including kubescape and OpenTelemetry) and a change in GetLabels to use RelatedKindMetadataKey and RelatedNameMetadataKey with corresponding WLID helper getters.

Changes

Cohort / File(s) Summary
Dependency Management
go.mod
Bumped versions for multiple dependencies: kubescape/k8s-interface v0.0.204 → v0.0.205, kubescape/storage v0.0.247 → v0.0.252, golang.org/x/sys v0.40.0 → v0.42.0, and several go.opentelemetry.io/otel modules (various packages) updated.
Metadata Key Updates
pkg/objectcache/shared_container_data.go
GetLabels switch cases changed to use RelatedKindMetadataKey and RelatedNameMetadataKey, deriving label values via wlid.GetKindFromWlid and wlid.GetNameFromWlid respectively.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

release

Poem

🐰 A tiny hop, dependencies climb the tree,
Related names and kinds now dance with glee,
Labels find their WLID song,
Versions hum along,
A rabbit cheers — small change, big spree! 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main changes: bumping k8s-interface dependency and updating renamed constants in the codebase.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/bump-k8s-interface-renamed-constants
📝 Coding Plan
  • Generate coding plan for human review comments

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.

@idohuber idohuber force-pushed the fix/bump-k8s-interface-renamed-constants branch from 129e489 to f00b41a Compare March 17, 2026 17:38
…enamed constants

KindMetadataKey → RelatedKindMetadataKey
NameMetadataKey → RelatedNameMetadataKey

Signed-off-by: idoh <idoh@armosec.io>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@idohuber idohuber force-pushed the fix/bump-k8s-interface-renamed-constants branch from f00b41a to bcc99e3 Compare March 17, 2026 18:19
@matthyx matthyx added release Create release and removed release Create release labels Mar 17, 2026
@matthyx matthyx moved this to Needs Reviewer in KS PRs tracking Mar 17, 2026
@matthyx
Copy link
Copy Markdown
Contributor

matthyx commented Mar 19, 2026

superseded by #754

@matthyx matthyx closed this Mar 19, 2026
@matthyx matthyx deleted the fix/bump-k8s-interface-renamed-constants branch March 19, 2026 08:16
@matthyx matthyx moved this from Needs Reviewer to To Archive in KS PRs tracking Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants