Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

Warn instead of blocking when selecting a storage class with no provisioner #381

Merged
1 commit merged into from Jan 14, 2021

Conversation

mturley
Copy link
Collaborator

@mturley mturley commented Jan 14, 2021

@mturley mturley requested a review from a team January 14, 2021 15:35
@konveyor-preview-bot
Copy link

🚀 Deployed Preview: http://konveyor-forklift-ui-pr-381-preview.surge.sh

Compare with current main branch: http://konveyor-forklift-ui-preview.surge.sh

@codecov
Copy link

codecov bot commented Jan 14, 2021

Codecov Report

Merging #381 (8f20836) into main (fa24254) will increase coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #381      +/-   ##
==========================================
+ Coverage   62.84%   62.87%   +0.03%     
==========================================
  Files         121      121              
  Lines        3488     3491       +3     
  Branches      797      798       +1     
==========================================
+ Hits         2192     2195       +3     
  Misses       1279     1279              
  Partials       17       17              
Impacted Files Coverage Δ
src/app/queries/mocks/storageClasses.mock.ts 100.00% <ø> (ø)
src/app/queries/storageClasses.ts 53.84% <ø> (ø)
.../components/MappingBuilder/MappingTargetSelect.tsx 73.68% <100.00%> (+1.46%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fa24254...8f20836. Read the comment docs.

@mturley mturley force-pushed the bz1916178-unblock-storage-classes branch from c2dad85 to 8f20836 Compare January 14, 2021 15:40
@ghost ghost merged commit 95ed51f into kubev2v:main Jan 14, 2021
@mturley mturley deleted the bz1916178-unblock-storage-classes branch January 14, 2021 15:47
@apinnick
Copy link

@mturley @fdupont-redhat

After checking OpenShift docs, I would like to change the error message slightly.

Could we change the first sentence to "This storage does not support dynamic provisioning"?

@mturley
Copy link
Collaborator Author

mturley commented Jan 18, 2021

@apinnick sure. fixed in #386 (backport: #387)

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants