[Engine-patches] Change in ovirt-engine[master]: engine: VdsDeploy cleanup 1

[lhornyak]

Laszlo Hornyak has uploaded a new change for review.

Change subject: engine: VdsDeploy cleanup 1
......................................................................

engine: VdsDeploy cleanup 1

- removed hunngarian notation from method names

Change-Id: I8aedeef3aa6a614881ad3a9b7dc46b90417d6169
Signed-off-by: Laszlo Hornyak <lhorn...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
1 file changed, 15 insertions(+), 15 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/72/12972/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
index 6d4e759..138f552 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
@@ -102,7 +102,7 @@
      * update the vds object.
      * @param vdsmid unique id read from host.
      */
-    private void _setVdsmId(String vdsmid) {
+    private void setVdsmId(String vdsmid) {
         if (vdsmid == null) {
             throw new SoftError("Cannot acquire node id");
         }
@@ -162,7 +162,7 @@
     /**
      * Set host to be node.
      */
-    private void _setNode() {
+    private void setNode() {
         _isNode = true;
 
         _vds.setVdsType(VDSType.oVirtNode);
@@ -181,7 +181,7 @@
      * For this simple task, no need to go via command mechanism.
      * @param status new status.
      */
-    private void _setVdsStatus(VDSStatus status) {
+    private void setVdsStatus(VDSStatus status) {
         _vds.setStatus(status);
 
         TransactionSupport.executeInNewTransaction(new 
TransactionMethod<Void>() {
@@ -318,7 +318,7 @@
                     InstallerMessages.Severity.INFO,
                     "Host is ovirt-node"
                 );
-                _setNode();
+                setNode();
             }
             return null;
         }},
@@ -375,7 +375,7 @@
             return null;
         }},
         new Callable<Object>() { public Object call() throws Exception {
-            _setVdsmId((String)_parser.cliEnvironmentGet(VdsmEnv.VDSM_ID));
+            setVdsmId((String)_parser.cliEnvironmentGet(VdsmEnv.VDSM_ID));
             return null;
         }},
         new Callable<Object>() { public Object call() throws Exception {
@@ -633,7 +633,7 @@
     /**
      * Execute the next termination vector entry.
      */
-    private void _nextTerminationEntry() throws Exception {
+    private void nextTerminationEntry() throws Exception {
         try {
             _terminationDialog[_terminationIndex++].call();
         }
@@ -646,7 +646,7 @@
      * Dialog implementation.
      * Handle events incoming from host.
      */
-    private void _threadMain() {
+    private void threadMain() {
         try {
             boolean terminate = false;
 
@@ -706,7 +706,7 @@
                         _nextCustomizationEntry();
                     }
                     else if (Queries.TERMINATION_COMMAND.equals(event.name)) {
-                        _nextTerminationEntry();
+                        nextTerminationEntry();
                     }
                     else {
                         throw new Exception(
@@ -795,7 +795,7 @@
             new Runnable() {
                 @Override
                 public void run() {
-                    _threadMain();
+                    threadMain();
                 }
             },
             "VdsDeploy"
@@ -892,7 +892,7 @@
     public void execute() throws Exception {
         InputStream in = null;
         try {
-            _setVdsStatus(VDSStatus.Installing);
+            setVdsStatus(VDSStatus.Installing);
 
             _dialog.setHost(_vds.getHostName());
             _dialog.connect();
@@ -940,13 +940,13 @@
                 );
             }
             else if (_goingToReboot) {
-                _setVdsStatus(VDSStatus.Reboot);
+                setVdsStatus(VDSStatus.Reboot);
             }
             else if (_installIncomplete) {
-                _setVdsStatus(VDSStatus.InstallFailed);
+                setVdsStatus(VDSStatus.InstallFailed);
             }
             else {
-                _setVdsStatus(VDSStatus.NonResponsive);
+                setVdsStatus(VDSStatus.NonResponsive);
             }
         }
         catch (TimeLimitExceededException e){
@@ -959,7 +959,7 @@
                 InstallerMessages.Severity.ERROR,
                 "Processing stopped due to timeout"
             );
-            _setVdsStatus(VDSStatus.InstallFailed);
+            setVdsStatus(VDSStatus.InstallFailed);
             throw e;
         }
         catch(Exception e) {
@@ -968,7 +968,7 @@
                 _vds.getHostName(),
                 e
             );
-            _setVdsStatus(VDSStatus.InstallFailed);
+            setVdsStatus(VDSStatus.InstallFailed);
 
             if (_failException == null) {
                 throw e;


--
To view, visit http://gerrit.ovirt.org/12972
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8aedeef3aa6a614881ad3a9b7dc46b90417d6169
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Laszlo Hornyak <lhorn...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches