Skip to content

Configuration settings validation improvements #10998

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.

Sign up for GitHub

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

Draft
wants to merge 1 commit into
base: 4.19
Choose a base branch
from

Configuration validation improvements, and some code refactoring

b7a23e2
Select commit
Loading
Failed to load commit list.
Draft

Configuration settings validation improvements #10998

Configuration validation improvements, and some code refactoring
b7a23e2
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jun 10, 2025 in 1s

41.02% of diff hit (target 15.18%)

View this Pull Request on Codecov

41.02% of diff hit (target 15.18%)

Annotations

Check warning on line 1319 in server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java#L1318-L1319 

Added lines #L1318 - L1319 were not covered by tests

Check warning on line 1321 in server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java#L1321 

Added line #L1321 was not covered by tests

Check warning on line 1326 in server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java#L1323-L1326 

Added lines #L1323 - L1326 were not covered by tests

Check warning on line 1395 in server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java#L1394-L1395 

Added lines #L1394 - L1395 were not covered by tests

Check warning on line 1398 in server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java#L1397-L1398 

Added lines #L1397 - L1398 were not covered by tests

Check warning on line 1401 in server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java#L1400-L1401 

Added lines #L1400 - L1401 were not covered by tests

Check warning on line 1403 in server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java#L1403 

Added line #L1403 was not covered by tests

Check warning on line 1406 in server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

server/src/main/java/com/cloud/configuration/ConfigurationManagerImpl.java#L1405-L1406 

Added lines #L1405 - L1406 were not covered by tests
View more details on Codecov