Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ease the bound for reflector to only request identifying metadata #1393

Merged
merged 6 commits into from Mar 5, 2024

Conversation

SOF3
Copy link
Contributor

@SOF3 SOF3 commented Jan 15, 2024

Fixes #1389, see issue for details

Copy link

codecov bot commented Jan 17, 2024

Codecov Report

Attention: Patch coverage is 94.59459% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 72.2%. Comparing base (c2983f6) to head (a5b5010).

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #1393     +/-   ##
=======================================
+ Coverage   72.1%   72.2%   +0.1%     
=======================================
  Files         75      75             
  Lines       6470    6485     +15     
=======================================
+ Hits        4663    4680     +17     
+ Misses      1807    1805      -2     
Files Coverage Δ
kube-core/src/resource.rs 63.6% <100.0%> (+5.2%) ⬆️
kube-runtime/src/reflector/mod.rs 100.0% <ø> (ø)
kube-runtime/src/reflector/store.rs 99.0% <100.0%> (-<0.1%) ⬇️
kube-runtime/src/reflector/object_ref.rs 69.4% <92.4%> (+3.6%) ⬆️

@clux

This comment was marked as off-topic.

@clux clux added the changelog-change changelog change category for prs label Feb 9, 2024
SOF3 added a commit to SOF3/kube-rs that referenced this pull request Feb 11, 2024
Motivation explained in kube-rs#1389

Depends on kube-rs#1393 to ease reflector bounds
SOF3 added a commit to SOF3/kube-rs that referenced this pull request Feb 11, 2024
Motivation explained in kube-rs#1389

Depends on kube-rs#1393 to ease reflector bounds
@SOF3

This comment was marked as off-topic.

@clux

This comment was marked as off-topic.

SOF3 added a commit to SOF3/kube-rs that referenced this pull request Feb 12, 2024
Motivation explained in kube-rs#1389

Depends on kube-rs#1393 to ease reflector bounds

Signed-off-by: SOFe <sofe2038@gmail.com>
Copy link
Member

@clux clux left a comment

Choose a reason for hiding this comment

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

Sorry again for slow cycle time on this.
I am ultimately happy with this, just naming stuff that I felt uneasy about. 🙃

have put in a name suggestion, wdyt?

kube-runtime/src/reflector/object_ref.rs Outdated Show resolved Hide resolved
@clux clux added this to the 0.89.0 milestone Feb 22, 2024
@SOF3
Copy link
Contributor Author

SOF3 commented Mar 4, 2024

Renamed to Lookup, expecting users to reference it as reflector::Lookup.

clippy_nightly is failing due to GitHub permission issues, not relevant to this PR.

Copy link
Member

@clux clux left a comment

Choose a reason for hiding this comment

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

Thanks! Two tiny nits that I am spotting, but beyond that I am all on board here.

kube-runtime/src/reflector/object_ref.rs Outdated Show resolved Hide resolved
kube-runtime/src/reflector/object_ref.rs Outdated Show resolved Hide resolved
Co-authored-by: Eirik A <sszynrae@gmail.com>
Signed-off-by: Jonathan Chan Kwan Yin <sofe2038@gmail.com>
Copy link
Member

@clux clux left a comment

Choose a reason for hiding this comment

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

This is all good with me. Thanks a lot!

Signed-off-by: Jonathan Chan Kwan Yin <sofe2038@gmail.com>
@clux clux merged commit 8fa3058 into kube-rs:main Mar 5, 2024
15 of 17 checks passed
@SOF3 SOF3 deleted the reflector-toobjectref branch March 6, 2024 01:23
SOF3 added a commit to SOF3/kube-rs that referenced this pull request Mar 9, 2024
Motivation explained in kube-rs#1389

Depends on kube-rs#1393 to ease reflector bounds

Signed-off-by: SOFe <sofe2038@gmail.com>
SOF3 added a commit to SOF3/kube-rs that referenced this pull request May 8, 2024
Motivation explained in kube-rs#1389

Depends on kube-rs#1393 to ease reflector bounds

Signed-off-by: SOFe <sofe2038@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-change changelog change category for prs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reflector should not request the whole meta
2 participants