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

fix(apiserver): filter VolumeState when list replicas #1202

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

hikariwo
Copy link
Member

@hikariwo hikariwo commented Oct 27, 2023

What this PR does / why we need it:

fix #1200

Special notes for your reviewer:

Does this PR introduce a user-facing change?


@codecov-commenter
Copy link

codecov-commenter commented Oct 27, 2023

Codecov Report

Merging #1202 (e55588a) into main (b2483b7) will not change coverage.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@           Coverage Diff           @@
##             main    #1202   +/-   ##
=======================================
  Coverage   35.81%   35.81%           
=======================================
  Files          28       28           
  Lines        2178     2178           
=======================================
  Hits          780      780           
  Misses       1295     1295           
  Partials      103      103           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Signed-off-by: lancerxiu <1377004871@qq.com>
@hikariwo hikariwo changed the title fix localvolume_controller.go::GetVolumeReplicas filter error and add more details on udev_test fix localvolume_controller.go::GetVolumeReplicas filter error Oct 30, 2023
@SSmallMonster
Copy link
Member

@peng9808 can you take a look? I am not sure if VolumeState is used in our front code, if not used, this change may cause empty response to the front.

@SSmallMonster SSmallMonster changed the title fix localvolume_controller.go::GetVolumeReplicas filter error fix(apiserver): filter VolumeState when list replicas Oct 30, 2023
@hikariwo
Copy link
Member Author

@peng9808 would you like to review this PR?

@peng9808
Copy link
Collaborator

@SSmallMonster VolumeState is used in our front code

@SSmallMonster
Copy link
Member

lgtm

@SSmallMonster SSmallMonster merged commit ac36b66 into hwameistor:main Nov 1, 2023
3 checks passed
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.

specified VolumeState filter error
4 participants