Skip to content

Conversation

@siegfriedweber
Copy link
Member

Description

Depend on the patched version of kube-rs available at https://github.com/stackabletech/kube-rs, ensuring the operators automatically benefit from the fixes.

The patched version of kube-rs avoids that null entries are not added to enums, see stackabletech/kube-rs#1. These null entries currently prevent the CRD documentations from being rendered.

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes

Author

Reviewer

  • Code contains useful comments
  • Code contains useful logging statements
  • (Integration-)Test cases added
  • Documentation added or updated. Follows the style guide.
  • Changelog updated
  • Cargo.toml only contains references to git tags (not specific commits or branches)

Acceptance

  • Feature Tracker has been updated
  • Proper release label has been added

@siegfriedweber siegfriedweber requested a review from a team December 8, 2025 15:21
@siegfriedweber siegfriedweber self-assigned this Dec 8, 2025
@siegfriedweber siegfriedweber moved this to Development: Waiting for Review in Stackable Engineering Dec 8, 2025
Copy link
Member

@razvan razvan left a comment

Choose a reason for hiding this comment

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

lgtm!

@siegfriedweber siegfriedweber added this pull request to the merge queue Dec 9, 2025
@siegfriedweber siegfriedweber moved this from Development: Waiting for Review to Development: Done in Stackable Engineering Dec 9, 2025
Merged via the queue into main with commit e10d2c5 Dec 9, 2025
8 checks passed
@siegfriedweber siegfriedweber deleted the use-patched-kube-rs branch December 9, 2025 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Done

Development

Successfully merging this pull request may close these issues.

3 participants