Skip to content

Commit 82c9a0c

Browse files
committed
Address reviews
1 parent 8306d7b commit 82c9a0c

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

server/src/main/java/org/apache/cloudstack/storage/heuristics/HeuristicRuleHelper.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ protected void injectPresetVariables(JsInterpreter jsInterpreter, PresetVariable
165165
}
166166

167167
if (presetVariables.getBackup() != null) {
168-
jsInterpreter.injectVariable("backup", presetVariables.getBackup().toString());
168+
jsInterpreter.injectVariable("backup", presetVariables.getBackup());
169169
}
170170

171171
if (presetVariables.getAccount() != null) {

tools/marvin/marvin/lib/base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6312,7 +6312,7 @@ def createVMFromBackup(cls, apiclient, services, mode, backupid, accountname, do
63126312

63136313
@classmethod
63146314
def downloadValidationScreenshot(self, apiclient, backupid):
6315-
"""Delete VM backup"""
6315+
"""Download Validation Screenshot"""
63166316

63176317
cmd = downloadValidationScreenshot.downloadValidationScreenshotCmd()
63186318
cmd.backupid = backupid

utils/src/main/java/com/cloud/utils/exception/BackupException.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020

2121
public class BackupException extends RuntimeException {
2222

23-
boolean isVmConsistent;
23+
private final boolean isVmConsistent;
2424

2525
public BackupException(String message, Throwable cause, boolean isVmConsistent) {
2626
super(message, cause);

0 commit comments

Comments
 (0)