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

fio: Move pmem create code out of fio test #2695

Open
abdhaleegit opened this issue Aug 25, 2023 · 3 comments
Open

fio: Move pmem create code out of fio test #2695

abdhaleegit opened this issue Aug 25, 2023 · 3 comments

Comments

@abdhaleegit
Copy link
Collaborator

As currently the fio code has pmem configuration code, so in long term we should move tihs out of fio

@PraveenPenguin
Copy link
Member

@abdhaleegit I feel it should be targeted to fio test as well as PMEM just a device configuration that can be YAML input so having different test cases will increase maintenance cost

@abdhaleegit
Copy link
Collaborator Author

abdhaleegit commented Sep 19, 2023

@PraveenPenguin I mean pmem device creation should be out of fio, only fio takes pmem device name.. which is already created. and the pmem can be created manually or by calling pmem code explicitly in cfg

@PraveenPenguin
Copy link
Member

@PraveenPenguin I mean pmem device creation should be out of fio, only fio takes pmem device name.. which is already created. and the pmem can be created manually or by calling pmem code explicitly in cfg

If not implement that had take on this but why to drop the code and have this user pre-requisite, as we have Avocado API which does these operation

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

No branches or pull requests

2 participants