Allon Mureinik has submitted this change and it was merged.
Change subject: core: Remove redundant "constants"
......................................................................
core: Remove redundant "constants"
Removed the redundant WARNING_LOW_SPACE_INDICATOR and
CRITICAL_SPACE_ACTION_BLOCKER "constants". They are only used to inject
config values (via the MockConfigRule) to config values which have
names that are descriptive enough. The double declaration doesn't add
anything besides lines of code.
Change-Id: I5068bae9b3414d3ed9eb69c4cc56a703de73e58f
Signed-off-by: Allon Mureinik <amureini(a)redhat.com>
---
M
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/AddExistingBlockStorageDomainCommandTest.java
1 file changed, 2 insertions(+), 4 deletions(-)
Approvals:
Tal Nisan: Looks good to me, approved
Allon Mureinik: Verified; Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/42825
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5068bae9b3414d3ed9eb69c4cc56a703de73e58f
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org