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

Allow to create bcache without caching set #838

Merged
merged 7 commits into from Feb 8, 2019

Conversation

joseivanlopez
Copy link
Contributor

@joseivanlopez joseivanlopez commented Feb 6, 2019

Problem

Bcache devices can be created without an associated caching set. In fact, this is a recommended practice if you are thinking about using bcache later. The idea is to setup all your slow devices as bcache backing devices without a cache, and you can choose to add a caching device later.

The Expert Partitioner needs to be adapted to allow bcache creation without a caching set.

Solution

The widget to select the caching set now has a new option: "Without caching".

Testing

  • Added unit tests.
  • Manually tested.

Screenshots

Option to not select caching device

virtualbox_opensuse tumbleweed_07_02_2019_15_59_47

Error when there are no suitable backing devices

virtualbox_opensuse tumbleweed_08_02_2019_09_50_30

Added help

virtualbox_opensuse tumbleweed_08_02_2019_10_02_39

@joseivanlopez joseivanlopez changed the title Allow to create bcache without caching set [WIP] Allow to create bcache without caching set Feb 6, 2019
@coveralls
Copy link

coveralls commented Feb 7, 2019

Coverage Status

Coverage increased (+0.03%) to 97.387% when pulling 84440ae on joseivanlopez:bcache_without_cset into d097cd5 on yast:master.

@joseivanlopez joseivanlopez changed the title [WIP] Allow to create bcache without caching set Allow to create bcache without caching set Feb 7, 2019
Copy link
Contributor

@ancorgs ancorgs left a comment

Choose a reason for hiding this comment

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

LGTM

@joseivanlopez joseivanlopez merged commit c61fa37 into yast:master Feb 8, 2019
@yast-bot
Copy link

yast-bot commented Feb 8, 2019

❌ Public Jenkins job #99 failed

@yast-bot
Copy link

yast-bot commented Feb 8, 2019

✔️ Internal Jenkins job #108 successfully finished
✔️ Created IBS submit request #183974

@yast yast deleted a comment from yast-bot Feb 8, 2019
@yast-bot
Copy link

yast-bot commented Feb 8, 2019

❌ Public Jenkins job #101 failed

@yast-bot
Copy link

yast-bot commented Feb 8, 2019

✔️ Public Jenkins job #102 successfully finished
✔️ Created OBS submit request #672840

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

4 participants