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

Add couple utility in disk utility #5928

Merged
merged 2 commits into from May 14, 2024

Conversation

PraveenPenguin
Copy link
Collaborator

1- added a utility which Returns io scheduler available for the IO Device
2-Added utility in disk library help to Set io scheduler to a device

@mr-avocado
Copy link

mr-avocado bot commented May 8, 2024

Dear contributor,
Avocado is currently at the end of sprint #105, therefore we are in feature freeze state.
Please avoid merging changes that do not fall into these categories:

  • Bug fixes
  • Documentation updates

The feature freeze will be active until the release planned on 05/06/2024.

@PraveenPenguin PraveenPenguin changed the title Add disk couple of disk utility Add couple utility in disk utility May 8, 2024
@PraveenPenguin PraveenPenguin force-pushed the add_disk branch 3 times, most recently from 00f5729 to f38bb3b Compare May 8, 2024 06:05
@richtja
Copy link
Contributor

richtja commented May 10, 2024

Hi @PraveenPenguin, alongside the static check and codespell it LGTM. Can you please make your code align with static checks and add hda to spell.ignore thank you.

@PraveenPenguin PraveenPenguin force-pushed the add_disk branch 4 times, most recently from fe92ea2 to 595078c Compare May 13, 2024 12:52
added a utility which Returns io scheduler available for the IO Device

Signed-off-by: Praveen K Pandey <praveen@linux.vnet.ibm.com>
@PraveenPenguin PraveenPenguin force-pushed the add_disk branch 2 times, most recently from 3d4df0f to d22bae0 Compare May 13, 2024 13:22
Added utility in disk library help to Set io scheduler to a device

Signed-off-by: Praveen K Pandey <praveen@linux.vnet.ibm.com>
@PraveenPenguin
Copy link
Collaborator Author

@richtja resolved CI issue can you please review

Copy link
Contributor

@richtja richtja left a comment

Choose a reason for hiding this comment

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

LGMT, thank you @PraveenPenguin

@richtja richtja merged commit bf9bc74 into avocado-framework:master May 14, 2024
60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done 106
Development

Successfully merging this pull request may close these issues.

None yet

2 participants