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

revert!: disable device localpv #172

Merged
merged 2 commits into from
Mar 21, 2024
Merged

Conversation

Abhinandan-Purkait
Copy link
Member

DEPRECATE LOCAL-PV DEVICE

This PR includes the bits that would contribute to the deprecation of localpv device. Henceforth all the dependent charts needed for localpv device ex. NDM would be removed. Provisioning and de-provisioning of local device volumes will no longer be supported.

Since this is a breaking change, we are bumping up the major version from 3.5.x to 4.0.x.

@codecov-commenter
Copy link

codecov-commenter commented Feb 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 38.20%. Comparing base (4123f5f) to head (486df55).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #172      +/-   ##
===========================================
+ Coverage    38.03%   38.20%   +0.17%     
===========================================
  Files           36       36              
  Lines         3363     3348      -15     
===========================================
  Hits          1279     1279              
+ Misses        2002     1987      -15     
  Partials        82       82              

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

@Abhinandan-Purkait Abhinandan-Purkait force-pushed the disable-localpv-device branch 10 times, most recently from fa653a7 to 569b939 Compare February 29, 2024 09:27
Copy link
Member

@niladrih niladrih left a comment

Choose a reason for hiding this comment

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

Requires changelog directory entry and CHANGELOG.md try.

README.md Outdated Show resolved Hide resolved
Copy link
Member

@niladrih niladrih left a comment

Choose a reason for hiding this comment

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

Let's remove the device-integration-test makefile target, as those would no longer function after the changes to the ginkgo tests.

@Abhinandan-Purkait
Copy link
Member Author

Abhinandan-Purkait commented Mar 7, 2024

Let's remove the device-integration-test makefile target, as those would no longer function after the changes to the ginkgo tests.

Have already removed it. Should I remove the integration-test as well? Since we only use hostpath-integration-test

Copy link

@tiagolobocastro tiagolobocastro left a comment

Choose a reason for hiding this comment

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

it's not deprecating the code, it's removing it completely right?

revert!: disable device localpv

EDIT: maybe disable is more appropriate actually?

@Abhinandan-Purkait Abhinandan-Purkait changed the title chore: deprecate device localpv revert!: disable device localpv Mar 7, 2024
Signed-off-by: Abhinandan Purkait <purkaitabhinandan@gmail.com>
Copy link
Member

@avishnu avishnu left a comment

Choose a reason for hiding this comment

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

lgtm

@niladrih
Copy link
Member

niladrih commented Mar 7, 2024

Let's remove the device-integration-test makefile target, as those would no longer function after the changes to the ginkgo tests.

Have already removed it. Should I remove the integration-test as well? Since we only use hostpath-integration-test

Yes, it won't work I think.

Signed-off-by: Abhinandan Purkait <purkaitabhinandan@gmail.com>
@Abhinandan-Purkait Abhinandan-Purkait merged commit 24fd7cd into develop Mar 21, 2024
8 checks passed
@niladrih niladrih deleted the disable-localpv-device branch March 21, 2024 07:08
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.

None yet

5 participants