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

Feature: Add support to ignore empty checks on UsedPartitionSpace plugin and ignore Unknowns #85

Merged

Conversation

LordHepipud
Copy link
Collaborator

@LordHepipud LordHepipud commented Oct 15, 2020

We should add support to ignore empty check packages in case we filtered all partitions out on the check. This might be required in certain scenarios and can be very helpful.

In addition it might be helpful to convert Unknown partition data to Ok for not causing false positives.

@LordHepipud LordHepipud force-pushed the feature/allow_empty_checks_on_usedpartitionspace_plugin branch from a0c0637 to b81fc8f Compare October 15, 2020 13:16
@LordHepipud LordHepipud self-assigned this Oct 15, 2020
@LordHepipud LordHepipud added the enhancement New feature or request label Oct 15, 2020
@LordHepipud LordHepipud added this to the v1.3.0 milestone Oct 15, 2020
@LordHepipud LordHepipud changed the title Feature: Add support to ignore empty checks on UsedPartitionSpace plugin Feature: Add support to ignore empty checks on UsedPartitionSpace plugin and ignore Unknowns Nov 6, 2020
@LordHepipud LordHepipud force-pushed the feature/allow_empty_checks_on_usedpartitionspace_plugin branch 2 times, most recently from 6c2ff87 to d321463 Compare November 6, 2020 14:39
* Plugin now supports to turn empty check packages from Unknown to Ok
* Plugin now supports to set Unknown of partitions without metrics to Ok
@LordHepipud LordHepipud force-pushed the feature/allow_empty_checks_on_usedpartitionspace_plugin branch from d321463 to adf0a97 Compare November 6, 2020 14:41
@LordHepipud LordHepipud merged commit 18098f8 into master Nov 6, 2020
@LordHepipud LordHepipud deleted the feature/allow_empty_checks_on_usedpartitionspace_plugin branch November 6, 2020 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant