Skip to content

Log previous and new value of configuration when reset/update API is called #10769

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

Merged
merged 4 commits into from
Jun 4, 2025

Log previous and new value of configuration before reset/update

7280d5b
Select commit
Loading
Failed to load commit list.
Merged

Log previous and new value of configuration when reset/update API is called #10769

Log previous and new value of configuration before reset/update
7280d5b
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Apr 24, 2025 in 1s

38.46% of diff hit (target 16.31%)

View this Pull Request on Codecov

38.46% of diff hit (target 16.31%)

Annotations

Check warning on line 982 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#L982 

Added line #L982 was not covered by tests

Check warning on line 987 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#L987 

Added line #L987 was not covered by tests

Check warning on line 992 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#L992 

Added line #L992 was not covered by tests

Check warning on line 999 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#L999 

Added line #L999 was not covered by tests

Check warning on line 1005 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#L1005 

Added line #L1005 was not covered by tests

Check warning on line 1010 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#L1010 

Added line #L1010 was not covered by tests

Check warning on line 1015 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#L1015 

Added line #L1015 was not covered by tests

Check warning on line 1031 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#L1030-L1031 

Added lines #L1030 - L1031 were not covered by tests

Check warning on line 1035 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#L1033-L1035 

Added lines #L1033 - L1035 were not covered by tests

Check warning on line 1229 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#L1229 

Added line #L1229 was not covered by tests
View more details on Codecov