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

[PWX-37088] Cherry-picking Stork-Config changes #1536

Merged
merged 4 commits into from
May 2, 2024

Conversation

olavangad-px
Copy link
Contributor

@olavangad-px olavangad-px commented Apr 30, 2024

Signed-off-by: Omkar Lavangad olavangad@purestorage.com

What this PR does / why we need it:
KubeSchedulerConfiguration v1beta3 is deprecated in v1.26 and is removed in v1.29. We must migrate to KubeSchedulerConfiguration to v1.
This PR is to cherry-pick the changes made as part of #1395 to 24.1.0 release branch

Which issue(s) this PR fixes (optional)
Closes PWX-37088, PWX-37080, PWX-35604

Special notes for your reviewer:
These changes have already been reviewed in #1395 and merged in 23.10.4 branch.
Here, I have cherry-picked the changes from master branch.

Also, since Zoran merged after me in 23.10.4 there were some extra changes introduced in these files.
I have added them in this PR as well in the second commit to keep it uniform with 23.10.4

Testing Details
Have verified with custom image locally that changes are working as expected in cluster with k8s version 1.29.0

olavangad-px and others added 2 commits April 30, 2024 07:52
…on 1.25+ (#1395)

* Update KubeSchedulerConfiguration to api v1 for k8s version 1.25+

* Addressing comments
Copy link

codecov bot commented Apr 30, 2024

Codecov Report

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

Project coverage is 80.01%. Comparing base (18b9138) to head (9c0c7d1).

Files Patch % Lines
pkg/controller/storagecluster/stork.go 92.94% 4 Missing and 2 partials ⚠️
Additional details and impacted files
@@                Coverage Diff                 @@
##           release-24.1.0    #1536      +/-   ##
==================================================
- Coverage           80.30%   80.01%   -0.30%     
==================================================
  Files                  60       60              
  Lines               17824    17854      +30     
==================================================
- Hits                14314    14286      -28     
- Misses               2523     2578      +55     
- Partials              987      990       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@olavangad-px olavangad-px merged commit b6f3a7e into release-24.1.0 May 2, 2024
7 of 8 checks passed
@nikita-bhatia nikita-bhatia deleted the PWX-37088 branch May 2, 2024 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants